<div dir="ltr"><div dir="ltr">Hi, <div><br></div><div>Is this tested on VFs? I saw crash on both PF as well as VF, on 20.11.1, but did not see PF crash on 20.11.3, </div><div>I'm seeing a crash on the secondary process when it receives packet on the VF, but PF doesn't crash now on 20.11.3</div><div><br></div><div>So I suspect this is the patch which is fixing the issue on ICE secondary process crash on PF.</div><div><br></div><div>Thanks & Regards,</div><div>Navin Srinivas</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 3, 2021 at 3:51 PM Zhang, Qi Z <<a href="mailto:qi.z.zhang@intel.com">qi.z.zhang@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"><br>
<br>
> -----Original Message-----<br>
> From: Wang, Yixue <<a href="mailto:yixue.wang@intel.com" target="_blank">yixue.wang@intel.com</a>><br>
> Sent: Thursday, June 3, 2021 6:04 PM<br>
> To: Zhang, Qi Z <<a href="mailto:qi.z.zhang@intel.com" target="_blank">qi.z.zhang@intel.com</a>>; Yang, Qiming<br>
> <<a href="mailto:qiming.yang@intel.com" target="_blank">qiming.yang@intel.com</a>><br>
> Cc: Zhang, Liheng <<a href="mailto:liheng.zhang@intel.com" target="_blank">liheng.zhang@intel.com</a>>; Dong, Yao<br>
> <<a href="mailto:yao.dong@intel.com" target="_blank">yao.dong@intel.com</a>>; <a href="mailto:dev@dpdk.org" target="_blank">dev@dpdk.org</a>; <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
> Subject: RE: [PATCH] net/ice: fix wrong data path selection in secondary<br>
> process<br>
> <br>
> Hi, Qi<br>
> <br>
> I've tested this patch and it works.<br>
> <br>
> Best Regards,<br>
> Yixue.<br>
> <br>
> > -----Original Message-----<br>
> > From: Zhang, Qi Z <<a href="mailto:qi.z.zhang@intel.com" target="_blank">qi.z.zhang@intel.com</a>><br>
> > Sent: Monday, May 24, 2021 17:08<br>
> > To: Yang, Qiming <<a href="mailto:qiming.yang@intel.com" target="_blank">qiming.yang@intel.com</a>><br>
> > Cc: Zhang, Liheng <<a href="mailto:liheng.zhang@intel.com" target="_blank">liheng.zhang@intel.com</a>>; Wang, Yixue<br>
> > <<a href="mailto:yixue.wang@intel.com" target="_blank">yixue.wang@intel.com</a>>; Dong, Yao <<a href="mailto:yao.dong@intel.com" target="_blank">yao.dong@intel.com</a>>; <a href="mailto:dev@dpdk.org" target="_blank">dev@dpdk.org</a>;<br>
> > Zhang, Qi Z <<a href="mailto:qi.z.zhang@intel.com" target="_blank">qi.z.zhang@intel.com</a>>; <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
> > Subject: [PATCH] net/ice: fix wrong data path selection in secondary<br>
> > process<br>
> ><br>
> > The flag use_avx2 and use_avx512 are defined as local variables, they<br>
> > will not be aware by the secondary process, then wrong data path is<br>
> > selected. Fix the issue by moving them into struct ice_adapter.<br>
> ><br>
> > Fixes: ae60d3c9b227 ("net/ice: support Rx AVX2 vector")<br>
> > Fixes: 2d5f6953d56d ("net/ice: support vector AVX2 in Tx")<br>
> > Fixes: 7f85d5ebcfe1 ("net/ice: add AVX512 vector path")<br>
> > Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
> ><br>
> > Reported-by: Yixue Wang <<a href="mailto:yixue.wang@intel.com" target="_blank">yixue.wang@intel.com</a>><br>
> > Signed-off-by: Qi Zhang <<a href="mailto:qi.z.zhang@intel.com" target="_blank">qi.z.zhang@intel.com</a>><br>
<br>
Tested-by: Yixue Wang <<a href="mailto:yixue.wang@intel.com" target="_blank">yixue.wang@intel.com</a>><br>
<br>
Applied to dpdk-next-net-intel.<br>
<br>
Thanks<br>
Qi<br>
<br>
</blockquote></div></div>