Re: [PATCH 5/7] riscv: Add fence to available_harts_lock

On Tue, Sep 8, 2020 at 2:17 AM Sean Anderson seanga2@gmail.com wrote:

Without these fences, it is perfectly valid for an out-of-order core to re-order memory accesses to outside of the available_harts_lock critical section.

The commit message should be reworded, because current codes do nothing wrong as "fence" instruction is used. What was done in this patch is using .aq / .rl to replace "fence".

Signed-off-by: Sean Anderson seanga2@gmail.com

arch/riscv/cpu/start.S | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

Regards, Bin