On Mon, Oct 07, 2019 at 11:45:36AM +0100, Mark Rutland wrote: > Both multi_cpu_stop() and set_state() access multi_stop_data::state > racily using plain accesses. These are subject to compiler > transformations which could break the intended behaviour of the code, > and this situation is detected by KCSAN on both arm64 and x86 (splats > below).
I really don't think there is anything the compiler can do wrong here. That is, I'm thinking I'd like to get this called out as a false-positive. That said, the patch looks obviously fine and will help with the validation effort so no real objection there.