On Tue, Nov 16, 2021 at 10:21 PM Honnappa Nagarahalli <honnappa.nagaraha...@arm.com> wrote: > > Joyce, Honnappa, > > > > On Tue, Nov 16, 2021 at 10:43 AM Joyce Kong <joyce.k...@arm.com> wrote: > > > > > > Convert rte_atomic usages to compiler atomic built-ins for lcore_state > > > and collisions sync. > > > > > > Also, move 'main_init_workers' outside of 'timer_stress2_main_loop' to > > > guarantee lcore_state initialized correctly before the threads > > > launched. > > > > Is this "also" part actually related to the change? > > Or is it a separate fix? > 'Also' part is not fixing a different problem (i.e. the code earlier was not > having any issues). This 'also' part just helps to keep the code simple.
This is indeed better this way. Thanks. -- David Marchand
