On Thu, Aug 29, 2019 at 04:30:40PM +1000, Dave Chinner wrote:
> + /*
> + * Now we know if we must allocate blocks, so if we are checking whether
> + * we can insert without allocation then we can return now.
> + */
> + if (args->op_flags & XFS_DA_OP_JUSTCHECK) {
> + if (dbno != -1)
> + return 0;
> + return -ENOSPC;
> + }
Nit: I'd invert the check to rturn -ENOSPC in the branch if dbno is
-1 to make the flow a littler easier.
Otherwise this looks great and makes the code much easier to read:
Reviewed-by: Christoph Hellwig <hch@xxxxxx>
