* [PATCH] *** Intel iavf: Return in the case of ADD/DEL ETH address ***
@ 2023-01-13 13:19 Vipin P R
2023-01-13 13:19 ` [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address Vipin P R
0 siblings, 1 reply; 6+ messages in thread
From: Vipin P R @ 2023-01-13 13:19 UTC (permalink / raw)
To: jingjing.wu, beilei.xing; +Cc: dev, Vipin P R
***
Intel iavf: Return in the case of ADD/DEL ETH address
In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR are unsupported.
i40evf_execute_vf_cmd is invoked with these operations as part of i40evf_set_mc_addr_list()
The cases are not handled in i40evf_execute_vf_cmd() thus hitting the default case.
There is a retry logic of upto 200 times (2000 in iavf) with a delay of 10ms (1ms in iavf).
This results in a needless delay of 2s in the init phase for each VNIC.
The patch aims to rectify that delay.
In fe2a571c70cc397f2ad4e280f8d084148fea5d62, i40e_ethdev_vf.c was deleted. Hence adding this in iavf_vchnl.c.
***
Vipin P R (1):
Intel iavf: Return in the case of ADD/DEL ETH address
drivers/net/iavf/iavf_vchnl.c | 8 ++++++++
1 file changed, 8 insertions(+)
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
2023-01-13 13:19 [PATCH] *** Intel iavf: Return in the case of ADD/DEL ETH address *** Vipin P R
@ 2023-01-13 13:19 ` Vipin P R
2023-01-16 17:55 ` Stephen Hemminger
2023-01-17 1:54 ` Zhang, Qi Z
0 siblings, 2 replies; 6+ messages in thread
From: Vipin P R @ 2023-01-13 13:19 UTC (permalink / raw)
To: jingjing.wu, beilei.xing; +Cc: dev, Vipin P R, stable
In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR are unsupported.
i40evf_execute_vf_cmd is invoked with these operations as part of i40evf_set_mc_addr_list()
The cases are not handled in i40evf_execute_vf_cmd() thus hitting the default case.
There is a retry logic of upto 200 times (2000 in iavf) with a delay of 10ms (1ms in iavf).
This results in a needless delay of 2s in the init phase for each VNIC.
The patch aims to rectify that delay.
In fe2a571c70cc397f2ad4e280f8d084148fea5d62, i40e_ethdev_vf.c was deleted. Hence adding this in iavf_vchnl.c.
Cc: stable@dpdk.org
Signed-off-by: Vipin P R <vipinp@vmware.com>
---
drivers/net/iavf/iavf_vchnl.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c
index f92daf9..e2f65f5 100644
--- a/drivers/net/iavf/iavf_vchnl.c
+++ b/drivers/net/iavf/iavf_vchnl.c
@@ -367,6 +367,14 @@ iavf_execute_vf_cmd(struct iavf_adapter *adapter, struct iavf_cmd_info *args,
}
_clear_cmd(vf);
break;
+
+ case VIRTCHNL_OP_ADD_ETH_ADDR:
+ case VIRTCHNL_OP_DEL_ETH_ADDR:
+ PMD_DRV_LOG(WARNING, "OP_{ADD/DEL}_ETH_ADDR unsupported");
+ err = 0;
+ _clear_cmd(vf);
+ break;
+
default:
/* For other virtchnl ops in running time,
* wait for the cmd done flag.
--
2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
2023-01-13 13:19 ` [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address Vipin P R
@ 2023-01-16 17:55 ` Stephen Hemminger
2023-01-17 1:54 ` Zhang, Qi Z
1 sibling, 0 replies; 6+ messages in thread
From: Stephen Hemminger @ 2023-01-16 17:55 UTC (permalink / raw)
To: Vipin P R; +Cc: jingjing.wu, beilei.xing, dev, stable
On Fri, 13 Jan 2023 13:19:23 +0000
Vipin P R <vipinp@vmware.com> wrote:
> In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR are unsupported.
> i40evf_execute_vf_cmd is invoked with these operations as part of i40evf_set_mc_addr_list()
>
> The cases are not handled in i40evf_execute_vf_cmd() thus hitting the default case.
> There is a retry logic of upto 200 times (2000 in iavf) with a delay of 10ms (1ms in iavf).
> This results in a needless delay of 2s in the init phase for each VNIC.
>
> The patch aims to rectify that delay.
> In fe2a571c70cc397f2ad4e280f8d084148fea5d62, i40e_ethdev_vf.c was deleted. Hence adding this in iavf_vchnl.c.
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Vipin P R <vipinp@vmware.com>
> ---
> drivers/net/iavf/iavf_vchnl.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
Please run checkpatch, the DPDK style is to indent with tabs not spaces.
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
2023-01-13 13:19 ` [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address Vipin P R
2023-01-16 17:55 ` Stephen Hemminger
@ 2023-01-17 1:54 ` Zhang, Qi Z
2023-02-07 10:42 ` Vipin P R
1 sibling, 1 reply; 6+ messages in thread
From: Zhang, Qi Z @ 2023-01-17 1:54 UTC (permalink / raw)
To: Vipin P R, Wu, Jingjing, Xing, Beilei; +Cc: dev, stable
> -----Original Message-----
> From: Vipin P R <vipinp@vmware.com>
> Sent: Friday, January 13, 2023 9:19 PM
> To: Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Cc: dev@dpdk.org; Vipin P R <vipinp@vmware.com>; stable@dpdk.org
> Subject: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
>
> In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and
> VIRTCHNL_OP_ADD_ETH_ADDR are unsupported.
> i40evf_execute_vf_cmd is invoked with these operations as part of
> i40evf_set_mc_addr_list()
>
> The cases are not handled in i40evf_execute_vf_cmd() thus hitting the
> default case.
> There is a retry logic of upto 200 times (2000 in iavf) with a delay of 10ms (1ms
> in iavf).
> This results in a needless delay of 2s in the init phase for each VNIC.
>
Sorry I didn't get this, why this is related with i40evf? I40evf PMD has been replaced by iavf PMD.
The iavf PMD works with both i40e and ice, does this will break ice's usage?
> The patch aims to rectify that delay.
> In fe2a571c70cc397f2ad4e280f8d084148fea5d62, i40e_ethdev_vf.c was
> deleted. Hence adding this in iavf_vchnl.c.
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Vipin P R <vipinp@vmware.com>
> ---
> drivers/net/iavf/iavf_vchnl.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c index
> f92daf9..e2f65f5 100644
> --- a/drivers/net/iavf/iavf_vchnl.c
> +++ b/drivers/net/iavf/iavf_vchnl.c
> @@ -367,6 +367,14 @@ iavf_execute_vf_cmd(struct iavf_adapter *adapter,
> struct iavf_cmd_info *args,
> }
> _clear_cmd(vf);
> break;
> +
> + case VIRTCHNL_OP_ADD_ETH_ADDR:
> + case VIRTCHNL_OP_DEL_ETH_ADDR:
> + PMD_DRV_LOG(WARNING, "OP_{ADD/DEL}_ETH_ADDR
> unsupported");
> + err = 0;
> + _clear_cmd(vf);
> + break;
> +
> default:
> /* For other virtchnl ops in running time,
> * wait for the cmd done flag.
> --
> 2.7.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
2023-01-17 1:54 ` Zhang, Qi Z
@ 2023-02-07 10:42 ` Vipin P R
2023-02-13 1:31 ` Zhang, Qi Z
0 siblings, 1 reply; 6+ messages in thread
From: Vipin P R @ 2023-02-07 10:42 UTC (permalink / raw)
To: Zhang, Qi Z, Wu, Jingjing, Xing, Beilei; +Cc: dev, stable
[-- Attachment #1: Type: text/plain, Size: 2837 bytes --]
________________________________
From: Zhang, Qi Z <qi.z.zhang@intel.com>
Sent: 17 January 2023 07:24
To: Vipin P R <vipinp@vmware.com>; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
Cc: dev@dpdk.org <dev@dpdk.org>; stable@dpdk.org <stable@dpdk.org>
Subject: RE: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
!! External Email
> -----Original Message-----
> From: Vipin P R <vipinp@vmware.com>
> Sent: Friday, January 13, 2023 9:19 PM
> To: Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Cc: dev@dpdk.org; Vipin P R <vipinp@vmware.com>; stable@dpdk.org
> Subject: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
>
> In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and
> VIRTCHNL_OP_ADD_ETH_ADDR are unsupported.
> i40evf_execute_vf_cmd is invoked with these operations as part of
> i40evf_set_mc_addr_list()
>
> The cases are not handled in i40evf_execute_vf_cmd() thus hitting the
> default case.
> There is a retry logic of upto 200 times (2000 in iavf) with a delay of 10ms (1ms
> in iavf).
> This results in a needless delay of 2s in the init phase for each VNIC.
>
Sorry I didn't get this, why this is related with i40evf? I40evf PMD has been replaced by iavf PMD.
The iavf PMD works with both i40e and ice, does this will break ice's usage?
Hi, this is problem was discovered while using i40vf (which was deprecated and replaced by iavf). Upon inspecting the code, this problem would also be hit in iavf, if the VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR are not supported.
> The patch aims to rectify that delay.
> In fe2a571c70cc397f2ad4e280f8d084148fea5d62, i40e_ethdev_vf.c was
> deleted. Hence adding this in iavf_vchnl.c.
>
> Cc: stable@dpdk.org
>
> Signed-off-by: Vipin P R <vipinp@vmware.com>
> ---
> drivers/net/iavf/iavf_vchnl.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c index
> f92daf9..e2f65f5 100644
> --- a/drivers/net/iavf/iavf_vchnl.c
> +++ b/drivers/net/iavf/iavf_vchnl.c
> @@ -367,6 +367,14 @@ iavf_execute_vf_cmd(struct iavf_adapter *adapter,
> struct iavf_cmd_info *args,
> }
> _clear_cmd(vf);
> break;
> +
> + case VIRTCHNL_OP_ADD_ETH_ADDR:
> + case VIRTCHNL_OP_DEL_ETH_ADDR:
> + PMD_DRV_LOG(WARNING, "OP_{ADD/DEL}_ETH_ADDR
> unsupported");
> + err = 0;
> + _clear_cmd(vf);
> + break;
> +
> default:
> /* For other virtchnl ops in running time,
> * wait for the cmd done flag.
> --
> 2.7.4
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
[-- Attachment #2: Type: text/html, Size: 6137 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
2023-02-07 10:42 ` Vipin P R
@ 2023-02-13 1:31 ` Zhang, Qi Z
0 siblings, 0 replies; 6+ messages in thread
From: Zhang, Qi Z @ 2023-02-13 1:31 UTC (permalink / raw)
To: Vipin P R, Wu, Jingjing, Xing, Beilei; +Cc: dev, stable
Please always apply text format in mailing list.
Sorry I have to reject this patch, as a device agnostic driver, iavf can't just simply assume kernel PF driver don't support VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR.
A better solution is to introduce some VF/PF negotiation during init or learning from runtime.(if get an unsupported error from PF after the first call)
From: Vipin P R <vipinp@vmware.com>
Sent: Tuesday, February 7, 2023 6:43 PM
To: Zhang, Qi Z <qi.z.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
Cc: dev@dpdk.org; stable@dpdk.org
Subject: Re: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
________________________________________
From: Zhang, Qi Z <mailto:qi.z.zhang@intel.com>
Sent: 17 January 2023 07:24
To: Vipin P R <mailto:vipinp@vmware.com>; Wu, Jingjing <mailto:jingjing.wu@intel.com>; Xing, Beilei <mailto:beilei.xing@intel.com>
Cc: mailto:dev@dpdk.org <mailto:dev@dpdk.org>; mailto:stable@dpdk.org <mailto:stable@dpdk.org>
Subject: RE: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
!! External Email
> -----Original Message-----
> From: Vipin P R <mailto:vipinp@vmware.com>
> Sent: Friday, January 13, 2023 9:19 PM
> To: Wu, Jingjing <mailto:jingjing.wu@intel.com>; Xing, Beilei <mailto:beilei.xing@intel.com>
> Cc: mailto:dev@dpdk.org; Vipin P R <mailto:vipinp@vmware.com>; mailto:stable@dpdk.org
> Subject: [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address
>
> In case of i40vf, VIRTCHNL_OP_DEL_ETH_ADDR and
> VIRTCHNL_OP_ADD_ETH_ADDR are unsupported.
> i40evf_execute_vf_cmd is invoked with these operations as part of
> i40evf_set_mc_addr_list()
>
> The cases are not handled in i40evf_execute_vf_cmd() thus hitting the
> default case.
> There is a retry logic of upto 200 times (2000 in iavf) with a delay of 10ms (1ms
> in iavf).
> This results in a needless delay of 2s in the init phase for each VNIC.
>
Sorry I didn't get this, why this is related with i40evf? I40evf PMD has been replaced by iavf PMD.
The iavf PMD works with both i40e and ice, does this will break ice's usage?
Hi, this is problem was discovered while using i40vf (which was deprecated and replaced by iavf). Upon inspecting the code, this problem would also be hit in iavf, if the VIRTCHNL_OP_DEL_ETH_ADDR and VIRTCHNL_OP_ADD_ETH_ADDR are not supported.
> The patch aims to rectify that delay.
> In fe2a571c70cc397f2ad4e280f8d084148fea5d62, i40e_ethdev_vf.c was
> deleted. Hence adding this in iavf_vchnl.c.
>
> Cc: mailto:stable@dpdk.org
>
> Signed-off-by: Vipin P R <mailto:vipinp@vmware.com>
> ---
> drivers/net/iavf/iavf_vchnl.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/net/iavf/iavf_vchnl.c b/drivers/net/iavf/iavf_vchnl.c index
> f92daf9..e2f65f5 100644
> --- a/drivers/net/iavf/iavf_vchnl.c
> +++ b/drivers/net/iavf/iavf_vchnl.c
> @@ -367,6 +367,14 @@ iavf_execute_vf_cmd(struct iavf_adapter *adapter,
> struct iavf_cmd_info *args,
> }
> _clear_cmd(vf);
> break;
> +
> + case VIRTCHNL_OP_ADD_ETH_ADDR:
> + case VIRTCHNL_OP_DEL_ETH_ADDR:
> + PMD_DRV_LOG(WARNING, "OP_{ADD/DEL}_ETH_ADDR
> unsupported");
> + err = 0;
> + _clear_cmd(vf);
> + break;
> +
> default:
> /* For other virtchnl ops in running time,
> * wait for the cmd done flag.
> --
> 2.7.4
!! External Email: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-02-13 1:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-13 13:19 [PATCH] *** Intel iavf: Return in the case of ADD/DEL ETH address *** Vipin P R
2023-01-13 13:19 ` [PATCH] Intel iavf: Return in the case of ADD/DEL ETH address Vipin P R
2023-01-16 17:55 ` Stephen Hemminger
2023-01-17 1:54 ` Zhang, Qi Z
2023-02-07 10:42 ` Vipin P R
2023-02-13 1:31 ` Zhang, Qi Z
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).