Re: [PATCH v2 5/8] buffer: Add kernel-doc for brelse() and __brelse()

From: Pankaj Raghav (Samsung)
Date: Wed Jan 10 2024 - 09:31:20 EST


> + * If all buffers on a folio have zero reference count, are clean
> + * and unlocked, and if the folio is clean and unlocked then

IIUC from your [PATCH 3/8], folio only needs to be unlocked to free the
buffers as try_to_free_buffers() will remove the dirty flag and "clean"
the folio?
So:
s/if folio is clean and unlocked/if folio is unlocked

> + * try_to_free_buffers() may strip the buffers from the folio in
> + * preparation for freeing it (sometimes, rarely, buffers are removed
> + * from a folio but it ends up not being freed, and buffers may later
> + * be reattached).
> + *
> + * Context: Any context.
> + */
> static inline void brelse(struct buffer_head *bh)
> {
> if (bh)
> --
> 2.43.0
>