Re: [PATCH v4 2/8] eal: fixes for re-initialization issues

2023-08-15 Thread Sinan Kaya
On Tue, 2023-08-15 at 10:49 -0700, Stephen Hemminger wrote: > > > > + __atomic_store_n(&run_once, 0, __ATOMIC_RELAXED); > >return 0; > > > Interesting, other flags don't use atomic. Why here? > > > > And is already set elsewhere? Looking at the history, this variable used to be

Re: [PATCH v4 2/8] eal: fixes for re-initialization issues

2023-08-15 Thread Stephen Hemminger
On Tue, 15 Aug 2023 10:50:17 -0400 ok...@kernel.org wrote: > +static uint32_t run_once; > + > int > eal_clean_runtime_dir(void) > { > @@ -505,6 +507,7 @@ eal_parse_socket_arg(char *strval, volatile uint64_t > *socket_arg) > socket_arg[i] = val; > } > > + __atomic_stor

[PATCH v4 2/8] eal: fixes for re-initialization issues

2023-08-15 Thread okaya
From: Graham Whyte reinitialize the solib link list and clean the globals holding state for parsing. Signed-off-by: Sinan Kaya Signed-off-by: Graham Whyte --- lib/eal/common/eal_common_options.c | 8 lib/eal/linux/eal.c | 4 +++- 2 files changed, 11 insertions(+), 1 d