Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [RFC: 0/9] No context-switching
@ 2008-08-16 15:29 Pedro Alves
  2008-08-18 12:10 ` Joel Brobecker
  2008-09-02 23:44 ` Daniel Jacobowitz
  0 siblings, 2 replies; 4+ messages in thread
From: Pedro Alves @ 2008-08-16 15:29 UTC (permalink / raw)
  To: gdb-patches

This series of patches removes inferior control globals, in favour of 
accessing thread_info members directly, effectivelly, getting rid of
the need to context-switch.

I've layed out the series so it is always functional at all of
its incremental steps.  At the end of the series, context_switch will
just be a wrapper around switch_to_thread.  I still left that function
around, mainly, for "set debug infrun 1" purposes (it prints an
useful message).

I've tested the whole series on x86_64-unknown-linux-gnu,
i386-unknown-openbsd4.3, i386-pc-solaris2.11, i686-pc-cygwin,
and arm-none-linux-gnu (to cover software single-step), I found no
regressions.

This series depends on having all targets register the main thread
in the the thread list, even when debugging a single-threaded
inferior.  Patches for most of the targets have been posted
already, most approved.

I believe the only missing targets to adjust are AIX,
remote-mips.c, nto-procfs.c.  I've talked with Aleksandar about
nto, and we came to the conclusion that nto is easy to fix.
remote-mips.c should be as similarly easy to remote-m32-sdi.c,
monitor.c or remote-sim.c.  I'll try to give another look at
AIX this weekend, and browse over IBM's documentation, to see if
I can come up with an initial patch, but it doesn't seem likelly,
with no AIX access.

In the mean time, I thought I'd post this series as an RFC.

-- 
Pedro Alves


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC: 0/9] No context-switching
  2008-08-16 15:29 [RFC: 0/9] No context-switching Pedro Alves
@ 2008-08-18 12:10 ` Joel Brobecker
  2008-09-02 23:44 ` Daniel Jacobowitz
  1 sibling, 0 replies; 4+ messages in thread
From: Joel Brobecker @ 2008-08-18 12:10 UTC (permalink / raw)
  To: Pedro Alves; +Cc: gdb-patches

> This series of patches removes inferior control globals, in favour of 
> accessing thread_info members directly, effectivelly, getting rid of
> the need to context-switch.

I haven't had time to look at the specifics, but I think this will
bring great progress in our code. Thanks for doing that! I'm really
swamped right now, and trying to help with patch reviews, since we
seem to be quite backed up, but I'll take a look asap, hopefully soon.

-- 
Joel


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC: 0/9] No context-switching
  2008-08-16 15:29 [RFC: 0/9] No context-switching Pedro Alves
  2008-08-18 12:10 ` Joel Brobecker
@ 2008-09-02 23:44 ` Daniel Jacobowitz
  2008-09-08 22:15   ` Pedro Alves
  1 sibling, 1 reply; 4+ messages in thread
From: Daniel Jacobowitz @ 2008-09-02 23:44 UTC (permalink / raw)
  To: gdb-patches

On Sat, Aug 16, 2008 at 04:29:11PM +0100, Pedro Alves wrote:
> This series of patches removes inferior control globals, in favour of 
> accessing thread_info members directly, effectivelly, getting rid of
> the need to context-switch.

Aside from the spelling error in 3/9, I've reviewed all these patches
and they look fine to me - go ahead!

Some of the weirder things you turned up, like which thread we pass a
signal to, should probably be fixed... separately.

-- 
Daniel Jacobowitz
CodeSourcery


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC: 0/9] No context-switching
  2008-09-02 23:44 ` Daniel Jacobowitz
@ 2008-09-08 22:15   ` Pedro Alves
  0 siblings, 0 replies; 4+ messages in thread
From: Pedro Alves @ 2008-09-08 22:15 UTC (permalink / raw)
  To: gdb-patches; +Cc: Daniel Jacobowitz

On Wednesday 03 September 2008 00:44:09, Daniel Jacobowitz wrote:

> Aside from the spelling error in 3/9, I've reviewed all these patches
> and they look fine to me - go ahead!

Fixed the spelling error, and checked the whole series in, in addition
to all its (already approved) dependencies.

If you notice any regression related to this, please let me know
and I'll look at it ASAP.

-- 
Pedro Alves


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-09-08 22:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-08-16 15:29 [RFC: 0/9] No context-switching Pedro Alves
2008-08-18 12:10 ` Joel Brobecker
2008-09-02 23:44 ` Daniel Jacobowitz
2008-09-08 22:15   ` Pedro Alves

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox