On Mon, Oct 21, 2024 at 2:12 PM Clement LE GOFFIC <clement.legoffic@xxxxxxxxxxx> 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
