[Crash-utility] [PATCH] Use /usr/bin/nm -D

Dave Anderson anderson at redhat.com
Tue Aug 26 12:30:05 UTC 2008


----- "Bernhard Walle" <bwalle at suse.de> wrote:

> * Dave Anderson [2008-08-25 16:07]:
> > 
> > I'm not sure what's worse -- not doing the translation at all
> > or showing potentially bogus entries.
> 
> *arg*, yes, I didn't check for such static functions. The functions
> in
> my test were only global, so everything worked.
> 
> However, I now modified the patch to check the size (with nm -S), and
> report "unknown" in such cases. So we still have the global symbols
> right, but no "false positives".
> 
> Is that okay?

Looks OK -- I'll give it run today, and if no problems arise, 
I'll queue it as is.

Thanks,
  Dave




More information about the Crash-utility mailing list