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