On Thu, Aug 29, 2019 at 01:10:13AM -0700, Christoph Hellwig wrote:
> 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.
Fixed.
-Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
