On Fri, Sep 7, 2018 at 6:06 PM, Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx> wrote: > > > On 08/29/2018 02:35 PM, Andrey Konovalov wrote: > >> 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!