[dpdk-dev] [PATCH 0/2] have net drivers use a standard map file name format

[dpdk-dev] [PATCH 0/2] have net drivers use a standard map file name format

[Bruce Richardson]

rename some of the version.map files that don't match the standard naming
convention. For future meson build system, this will save us having to
specify for each driver what the filename of its map file is.

Bruce Richardson (2):
  net/bonding: rename map file to standard name
  net/xenvirt: rename map file to standard name

 drivers/net/bonding/Makefile                                            | 2 +-
 .../net/bonding/{rte_eth_bond_version.map => rte_pmd_bond_version.map}  | 0
 drivers/net/xenvirt/Makefile                                            | 2 +-
 .../{rte_eth_xenvirt_version.map => rte_pmd_xenvirt_version.map}        | 0
 4 files changed, 2 insertions(+), 2 deletions(-)
 rename drivers/net/bonding/{rte_eth_bond_version.map => rte_pmd_bond_version.map} (100%)
 rename drivers/net/xenvirt/{rte_eth_xenvirt_version.map => rte_pmd_xenvirt_version.map} (100%)

-- 
2.13.5

[dpdk-dev] [PATCH 1/2] net/bonding: rename map file to standard name

[Bruce Richardson]

Naming convention for net drivers is "rte_pmd_<name>_version.map"

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/net/bonding/Makefile                                            | 2 +-
 .../net/bonding/{rte_eth_bond_version.map => rte_pmd_bond_version.map}  | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename drivers/net/bonding/{rte_eth_bond_version.map => rte_pmd_bond_version.map} (100%)

diff --git a/drivers/net/bonding/Makefile b/drivers/net/bonding/Makefile
index 910c932da..ab8b6894a 100644
--- a/drivers/net/bonding/Makefile
+++ b/drivers/net/bonding/Makefile
@@ -39,7 +39,7 @@ LIB = librte_pmd_bond.a
 CFLAGS += -O3
 CFLAGS += $(WERROR_FLAGS)
 
-EXPORT_MAP := rte_eth_bond_version.map
+EXPORT_MAP := rte_pmd_bond_version.map
 
 LIBABIVER := 1
 
diff --git a/drivers/net/bonding/rte_eth_bond_version.map b/drivers/net/bonding/rte_pmd_bond_version.map
similarity index 100%
rename from drivers/net/bonding/rte_eth_bond_version.map
rename to drivers/net/bonding/rte_pmd_bond_version.map
-- 
2.13.5

Re: [dpdk-dev] [PATCH 1/2] net/bonding: rename map file to standard name

[Ferruh Yigit]

On 9/14/2017 4:57 PM, Bruce Richardson wrote:
> Naming convention for net drivers is "rte_pmd_<name>_version.map"
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

Applied to dpdk-next-net/master, thanks.

[dpdk-dev] [PATCH 2/2] net/xenvirt: rename map file to standard name

[Bruce Richardson]

Naming convention for net drivers is "rte_pmd_<name>_version.map"

Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
---
 drivers/net/xenvirt/Makefile                                            | 2 +-
 .../{rte_eth_xenvirt_version.map => rte_pmd_xenvirt_version.map}        | 0
 2 files changed, 1 insertion(+), 1 deletion(-)
 rename drivers/net/xenvirt/{rte_eth_xenvirt_version.map => rte_pmd_xenvirt_version.map} (100%)

diff --git a/drivers/net/xenvirt/Makefile b/drivers/net/xenvirt/Makefile
index 8b4b8f037..bf5db050f 100644
--- a/drivers/net/xenvirt/Makefile
+++ b/drivers/net/xenvirt/Makefile
@@ -40,7 +40,7 @@ CFLAGS += -O3
 CFLAGS += $(WERROR_FLAGS)
 LDLIBS += -lxenstore
 
-EXPORT_MAP := rte_eth_xenvirt_version.map
+EXPORT_MAP := rte_pmd_xenvirt_version.map
 
 LIBABIVER := 1
 
diff --git a/drivers/net/xenvirt/rte_eth_xenvirt_version.map b/drivers/net/xenvirt/rte_pmd_xenvirt_version.map
similarity index 100%
rename from drivers/net/xenvirt/rte_eth_xenvirt_version.map
rename to drivers/net/xenvirt/rte_pmd_xenvirt_version.map
-- 
2.13.5

Re: [dpdk-dev] [PATCH 2/2] net/xenvirt: rename map file to standard name

[Ferruh Yigit]

On 9/14/2017 4:57 PM, Bruce Richardson wrote:
> Naming convention for net drivers is "rte_pmd_<name>_version.map"
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>

This one not get, because xenvirt will be removed this release.

Re: [dpdk-dev] [PATCH 0/2] have net drivers use a standard map file name format

[Ferruh Yigit]

On 9/14/2017 4:57 PM, Bruce Richardson wrote:
> rename some of the version.map files that don't match the standard naming
> convention. For future meson build system, this will save us having to
> specify for each driver what the filename of its map file is.
> 
> Bruce Richardson (2):
>   net/bonding: rename map file to standard name
>   net/xenvirt: rename map file to standard name

+1 for bonding,
but xenvirt is going away in this release, can we drop that one?

> 
>  drivers/net/bonding/Makefile                                            | 2 +-
>  .../net/bonding/{rte_eth_bond_version.map => rte_pmd_bond_version.map}  | 0
>  drivers/net/xenvirt/Makefile                                            | 2 +-
>  .../{rte_eth_xenvirt_version.map => rte_pmd_xenvirt_version.map}        | 0
>  4 files changed, 2 insertions(+), 2 deletions(-)
>  rename drivers/net/bonding/{rte_eth_bond_version.map => rte_pmd_bond_version.map} (100%)
>  rename drivers/net/xenvirt/{rte_eth_xenvirt_version.map => rte_pmd_xenvirt_version.map} (100%)
> 

Re: [dpdk-dev] [PATCH 0/2] have net drivers use a standard map file name format

[Richardson, Bruce]

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Thursday, September 14, 2017 5:34 PM
> To: Richardson, Bruce <bruce.richardson@intel.com>; Tan, Jianfeng
> <jianfeng.tan@intel.com>; Doherty, Declan <declan.doherty@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [PATCH 0/2] have net drivers use a standard map file name
> format
> 
> On 9/14/2017 4:57 PM, Bruce Richardson wrote:
> > rename some of the version.map files that don't match the standard
> > naming convention. For future meson build system, this will save us
> > having to specify for each driver what the filename of its map file is.
> >
> > Bruce Richardson (2):
> >   net/bonding: rename map file to standard name
> >   net/xenvirt: rename map file to standard name
> 
> +1 for bonding,
> but xenvirt is going away in this release, can we drop that one?
> 
Good point. Please ignore patch 2, and take only patch 1.