Hi Pankai,

I'd consider a patch which adds RETA controls as well. IMHO it
is a major feature which could be interesting to end users.

Regards,
Andrew.

On 5/23/22 23:56, Pankaj Gupta wrote:
Hi Andrew,

New features were introduced in following two patches so I will update release_22_07.rst for these two patches.

[PATCH v6 1/9] net/vmxnet3: add version 5 support

[PATCH v6 6/9] net/vmxnet3: add version 6 support

Let me know if I am missing anything.

Thanks,

Pankaj

*From: *Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>
*Date: *Thursday, May 19, 2022 at 1:07 AM
*To: *Pankaj Gupta <pagu...@vmware.com>, Jochen Behrens <jbehr...@vmware.com>
*Cc: *dev@dpdk.org <dev@dpdk.org>
*Subject: *Re: [PATCH v6 0/9] net/vmxnet3: support versions 5 and 6

⚠External Email

Hi Pankaj,

I did final cleanup before applying the patch series, but
realized that release notes update is missing.
Please, pick up my changes and amend patches which support
new features with release notes update in the documentation.

Everything else LGTM now.

Thanks,
Andrew.

On 5/19/22 11:04, Andrew Rybchenko wrote:
v6 (Andrew Rybchenko):
      Minor style changes in patches description
      Split FW version get patch to have callbacks reorder seperately
      Minor fix in FW version get patch to put callback definition,
      assignment and implementation just after device info get

v5:
      Use RTE_MIN instead of defining MIN in 
drivers/net/vmxnet3/vmxnet3_ethdev.c
      User RETA instead of reta in commit message
      Fix the issue thrown by checkpatches.sh in file 
drivers/net/vmxnet3/vmxnet3_ethdev.c

v4:
      Changing the subject for [Patch 8/8] from "net/vmxnet3: fix merge error
      in initialization for rxDataRing feature" to
      "Fixes: 046f11619567 ("net/vmxnet3: support MSI-X interrupt")"

v3:
      adding Reviewed-by info in all the patches

v2:
      address review comments from Andrew Rybchenko 
andrew.rybche...@oktetlabs.ru

Pankaj Gupta (9):
    net/vmxnet3: add version 5 support
    net/vmxnet3: implement RETA query and RETA update
    net/vmxnet3: add Rx queue usage count utility
    net/vmxnet3: fix ethdev callbacks init order
    net/vmxnet3: report HW version on FW version get
    net/vmxnet3: add version 6 support
    net/vmxnet3: advertise RETA size in device info
    net/vmxnet3: set packet type for fragmented packet
    net/vmxnet3: fix merge error in Rx data ring initialization

   drivers/net/vmxnet3/base/vmxnet3_defs.h |  73 ++++-
   drivers/net/vmxnet3/vmxnet3_ethdev.c    | 339 +++++++++++++++++++-----
   drivers/net/vmxnet3/vmxnet3_ethdev.h    |  15 +-
   drivers/net/vmxnet3/vmxnet3_rxtx.c      |  49 +++-
   4 files changed, 388 insertions(+), 88 deletions(-)



________________________________

⚠External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.


Reply via email to