On Fri, Sep 7, 2018 at 6:06 PM, Andrey Ryabinin [off-list ref] wrote:

On 08/29/2018 02:35 PM, Andrey Konovalov wrote:

quoted

 void kasan_poison_slab(struct page *page)
 {
+     unsigned long i;
+
+     if (IS_ENABLED(CONFIG_SLAB))
+             page->s_mem = reset_tag(page->s_mem);

Why reinitialize here, instead of single initialization in alloc_slabmgmt()?

Hm, don't see why I did it this way, looks odd to me as well. Will fix
in v7, thanks!