[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
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)
Main Index |
Thread Index |