On Mon, Nov 25, 2024 at 09:28:37PM -0800, Christoph Hellwig wrote: > > if (test_bit(XFS_LI_IN_AIL, &lip->li_flags) && > > - ((lip->li_lsn == qip->qli_flush_lsn) || > > + ((lip->li_lsn == qlip->qli_flush_lsn) || > > Let's drop the superfluous inner braces here while you're at it. Done. > Otherwise looks good: > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> Thank you! --D