<div dir="auto"><div><div>Hi,</div><br><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Sep 11, 2025, 22:53 Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 11 Sep 2025 03:25:01 -0700<br>
Khadem Ullah <<a href="mailto:14pwcse1224@uetpeshawar.edu.pk" target="_blank" rel="noreferrer">14pwcse1224@uetpeshawar.edu.pk</a>> wrote:<br>
<br>
> When the secondary process testpmd application running any fwd_engine<br>
> and after that primary has exited will cause a crash.<br>
> <br>
> This patch forces secondary process forwarding lcores<br>
> in fwd_engine to stop before the primary process exits.<br>
> <br>
> Fixes: a550baf24af90f ("app/testpmd: support multi-process")<br>
> Cc: <a href="mailto:stable@dpdk.org" target="_blank" rel="noreferrer">stable@dpdk.org</a><br>
> <br>
> Signed-off-by: Khadem Ullah <<a href="mailto:14pwcse1224@uetpeshawar.edu.pk" target="_blank" rel="noreferrer">14pwcse1224@uetpeshawar.edu.pk</a>><br>
<br>
NAK<br>
As you said, you did not test it on Windows, yet you added #ifdef<br>
for Windows support. Since Windows doesn't support secondary<br>
processes (yet) how could you.<br>
<br>
Adding Windows here has to wait until it can be tested.<br></blockquote></div></div><div dir="auto">Can you please double check the code? </div><div dir="auto">I have used the following macro everywhere which means the code is only targeted to Linux. </div><div dir="auto">#ifndef RTE_EXEC_ENV_WINDOWS</div><div dir="auto"><br></div><div dir="auto">I will add a newer version to if you agree to add additional comments to further clarify that the solution is only targeted for Linux. </div><div dir="auto"><br></div><div dir="auto">Best regards, </div><div dir="auto">Khadem </div><div dir="auto"><div class="gmail_quote gmail_quote_container"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>