Hi Thomas,

On Thu, Sep 11, 2025, 22:53 Stephen Hemminger <stephen@networkplumber.org>
wrote:

> On Thu, 11 Sep 2025 03:25:01 -0700
> Khadem Ullah <14pwcse1224@uetpeshawar.edu.pk> wrote:
>
> > When the secondary process testpmd application running any fwd_engine
> > and after that primary has exited will cause a crash.
> >
> > This patch forces secondary process forwarding lcores
> > in fwd_engine to stop before the primary process exits.
> >
> > Fixes: a550baf24af90f ("app/testpmd: support multi-process")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Khadem Ullah <14pwcse1224@uetpeshawar.edu.pk>
>
> NAK
> As you said, you did not test it on Windows, yet you added #ifdef
> for Windows support. Since Windows doesn't support secondary
> processes (yet) how could you.
>
> Adding Windows here has to wait until it can be tested.
>
Can you please check if my understanding is correct that #ifndef
RTE_EXEC_ENV_WINDOWS means only Linux based systems ?
I have added all the codes under this macro :)


Best regards,
Khadem

>