[PATCH next] drm/i915: Fix NULL vs IS_ERR() check in __shmem_writeback()

Andi Shyti andi.shyti at linux.intel.com
Thu Feb 6 11:16:21 UTC 2025


> The filemap_lock_folio() function doesn't return NULL, it returns error
> pointers.
> 
> Fixes: 25dd342f0cc8 ("drm/i915/gem: convert __shmem_writeback() to folios")
> Signed-off-by: Dan Carpenter <dan.carpenter at linaro.org>

...

>  		struct folio *folio;
>  
>  		folio = filemap_lock_folio(mapping, i);
> -		if (!folio)
> +		if (IS_ERR(folio))

ops! Thanks, Dan

Reviewed-by: Andi Shyti <andi.shyti at linux.intel.com>

Andi


More information about the dri-devel mailing list