Re: [dpdk-dev] [PATCH v10 0/3] pmdinfogen: rewrite in Python

Hi,

-----Original Message-----
From: dev <redacted> On Behalf Of Dmitry Kozlyuk
Sent: Sunday, January 24, 2021 10:52 PM
To: dev@dpdk.org
Cc: Maxime Coquelin <redacted>; Bruce Richardson
[off-list ref]; NBU-Contact-Thomas Monjalon
[off-list ref]; Dmitry Kozlyuk [off-list ref]; Neil
Horman [off-list ref]; Jie Zhou [off-list ref]
Subject: [dpdk-dev] [PATCH v10 0/3] pmdinfogen: rewrite in Python

This patchset implements existing pmdinfogen logic in Python, replaces and
removes the old code. The goals of rewriting are:

* easier maintenance by using a more high-level language,
* simpler build process without host application and libelf,
* foundation for adding Windows support.

Identity of generated PMD information is checked by comparing output of
pmdinfo before and after the patch:

    find build/drivers -name '*.so' -exec usertools/dpdk-pmdinfo.py

Acked-by: Neil Horman <nhorman@tuxdriver.com>
Tested-by: Jie Zhou <redacted>

---

Please see the following build errors:
https://github.com/DPDK/dpdk/actions/runs/509771118
"""
FAILED: drivers/rte_common_cpt.pmd.c 
/usr/bin/python3 ../buildtools/gen-pmdinfo-cfile.py ar /home/runner/work/dpdk/dpdk/build/drivers/libtmp_rte_common_cpt.a drivers/rte_common_cpt.pmd.c /usr/bin/python3 ../buildtools/pmdinfogen.py elf
Traceback (most recent call last):
  File "../buildtools/gen-pmdinfo-cfile.py", line 14, in <module>
    [ar, "xv", os.path.abspath(archive)], capture_output=True, check=True, cwd=temp
  File "/usr/lib/python3.6/subprocess.py", line 423, in run
    with Popen(*popenargs, **kwargs) as process:
TypeError: __init__() got an unexpected keyword argument 'capture_output'
"""

I also see the exception in Ubuntu 16.04 (Python 3.5.2, pyelftools 0.27), CentOS 7 (Python 3.6.8, pyelftools 0.27), CentOS 8 (Python 3.6.8, pyelftools 0.27) and OpenSUSE Leap 15.2 (Python 3.6.12, pyelftools 0.26).

Thanks,
Ali