DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] remove rte_pci_dev_ids.h
@ 2016-07-08 14:31 Thomas Monjalon
  0 siblings, 0 replies; only message in thread
From: Thomas Monjalon @ 2016-07-08 14:31 UTC (permalink / raw)
  To: dev, Stephen Hurd, Sony Chacko, Harish Patil, Rasesh Mody,
	Wenzhuo Lu, John Daley, Nelson Escobar, Jing Chen, Helin Zhang,
	Jingjing Wu, Huawei Xie, Yuanhan Liu, Yong Wang
  Cc: ferruh.yigit

Some drivers define the supported PCI IDs, others define them
in the common file rte_pci_dev_ids.h.
This situation brings some confusion in the layering design.
In order to clean up things, I've suggested a plan to remove
rte_pci_dev_ids.h:
	http://dpdk.org/ml/archives/dev/2016-July/043596.html

I bring it here in a new thread to give more visibility.

There were 4 usages of this file:
- PMDs
- KNI ethtool (igb/ixgbe only)
- testpmd to use ixgbe bypass
- PCI autotest

This is the plan to get rid of them:
1/ remove need in PCI autotest (done: http://dpdk.org/commit/1dbba165)
2/ move PCI ids in bnx2x, bnxt, e1000, enic, fm10k, i40e, virtio, vmxnet3
3/ remove KNI ethtool (only igb/ixgbe support)
4/ remove bypass API or move it to ixgbe specific API
5/ move remaining PCI ids in igb and ixgbe PMDs

To make it short, please driver maintainers, move your PCI ids in your
drivers as soon as possible, in order to achieve step 2.
Thanks

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2016-07-08 14:31 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-08 14:31 [dpdk-dev] remove rte_pci_dev_ids.h Thomas Monjalon

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).