Skip to content

RFC: implement VF reset for i40e, e1000 and ixgbe

[email protected] dpdk-dev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3net/i40e: implement VF resetdrivers/net/i40e/i40e_ethdev.h drivers/net/i40e/i40e_ethdev_vf.c drivers/net/i40e/i40e_rxtx.c +1 more+70 -40 replies
2/3net/e1000: implement VF resetdrivers/net/e1000/igb_ethdev.c+59 -00 replies
3/3net/ixgbe: implement VF resetdrivers/net/ixgbe/ixgbe_ethdev.c drivers/net/ixgbe/ixgbe_ethdev.h drivers/net/ixgbe/ixgbe_rxtx.c+63 -110 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/3] RFC: implement VF reset for i40e, e1000 and ixgbe

From: Luca Boccassi <[email protected]>

These patches were originally sent by Wenzhuo Lu:

http://dpdk.org/dev/patchwork/patch/14009/
http://dpdk.org/dev/patchwork/patch/14010/
http://dpdk.org/dev/patchwork/patch/14011/

The current rte_eth_dev_reset API does not correctly reset the VF when the PF flaps on the host. With these patches, at least the ixgbe driver with a X540 card (Linux kernel 4.9 on the host) appears to work correctly.
The test is as simple ip link down/up on the host, and then check that traffic still flows through the VF in the guest.

I am not an expert in these PMDs hence the RFC mark - I would like to ask for feedback and help from the PMD maintainers and developers.

Thanks!

Luca Boccassi (3):
net/i40e: implement VF reset
net/e1000: implement VF reset
net/ixgbe: implement VF reset

 drivers/net/e1000/igb_ethdev.c    | 59 ++++++++++++++++++++++++++++++++++++++
 drivers/net/i40e/i40e_ethdev.h    |  3 ++
 drivers/net/i40e/i40e_ethdev_vf.c | 56 +++++++++++++++++++++++++++++++++---
 drivers/net/i40e/i40e_rxtx.c      | 11 +++++++
 drivers/net/i40e/i40e_rxtx.h      |  4 +++
 drivers/net/ixgbe/ixgbe_ethdev.c  | 60 ++++++++++++++++++++++++++++++++++-----
 drivers/net/ixgbe/ixgbe_ethdev.h  |  2 +-
 drivers/net/ixgbe/ixgbe_rxtx.c    | 12 ++++++--
 8 files changed, 192 insertions(+), 15 deletions(-)
Signature · 1 line
2.11.0