DPDK-dev Archive on lore.kernel.org
 help / color / Atom feed
* [dpdk-dev] [PATCH] kni: remove PCI related information
@ 2019-06-06 13:26 Igor Ryzhov
  2019-06-06 14:16 ` Ferruh Yigit
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Igor Ryzhov @ 2019-06-06 13:26 UTC (permalink / raw)
  To: dev

As there is no ethtool support in KNI anymore,
PCI related information is no longer needed.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
---
 kernel/linux/kni/kni_dev.h                        | 3 ---
 kernel/linux/kni/kni_misc.c                       | 6 ------
 lib/librte_eal/linux/eal/include/rte_kni_common.h | 7 -------
 lib/librte_kni/rte_kni.c                          | 4 ----
 4 files changed, 20 deletions(-)

diff --git a/kernel/linux/kni/kni_dev.h b/kernel/linux/kni/kni_dev.h
index 44a5a61f7..d57bce647 100644
--- a/kernel/linux/kni/kni_dev.h
+++ b/kernel/linux/kni/kni_dev.h
@@ -50,9 +50,6 @@ struct kni_dev {
 	wait_queue_head_t wq;
 	struct mutex sync_lock;
 
-	/* PCI device id */
-	uint16_t device_id;
-
 	/* kni device */
 	struct net_device *net_dev;
 
diff --git a/kernel/linux/kni/kni_misc.c b/kernel/linux/kni/kni_misc.c
index af18c67c4..1fc5eeb9c 100644
--- a/kernel/linux/kni/kni_misc.c
+++ b/kernel/linux/kni/kni_misc.c
@@ -377,12 +377,6 @@ kni_ioctl_create(struct net *net, uint32_t ioctl_num,
 		(unsigned long long) dev_info.resp_phys, kni->resp_q);
 	pr_debug("mbuf_size:    %u\n", kni->mbuf_size);
 
-	pr_debug("PCI: %02x:%02x.%02x %04x:%04x\n",
-					dev_info.bus,
-					dev_info.devid,
-					dev_info.function,
-					dev_info.vendor_id,
-					dev_info.device_id);
 	/* if user has provided a valid mac address */
 	if (is_valid_ether_addr(dev_info.mac_addr))
 		memcpy(net_dev->dev_addr, dev_info.mac_addr, ETH_ALEN);
diff --git a/lib/librte_eal/linux/eal/include/rte_kni_common.h b/lib/librte_eal/linux/eal/include/rte_kni_common.h
index 5db5a1333..91a1c1408 100644
--- a/lib/librte_eal/linux/eal/include/rte_kni_common.h
+++ b/lib/librte_eal/linux/eal/include/rte_kni_common.h
@@ -111,13 +111,6 @@ struct rte_kni_device_info {
 	void * mbuf_va;
 	phys_addr_t mbuf_phys;
 
-	/* PCI info */
-	uint16_t vendor_id;           /**< Vendor ID or PCI_ANY_ID. */
-	uint16_t device_id;           /**< Device ID or PCI_ANY_ID. */
-	uint8_t bus;                  /**< Device bus */
-	uint8_t devid;                /**< Device ID */
-	uint8_t function;             /**< Device function. */
-
 	uint16_t group_id;            /**< Group ID */
 	uint32_t core_id;             /**< core ID to bind for kernel thread */
 
diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c
index a0f1e3767..e29d0cc7d 100644
--- a/lib/librte_kni/rte_kni.c
+++ b/lib/librte_kni/rte_kni.c
@@ -252,10 +252,6 @@ rte_kni_alloc(struct rte_mempool *pktmbuf_pool,
 
 	strlcpy(dev_info.name, conf->name, RTE_KNI_NAMESIZE);
 
-	RTE_LOG(INFO, KNI, "pci: %02x:%02x:%02x \t %02x:%02x\n",
-		dev_info.bus, dev_info.devid, dev_info.function,
-			dev_info.vendor_id, dev_info.device_id);
-
 	ret = kni_reserve_mz(kni);
 	if (ret < 0)
 		goto mz_fail;
-- 
2.21.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] kni: remove PCI related information
  2019-06-06 13:26 [dpdk-dev] [PATCH] kni: remove PCI related information Igor Ryzhov
@ 2019-06-06 14:16 ` Ferruh Yigit
  2019-06-17 11:24   ` Thomas Monjalon
  2019-06-18 16:04 ` Stephen Hemminger
  2019-06-18 16:06 ` Stephen Hemminger
  2 siblings, 1 reply; 5+ messages in thread
From: Ferruh Yigit @ 2019-06-06 14:16 UTC (permalink / raw)
  To: Igor Ryzhov, dev

On 6/6/2019 2:26 PM, Igor Ryzhov wrote:
> As there is no ethtool support in KNI anymore,
> PCI related information is no longer needed.
> 
> Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Fixes: ea6b39b5b847 ("kni: remove ethtool support")

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] kni: remove PCI related information
  2019-06-06 14:16 ` Ferruh Yigit
@ 2019-06-17 11:24   ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2019-06-17 11:24 UTC (permalink / raw)
  To: Igor Ryzhov; +Cc: dev, Ferruh Yigit

06/06/2019 17:16, Ferruh Yigit:
> On 6/6/2019 2:26 PM, Igor Ryzhov wrote:
> > As there is no ethtool support in KNI anymore,
> > PCI related information is no longer needed.
> > 
> > Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
> 
> Fixes: ea6b39b5b847 ("kni: remove ethtool support")
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied, thanks




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] kni: remove PCI related information
  2019-06-06 13:26 [dpdk-dev] [PATCH] kni: remove PCI related information Igor Ryzhov
  2019-06-06 14:16 ` Ferruh Yigit
@ 2019-06-18 16:04 ` Stephen Hemminger
  2019-06-18 16:06 ` Stephen Hemminger
  2 siblings, 0 replies; 5+ messages in thread
From: Stephen Hemminger @ 2019-06-18 16:04 UTC (permalink / raw)
  To: Igor Ryzhov; +Cc: dev

On Thu,  6 Jun 2019 16:26:26 +0300
Igor Ryzhov <iryzhov@nfware.com> wrote:

> As there is no ethtool support in KNI anymore,
> PCI related information is no longer needed.
> 
> Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Several other fields are unused, and are removed by a patch I sent
a week ago.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [dpdk-dev] [PATCH] kni: remove PCI related information
  2019-06-06 13:26 [dpdk-dev] [PATCH] kni: remove PCI related information Igor Ryzhov
  2019-06-06 14:16 ` Ferruh Yigit
  2019-06-18 16:04 ` Stephen Hemminger
@ 2019-06-18 16:06 ` Stephen Hemminger
  2 siblings, 0 replies; 5+ messages in thread
From: Stephen Hemminger @ 2019-06-18 16:06 UTC (permalink / raw)
  To: Igor Ryzhov; +Cc: dev

On Thu,  6 Jun 2019 16:26:26 +0300
Igor Ryzhov <iryzhov@nfware.com> wrote:

> As there is no ethtool support in KNI anymore,
> PCI related information is no longer needed.
> 
> Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

See http://patchwork.dpdk.org/patch/54627/

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, back to index

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-06 13:26 [dpdk-dev] [PATCH] kni: remove PCI related information Igor Ryzhov
2019-06-06 14:16 ` Ferruh Yigit
2019-06-17 11:24   ` Thomas Monjalon
2019-06-18 16:04 ` Stephen Hemminger
2019-06-18 16:06 ` Stephen Hemminger

DPDK-dev Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dpdk-dev/0 dpdk-dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dpdk-dev dpdk-dev/ https://lore.kernel.org/dpdk-dev \
		dev@dpdk.org dpdk-dev@archiver.kernel.org
	public-inbox-index dpdk-dev


Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/ public-inbox