Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Thomas Monjalon
25/01/2021 11:46, Kinsella, Ray: > On 25/01/2021 10:29, David Marchand wrote: > > The symbol itself can be hidden from the ABeyes. > > It is only a placeholder for the PMD_INFO_STRING= string used by > > usertools/dpdk-pmdinfo.py and maybe some other parsing tool. > > > > I guess a static symbol w

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Kinsella, Ray
On 25/01/2021 10:29, David Marchand wrote: > On Mon, Jan 25, 2021 at 11:01 AM Kinsella, Ray wrote: >> >> >> >> On 25/01/2021 09:25, Kinsella, Ray wrote: >>> >>> >>> On 23/01/2021 11:38, Thomas Monjalon wrote: 22/01/2021 23:24, Dmitry Kozlyuk: > On Fri, 22 Jan 2021 21:57:15 +0100, Thoma

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Dmitry Kozlyuk
On Mon, 25 Jan 2021 10:11:07 +, Kinsella, Ray wrote: > On 25/01/2021 10:05, Dmitry Kozlyuk wrote: > > On Mon, 25 Jan 2021 09:25:51 +, Kinsella, Ray wrote: > >> On 23/01/2021 11:38, Thomas Monjalon wrote: > >>> 22/01/2021 23:24, Dmitry Kozlyuk: > On Fri, 22 Jan 2021 21:57:15 +01

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread David Marchand
On Mon, Jan 25, 2021 at 11:01 AM Kinsella, Ray wrote: > > > > On 25/01/2021 09:25, Kinsella, Ray wrote: > > > > > > On 23/01/2021 11:38, Thomas Monjalon wrote: > >> 22/01/2021 23:24, Dmitry Kozlyuk: > >>> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: > 22/01/2021 21:31, Dmitry Ko

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Kinsella, Ray
On 25/01/2021 10:05, Dmitry Kozlyuk wrote: > On Mon, 25 Jan 2021 09:25:51 +, Kinsella, Ray wrote: >> On 23/01/2021 11:38, Thomas Monjalon wrote: >>> 22/01/2021 23:24, Dmitry Kozlyuk: On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: > 22/01/2021 21:31, Dmitry Kozlyuk:

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Dmitry Kozlyuk
On Mon, 25 Jan 2021 09:25:51 +, Kinsella, Ray wrote: > On 23/01/2021 11:38, Thomas Monjalon wrote: > > 22/01/2021 23:24, Dmitry Kozlyuk: > >> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: > >>> 22/01/2021 21:31, Dmitry Kozlyuk: > On Wed, 20 Jan 2021 11:24:21 +0100, Thoma

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Kinsella, Ray
On 25/01/2021 09:25, Kinsella, Ray wrote: > > > On 23/01/2021 11:38, Thomas Monjalon wrote: >> 22/01/2021 23:24, Dmitry Kozlyuk: >>> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: 22/01/2021 21:31, Dmitry Kozlyuk: > On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-25 Thread Kinsella, Ray
On 23/01/2021 11:38, Thomas Monjalon wrote: > 22/01/2021 23:24, Dmitry Kozlyuk: >> On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: >>> 22/01/2021 21:31, Dmitry Kozlyuk: On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote: > 20/01/2021 08:23, Dmitry Kozlyuk: >> O

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-24 Thread Dmitry Kozlyuk
On Sat, 23 Jan 2021 12:38:45 +0100, Thomas Monjalon wrote: > 22/01/2021 23:24, Dmitry Kozlyuk: > > On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: > > > 22/01/2021 21:31, Dmitry Kozlyuk: > > > > On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote: > > > > > 20/01/2021 08:23

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-23 Thread Thomas Monjalon
22/01/2021 23:24, Dmitry Kozlyuk: > On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: > > 22/01/2021 21:31, Dmitry Kozlyuk: > > > On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote: > > > > 20/01/2021 08:23, Dmitry Kozlyuk: > > > > > On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Mo

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-22 Thread Dmitry Kozlyuk
On Fri, 22 Jan 2021 21:57:15 +0100, Thomas Monjalon wrote: > 22/01/2021 21:31, Dmitry Kozlyuk: > > On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote: > > > 20/01/2021 08:23, Dmitry Kozlyuk: > > > > On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Monjalon wrote: > > > > > This is now the

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-22 Thread Thomas Monjalon
22/01/2021 21:31, Dmitry Kozlyuk: > On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote: > > 20/01/2021 08:23, Dmitry Kozlyuk: > > > On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Monjalon wrote: > > > > This is now the right timeframe to introduce this change > > > > with the new Python module

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-22 Thread Dmitry Kozlyuk
On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote: > 20/01/2021 08:23, Dmitry Kozlyuk: > > On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Monjalon wrote: > > > This is now the right timeframe to introduce this change > > > with the new Python module dependency. > > > Unfortunately, the ABI ch

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-20 Thread Thomas Monjalon
20/01/2021 08:23, Dmitry Kozlyuk: > On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Monjalon wrote: > > This is now the right timeframe to introduce this change > > with the new Python module dependency. > > Unfortunately, the ABI check is returning an issue: > > > > 'const char mlx5_common_pci_pmd_inf

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-19 Thread Dmitry Kozlyuk
On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Monjalon wrote: > This is now the right timeframe to introduce this change > with the new Python module dependency. > Unfortunately, the ABI check is returning an issue: > > 'const char mlx5_common_pci_pmd_info[62]' was changed > to 'const char mlx5_commo

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2021-01-19 Thread Thomas Monjalon
This is now the right timeframe to introduce this change with the new Python module dependency. Unfortunately, the ABI check is returning an issue: 'const char mlx5_common_pci_pmd_info[62]' was changed to 'const char mlx5_common_pci_pmd_info[60]' at rte_common_mlx5.pmd.c Few more comments below:

Re: [dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2020-10-21 Thread Bruce Richardson
On Tue, Oct 20, 2020 at 08:44:02PM +0300, Dmitry Kozlyuk wrote: > Use the same interpreter to run pmdinfogen as for other build scripts. > Adjust wrapper script accordingly and also don't suppress stderr from ar > and pmdinfogen. > > Add configure-time check for elftools Python module for Unix hos

[dpdk-dev] [PATCH v8 2/3] build: use Python pmdinfogen

2020-10-20 Thread Dmitry Kozlyuk
Use the same interpreter to run pmdinfogen as for other build scripts. Adjust wrapper script accordingly and also don't suppress stderr from ar and pmdinfogen. Add configure-time check for elftools Python module for Unix hosts. Add python3-pyelftools to CI configuration. The package is available