[dpdk-dev] [PATCH v3 0/9] Decouple ethdev from PCI device
Jan Blunck
jblunck at infradead.org
Wed Dec 21 16:12:11 CET 2016
On Wed, Dec 21, 2016 at 11:00 AM, Shreyansh Jain <shreyansh.jain at nxp.com> wrote:
> Hi Jan,
>
>
> On Tuesday 20 December 2016 04:41 PM, Jan Blunck wrote:
>>
>> This is a repost of the series I sent in November. I've addressed
>> Shreyansh's
>> review comments about the PCI device users I've missed.
>>
>> Jan Blunck (9):
>> eal: define container_of macro
>> ethdev: Helper to convert to struct rte_pci_device
>> drivers: Use ETH_DEV_PCI_DEV() helper
>> virtio: Don't fill dev_info->driver_name
>> virtio: Add vtpci_intr_handle() helper to get rte_intr_handle
>> virtio: Don't depend on struct rte_eth_dev's pci_dev
>> ethdev: Move filling of rte_eth_dev_info->pci_dev to dev_infos_get()
>> ethdev: Decouple interrupt handling from PCI device
>> ethdev: Decouple struct rte_eth_dev from struct rte_pci_device
>>
>> app/test/virtual_pmd.c | 4 +-
>> drivers/net/bnx2x/bnx2x_ethdev.c | 13 +++---
>> drivers/net/bnxt/bnxt_ethdev.c | 21 ++++++----
>> drivers/net/bnxt/bnxt_ring.c | 11 ++---
>> drivers/net/bonding/rte_eth_bond_args.c | 12 +++++-
>> drivers/net/cxgbe/cxgbe_ethdev.c | 4 +-
>> drivers/net/cxgbe/cxgbe_main.c | 4 +-
>> drivers/net/e1000/em_ethdev.c | 21 ++++++----
>> drivers/net/e1000/igb_ethdev.c | 52 +++++++++++++----------
>> drivers/net/e1000/igb_pf.c | 3 +-
>> drivers/net/ena/ena_ethdev.c | 4 +-
>> drivers/net/enic/enic_ethdev.c | 3 +-
>> drivers/net/fm10k/fm10k_ethdev.c | 56 +++++++++++++------------
>> drivers/net/i40e/i40e_ethdev.c | 45 ++++++++++----------
>> drivers/net/i40e/i40e_ethdev.h | 4 ++
>> drivers/net/i40e/i40e_ethdev_vf.c | 39 ++++++++---------
>> drivers/net/ixgbe/ixgbe_ethdev.c | 67
>> ++++++++++++++++++------------
>> drivers/net/ixgbe/ixgbe_pf.c | 2 +-
>> drivers/net/mlx4/mlx4.c | 2 +
>> drivers/net/mlx5/mlx5_ethdev.c | 2 +
>> drivers/net/nfp/nfp_net.c | 17 ++++----
>> drivers/net/qede/qede_ethdev.c | 18 ++++----
>> drivers/net/szedata2/rte_eth_szedata2.c | 28 +++++++------
>> drivers/net/szedata2/rte_eth_szedata2.h | 34 +++++++--------
>> drivers/net/thunderx/nicvf_ethdev.c | 7 +++-
>> drivers/net/virtio/virtio_ethdev.c | 38 ++++++++---------
>> drivers/net/virtio/virtio_pci.h | 6 +++
>> drivers/net/virtio/virtio_user_ethdev.c | 1 -
>> drivers/net/vmxnet3/vmxnet3_ethdev.c | 6 ++-
>> lib/librte_eal/common/include/rte_common.h | 20 +++++++++
>> lib/librte_eal/common/include/rte_pci.h | 6 +++
>> lib/librte_ether/rte_ethdev.c | 25 ++++++++---
>> lib/librte_ether/rte_ethdev.h | 9 +++-
>> 33 files changed, 355 insertions(+), 229 deletions(-)
>>
>
> Besides the MLX* and bnx2x compile errors (responses in individual patch
> email), all other changes look fine to me.
>
> Either you can directly add my series ACK to v4 or, for ML record purpose, I
> will do that once you post v4 - whichever way you prefer.
>
Shreyansh,
Thanks for the review. I've added acks for the patches that you have
review. I've broken out the changes a little bit better like Stephen
did in his series. For these patches I haven't added your acks yet.
Cheers,
Jan
More information about the dev
mailing list