diff mbox series

net/mlx4: advertise jumbo frame support in Rx

Message ID 1533053586-8551-1-git-send-email-motih@mellanox.com (mailing list archive)
State Accepted, archived
Delegated to: Shahaf Shuler
Headers
Series net/mlx4: advertise jumbo frame support in Rx |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Moti Haimovsky July 31, 2018, 4:13 p.m. UTC 
  This commit adds the missing Rx jumbo frame support advertisement
in Rx offload capabilities.

Signed-off-by: Moti Haimovsky <motih@mellanox.com>
---
 drivers/net/mlx4/mlx4_rxq.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Matan Azrad Aug. 1, 2018, 10:14 a.m. UTC | #1 
Hi Moti

From: Mordechay Haimovsky
> This commit adds the missing Rx jumbo frame support advertisement in Rx
> offload capabilities.
> 
> Signed-off-by: Moti Haimovsky <motih@mellanox.com>
> ---
>  drivers/net/mlx4/mlx4_rxq.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/mlx4/mlx4_rxq.c b/drivers/net/mlx4/mlx4_rxq.c index
> e8bef1d..9737da2 100644
> --- a/drivers/net/mlx4/mlx4_rxq.c
> +++ b/drivers/net/mlx4/mlx4_rxq.c
> @@ -679,7 +679,8 @@ struct mlx4_rss *
>  {
>  	uint64_t offloads = DEV_RX_OFFLOAD_SCATTER |
>  			    DEV_RX_OFFLOAD_CRC_STRIP |
> -			    DEV_RX_OFFLOAD_KEEP_CRC;
> +			    DEV_RX_OFFLOAD_KEEP_CRC |
> +			    DEV_RX_OFFLOAD_JUMBO_FRAME;
> 
>  	if (priv->hw_csum)
>  		offloads |= DEV_RX_OFFLOAD_CHECKSUM;
> --

This patch should be backported to the stable branch.

Shahaf, can you add cc stable in the integration?

Cc: stable@dpdk.org

Acked-by: Matan Azrad <matan@mellanox.com>
Shahaf Shuler Aug. 1, 2018, 12:51 p.m. UTC | #2 
Wednesday, August 1, 2018 1:15 PM, Matan Azrad:
> Subject: Re: [dpdk-dev] [PATCH] net/mlx4: advertise jumbo frame support in
> Rx
> 
> Hi Moti
> 
> From: Mordechay Haimovsky
> > This commit adds the missing Rx jumbo frame support advertisement in
> > Rx offload capabilities.
> >
> > Signed-off-by: Moti Haimovsky <motih@mellanox.com>
> > ---
> >  drivers/net/mlx4/mlx4_rxq.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/mlx4/mlx4_rxq.c b/drivers/net/mlx4/mlx4_rxq.c
> > index
> > e8bef1d..9737da2 100644
> > --- a/drivers/net/mlx4/mlx4_rxq.c
> > +++ b/drivers/net/mlx4/mlx4_rxq.c
> > @@ -679,7 +679,8 @@ struct mlx4_rss *
> >  {
> >  	uint64_t offloads = DEV_RX_OFFLOAD_SCATTER |
> >  			    DEV_RX_OFFLOAD_CRC_STRIP |
> > -			    DEV_RX_OFFLOAD_KEEP_CRC;
> > +			    DEV_RX_OFFLOAD_KEEP_CRC |
> > +			    DEV_RX_OFFLOAD_JUMBO_FRAME;
> >
> >  	if (priv->hw_csum)
> >  		offloads |= DEV_RX_OFFLOAD_CHECKSUM;
> > --
> 
> This patch should be backported to the stable branch.
> 
> Shahaf, can you add cc stable in the integration?
> 
> Cc: stable@dpdk.org
> 
> Acked-by: Matan Azrad <matan@mellanox.com>

Applied to next-net-mlx with Cc'ing stable. Thanks. 

>
diff mbox series

Patch

diff --git a/drivers/net/mlx4/mlx4_rxq.c b/drivers/net/mlx4/mlx4_rxq.c
index e8bef1d..9737da2 100644
--- a/drivers/net/mlx4/mlx4_rxq.c
+++ b/drivers/net/mlx4/mlx4_rxq.c
@@ -679,7 +679,8 @@  struct mlx4_rss *
 {
 	uint64_t offloads = DEV_RX_OFFLOAD_SCATTER |
 			    DEV_RX_OFFLOAD_CRC_STRIP |
-			    DEV_RX_OFFLOAD_KEEP_CRC;
+			    DEV_RX_OFFLOAD_KEEP_CRC |
+			    DEV_RX_OFFLOAD_JUMBO_FRAME;
 
 	if (priv->hw_csum)
 		offloads |= DEV_RX_OFFLOAD_CHECKSUM;