This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: [PATCH] elfclassify tool
- From: Florian Weimer <fweimer at redhat dot com>
- To: Mark Wielaard <mark at klomp dot org>
- Cc: elfutils-devel at sourceware dot org, Panu Matilainen <pmatilai at laiskiainen dot org>
- Date: Mon, 22 Jul 2019 17:54:57 +0200
- Subject: Re: [PATCH] elfclassify tool
- References: <87k1fz8c9q.fsf@oldenburg2.str.redhat.com> <2e6a27c552ae5e365db54ca6b432c77c9ad5b041.camel@klomp.org> <871s22yybt.fsf@oldenburg2.str.redhat.com> <8736mfzhob.fsf@oldenburg2.str.redhat.com> <ce73f5ebf076570ddc0c1dcd57a07ec39b7555b0.camel@klomp.org>
* Mark Wielaard:
> On Thu, 2019-04-18 at 13:17 +0200, Florian Weimer wrote:
>> * Florian Weimer:
>>
>> > > BTW. Florian, the extra options are certainly not required for you to
>> > > implement to get eu-elfclassify accepted. They are just suggestions,
>> > > which we might decide not to do/add. Or they can be added by others if
>> > > they think they are useful.
>> >
>> > Understood. I would rather fix the command line syntax as a priority,
>> > implement --unstripped, and add a test suite.
>>
>> The patch below, also available here:
>>
>> <https://pagure.io/fweimer/elfutils/commits/elfclassify>
>>
>> reworks the command line parser, implements filtering of file lists, and
>> adds the --unstripped option.
>
> That looks really good. I went ahead an fixed a couple of nits and
> added some of my suggestions. I'll respond to your other email
> explaining some of my reasoning. The changes I made are:
Wow, this is much more than I expected. Thanks!
> elfclassify: Fix bre -> be typo in "unstripped" option help text.
> elfclassify: When reading stdin make sure paths don't include newline.
> elfclassify: Allow inspecting compressed or (kernel) image files with -z.
> elfclassify: Always clean up ELF file and descriptor if one is still open.
> elfclassify: Don't treat errors in elf_open or run_classify as fatal.
> elfclassify: Add --quiet/-q to suppress error messages.
> elfclassify: Add \n to fputs debug output.
> elfclassify: Add --file/-f for testing just regular files.
> elfclassify: Require --elf by default. Add more classifications.
> elfclassify: Add elf_kind and elf_type strings for verbose output.
> elfclassify: Require PT_LOAD for loadable classification.
> elfclassify: Add --program classification.
> elfclassify: Don't use ARGP_NO_EXIT and document exit code expectation.
> elfclassify: Add --linux-kernel-module classification.
> elfclassify: Add --debug-only classification.
I went through these patches, albeit in a somewhat cursory fashion, and
they look okay to me.
Do you think this is enough to port over RPM's find-debuginfo.sh?
Thanks,
Florian