diff mbox series

[3/3] doc: fix vector mode info for bnxt

Message ID 20200929205935.20432-4-ajit.khaparde@broadcom.com (mailing list archive)
State Rejected, archived
Delegated to: Ajit Khaparde
Headers
Series patchset for bnxt PMD |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-testing success Testing PASS

Commit Message

Ajit Khaparde Sept. 29, 2020, 8:59 p.m. UTC 
  Vector mode is disabled when jumbo frames are enabled.
This usage guide was reporting it incorrectly.

Fixes: 1adaf0e0f2ee ("doc: update bnxt guide")
Cc:stable@dpdk.org

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 doc/guides/nics/bnxt.rst | 1 -
 1 file changed, 1 deletion(-)

Comments

Lance Richardson Sept. 30, 2020, 5:28 p.m. UTC | #1 
On Tue, Sep 29, 2020 at 5:01 PM Ajit Khaparde
<ajit.khaparde@broadcom.com> wrote:
>
> Vector mode is disabled when jumbo frames are enabled.
> This usage guide was reporting it incorrectly.
>
> Fixes: 1adaf0e0f2ee ("doc: update bnxt guide")
> Cc:stable@dpdk.org
>
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> ---
>  doc/guides/nics/bnxt.rst | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/doc/guides/nics/bnxt.rst b/doc/guides/nics/bnxt.rst
> index 129a16cfc..48ec6156d 100644
> --- a/doc/guides/nics/bnxt.rst
> +++ b/doc/guides/nics/bnxt.rst
> @@ -836,7 +836,6 @@ This improved performance comes from several optimizations:
>
>         DEV_RX_OFFLOAD_VLAN_STRIP
>         DEV_RX_OFFLOAD_KEEP_CRC
> -       DEV_RX_OFFLOAD_JUMBO_FRAME
>         DEV_RX_OFFLOAD_IPV4_CKSUM
>         DEV_RX_OFFLOAD_UDP_CKSUM
>         DEV_RX_OFFLOAD_TCP_CKSUM
> --
> 2.21.1 (Apple Git-122.3)
>

NAK - receive vector mode *is* allowed when jumbo is enabled, as
long as the MTU is not large enough to require scattered RX (which
also depends on the mbuf size).

   Lance
Ajit Khaparde Sept. 30, 2020, 5:37 p.m. UTC | #2 
On Wed, Sep 30, 2020 at 10:28 AM Lance Richardson
<lance.richardson@broadcom.com> wrote:
>
> On Tue, Sep 29, 2020 at 5:01 PM Ajit Khaparde
> <ajit.khaparde@broadcom.com> wrote:
> >
> > Vector mode is disabled when jumbo frames are enabled.
> > This usage guide was reporting it incorrectly.
> >
> > Fixes: 1adaf0e0f2ee ("doc: update bnxt guide")
> > Cc:stable@dpdk.org
> >
> > Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> > ---
> >  doc/guides/nics/bnxt.rst | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/doc/guides/nics/bnxt.rst b/doc/guides/nics/bnxt.rst
> > index 129a16cfc..48ec6156d 100644
> > --- a/doc/guides/nics/bnxt.rst
> > +++ b/doc/guides/nics/bnxt.rst
> > @@ -836,7 +836,6 @@ This improved performance comes from several optimizations:
> >
> >         DEV_RX_OFFLOAD_VLAN_STRIP
> >         DEV_RX_OFFLOAD_KEEP_CRC
> > -       DEV_RX_OFFLOAD_JUMBO_FRAME
> >         DEV_RX_OFFLOAD_IPV4_CKSUM
> >         DEV_RX_OFFLOAD_UDP_CKSUM
> >         DEV_RX_OFFLOAD_TCP_CKSUM
> > --
> > 2.21.1 (Apple Git-122.3)
> >
>
> NAK - receive vector mode *is* allowed when jumbo is enabled, as
> long as the MTU is not large enough to require scattered RX (which
> also depends on the mbuf size).
Ok. Fair enough. Let me drop this patch.

>
>    Lance
diff mbox series

Patch

diff --git a/doc/guides/nics/bnxt.rst b/doc/guides/nics/bnxt.rst
index 129a16cfc..48ec6156d 100644
--- a/doc/guides/nics/bnxt.rst
+++ b/doc/guides/nics/bnxt.rst
@@ -836,7 +836,6 @@  This improved performance comes from several optimizations:
 
        DEV_RX_OFFLOAD_VLAN_STRIP
        DEV_RX_OFFLOAD_KEEP_CRC
-       DEV_RX_OFFLOAD_JUMBO_FRAME
        DEV_RX_OFFLOAD_IPV4_CKSUM
        DEV_RX_OFFLOAD_UDP_CKSUM
        DEV_RX_OFFLOAD_TCP_CKSUM