mbox series

[v7,0/3] net/i40e: remove i40evf

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

Message

Robin Zhang Sept. 24, 2021, 6:22 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.

v7:
- rebase code.

v6:
- remove i40e_vf struct and related code, remove doc i40e_vf.ini.

v5:
- rebase code.

v4:
- resolve compile warning issue.

v3:
- remove VF related code in i40e_rxtx.c.

v2:
- update 21.11 release note, remove some missed documentation.

Robin Zhang (3):
  net/i40e: remove i40evf
  net/iavf: remove i40evf devargs option
  doc: remove i40evf related documentation

 doc/guides/howto/lm_bond_virtio_sriov.rst |    4 +-
 doc/guides/nics/features/i40e_vf.ini      |   40 -
 doc/guides/nics/intel_vf.rst              |    8 +-
 doc/guides/rel_notes/deprecation.rst      |    8 -
 doc/guides/rel_notes/release_21_11.rst    |    2 +
 drivers/net/i40e/base/i40e_osdep.h        |    1 -
 drivers/net/i40e/i40e_ethdev.h            |   78 +-
 drivers/net/i40e/i40e_ethdev_vf.c         | 3006 ---------------------
 drivers/net/i40e/i40e_rxtx.c              |   50 +-
 drivers/net/i40e/meson.build              |    1 -
 drivers/net/i40e/rte_pmd_i40e.c           |    9 +-
 drivers/net/iavf/iavf_ethdev.c            |   52 +-
 12 files changed, 31 insertions(+), 3228 deletions(-)
 delete mode 100644 doc/guides/nics/features/i40e_vf.ini
 delete mode 100644 drivers/net/i40e/i40e_ethdev_vf.c
  

Comments

Qi Zhang Sept. 26, 2021, 6:22 a.m. UTC | #1
> -----Original Message-----
> From: Zhang, RobinX <robinx.zhang@intel.com>
> Sent: Friday, September 24, 2021 2:22 PM
> To: dev@dpdk.org
> Cc: Wang, Haiyue <haiyue.wang@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>; mdr@ashroe.eu; Wu, Jingjing
> <jingjing.wu@intel.com>; Burakov, Anatoly <anatoly.burakov@intel.com>;
> Zhang, Qi Z <qi.z.zhang@intel.com>; Guo, Junfeng <junfeng.guo@intel.com>;
> Yang, SteveX <stevex.yang@intel.com>; Zhang, RobinX
> <robinx.zhang@intel.com>
> Subject: [PATCH v7 0/3] net/i40e: remove i40evf
> 
> 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.
> 
> v7:
> - rebase code.
> 
> v6:
> - remove i40e_vf struct and related code, remove doc i40e_vf.ini.
> 
> v5:
> - rebase code.
> 
> v4:
> - resolve compile warning issue.
> 
> v3:
> - remove VF related code in i40e_rxtx.c.
> 
> v2:
> - update 21.11 release note, remove some missed documentation.
> 
> Robin Zhang (3):
>   net/i40e: remove i40evf
>   net/iavf: remove i40evf devargs option
>   doc: remove i40evf related documentation
> 
>  doc/guides/howto/lm_bond_virtio_sriov.rst |    4 +-
>  doc/guides/nics/features/i40e_vf.ini      |   40 -
>  doc/guides/nics/intel_vf.rst              |    8 +-
>  doc/guides/rel_notes/deprecation.rst      |    8 -
>  doc/guides/rel_notes/release_21_11.rst    |    2 +
>  drivers/net/i40e/base/i40e_osdep.h        |    1 -
>  drivers/net/i40e/i40e_ethdev.h            |   78 +-
>  drivers/net/i40e/i40e_ethdev_vf.c         | 3006 ---------------------
>  drivers/net/i40e/i40e_rxtx.c              |   50 +-
>  drivers/net/i40e/meson.build              |    1 -
>  drivers/net/i40e/rte_pmd_i40e.c           |    9 +-
>  drivers/net/iavf/iavf_ethdev.c            |   52 +-
>  12 files changed, 31 insertions(+), 3228 deletions(-)  delete mode 100644
> doc/guides/nics/features/i40e_vf.ini
>  delete mode 100644 drivers/net/i40e/i40e_ethdev_vf.c
> 
> --
> 2.25.1

Acked-by: Qi Zhang <qi.z.zhang@intel.com>

Applied to dpdk-next-inet-intel.

Thanks
Qi
  
Thomas Monjalon Oct. 5, 2021, 9:43 a.m. UTC | #2
> > Robin Zhang (3):
> >   net/i40e: remove i40evf
> >   net/iavf: remove i40evf devargs option
> >   doc: remove i40evf related documentation
> 
> Acked-by: Qi Zhang <qi.z.zhang@intel.com>
> 
> Applied to dpdk-next-inet-intel.

The split of the patch is not logical.
"net/iavf: remove i40evf devargs option" should be first.
doc changes should be squashed in first 2 patches.
There are also some minor blank line issues.
I'll fix all while pulling.