<div dir="auto"><div>Hi Stephen,<br><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Sep 12, 2025, 04:51 Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Ok, the Windows #ifdef was confusing me then looked more closely.<br>
This patch is adding a set of rings so that primary and secondary<br>
can communicate, the adding one command.<br>
<br>
The idea is good, but there is a better way to handle this.<br>
There already exists a way for primary and secondary to communicate<br>
through the mp service. This is used for hotplug and pdump and probably<br>
other things as well.<br></blockquote></div></div><div dir="auto">Yeah, agree. Testpmd have also hutplug callbacks, I have tried that too, but it is in lower layer to call any stopping fwd_engines. </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"><br>
The communication can be either way, for example I proposed patches<br>
to pdump so that primary can tell secondary to participate.<br>
<br>
<a href="https://patchwork.dpdk.org/project/dpdk/patch/20250814165307.12786-7-stephen@networkplumber.org/" rel="noreferrer noreferrer" target="_blank">https://patchwork.dpdk.org/project/dpdk/patch/20250814165307.12786-7-stephen@networkplumber.org/</a><br>
<br>
This of testpmd can be done in similar way.<br>
The handler in secondary should be able to act same as<br>
when SIGINT is received. Set the flag f_exit which will cause the main<br>
loop to exit.<br>
<br>
This can then happen immediately, and the proc monitor alarm function<br>
is only then needed to handle when primary process crashes.<br></blockquote></div></div><div dir="auto">Yeah, we can try that, my only concern is that if it can handle and stop any numbers of secondary processes fwd_engines as the current solution can stop. </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"><br>
<br>
</blockquote></div></div></div>