> +static inline struct xfs_buf *
> +xfs_buf_find_fast(
I'd probably name this xfs_buf_lookup as the _fast suffix is a little
weird.
> +/*
> + * Insert the new_bp into the hash table. This consumes the perag reference
> + * taken for the lookup.
> + */
> +static int
> +xfs_buf_find_insert(
> + struct xfs_buftarg *btp,
> + struct xfs_perag *pag,
> + struct xfs_buf *new_bp)
> +{
> /* No match found */
> if (!new_bp) {
> xfs_perag_put(pag);
> + XFS_STATS_INC(btp->bt_mount, xb_miss_locked);
> return -ENOENT;
> }
Having this condition here vs in the caller seems a bit odd, but
I'll see what the rest of the series brings.
