Fix namespaces for VFIO and PCI
cover Fix namespaces for VFIO and PCI
Some symbols are lacking the proper rte_ prefix.
Gaetan Rivet (2):
eal: use proper namespace for VFIO symbols
pci: fix PCI lib prefix
drivers/bus/pci/bsd/pci.c | 2 +- drivers/bus/pci/linux/pci.c | 4 ++-- drivers/bus/pci/linux/pci_vfio.c | 14 +++++++------- drivers/bus/pci/pci_common.c | 8 ++++---- drivers/bus/pci/pci_common_uio.c | 4 ++-- lib/librte_eal/bsdapp/eal/eal.c | 20 ++++++++++---------- lib/librte_eal/common/include/rte_vfio.h | 10 +++++----- lib/librte_eal/linuxapp/eal/eal.c | 4 ++-- lib/librte_eal/linuxapp/eal/eal_vfio.c | 10 +++++----- lib/librte_eal/rte_eal_version.map | 10 +++++----- lib/librte_pci/rte_pci.c | 13 +++---------- lib/librte_pci/rte_pci.h | 22 +++------------------- lib/librte_pci/rte_pci_version.map | 6 +++--- 13 files changed, 52 insertions(+), 75 deletions(-)
Signature · 1 line
2.1.4
PATCH 1/2 eal: use proper namespace for VFIO symbols
Exposed VFIO functions simply uses a "vfio" prefix. Use the proper "rte_vfio" prefix for those symbols.
Fixes 279b581c897d ("vfio: expose functions")
Signed-off-by Gaetan Rivet <[email protected]>
--- drivers/bus/pci/linux/pci.c | 2 +- drivers/bus/pci/linux/pci_vfio.c | 12 ++++++------ lib/librte_eal/bsdapp/eal/eal.c | 20 ++++++++++---------- lib/librte_eal/common/include/rte_vfio.h | 10 +++++----- lib/librte_eal/linuxapp/eal/eal.c | 4 ++-- lib/librte_eal/linuxapp/eal/eal_vfio.c | 10 +++++----- lib/librte_eal/rte_eal_version.map | 10 +++++----- 7 files changed, 34 insertions(+), 34 deletions(-)
drivers/bus/pci/linux/pci.c +1 -1
@@ -594,7 +594,7 @@ rte_pci_get_iommu_class(void) has_iova_va =pci_one_device_has_iova_va (); is_bound_uio =pci_one_device_bound_uio ();#ifdef VFIO_PRESENT - is_vfio_noiommu_enabled =vfio_noiommu_is_enabled () == true ? + is_vfio_noiommu_enabled =rte_vfio_noiommu_is_enabled () == true ? true : false;#endif
drivers/bus/pci/linux/pci_vfio.c +6 -6
@@ -325,7 +325,7 @@ pci_vfio_is_ioport_bar(int vfio_dev_fd, int bar_index) }static int -pci_vfio_setup_device (struct rte_pci_device *dev,int vfio_dev_fd) +pci_rte_vfio_setup_device (struct rte_pci_device *dev,int vfio_dev_fd) {if (pci_vfio_setup_interrupts (dev, vfio_dev_fd) !=0 ) {RTE_LOG (ERR ,EAL ,"Error setting up interrupts!\n" );
@@ -465,7 +465,7 @@ pci_vfio_map_resource_primary(struct rte_pci_device *dev)snprintf (pci_addr,sizeof (pci_addr),PCI_PRI_FMT , loc->domain, loc->bus, loc->devid, loc->function); - ret =vfio_setup_device (pci_get_sysfs_path (), pci_addr, + ret =rte_vfio_setup_device (pci_get_sysfs_path (), pci_addr, &vfio_dev_fd, &device_info);if (ret)return ret;
@@ -546,7 +546,7 @@ pci_vfio_map_resource_primary(struct rte_pci_device *dev) dev->mem_resource[i].addr = maps[i].addr; } -if (pci_vfio_setup_device (dev, vfio_dev_fd) <0 ) { +if (pci_rte_vfio_setup_device (dev, vfio_dev_fd) <0 ) {RTE_LOG (ERR ,EAL ," %s setup device failed\n" , pci_addr); goto err_vfio_res; }
@@ -582,7 +582,7 @@ pci_vfio_map_resource_secondary(struct rte_pci_device *dev)snprintf (pci_addr,sizeof (pci_addr),PCI_PRI_FMT , loc->domain, loc->bus, loc->devid, loc->function); - ret =vfio_setup_device (pci_get_sysfs_path (), pci_addr, + ret =rte_vfio_setup_device (pci_get_sysfs_path (), pci_addr, &vfio_dev_fd, &device_info);if (ret)return ret;
@@ -662,7 +662,7 @@ pci_vfio_unmap_resource(struct rte_pci_device *dev)return -1 ; } - ret =vfio_release_device (pci_get_sysfs_path (), pci_addr, + ret =rte_vfio_release_device (pci_get_sysfs_path (), pci_addr, dev->intr_handle.vfio_dev_fd);if (ret <0 ) { RTE_LOG(ERR, EAL,
@@ -758,6 +758,6 @@ pci_vfio_ioport_unmap(struct rte_pci_ioport *p)int pci_vfio_is_enabled (void ) { -return vfio_is_enabled ("vfio_pci" ); +return rte_vfio_is_enabled ("vfio_pci" ); }#endif
lib/librte_eal/bsdapp/eal/eal.c +10 -10
@@ -743,14 +743,14 @@ rte_eal_vfio_intr_mode(void)struct vfio_device_info ;/* dummy prototypes. */ -int vfio_setup_device (const char *sysfs_base,const char *dev_addr, +int rte_vfio_setup_device (const char *sysfs_base,const char *dev_addr,int *vfio_dev_fd,struct vfio_device_info *device_info); -int vfio_release_device (const char *sysfs_base,const char *dev_addr,int fd); -int vfio_enable (const char *modname); -int vfio_is_enabled (const char *modname); -int vfio_noiommu_is_enabled (void ); +int rte_vfio_release_device (const char *sysfs_base,const char *dev_addr,int fd); +int rte_vfio_enable (const char *modname); +int rte_vfio_is_enabled (const char *modname); +int rte_vfio_noiommu_is_enabled (void ); -int vfio_setup_device(__rte_unused const char *sysfs_base, +int rte_vfio_setup_device(__rte_unused const char *sysfs_base, __rte_unused const char *dev_addr, __rte_unused int *vfio_dev_fd, __rte_unused struct vfio_device_info *device_info)
@@ -758,24 +758,24 @@ int vfio_setup_device(__rte_unused const char *sysfs_base,return -1 ; } -int vfio_release_device(__rte_unused const char *sysfs_base, +int rte_vfio_release_device(__rte_unused const char *sysfs_base, __rte_unused const char *dev_addr, __rte_unused int fd) {return -1 ; } -int vfio_enable(__rte_unused const char *modname) +int rte_vfio_enable(__rte_unused const char *modname) {return -1 ; } -int vfio_is_enabled(__rte_unused const char *modname) +int rte_vfio_is_enabled(__rte_unused const char *modname) {return 0 ; } -int vfio_noiommu_is_enabled (void ) +int rte_vfio_noiommu_is_enabled (void ) {return 0 ; }
lib/librte_eal/common/include/rte_vfio.h +5 -5
@@ -82,7 +82,7 @@* <0 on failure. * >1 if the device cannot be managed this way. */ -int vfio_setup_device (const char *sysfs_base,const char *dev_addr, +int rte_vfio_setup_device (const char *sysfs_base,const char *dev_addr,int *vfio_dev_fd,struct vfio_device_info *device_info);/**
@@ -104,7 +104,7 @@ int vfio_setup_device(const char *sysfs_base, const char *dev_addr,* 0 on success. * <0 on failure. */ -int vfio_release_device (const char *sysfs_base,const char *dev_addr,int fd); +int rte_vfio_release_device (const char *sysfs_base,const char *dev_addr,int fd);/** * Enable a VFIO-related kmod.
@@ -119,7 +119,7 @@ int vfio_release_device(const char *sysfs_base, const char *dev_addr, int fd);* 0 on success. * <0 on failure. */ -int vfio_enable (const char *modname); +int rte_vfio_enable (const char *modname);/** * Check whether a VFIO-related kmod is enabled.
@@ -134,7 +134,7 @@ int vfio_enable(const char *modname);* !0 if true. * 0 otherwise. */ -int vfio_is_enabled (const char *modname); +int rte_vfio_is_enabled (const char *modname);/** * Whether VFIO NOIOMMU mode is enabled.
@@ -146,7 +146,7 @@ int vfio_is_enabled(const char *modname);* !0 if true. * 0 otherwise. */ -int vfio_noiommu_is_enabled (void ); +int rte_vfio_noiommu_is_enabled (void );#endif /* VFIO_PRESENT */
lib/librte_eal/linuxapp/eal/eal.c +2 -2
@@ -710,9 +710,9 @@ static int rte_eal_vfio_setup(void) {int vfio_enabled =0 ; -if (vfio_enable ("vfio" )) +if (rte_vfio_enable ("vfio" ))return -1 ; - vfio_enabled =vfio_is_enabled ("vfio" ); + vfio_enabled =rte_vfio_is_enabled ("vfio" );if (vfio_enabled) {
lib/librte_eal/linuxapp/eal/eal_vfio.c +5 -5
@@ -284,7 +284,7 @@ clear_group(int vfio_group_fd) }int -vfio_setup_device (const char *sysfs_base,const char *dev_addr, +rte_vfio_setup_device (const char *sysfs_base,const char *dev_addr,int *vfio_dev_fd,struct vfio_device_info *device_info) {struct vfio_group_status group_status = {
@@ -416,7 +416,7 @@ vfio_setup_device(const char *sysfs_base, const char *dev_addr, }int -vfio_release_device (const char *sysfs_base,const char *dev_addr, +rte_vfio_release_device (const char *sysfs_base,const char *dev_addr,int vfio_dev_fd) {struct vfio_group_status group_status = {
@@ -478,7 +478,7 @@ vfio_release_device(const char *sysfs_base, const char *dev_addr, }int -vfio_enable (const char *modname) +rte_vfio_enable (const char *modname) {/* initialize group list */ int i;
@@ -523,7 +523,7 @@ vfio_enable(const char *modname) }int -vfio_is_enabled (const char *modname) +rte_vfio_is_enabled (const char *modname) {const int mod_available =rte_eal_check_module (modname);return vfio_cfg.vfio_enabled && mod_available;
@@ -841,7 +841,7 @@ vfio_noiommu_dma_map(int __rte_unused vfio_container_fd) }int -vfio_noiommu_is_enabled (void ) +rte_vfio_noiommu_is_enabled (void ) { int fd, ret, cnt __rte_unused;char c;
lib/librte_eal/rte_eal_version.map +5 -5
@@ -235,10 +235,10 @@ DPDK_17.11 { rte_eal_vfio_intr_mode; rte_lcore_has_role; rte_memcpy_ptr; - vfio_enable; - vfio_is_enabled; - vfio_noiommu_is_enabled; - vfio_release_device; - vfio_setup_device; + rte_vfio_enable; + rte_vfio_is_enabled; + rte_vfio_noiommu_is_enabled; + rte_vfio_release_device; + rte_vfio_setup_device; } DPDK_17.08;
Signature · 1 line
2.1.4
PATCH 2/2 pci: fix PCI lib prefix
In reply to Gaetan RivetSome symbols were introduced with the wrong prefix. Add the usual "rte_" prefix when needed.
Fixes c752998b5e2e ("pci: introduce library and driver")
Signed-off-by Gaetan Rivet <[email protected]>
--- drivers/bus/pci/bsd/pci.c | 2 +- drivers/bus/pci/linux/pci.c | 2 +- drivers/bus/pci/linux/pci_vfio.c | 2 +- drivers/bus/pci/pci_common.c | 8 ++++---- drivers/bus/pci/pci_common_uio.c | 4 ++-- lib/librte_pci/rte_pci.c | 13 +++---------- lib/librte_pci/rte_pci.h | 22 +++------------------- lib/librte_pci/rte_pci_version.map | 6 +++--- 8 files changed, 18 insertions(+), 41 deletions(-)
drivers/bus/pci/bsd/pci.c +1 -1
@@ -324,7 +324,7 @@ pci_scan_one(int dev_pci_fd, struct pci_conf *conf)int ret; TAILQ_FOREACH(dev2, &rte_pci_bus.device_list, next) { - ret =pci_addr_cmp (&dev->addr, &dev2->addr); + ret =rte_pci_addr_cmp (&dev->addr, &dev2->addr);if (ret >0 )continue ;else if (ret <0 ) {
drivers/bus/pci/linux/pci.c +1 -1
@@ -366,7 +366,7 @@ pci_scan_one(const char *dirname, const struct rte_pci_addr *addr)int ret; TAILQ_FOREACH(dev2, &rte_pci_bus.device_list, next) { - ret =pci_addr_cmp (&dev->addr, &dev2->addr); + ret =rte_pci_addr_cmp (&dev->addr, &dev2->addr);if (ret >0 )continue ;
drivers/bus/pci/linux/pci_vfio.c +1 -1
@@ -589,7 +589,7 @@ pci_vfio_map_resource_secondary(struct rte_pci_device *dev)/* if we're in a secondary process, just find our tailq entry */ TAILQ_FOREACH(vfio_res, vfio_res_list, next) { -if (pci_addr_cmp (&vfio_res->pci_addr, +if (rte_pci_addr_cmp (&vfio_res->pci_addr, &dev->addr))continue ;break ;
drivers/bus/pci/pci_common.c +4 -4
@@ -82,7 +82,7 @@ static struct rte_devargs *pci_devargs_lookup(struct rte_pci_device *dev)if (devargs->bus != pbus)continue ; devargs->bus->parse (devargs->name, &addr); -if (!pci_addr_cmp (&dev->addr, &addr)) +if (!rte_pci_addr_cmp (&dev->addr, &addr))return devargs; }return NULL ;
@@ -309,7 +309,7 @@ rte_pci_probe_one(const struct rte_pci_addr *addr) goto err_return;FOREACH_DEVICE_ON_PCIBUS (dev) { -if (pci_addr_cmp (&dev->addr, addr)) +if (rte_pci_addr_cmp (&dev->addr, addr))continue ; ret =pci_probe_all_drivers (dev);
@@ -339,7 +339,7 @@ rte_pci_detach(const struct rte_pci_addr *addr)return -1 ;FOREACH_DEVICE_ON_PCIBUS (dev) { -if (pci_addr_cmp (&dev->addr, addr)) +if (rte_pci_addr_cmp (&dev->addr, addr))continue ; ret =rte_pci_detach_dev (dev);
@@ -441,7 +441,7 @@ pci_parse(const char *name, void *addr)struct rte_pci_addr pci_addr;bool parse; - parse = (pci_addr_parse (name, &pci_addr) ==0 ); + parse = (rte_pci_addr_parse (name, &pci_addr) ==0 );if (parse && addr !=NULL ) *out = pci_addr;return parse == false;
drivers/bus/pci/pci_common_uio.c +2 -2
@@ -63,7 +63,7 @@ pci_uio_map_secondary(struct rte_pci_device *dev) TAILQ_FOREACH(uio_res, uio_res_list, next) {/* skip this element if it doesn't match our PCI address */ -if (pci_addr_cmp (&uio_res->pci_addr, &dev->addr)) +if (rte_pci_addr_cmp (&uio_res->pci_addr, &dev->addr))continue ;for (i =0 ; i != uio_res->nb_maps; i++) {
@@ -189,7 +189,7 @@ pci_uio_find_resource(struct rte_pci_device *dev) TAILQ_FOREACH(uio_res, uio_res_list, next) {/* skip this element if it doesn't match our PCI address */ -if (!pci_addr_cmp (&uio_res->pci_addr, &dev->addr)) +if (!rte_pci_addr_cmp (&uio_res->pci_addr, &dev->addr))return uio_res; }return NULL ;
lib/librte_pci/rte_pci.c +3 -10
@@ -127,13 +127,6 @@ eal_parse_pci_DomBDF(const char *input, struct rte_pci_addr *dev_addr)void rte_pci_device_name (const struct rte_pci_addr *addr, -char *output,size_t size) -{ -pci_device_name (addr, output, size); -} - -void -pci_device_name (const struct rte_pci_addr *addr,char *output,size_t size) {RTE_VERIFY (size >=PCI_PRI_STR_SIZE );
@@ -146,11 +139,11 @@ int rte_eal_compare_pci_addr(const struct rte_pci_addr *addr, const struct rte_pci_addr *addr2) { - return pci_addr_cmp(addr, addr2); + return rte_pci_addr_cmp(addr, addr2); }int -pci_addr_cmp (const struct rte_pci_addr *addr, +rte_pci_addr_cmp (const struct rte_pci_addr *addr,const struct rte_pci_addr *addr2) {uint64_t dev_addr, dev_addr2;
@@ -172,7 +165,7 @@ pci_addr_cmp(const struct rte_pci_addr *addr, }int -pci_addr_parse (const char *str,struct rte_pci_addr *addr) +rte_pci_addr_parse (const char *str,struct rte_pci_addr *addr) {if (pci_bdf_parse (str, addr) ==0 ||pci_dbdf_parse (str, addr) ==0 )
lib/librte_pci/rte_pci.h +3 -19
@@ -166,7 +166,6 @@ int eal_parse_pci_BDF(const char *input, struct rte_pci_addr *dev_addr);int eal_parse_pci_DomBDF (const char *input,struct rte_pci_addr *dev_addr);/** -* @deprecated * Utility function to write a pci device name, this device name can later be * used to retrieve the corresponding rte_pci_addr using eal_parse_pci_* * BDF helpers.
@@ -179,21 +178,6 @@ int eal_parse_pci_DomBDF(const char *input, struct rte_pci_addr *dev_addr);* The output buffer size */ void rte_pci_device_name (const struct rte_pci_addr *addr, -char *output,size_t size); - -/** -* Utility function to write a pci device name, this device name can later be -* used to retrieve the corresponding rte_pci_addr using eal_parse_pci_* -* BDF helpers. -* -* @param addr -* The PCI Bus-Device-Function address -* @param output -* The output buffer string -* @param size -* The output buffer size -*/ -void pci_device_name (const struct rte_pci_addr *addr,char *output,size_t size);/**
@@ -224,8 +208,8 @@ int rte_eal_compare_pci_addr(const struct rte_pci_addr *addr,* Positive on addr is greater than addr2. * Negative on addr is less than addr2, or error. */ -int pci_addr_cmp (const struct rte_pci_addr *addr, -const struct rte_pci_addr *addr2); +int rte_pci_addr_cmp (const struct rte_pci_addr *addr, +const struct rte_pci_addr *addr2);/**
@@ -240,7 +224,7 @@ int pci_addr_cmp(const struct rte_pci_addr *addr,* 0 on success * <0 otherwise */ -int pci_addr_parse (const char *str,struct rte_pci_addr *addr); +int rte_pci_addr_parse (const char *str,struct rte_pci_addr *addr);/** * Map a particular resource from a file.
lib/librte_pci/rte_pci_version.map +3 -3
@@ -3,9 +3,9 @@ DPDK_17.11 { eal_parse_pci_BDF; eal_parse_pci_DomBDF; - pci_addr_cmp; - pci_addr_parse; - pci_device_name; + rte_pci_addr_cmp; + rte_pci_addr_parse; + rte_pci_device_name; pci_map_resource; pci_unmap_resource; rte_eal_compare_pci_addr;
Signature · 1 line
2.1.4
Thomas Monjalon
Applied06/11/2017 17:08, Gaetan Rivet:
Quoted 5 lines
Some symbols are lacking the proper
rte_prefix.Gaetan Rivet (2):
eal: use proper namespace for VFIO symbols
pci: fix PCI lib prefix
Applied, thanks