On Mon, 15 Sep 2025 09:30:03 +0500 Sabyrzhan Tasbolatov <snovit...@gmail.com>
wrote:
> On Wed, Sep 3, 2025 at 6:01 PM Andrey Konovalov <andreyk...@gmail.com> wrote:
>
[400+ lines removed - people, please have mercy]
>
> > > @@ -246,7 +255,7 @@ static inline void poison_slab_object(struct
> > > kmem_cache *cache, void *object,
> > > bool __kasan_slab_pre_free(struct kmem_cache *cache, void *object,
> > > unsigned long ip)
> > > {
> > > - if (!kasan_arch_is_ready() || is_kfence_address(object))
> > > + if (is_kfence_address(object))
> > > return false;
> >
> > Why is the check removed here and in some other places below? This
> > need to be explained in the commit message.
>
> kasan_arch_is_ready which was unified with kasan_enabled, was removed
> here because
> __kasan_slab_pre_free is called from include/linux/kasan.h [1] where
> there's already kasan_enabled() check.
>
> [1] https://elixir.bootlin.com/linux/v6.16.7/source/include/linux/kasan.h#L198
>
> Please let me know if v7 is required with the change in the git commit
> message only.
Neither works - please send along the appropriate paragraph and I'll
paste it in, can't get easier than that.
> >
>
> [another ~250 lines snipped]
>
