[PATCH v5] usertools: rewrite pmdinfo

Ferruh Yigit ferruh.yigit at amd.com
Thu Sep 22 17:12:13 CEST 2022


On 9/22/2022 12:58 PM, Robin Jarry wrote:
> 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. Rewrite it so that it can produce valid JSON.
> 
> Remove the PCI database parsing for PCI-ID to Vendor-Device names
> conversion. This should be done by external scripts (if really needed).
> 
> The script passes flake8, black, isort and pylint checks.
> 
> I have tested this with a matrix of python/pyelftools versions:
> 
>                                   pyelftools
>                 0.22  0.23  0.24  0.25  0.26  0.27  0.28  0.29
>          3.6      ok    ok    ok    ok    ok    ok    ok    ok
>          3.7      ok    ok    ok    ok    ok    ok    ok    ok
>   Python 3.8      ok    ok    ok    ok    ok    ok    ok    ok
>          3.9      ok    ok    ok    ok    ok    ok    ok    ok
>          3.10   fail  fail  fail  fail    ok    ok    ok    ok
> 
> All failures with python 3.10 are related to the same issue:
> 
>    File "elftools/construct/lib/container.py", line 5, in <module>
>      from collections import MutableMapping
>    ImportError: cannot import name 'MutableMapping' from 'collections'
> 
> Python 3.10 support is only available since pyelftools 0.26. The script
> will only work with Python 3.6 and later. Update the minimal system
> requirements, docs and release notes.
> 
> Cc: Olivier Matz<olivier.matz at 6wind.com>
> Cc: Ferruh Yigit<ferruh.yigit at xilinx.com>
> Cc: Bruce Richardson<bruce.richardson at intel.com>
> Signed-off-by: Robin Jarry<rjarry at redhat.com>

Tested-by: Ferruh Yigit <ferruh.yigit at amd.com>


More information about the dev mailing list