Re: [PATCH v3 1/2] netfs: do not unlock and put the folio twice

[David Howells <dhowells@xxxxxxxxxx>]

xiubli@xxxxxxxxxx wrote:

> -error:
> +error_locked:
>  	folio_unlock(folio);
>  	folio_put(folio);
> +error_unlocked:

I would do:

	error:
		if (folio) {
			folio_unlock(folio);
			folio_put(folio);
		}

David