Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Pedro Alves <pedro_alves@portugalmail.pt>
To: Pedro Alves <pedro_alves@portugalmail.pt>
Cc: Lerele <lerele@champenstudios.com>,  gdb-patches@sourceware.org
Subject: Re: [Patch] Win32 gdbserver new interrupt support, and attach to     process  fix.
Date: Tue, 06 Mar 2007 00:04:00 -0000	[thread overview]
Message-ID: <45ECAFFC.4010700@portugalmail.pt> (raw)
In-Reply-To: <45EC7842.6020406@portugalmail.pt>

Pedro Alves escreveu:
> Lerele wrote:
>
>> It seems after all the solution proposed may be the best resort, 
>> instead of the last: more compatible, less interfereable with child, 
>> however a bit more difficult to write.
>> In fact, the first version of interrupt code I wrote several months 
>> ago worked like that (I still keep a copy somewhere). It worked, but 
>> it was buggy, so I dropped it and wrote that other simpler new version.
>> This other solution may be the one-for-all solution, because it uses 
>> rather more standard Win32 calls.
>
> The more I search for a *clean* way to do it in WinCE, the more I'm
> inclined to drop the create_a_remote_thread idea.  A bit of
> googling and reading old MSFT docs indicates that older debuggers either
> suspend all threads with SuspendThread (like you suggest, or get the
> main thread's PC (EIP) and set a breakpoint there, which should be
> simpler.  (I would still have to do a little more work for WinCE
> because the address might not be physically writable.)  I think I
> read somewhere that recent MSVC uses DebugBreakProcess, and hides the 
> fact
> that the break was inside ntdll.dll, or wherever, by switching threads...
>
> What do you think of just suspending of thread, and setting a breakpoint
> at the current PC, and resuming?
>

The GoVest debugger (*) uses something even simpler:
- Suspend main thread,
- Set single-step / trace flag (i386)
- Resume thread

(*) http://www.geocities.com/GoVest/

Cheers,
Pedro Alves



  reply	other threads:[~2007-03-06  0:04 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-23 23:52 Lerele
2007-02-24 12:07 ` Eli Zaretskii
2007-02-24 13:03   ` Lerele
2007-02-24 14:07     ` Eli Zaretskii
2007-02-24 15:23       ` Lerele
2007-02-24 19:10         ` Eli Zaretskii
2007-02-24 21:19         ` Pedro Alves
2007-02-24 21:44           ` Andreas Schwab
2007-02-24 23:35           ` Lerele
2007-02-25  0:15             ` Daniel Jacobowitz
2007-02-25  1:57             ` Pedro Alves
2007-02-25 22:46 ` Pedro Alves
2007-03-04 22:53   ` Lerele
2007-03-05  0:56     ` Pedro Alves
2007-03-05  1:21       ` Pedro Alves
2007-03-05 13:17       ` Lerele
2007-03-05 20:34         ` Lerele
2007-03-05 20:44         ` Pedro Alves
2007-03-06  0:04           ` Pedro Alves [this message]
2007-03-06 20:39           ` Pedro Alves
2007-03-06 22:18             ` Lerele
2007-03-06 23:22               ` Pedro Alves
2007-03-05 12:44     ` Daniel Jacobowitz
2007-03-05 20:30       ` Lerele

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=45ECAFFC.4010700@portugalmail.pt \
    --to=pedro_alves@portugalmail.pt \
    --cc=gdb-patches@sourceware.org \
    --cc=lerele@champenstudios.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