On Tue, 29 Jun 2021 at 15:00, Alexander Potapenko [off-list ref] wrote:
[...]

+       /*
+        * Skip DMA allocations. These must reside in the low memory, which we
+        * cannot guarantee.
+        */
+       if (flags & (__GFP_DMA | __GFP_DMA32) ||

I think we want braces around "flags & (...)", so that this becomes:

    if ((flags & (...)) ||

+           (s->flags & (SLAB_CACHE_DMA | SLAB_CACHE_DMA32)))
+               return NULL;

Thanks,
-- Marco