Re: [PATCH v4 8/8] eal: initialize worker threads once

2023-08-15 Thread Sinan Kaya
On Tue, 2023-08-15 at 10:46 -0700, Stephen Hemminger wrote: > > static uint32_t run_once; > > +static bool worker_initialized; > > > > > I see a pattern here. Many places are using static to > > only initialize once. Could you name these variables the > > same; suggest using run_once every

Re: [PATCH v4 8/8] eal: initialize worker threads once

2023-08-15 Thread Stephen Hemminger
On Tue, 15 Aug 2023 10:50:23 -0400 ok...@kernel.org wrote: > From: Sinan Kaya > > Initialize worker threads once and keep a flag > for other init calls. > > Signed-off-by: Sinan Kaya > --- > lib/eal/linux/eal.c | 60 - > 1 file changed, 32 insertion

[PATCH v4 8/8] eal: initialize worker threads once

2023-08-15 Thread okaya
From: Sinan Kaya Initialize worker threads once and keep a flag for other init calls. Signed-off-by: Sinan Kaya --- lib/eal/linux/eal.c | 60 - 1 file changed, 32 insertions(+), 28 deletions(-) diff --git a/lib/eal/linux/eal.c b/lib/eal/linux/eal.c