<div dir="ltr"><div><div style="font-family:verdana,sans-serif;color:rgb(103,78,167)" class="gmail_default">Sure Ciara. Thanks.</div><div style="font-family:verdana,sans-serif;color:rgb(103,78,167)" class="gmail_default"><br></div><div style="font-family:verdana,sans-serif;color:rgb(103,78,167)" class="gmail_default">I will raise a fresh new patch for bug Id: 1735. That would be better.</div><br clear="all"></div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><span style="font-family:verdana,sans-serif">Thanks</span></div><div><span style="font-family:verdana,sans-serif">Amiya</span><br></div></div></div></div><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, 3 Jul 2025 at 13:48, Loftus, Ciara <<a href="mailto:ciara.loftus@intel.com">ciara.loftus@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Subject: [PATCH v3] net/iavf: fix VLAN offload strip flag<br>
> <br>
> For i40e kernel drivers which support either vlan(v1) or vlan(v2)<br>
> VIRTCHNL OP,it will set strip on when setting filter on. But dpdk<br>
> side will not change strip flag. To be consistent with dpdk side,<br>
> explicitly disable strip again.<br>
> <br>
> Bugzilla ID:1725<br>
> Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
> <br>
> v3:<br>
> - Fixed the vlan(v2) scenario by calling appropriate api<br>
> - Addresses Bugzilla ID: 1735<br>
> v2:<br>
> - Fixed indentation in commit message<br>
> <br>
> Signed-off-by: Amiya Ranjan Mohakud <<a href="mailto:amiyaranjan.mohakud@gmail.com" target="_blank">amiyaranjan.mohakud@gmail.com</a>><br>
<br>
Hi Amiya,<br>
<br>
Thanks for the patch.<br>
Since your first patch was already merged into the main branch, this patch is considered a new patch (v1).<br>
If you submit another revision, it should be labelled v2.<br>
<br>
Since it's a new patch, it needs a new commit message. Something like:<br>
<br>
net/iavf: fix VLAN strip disabling for v2 capability<br>
<br>
Ensure the correct virtchnl op is called for disabling vlan stripping by checking if the device supports either v1 or v2 vlan capability and choosing the op accordingly.<br>
<br>
You should include the below line in the commit message which indicates it is a fix for the first patch:<br>
Fixes: 3bfad066f9b4 ("net/iavf: fix VLAN strip setting after enabling filter")<br>
<br>
Thanks,<br>
Ciara<br>
<br>
> ---<br>
>  drivers/net/intel/iavf/iavf_ethdev.c | 6 +++++-<br>
>  1 file changed, 5 insertions(+), 1 deletion(-)<br>
> <br>
> diff --git a/drivers/net/intel/iavf/iavf_ethdev.c<br>
> b/drivers/net/intel/iavf/iavf_ethdev.c<br>
> index c33fdd9069..335a8126c4 100644<br>
> --- a/drivers/net/intel/iavf/iavf_ethdev.c<br>
> +++ b/drivers/net/intel/iavf/iavf_ethdev.c<br>
> @@ -1388,6 +1388,7 @@ iavf_disable_vlan_strip_ex(struct rte_eth_dev<br>
> *dev, int on)<br>
>        */<br>
>       struct iavf_adapter *adapter =<br>
>               IAVF_DEV_PRIVATE_TO_ADAPTER(dev->data->dev_private);<br>
> +     struct iavf_info *vf = IAVF_DEV_PRIVATE_TO_VF(adapter);<br>
>       struct rte_eth_conf *dev_conf = &dev->data->dev_conf;<br>
>       int err;<br>
> <br>
> @@ -1395,7 +1396,10 @@ iavf_disable_vlan_strip_ex(struct rte_eth_dev<br>
> *dev, int on)<br>
>           adapter->hw.mac.type == IAVF_MAC_VF ||<br>
>           adapter->hw.mac.type == IAVF_MAC_X722_VF) {<br>
>               if (on && !(dev_conf->rxmode.offloads &<br>
> RTE_ETH_RX_OFFLOAD_VLAN_STRIP)) {<br>
> -                     err = iavf_disable_vlan_strip(adapter);<br>
> +                     if (vf->vf_res->vf_cap_flags &<br>
> VIRTCHNL_VF_OFFLOAD_VLAN_V2)<br>
> +                             err = iavf_config_vlan_strip_v2(adapter, false);<br>
> +                     else<br>
> +                             err = iavf_disable_vlan_strip(adapter);<br>
>                       if (err)<br>
>                               return -EIO;<br>
>               }<br>
> --<br>
> 2.39.5 (Apple Git-154)<br>
<br>
</blockquote></div>