21 Oct
2024
21 Oct
'24
12:46 p.m.
On Mon, Oct 21, 2024 at 2:12 PM Clement LE GOFFIC clement.legoffic@foss.st.com wrote:
I saw your email about Melon's patch targeting the same subject. If we don't enable KASAN either you patch or Melon's one do not compile.
[...]
if (IS_ENABLED(CONFIG_KASAN_VMALLOC))[...]
Should be replaced with an #ifdef directive. `kasan_mem_to_shadow` symbol is hiden behind :
include/linux/kasan.h:32:#if defined(CONFIG_KASAN_GENERIC) || defined(CONFIG_KASAN_SW_TAGS)
So symbol doesn't exist without KASAN enabled.
Yeah sorry for missing this. :(
The absence of stubs in the Kasan header makes it necessary to rely on ifdefs.
I will fold the ideas from Melon's patch into mine and also develop a version that works with ifdefs.
Yours, Linus Walleij