A Friday 14 March 2008 14:38:45, Daniel Jacobowitz wrote:
> On Fri, Mar 14, 2008 at 08:02:01AM +0000, Pedro Alves wrote:
> > We do have a target method for this now.  Obviouly for every
> > async target other than "target (extended-)async", this wasn't
> > working.
> >
> > Installed as obvious (after testing with a local gdbserver with
> > the testsuite and manually).
>
> Sorry, but I don't think this is right.  Or else it deserves a better
> explanation :-)  async_remote_interrupt_twice didn't stop the target;
> it was used to escape from a hung remote target (see interrupt_query).
>
> You can trace the twisty path that got here through
> sigint_remote_twice_token.  I'm not sure what INF_QUIT_REQ should do,
> or if it's necessary - I find inferior_event_handler a bit weird.

Uuugh, I'm the one who has to say sorry.
In fact, the twisted maze that got me to needing this was a local hack
I used to have that is no longer needed.  My patch was definitelly
wrong.

I reverted the patch with the attached.  (I even had missed
Makefile.in that's why you don't see it in this patch.  Sigh...)

-- 
Pedro Alves