On 05.06.22 17:25, Daniel Vetter wrote: > It only does a might_sleep_if(GFP_RECLAIM) check, which is already > covered by the might_alloc() in slab_pre_alloc_hook(). And all callers > of cache_alloc_debugcheck_before() call that beforehand already. > > Signed-off-by: Daniel Vetter <daniel.vet...@intel.com> > Cc: Christoph Lameter <c...@linux.com> > Cc: Pekka Enberg <penb...@kernel.org> > Cc: David Rientjes <rient...@google.com> > Cc: Joonsoo Kim <iamjoonsoo....@lge.com> > Cc: Andrew Morton <a...@linux-foundation.org> > Cc: Vlastimil Babka <vba...@suse.cz> > Cc: Roman Gushchin <roman.gushc...@linux.dev> > Cc: linux...@kvack.org > ---
LGTM Reviewed-by: David Hildenbrand <da...@redhat.com> -- Thanks, David / dhildenb