This is the mail archive of the gdb-patches@sources.redhat.com 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: PING: [RFA] Runtime Dwarf2 CFI engine cleanup


On Wed, Feb 26, 2003 at 01:22:49PM -0500, Andrew Cagney wrote:
> >
> >
> >I don't think that the way you're selecting objfiles to unload is ideal
> >- you're duplicating logic from objfile_purge_solibs.  We need a more
> >general hook to free objfile-specific information.  However, I believe
> >that can be addressed separately, and this will definitely fix some
> >crashes as-is and not introduce any new ones.
> >
> >I don't have any problem with linking dwarf2cfi.o in for all targets. 
> >Let's sit on this for another day and see if anyone else objects, and
> >then you can commit it.
> 
> A quick look at the bigger picture reveals:
> 
> - the underlying code needs to be fixed before it is linked into all 
> targets.

Why?  The code can be fixed as it is activated for other targets;
linking and use aren't the same thing.  I'd even call this the first
step in fixing it - doesn't break anything and makes it easier for
other targets to test it.

> - if it needs to be modified of changes in another part of gdb then it 
> should use an observer.  It should not introduce another hack.
> 
> You'll note that Joel and I are currently working through this.  Yes it 
> means that the posted patch doesn't go in.  It also means that GDB is 
> that bit more (not less) maintainable.

OK, if that's your preference, Michal please do not check in this
patch.  I think it's a bit silly to reject a patch which fixes a
"break; run; run" segfault on this basis, especially one two months
or more old, though.

This is one of the two major issues I know about that are forcing
Michal to do x86-64 development on 5.3 instead of HEAD.  I applaud your
vision in preserving future maintainability; I'm tired of that vision
interfering with having a working GDB.

> >You made a line in Makefile.in too wide; please fix that before you
> >check it in.
> >
> >[List: my assumption in approving this is that the dwarf2cfi support is
> >not part of the dwarf2 reader, so I'm not stepping on anyone's toes. 
> >If you disagree, please tell me so.]
> 
> Puzzled expression.

This is the problem with vague lines of authority.  It's not always
clear who can approve a patch even to people who do the approving.  I'm
tired of exercising what seems to be pretty clear authority to approve
something and getting prickly mail from someone else overturning my
decision.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer


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