On Mon, May 20, 2019 at 09:12:33AM -0400, Brian Foster wrote: > > + spin_unlock(&iclog->ic_callback_lock); > > + xlog_cil_process_commited(&tmp, aborted); > > s/commited/committed/ please. Ok. > > + while ((ctx = list_first_entry_or_null(list, > > Are double braces necessary here? Without them gcc is unhappy: fs/xfs/xfs_log_cil.c: In function ‘xlog_cil_process_commited’: fs/xfs/xfs_log_cil.c:624:9: warning: suggest parentheses around assignment used as truth value [-Wparentheses] while (ctx = list_first_entry_or_null(list, > > /* attach all the transactions w/ busy extents to iclog */ > > Any idea what this ^ comment means? ISTM it's misplaced or stale. If so, > we might as well toss/replace it. No idea. We can probbaly remove it.
