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

22/01/2021 21:31, Dmitry Kozlyuk:

On Wed, 20 Jan 2021 11:24:21 +0100, Thomas Monjalon wrote:

quoted

20/01/2021 08:23, Dmitry Kozlyuk:

quoted

On Wed, 20 Jan 2021 01:05:59 +0100, Thomas Monjalon wrote:  

quoted

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  

Will investigate and fix ASAP.

Now that I think of it: strings like this change every time new PCI IDs are
added to a PMD, but AFAIK adding PCI IDs is not considered an ABI breakage,
is it? One example is 28c9a7d7b48e ("net/mlx5: add ConnectX-6 Lx device ID")
added 2020-07-08, i.e. clearly outside of ABI change window.

You're right.

"xxx_pmd_info" changes are due to JSON formatting (new is more canonical),
which can be worked around easily, if the above is wrong.

If the new format is better, please keep it.
What we need is an exception for the pmdinfo symbols
in the file devtools/libabigail.abignore.
You can probably use a regex for these symbols.

quoted

quoted

quoted

quoted

--- a/meson.build
+++ b/meson.build
-subdir('buildtools/pmdinfogen')    

This could be in patch 3 (removing the code).  

It would redefine "pmdinfogen" variable to old pmdinfogen.
Besides, why build what's not used at this patch already?  

Just trying to find the best patch split.
If needed, OK to keep as is.

I even don't mind squashing all three commits into one.
The split is done to ease the review.

I think the split is good as is.