DPDK patches and discussions
 help / color / mirror / Atom feed
From: "Gaëtan Rivet" <gaetan.rivet@6wind.com>
To: Thomas Monjalon <thomas@monjalon.net>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] bus/pci: fix namespace of sysfs path function
Date: Mon, 6 Nov 2017 09:43:34 +0100	[thread overview]
Message-ID: <20171106084334.GN10890@bidouze.vm.6wind.com> (raw)
In-Reply-To: <20171106020847.10462-1-thomas@monjalon.net>

There are others namespace inconsistencies with my pci move, I will send
additional fixes later.

On Mon, Nov 06, 2017 at 03:08:47AM +0100, Thomas Monjalon wrote:
> The function pci_get_sysfs_path was moved from EAL to the PCI driver.
> 
> The namespace is now fixed by adding "rte_" prefix.
> The map files are fixed by removing the symbol from EAL and adding
> it to the PCI driver.
> 
> It is an API break but it is probably not used by applications.
> Anyway this API is already broken by the move in a new header file.
> 
> Fixes: c752998b5e2e ("pci: introduce library and driver")
> 
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: Gaetan Rivet <gaetan.rivet@6wind.com>
> ---
>  drivers/bus/pci/linux/pci.c             |  6 +++---
>  drivers/bus/pci/linux/pci_uio.c         | 10 +++++-----
>  drivers/bus/pci/linux/pci_vfio.c        |  6 +++---
>  drivers/bus/pci/pci_common.c            |  2 +-
>  drivers/bus/pci/rte_bus_pci.h           |  2 +-
>  drivers/bus/pci/rte_bus_pci_version.map |  1 +
>  drivers/net/szedata2/rte_eth_szedata2.c |  2 +-
>  lib/librte_eal/rte_eal_version.map      |  1 -
>  8 files changed, 15 insertions(+), 15 deletions(-)
> 
> diff --git a/drivers/bus/pci/linux/pci.c b/drivers/bus/pci/linux/pci.c
> index cdf810693..e5cbc55b4 100644
> --- a/drivers/bus/pci/linux/pci.c
> +++ b/drivers/bus/pci/linux/pci.c
> @@ -395,7 +395,7 @@ pci_update_device(const struct rte_pci_addr *addr)
>  	char filename[PATH_MAX];
>  
>  	snprintf(filename, sizeof(filename), "%s/" PCI_PRI_FMT,
> -		 pci_get_sysfs_path(), addr->domain, addr->bus, addr->devid,
> +		 rte_pci_get_sysfs_path(), addr->domain, addr->bus, addr->devid,
>  		 addr->function);
>  
>  	return pci_scan_one(filename, addr);
> @@ -468,7 +468,7 @@ rte_pci_scan(void)
>  		RTE_LOG(DEBUG, EAL, "VFIO PCI modules not loaded\n");
>  #endif
>  
> -	dir = opendir(pci_get_sysfs_path());
> +	dir = opendir(rte_pci_get_sysfs_path());
>  	if (dir == NULL) {
>  		RTE_LOG(ERR, EAL, "%s(): opendir failed: %s\n",
>  			__func__, strerror(errno));
> @@ -483,7 +483,7 @@ rte_pci_scan(void)
>  			continue;
>  
>  		snprintf(dirname, sizeof(dirname), "%s/%s",
> -				pci_get_sysfs_path(), e->d_name);
> +				rte_pci_get_sysfs_path(), e->d_name);
>  
>  		if (pci_scan_one(dirname, &addr) < 0)
>  			goto error;
> diff --git a/drivers/bus/pci/linux/pci_uio.c b/drivers/bus/pci/linux/pci_uio.c
> index 8cf6218fb..92b7f0275 100644
> --- a/drivers/bus/pci/linux/pci_uio.c
> +++ b/drivers/bus/pci/linux/pci_uio.c
> @@ -164,14 +164,14 @@ pci_get_uio_dev(struct rte_pci_device *dev, char *dstbuf,
>  	 * or uio:uioX */
>  
>  	snprintf(dirname, sizeof(dirname),
> -			"%s/" PCI_PRI_FMT "/uio", pci_get_sysfs_path(),
> +			"%s/" PCI_PRI_FMT "/uio", rte_pci_get_sysfs_path(),
>  			loc->domain, loc->bus, loc->devid, loc->function);
>  
>  	dir = opendir(dirname);
>  	if (dir == NULL) {
>  		/* retry with the parent directory */
>  		snprintf(dirname, sizeof(dirname),
> -				"%s/" PCI_PRI_FMT, pci_get_sysfs_path(),
> +				"%s/" PCI_PRI_FMT, rte_pci_get_sysfs_path(),
>  				loc->domain, loc->bus, loc->devid, loc->function);
>  		dir = opendir(dirname);
>  
> @@ -323,7 +323,7 @@ pci_uio_map_resource_by_index(struct rte_pci_device *dev, int res_idx,
>  	/* update devname for mmap  */
>  	snprintf(devname, sizeof(devname),
>  			"%s/" PCI_PRI_FMT "/resource%d",
> -			pci_get_sysfs_path(),
> +			rte_pci_get_sysfs_path(),
>  			loc->domain, loc->bus, loc->devid,
>  			loc->function, res_idx);
>  
> @@ -432,7 +432,7 @@ pci_uio_ioport_map(struct rte_pci_device *dev, int bar,
>  
>  	/* open and read addresses of the corresponding resource in sysfs */
>  	snprintf(filename, sizeof(filename), "%s/" PCI_PRI_FMT "/resource",
> -		pci_get_sysfs_path(), dev->addr.domain, dev->addr.bus,
> +		rte_pci_get_sysfs_path(), dev->addr.domain, dev->addr.bus,
>  		dev->addr.devid, dev->addr.function);
>  	f = fopen(filename, "r");
>  	if (f == NULL) {
> @@ -454,7 +454,7 @@ pci_uio_ioport_map(struct rte_pci_device *dev, int bar,
>  		goto error;
>  	}
>  	snprintf(filename, sizeof(filename), "%s/" PCI_PRI_FMT "/resource%d",
> -		pci_get_sysfs_path(), dev->addr.domain, dev->addr.bus,
> +		rte_pci_get_sysfs_path(), dev->addr.domain, dev->addr.bus,
>  		dev->addr.devid, dev->addr.function, bar);
>  
>  	/* mmap the pci resource */
> diff --git a/drivers/bus/pci/linux/pci_vfio.c b/drivers/bus/pci/linux/pci_vfio.c
> index 3ca35b6db..4b015b1ae 100644
> --- a/drivers/bus/pci/linux/pci_vfio.c
> +++ b/drivers/bus/pci/linux/pci_vfio.c
> @@ -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 = vfio_setup_device(rte_pci_get_sysfs_path(), pci_addr,
>  					&vfio_dev_fd, &device_info);
>  	if (ret)
>  		return ret;
> @@ -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 = vfio_setup_device(rte_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 = vfio_release_device(rte_pci_get_sysfs_path(), pci_addr,
>  				  dev->intr_handle.vfio_dev_fd);
>  	if (ret < 0) {
>  		RTE_LOG(ERR, EAL,
> diff --git a/drivers/bus/pci/pci_common.c b/drivers/bus/pci/pci_common.c
> index 012420a9c..8b6a8bd88 100644
> --- a/drivers/bus/pci/pci_common.c
> +++ b/drivers/bus/pci/pci_common.c
> @@ -59,7 +59,7 @@ extern struct rte_pci_bus rte_pci_bus;
>  
>  #define SYSFS_PCI_DEVICES "/sys/bus/pci/devices"
>  
> -const char *pci_get_sysfs_path(void)
> +const char *rte_pci_get_sysfs_path(void)
>  {
>  	const char *path = NULL;
>  
> diff --git a/drivers/bus/pci/rte_bus_pci.h b/drivers/bus/pci/rte_bus_pci.h
> index c0b619f29..d4a299641 100644
> --- a/drivers/bus/pci/rte_bus_pci.h
> +++ b/drivers/bus/pci/rte_bus_pci.h
> @@ -60,7 +60,7 @@ extern "C" {
>  #include <rte_pci.h>
>  
>  /** Pathname of PCI devices directory. */
> -const char *pci_get_sysfs_path(void);
> +const char *rte_pci_get_sysfs_path(void);
>  
>  /* Forward declarations */
>  struct rte_pci_device;
> diff --git a/drivers/bus/pci/rte_bus_pci_version.map b/drivers/bus/pci/rte_bus_pci_version.map
> index ee67033b0..27e9c4f10 100644
> --- a/drivers/bus/pci/rte_bus_pci_version.map
> +++ b/drivers/bus/pci/rte_bus_pci_version.map
> @@ -2,6 +2,7 @@ DPDK_17.11 {
>  	global:
>  
>  	rte_pci_dump;
> +	rte_pci_get_sysfs_path;
>  	rte_pci_ioport_map;
>  	rte_pci_ioport_read;
>  	rte_pci_ioport_unmap;
> diff --git a/drivers/net/szedata2/rte_eth_szedata2.c b/drivers/net/szedata2/rte_eth_szedata2.c
> index c5486b72d..403cfdbb3 100644
> --- a/drivers/net/szedata2/rte_eth_szedata2.c
> +++ b/drivers/net/szedata2/rte_eth_szedata2.c
> @@ -1540,7 +1540,7 @@ rte_szedata2_eth_dev_init(struct rte_eth_dev *dev)
>  		return -EINVAL;
>  	}
>  	snprintf(rsc_filename, PATH_MAX,
> -		"%s/" PCI_PRI_FMT "/resource%u", pci_get_sysfs_path(),
> +		"%s/" PCI_PRI_FMT "/resource%u", rte_pci_get_sysfs_path(),
>  		pci_addr->domain, pci_addr->bus,
>  		pci_addr->devid, pci_addr->function, PCI_RESOURCE_NUMBER);
>  	fd = open(rsc_filename, O_RDWR);
> diff --git a/lib/librte_eal/rte_eal_version.map b/lib/librte_eal/rte_eal_version.map
> index b4d7cd93c..885892188 100644
> --- a/lib/librte_eal/rte_eal_version.map
> +++ b/lib/librte_eal/rte_eal_version.map
> @@ -124,7 +124,6 @@ DPDK_16.04 {
>  DPDK_16.07 {
>  	global:
>  
> -	pci_get_sysfs_path;
>  	rte_keepalive_mark_sleep;
>  	rte_keepalive_register_relay_callback;
>  	rte_rtm_supported;
> -- 
> 2.14.2
> 

-- 
Gaëtan Rivet
6WIND

  reply	other threads:[~2017-11-06  8:43 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-06  2:08 Thomas Monjalon
2017-11-06  8:43 ` Gaëtan Rivet [this message]
2017-11-06 23:44   ` Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171106084334.GN10890@bidouze.vm.6wind.com \
    --to=gaetan.rivet@6wind.com \
    --cc=dev@dpdk.org \
    --cc=thomas@monjalon.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).