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: RFC: Updates support for breakpoints that generate SIGILL


On Mon, Feb 08, 2010 at 08:35:17PM +0100, Ulrich Weigand wrote:
> Unfortunately this change broke Cell/B.E. debugging.
> 
> The problem is that with the change, handle_inferior_event will now
> *always* look up a regcache, even when the process has exited.
> 
> Before the patch, the code has always taken care to avoid that; while
> some targets don't mind, others will break (in particular, the Cell/B.E.
> multi-arch target does).
> 
> The patch below changes this to only look up the regcache when needed
> (in particular, this implies the process is still there).
> 
> Tested on Cell/B.E. (ppc32 and ppc64) with no regressions.
> 
> Does this look OK to you?

Sorry for the breakage.  Yes, your fix looks good to me.

-- 
Daniel Jacobowitz
CodeSourcery


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