diff mbox series

net/i40e: fix Virtchnl Offload Caps missing

Message ID 20210312085209.983526-1-robinx.zhang@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Qi Zhang
Headers show
Series net/i40e: fix Virtchnl Offload Caps missing | expand

Checks

Context Check Description
ci/intel-Testing success Testing PASS
ci/iol-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-mellanox-Functional fail Functional Testing issues
ci/iol-mellanox-Performance success Performance Testing PASS
ci/github-robot success github build: passed
ci/travis-robot success travis build: passed
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Robin Zhang March 12, 2021, 8:52 a.m. UTC
A new feature requesting additional queues from PF is added in iavf, before
sending VIRTCHNL_OP_REQUEST_QUEUES op code, the offload capability flag
VIRTCHNL_VF_OFFLOAD_REQ_QUEUES will be checked. And due to DPDK PF is
still used by some cases, so add this offload capability flag in i40e pf.

Fixes: cbdbd360f77f ("net/i40e: support AVF basic interface")

Signed-off-by: Robin Zhang <robinx.zhang@intel.com>
---
 drivers/net/i40e/i40e_pf.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Guo, Jia March 12, 2021, 9:29 a.m. UTC | #1
Acked-by: Jeff Guo <jia.guo@intel.com>

> -----Original Message-----
> From: Zhang, RobinX <robinx.zhang@intel.com>
> Sent: Friday, March 12, 2021 4:52 PM
> To: dev@dpdk.org
> Cc: Xing, Beilei <beilei.xing@intel.com>; Guo, Jia <jia.guo@intel.com>; Zhang,
> RobinX <robinx.zhang@intel.com>
> Subject: [PATCH] net/i40e: fix Virtchnl Offload Caps missing
> 
> A new feature requesting additional queues from PF is added in iavf, before
> sending VIRTCHNL_OP_REQUEST_QUEUES op code, the offload capability
> flag VIRTCHNL_VF_OFFLOAD_REQ_QUEUES will be checked. And due to
> DPDK PF is still used by some cases, so add this offload capability flag in i40e
> pf.
> 
> Fixes: cbdbd360f77f ("net/i40e: support AVF basic interface")
> 
> Signed-off-by: Robin Zhang <robinx.zhang@intel.com>
> ---
>  drivers/net/i40e/i40e_pf.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/net/i40e/i40e_pf.c b/drivers/net/i40e/i40e_pf.c index
> f3d6d899f6..9804ed4253 100644
> --- a/drivers/net/i40e/i40e_pf.c
> +++ b/drivers/net/i40e/i40e_pf.c
> @@ -333,6 +333,10 @@ i40e_pf_host_process_cmd_get_vf_resource(struct
> i40e_pf_vf *vf, uint8_t *msg,
> 
>  	vf_res->vf_cap_flags = vf->request_caps &
>  				   I40E_VIRTCHNL_OFFLOAD_CAPS;
> +
> +	if (vf->request_caps & VIRTCHNL_VF_OFFLOAD_REQ_QUEUES)
> +		vf_res->vf_cap_flags |=
> VIRTCHNL_VF_OFFLOAD_REQ_QUEUES;
> +
>  	/* For X722, it supports write back on ITR
>  	 * without binding queue to interrupt vector.
>  	 */
> --
> 2.25.1
Qi Zhang March 25, 2021, 8:15 a.m. UTC | #2
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Guo, Jia
> Sent: Friday, March 12, 2021 5:29 PM
> To: Zhang, RobinX <robinx.zhang@intel.com>; dev@dpdk.org
> Cc: Xing, Beilei <beilei.xing@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] net/i40e: fix Virtchnl Offload Caps missing
> 
> Acked-by: Jeff Guo <jia.guo@intel.com>
> 
> > -----Original Message-----
> > From: Zhang, RobinX <robinx.zhang@intel.com>
> > Sent: Friday, March 12, 2021 4:52 PM
> > To: dev@dpdk.org
> > Cc: Xing, Beilei <beilei.xing@intel.com>; Guo, Jia
> > <jia.guo@intel.com>; Zhang, RobinX <robinx.zhang@intel.com>
> > Subject: [PATCH] net/i40e: fix Virtchnl Offload Caps missing
> >
> > A new feature requesting additional queues from PF is added in iavf,
> > before sending VIRTCHNL_OP_REQUEST_QUEUES op code, the offload
> > capability flag VIRTCHNL_VF_OFFLOAD_REQ_QUEUES will be checked. And
> > due to DPDK PF is still used by some cases, so add this offload
> > capability flag in i40e pf.
> >
> > Fixes: cbdbd360f77f ("net/i40e: support AVF basic interface")
> >
> > Signed-off-by: Robin Zhang <robinx.zhang@intel.com>
> > ---

Applied to dpdk-next-net-intel.

Thanks
Qi
diff mbox series

Patch

diff --git a/drivers/net/i40e/i40e_pf.c b/drivers/net/i40e/i40e_pf.c
index f3d6d899f6..9804ed4253 100644
--- a/drivers/net/i40e/i40e_pf.c
+++ b/drivers/net/i40e/i40e_pf.c
@@ -333,6 +333,10 @@  i40e_pf_host_process_cmd_get_vf_resource(struct i40e_pf_vf *vf, uint8_t *msg,
 
 	vf_res->vf_cap_flags = vf->request_caps &
 				   I40E_VIRTCHNL_OFFLOAD_CAPS;
+
+	if (vf->request_caps & VIRTCHNL_VF_OFFLOAD_REQ_QUEUES)
+		vf_res->vf_cap_flags |= VIRTCHNL_VF_OFFLOAD_REQ_QUEUES;
+
 	/* For X722, it supports write back on ITR
 	 * without binding queue to interrupt vector.
 	 */