Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/dev



On Thu, 26 May 2011, Masao Uebayashi wrote:

> > I wonder if that can be worked around by stripping or ignoring the thing
> > you want to ignore (non-global symbols), rather than potentially
> > introducing type inconsistency bugs? ("objcopy -x" will discard all
> > non-global symbols, I don't know if that is too much)
>
> - I want to keep reproducibility even in intermediate files for better
> traceability.

how do you detect different signatures?  diff -I '__func__\.[0-9]*' may
also suffice, if you were using objdump -D for that..

> - I can't think of how cfdriver decls can lead to type inconsistency bugs.

Ok, somebody makes a mistake and writes

extern struct cfdata foo_cd;

and it works by chance because the address was zero anyway, then later
something else is changed around and all of a sudden, an innocent code
causes a kernel panic..

> > you might note that the bthub.o file (at least, I didn't check the others)
> > does not actually use __func__ (unless DEBUG) and in fact the objdump does
> > not change with the removal of "ioconf.h"
>
> Right, bthub.o is not affected.  fwmem.o is.

ok but why change files not affected?  You did not actually remove all
inclusions of "ioconf.h" in any case.. (grep ioconf.h *.d finds two more
in i386/GENERIC, I didn't look at any other arch)

iain


Home | Main Index | Thread Index | Old Index