Re: [PATCH] riscv/fence: Consolidate fence definitions and define __{mb,rmb,wmb}

From: Eric Chan
Date: Tue Feb 13 2024 - 10:06:23 EST


Hi Conor,

Thank you, I should not have just used defconfig without allesconfig.
It has been corrected in PATCH v2 [0] as follows.

v1 -> v2: makes compilation pass with allyesconfig instead of
defconfig only, also satisfy scripts/checkpatch.pl.
- (__asm__ __volatile__ (RISCV_FENCE_ASM(p, s) : : : "memory"))
+ ({ __asm__ __volatile__ (RISCV_FENCE_ASM(p, s) : : : "memory"); })

I've updated patch v3[https://marc.info/?l=linux-kernel&m=170783431129245]
Please let me know if you have any further questions or suggestions.

[0] https://lore.kernel.org/lkml/20240212105946.1241100-1-ericchancf@xxxxxxxxxx/

Sincerely,
Eric Chan