[PATCH 3/8] xfs: bulk AIL insertion during transaction commit

[Christoph Hellwig]

Generally looks good, a few minor comments below:

> +void
> +xfs_trans_committed_bulk(
> +	struct xfs_ail		*ailp,
> +	struct xfs_log_vec	*log_vector,
> +	xfs_lsn_t		commit_lsn,
> +	int			aborted)
> +{
> +#define LGIA_SIZE	32
> +	struct xfs_log_item	*lgia[LGIA_SIZE];

The lgia name seems a bit to obscure to me.  What about just log_items
as variable name, and LOG_ITEM_BATCH_SIZE or similar for the size?

> +			spin_lock(&ailp->xa_lock);
> +			xfs_trans_ail_update_bulk(ailp, lgia, LGIA_SIZE,
> +							commit_lsn);
> +			for (i = 0; i < LGIA_SIZE; i++)
> +				IOP_UNPIN(lgia[i], 0);

> +		spin_lock(&ailp->xa_lock);
> +		xfs_trans_ail_update_bulk(ailp, lgia, i, commit_lsn);
> +		for (j = 0; j < i; j++)
> +			IOP_UNPIN(lgia[j], 0);

It might be worth to factor these two little sniplets into a little
helper.

> + * Bulk update version of xfs_trans_ail_update.

That won't be a very useful comment anymore once xfs_trans_ail_update is
implemented as a wrapper around this function..

> +	struct xfs_log_item	**lgia,

Same naming comment here.