Re: [PATCH v3 4/6] xfs: move xfs_dialloc_roll() into xfs_dialloc()

Christoph Hellwig
2020-12-07T13:54:41Z

>
> if (ialloced) {
> /*
> + * We successfully allocated some inodes, roll the
> + * transaction so they can allocate one of the free
> + * inodes we just prepared for them.

Maybe:

/*
* We successfully allocated space for an inode cluster
* in this AG. Roll the transaction so that we can
* allocate one of the new inodes.
*/

Otherwise looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>