mbox series

[0/3] net/i40e: remove i40evf

Message ID 20210830031215.557238-1-robinx.zhang@intel.com (mailing list archive)
Headers
Series net/i40e: remove i40evf |

Message

Robin Zhang Aug. 30, 2021, 3:12 a.m. UTC 
  In DPDK 21.05, iavf already became the default VF for i40e devices.
So remove i40evf due to it's no need to maintain now.

Robin Zhang (3):
  net/i40e: remove i40evf
  net/iavf: remove i40evf devargs option
  doc: remove old deprecation notice for i40e

 doc/guides/rel_notes/deprecation.rst |    8 -
 drivers/net/i40e/i40e_ethdev.h       |    1 -
 drivers/net/i40e/i40e_ethdev_vf.c    | 3015 --------------------------
 drivers/net/i40e/meson.build         |    1 -
 drivers/net/i40e/rte_pmd_i40e.c      |    9 +-
 drivers/net/iavf/iavf_ethdev.c       |   52 +-
 6 files changed, 5 insertions(+), 3081 deletions(-)
 delete mode 100644 drivers/net/i40e/i40e_ethdev_vf.c

Comments

David Marchand Aug. 30, 2021, 11:41 a.m. UTC | #1 
On Mon, Aug 30, 2021 at 5:25 AM Robin Zhang <robinx.zhang@intel.com> wrote:
>
> In DPDK 21.05, iavf already became the default VF for i40e devices.
> So remove i40evf due to it's no need to maintain now.
>
> Robin Zhang (3):
>   net/i40e: remove i40evf
>   net/iavf: remove i40evf devargs option
>   doc: remove old deprecation notice for i40e

Release notes must be updated.

Are you leaving the PF support for servicing i40evf devices on purpose?
Idem, driver-specific API wrt to VF is still present.
Robin Zhang Aug. 31, 2021, 9:40 a.m. UTC | #2 
From: robinx.zhang@intel.com

Hi David

> -----Original Message-----
> From: David Marchand <david.marchand@redhat.com>
> Sent: Monday, August 30, 2021 7:41 PM
> To: Zhang, RobinX <robinx.zhang@intel.com>
> Cc: dev <dev@dpdk.org>; Xing, Beilei <beilei.xing@intel.com>; Burakov,
> Anatoly <anatoly.burakov@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Ray Kinsella <mdr@ashroe.eu>; Zhang, Qi Z
> <qi.z.zhang@intel.com>; Guo, Junfeng <junfeng.guo@intel.com>; Yang,
> SteveX <stevex.yang@intel.com>; Kevin Traynor <ktraynor@redhat.com>
> Subject: Re: [dpdk-dev] [PATCH 0/3] net/i40e: remove i40evf
> 
> On Mon, Aug 30, 2021 at 5:25 AM Robin Zhang <robinx.zhang@intel.com>
> wrote:
> >
> > In DPDK 21.05, iavf already became the default VF for i40e devices.
> > So remove i40evf due to it's no need to maintain now.
> >
> > Robin Zhang (3):
> >   net/i40e: remove i40evf
> >   net/iavf: remove i40evf devargs option
> >   doc: remove old deprecation notice for i40e
> 
> Release notes must be updated.

Will update in v2 patch.

> 
> Are you leaving the PF support for servicing i40evf devices on purpose?
> Idem, driver-specific API wrt to VF is still present.

I40evf driver is the old VF driver for i40e devices, now it's replaced by iavf driver.
We still need the i40e PF support and driver-specific API work with iavf on i40e devices.

> 
> 
> --
> David Marchand
Ray Kinsella Aug. 31, 2021, 3:26 p.m. UTC | #3 
Please check dpdk documentation for references to i40evf also.

See guides/nics/intel_vf.rst and guides/howto/lm_bond_virtio_sriov.rst.

On 30/08/2021 04:12, Robin Zhang wrote:
> In DPDK 21.05, iavf already became the default VF for i40e devices.
> So remove i40evf due to it's no need to maintain now.
> 
> Robin Zhang (3):
>   net/i40e: remove i40evf
>   net/iavf: remove i40evf devargs option
>   doc: remove old deprecation notice for i40e
> 
>  doc/guides/rel_notes/deprecation.rst |    8 -
>  drivers/net/i40e/i40e_ethdev.h       |    1 -
>  drivers/net/i40e/i40e_ethdev_vf.c    | 3015 --------------------------
>  drivers/net/i40e/meson.build         |    1 -
>  drivers/net/i40e/rte_pmd_i40e.c      |    9 +-
>  drivers/net/iavf/iavf_ethdev.c       |   52 +-
>  6 files changed, 5 insertions(+), 3081 deletions(-)
>  delete mode 100644 drivers/net/i40e/i40e_ethdev_vf.c
>