On Mon, Mar 25, 2024 at 04:55:43PM -0700, Darrick J. Wong wrote:
> > + /*
> > + * Steal as many blocks as we can to try and satisfy the worst
> > + * case indlen for both new extents.
> > + */
> > + da_new = got_indlen + new_indlen;
> > + if (da_new > da_old) {
> > + stolen = XFS_FILBLKS_MIN(da_new - da_old,
> > + new.br_blockcount);
>
> Huh. We used to pass del->blockcount as one of the constraints on the
> stolen block count. Why pass new.br_blockcount instead?
.. we probably shouldn't. Let me take another closer look.
