Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Ulrich Weigand <Ulrich.Weigand@de.ibm.com>
To: "akamath996@gmail.com" <akamath996@gmail.com>,
	"tom@tromey.com" <tom@tromey.com>,
	"simon.marchi@polymtl.ca" <simon.marchi@polymtl.ca>
Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>,
	SANGAMESH MALLAYYA <sangamesh.swamy@in.ibm.com>,
	Aditya Kamath <Aditya.Kamath1@ibm.com>
Subject: Re: [PATCH v2] Speed up next/step while debugging multithreaded programs on AIX.
Date: Thu, 10 Sep 2026 13:14:15 +0000	[thread overview]
Message-ID: <f341ce7e2c62377bc3c9580e2229c1a86de42183.camel@de.ibm.com> (raw)
In-Reply-To: <20260910101643.85955-2-akamath996@gmail.com>

Aditya Vidyadhar Kamath <akamath996@gmail.com> wrote:

+      /* Attempt to sync_threadlists() only when debugging object
files
+	 and not core files since the list of threads never changes
for
+	 core files.  Even for live inferiors, skip the sync when the
+	 thread count matches the previous stop.  */

This last part seems racy to me.  In principle, it is possible
that one thread terminated and simultaneously one new thread was
created, since the last time we stopped.  In this case, the thread
count would be the same, but the thread list still incorrect.

Given the other test for single-stepping, is this particular
check even still resulting in a noticeable performance
difference?

Bye,
Ulrich

  reply	other threads:[~2026-09-10 13:14 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-10 10:16 Aditya Vidyadhar Kamath
2026-09-10 13:14 ` Ulrich Weigand [this message]
2026-09-10 13:45   ` Simon Marchi
2026-09-10 13:25 ` Simon Marchi
2026-09-10 17:32 ` Abhay Kandpal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f341ce7e2c62377bc3c9580e2229c1a86de42183.camel@de.ibm.com \
    --to=ulrich.weigand@de.ibm.com \
    --cc=Aditya.Kamath1@ibm.com \
    --cc=akamath996@gmail.com \
    --cc=gdb-patches@sourceware.org \
    --cc=sangamesh.swamy@in.ibm.com \
    --cc=simon.marchi@polymtl.ca \
    --cc=tom@tromey.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox