> 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. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>