Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-12 Thread Thomas Monjalon
12/10/2022 18:48, Robin Jarry: > Thomas Monjalon, Oct 12, 2022 at 18:44: > > OK for executable, > > but can we expect .so to be always executable? > > I think this is the default when linking. Whether dynamic libraries or > executable programs. > > Also, the "must be executable" limitation only a

Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-12 Thread Robin Jarry
Thomas Monjalon, Oct 12, 2022 at 18:44: > OK for executable, > but can we expect .so to be always executable? I think this is the default when linking. Whether dynamic libraries or executable programs. Also, the "must be executable" limitation only applies on the files specified on the command li

Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-12 Thread Thomas Monjalon
12/10/2022 18:30, Robin Jarry: > Thomas Monjalon, Oct 12, 2022 at 18:16: > > What about implementing both? > > If ldd is available, use it, > > otherwise use LD_TRACE_LOADED_OBJECTS variable. > > This is a bit overkill in my opinion. Also it would make the behaviour > somewhat different whether ld

Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-12 Thread Robin Jarry
Thomas Monjalon, Oct 12, 2022 at 18:16: > What about implementing both? > If ldd is available, use it, > otherwise use LD_TRACE_LOADED_OBJECTS variable. This is a bit overkill in my opinion. Also it would make the behaviour somewhat different whether ldd is available and/or the analyzed binaries a

Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-12 Thread Thomas Monjalon
12/10/2022 17:16, Olivier Matz: > On Tue, Oct 11, 2022 at 12:44:56AM +0200, Thomas Monjalon wrote: > As discussed off-list with Robin, it appears that "ldd" is not available > on buildroot-based images. See: > http://lists.busybox.net/pipermail/buildroot/2013-July/074927.html > > The link is quite

Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-12 Thread Olivier Matz
Hi, On Tue, Oct 11, 2022 at 12:44:56AM +0200, Thomas Monjalon wrote: > 04/10/2022 21:29, Robin Jarry: > > dpdk-pmdinfo.py does not produce any parseable output. The -r/--raw flag > > merely prints multiple independent JSON lines which cannot be fed > > directly to any JSON parser. Moreover, the sc

Re: [PATCH v7] usertools: rewrite pmdinfo

2022-10-10 Thread Thomas Monjalon
04/10/2022 21:29, Robin Jarry: > dpdk-pmdinfo.py does not produce any parseable output. The -r/--raw flag > merely prints multiple independent JSON lines which cannot be fed > directly to any JSON parser. Moreover, the script complexity is rather > high for such a simple task: extracting PMD_INFO_S

[PATCH v7] usertools: rewrite pmdinfo

2022-10-04 Thread Robin Jarry
dpdk-pmdinfo.py does not produce any parseable output. The -r/--raw flag merely prints multiple independent JSON lines which cannot be fed directly to any JSON parser. Moreover, the script complexity is rather high for such a simple task: extracting PMD_INFO_STRING from .rodata ELF sections. Rewrit