<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:SimSun;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:DengXian;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"\@DengXian";
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"\@SimSun";
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
text-align:justify;
text-justify:inter-ideograph;
font-size:10.5pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}
p.MsoPlainText, li.MsoPlainText, div.MsoPlainText
{mso-style-priority:99;
mso-style-link:"Plain Text Char";
margin:0cm;
font-size:10.5pt;
font-family:"Calibri",sans-serif;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri",sans-serif;
color:windowtext;}
span.PlainTextChar
{mso-style-name:"Plain Text Char";
mso-style-priority:99;
mso-style-link:"Plain Text";
font-family:"Calibri",sans-serif;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri",sans-serif;}
.MsoPapDefault
{mso-style-type:export-only;
text-align:justify;
text-justify:inter-ideograph;}
/* Page Definitions */
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="ZH-CN" link="#0563C1" vlink="#954F72" style="word-wrap:break-word;text-justify-trim:punctuation">
<div class="WordSection1">
<p class="MsoNormal" align="left" style="margin-bottom:7.5pt;text-align:left;background:white;word-break:break-all">
<span lang="EN-US" style="color:black">> -----Original Message-----</span><span lang="EN-US" style="font-size:10.0pt;font-family:Consolas;color:#333333"><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> From: Mingjin Ye <<a href="mailto:mingjinx.ye@intel.com">mingjinx.ye@intel.com</a>>
<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Sent: Tuesday, April 11, 2023 10:09 AM<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> To: <a href="mailto:dev@dpdk.org">dev@dpdk.org</a><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Cc: Yang, Qiming <<a href="mailto:qiming.yang@intel.com">qiming.yang@intel.com</a>>;
<a href="mailto:stable@dpdk.org">stable@dpdk.org</a>; Zhou, YidingX <<a href="mailto:yidingx.zhou@intel.com">yidingx.zhou@intel.com</a>>; Ye, MingjinX <<a href="mailto:mingjinx.ye@intel.com">mingjinx.ye@intel.com</a>>; Zhang, Ke1X <<a href="mailto:ke1x.zhang@intel.com">ke1x.zhang@intel.com</a>>;
Zhang, Qi Z <<a href="mailto:qi.z.zhang@intel.com">qi.z.zhang@intel.com</a>><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Subject: [PATCH v7] net/ice: fix ice dcf control thread crash<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">><o:p> </o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> The control thread accesses the hardware resources after the resources were released, which results in a segment error.<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">><o:p> </o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> The 'ice-reset' threads are detached, so thread resources cannot be reclaimed by `pthread_join` calls.<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">><o:p> </o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> This commit synchronizes the number of "ice-reset" threads by adding a variable ("vsi_update_thread_num") to the "struct ice_dcf_hw" and performing an atomic operation on this variable. When releasing HW resources,
we wait for the number of "ice-reset" threads to be reduced to 0 before releasing the resources.<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">><o:p> </o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Fixes: c7e1a1a3bfeb ("net/ice: refactor DCF VLAN handling")<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Fixes: 3b3757bda3c3 ("net/ice: get VF hardware index in DCF")<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Fixes: 7564d5509611 ("net/ice: add DCF hardware initialization")<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Fixes: 0b02c9519432 ("net/ice: handle PF initialization by DCF")<o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Cc: <a href="mailto:stable@dpdk.org">
stable@dpdk.org</a><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">><o:p> </o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Signed-off-by: Ke Zhang <<a href="mailto:ke1x.zhang@intel.com">ke1x.zhang@intel.com</a>><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">> Signed-off-by: Mingjin Ye <<a href="mailto:mingjinx.ye@intel.com">mingjinx.ye@intel.com</a>><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US">Tested-by: </span><span lang="EN-US" style="font-size:11.0pt">Tingting Liao <<a href="mailto:tingtingx.liao@intel.com">tingtingx.liao@intel.com</a>></span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoPlainText"><span lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
</div>
</body>
</html>