This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Async mode fixes.


On Saturday 08 March 2008 01:32:47 Daniel Jacobowitz wrote:
> Hi Nick, thanks for looking at this.  I was hoping you would.
> 
> On Sat, Mar 08, 2008 at 11:18:47AM +1300, Nick Roberts wrote:
> > Like my patch, IMO this one looks a bit like a dog's breakfast.  It's got some
> > good ideas, though, and has certainly increased my understanding of the
> > asynchronous code.  Perhaps a combination of the two patches would create
> > something useful.
> 
> As I understand it, it already is combined - I know Vladimir started
> with a copy of your most recent async patch.

Yes. Nick patch has two bits:
1. Linux native async mode (which Pedro is looking at as we speak)
2. Core async mode changes.

My patches addresses (2), and I've used both Nick's patch and Apple
branch source as reference.

> I haven't looked at his 
> patch yet but it may need to be broken down into some pieces.

I surely can break it into 10 separate patches, *if* that will make it
more easy to review for you.

> > It may reduce the failures, but I suspect that's partly because it's not really
> > running asynchronously.  I don't understand how it really could without adding
> > another file handler for inferior events.  That doesn't mean it can't, just
> > that I don't understand how it could.
> 
> Vladimir, what target were you using to test - was it "target async"?

Yes, it's target async, and it's really running asynchronously.

- Volodya


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]