On Fri, 26 Feb 2021 16:30:47 -0800 Wei Wang wrote: > thread = READ_ONCE(napi->thread); > if (thread) { > + set_bit(NAPI_STATE_SCHED_THREADED, &napi->state); > wake_up_process(thread);
What about the version which checks RUNNING? As long as wake_up_process() implies a barrier I _think_ it should work as well. Am I missing some case, or did you decide to go with the simpler/safer approach?