patches for DPDK stable branches
 help / color / mirror / Atom feed
* [dpdk-stable] [PATCH] net/ixgbe: check the result of setting VF MTU
@ 2020-12-02  9:48 Zhang,Alvin
  2020-12-02 10:54 ` Wang, Haiyue
  0 siblings, 1 reply; 3+ messages in thread
From: Zhang,Alvin @ 2020-12-02  9:48 UTC (permalink / raw)
  To: jia.guo, haiyue.wang; +Cc: dev, Alvin Zhang, stable

From: Alvin Zhang <alvinx.zhang@intel.com>

If a VF request to set a invalid maximum packet length value,
The PF kernel driver may disable its reception.

This patch add codes to output information and return the error status.

Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>

Fixes: 12cd0cccc3db ("ixgbevf: allow to set MTU")
Cc: stable@dpdk.org
---
 drivers/net/ixgbe/ixgbe_ethdev.c | 3 ++-
 drivers/net/ixgbe/ixgbe_rxtx.c   | 8 ++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
index 9a47a8b..d7a1806 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.c
+++ b/drivers/net/ixgbe/ixgbe_ethdev.c
@@ -6555,7 +6555,8 @@ static void ixgbevf_set_vfta_all(struct rte_eth_dev *dev, bool on)
 	 * prior to 3.11.33 which contains the following change:
 	 * "ixgbe: Enable jumbo frames support w/ SR-IOV"
 	 */
-	ixgbevf_rlpml_set_vf(hw, max_frame);
+	if (ixgbevf_rlpml_set_vf(hw, max_frame))
+		return -EINVAL;
 
 	/* update max frame size */
 	dev->data->dev_conf.rxmode.max_rx_pkt_len = max_frame;
diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index 6cfbb58..7bb8460 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -5634,8 +5634,12 @@ void ixgbe_configure_dcb(struct rte_eth_dev *dev)
 	 * ixgbevf_rlpml_set_vf even if jumbo frames are not used. This way,
 	 * VF packets received can work in all cases.
 	 */
-	ixgbevf_rlpml_set_vf(hw,
-		(uint16_t)dev->data->dev_conf.rxmode.max_rx_pkt_len);
+	if (ixgbevf_rlpml_set_vf(hw,
+	    (uint16_t)dev->data->dev_conf.rxmode.max_rx_pkt_len)) {
+		PMD_INIT_LOG(ERR, "Set max packet length to %d failed.",
+			     dev->data->dev_conf.rxmode.max_rx_pkt_len);
+		return -EINVAL;
+	}
 
 	/*
 	 * Assume no header split and no VLAN strip support
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-stable] [PATCH] net/ixgbe: check the result of setting VF MTU
  2020-12-02  9:48 [dpdk-stable] [PATCH] net/ixgbe: check the result of setting VF MTU Zhang,Alvin
@ 2020-12-02 10:54 ` Wang, Haiyue
  2020-12-11  2:32   ` Zhang, Qi Z
  0 siblings, 1 reply; 3+ messages in thread
From: Wang, Haiyue @ 2020-12-02 10:54 UTC (permalink / raw)
  To: Zhang, AlvinX, Guo, Jia; +Cc: dev, Zhang, AlvinX, stable

> -----Original Message-----
> From: Zhang,Alvin <alvinx.zhang@intel.com>
> Sent: Wednesday, December 2, 2020 17:48
> To: Guo, Jia <jia.guo@intel.com>; Wang, Haiyue <haiyue.wang@intel.com>
> Cc: dev@dpdk.org; Zhang, AlvinX <alvinx.zhang@intel.com>; stable@dpdk.org
> Subject: [PATCH] net/ixgbe: check the result of setting VF MTU
> 
> From: Alvin Zhang <alvinx.zhang@intel.com>
> 
> If a VF request to set a invalid maximum packet length value,
> The PF kernel driver may disable its reception.
> 
> This patch add codes to output information and return the error status.
> 
> Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
> 
> Fixes: 12cd0cccc3db ("ixgbevf: allow to set MTU")
> Cc: stable@dpdk.org
> ---
>  drivers/net/ixgbe/ixgbe_ethdev.c | 3 ++-
>  drivers/net/ixgbe/ixgbe_rxtx.c   | 8 ++++++--
>  2 files changed, 8 insertions(+), 3 deletions(-)
> 

Acked-by: Haiyue Wang <haiyue.wang@intel.com>

> --
> 1.8.3.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-stable] [PATCH] net/ixgbe: check the result of setting VF MTU
  2020-12-02 10:54 ` Wang, Haiyue
@ 2020-12-11  2:32   ` Zhang, Qi Z
  0 siblings, 0 replies; 3+ messages in thread
From: Zhang, Qi Z @ 2020-12-11  2:32 UTC (permalink / raw)
  To: Wang, Haiyue, Zhang, AlvinX, Guo, Jia; +Cc: dev, Zhang, AlvinX, stable



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Wang, Haiyue
> Sent: Wednesday, December 2, 2020 6:55 PM
> To: Zhang, AlvinX <alvinx.zhang@intel.com>; Guo, Jia <jia.guo@intel.com>
> Cc: dev@dpdk.org; Zhang, AlvinX <alvinx.zhang@intel.com>; stable@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] net/ixgbe: check the result of setting VF MTU
> 
> > -----Original Message-----
> > From: Zhang,Alvin <alvinx.zhang@intel.com>
> > Sent: Wednesday, December 2, 2020 17:48
> > To: Guo, Jia <jia.guo@intel.com>; Wang, Haiyue <haiyue.wang@intel.com>
> > Cc: dev@dpdk.org; Zhang, AlvinX <alvinx.zhang@intel.com>;
> > stable@dpdk.org
> > Subject: [PATCH] net/ixgbe: check the result of setting VF MTU
> >
> > From: Alvin Zhang <alvinx.zhang@intel.com>
> >
> > If a VF request to set a invalid maximum packet length value, The PF
> > kernel driver may disable its reception.
> >
> > This patch add codes to output information and return the error status.
> >
> > Signed-off-by: Alvin Zhang <alvinx.zhang@intel.com>
> >
> > Fixes: 12cd0cccc3db ("ixgbevf: allow to set MTU")
> > Cc: stable@dpdk.org
> > ---
> >  drivers/net/ixgbe/ixgbe_ethdev.c | 3 ++-
> >  drivers/net/ixgbe/ixgbe_rxtx.c   | 8 ++++++--
> >  2 files changed, 8 insertions(+), 3 deletions(-)
> >
> 
> Acked-by: Haiyue Wang <haiyue.wang@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi
> 
> > --
> > 1.8.3.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-12-11  2:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-02  9:48 [dpdk-stable] [PATCH] net/ixgbe: check the result of setting VF MTU Zhang,Alvin
2020-12-02 10:54 ` Wang, Haiyue
2020-12-11  2:32   ` Zhang, Qi Z

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ http://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git