Page MenuHomeFreeBSD

readelf: use table-based DT_FLAGS and DT_FLAGS_1 decoding

Authored by emaste on Fri, Feb 1, 3:09 PM.


Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

emaste created this revision.Fri, Feb 1, 3:09 PM
brooks accepted this revision.Fri, Feb 1, 6:18 PM
brooks added a subscriber: brooks.


2741 ↗(On Diff #53509)

Would this make more sense to the user as something like:

printf(" unknown(0x%jx)", (uintmax_t)val);
This revision is now accepted and ready to land.Fri, Feb 1, 6:18 PM
kib added a subscriber: kib.Fri, Feb 1, 6:44 PM
kib added inline comments.
2736 ↗(On Diff #53509)

This is weird, why not do &= ~ ?

2741 ↗(On Diff #53509)

It would make more sense to print each unknown flag individually. At least I do find it more useful, from my experience with the Intel CPUID bits and DMAR capabilities reports.

This revision was automatically updated to reflect the committed changes.
emaste added inline comments.Fri, Feb 1, 8:30 PM
2741 ↗(On Diff #53509)

Not a bad idea, I will keep it in mind for a followup