Re: [patch] xfs: remove an unneeded NULL check

Dave Chinner
2012-02-01T21:13:26Z

On Wed, Feb 01, 2012 at 05:45:01AM -0500, Christoph Hellwig wrote:
> On Wed, Feb 01, 2012 at 09:56:01AM +1100, Dave Chinner wrote:
> > > /* lock out background commit */
> > > down_read(&log->l_cilp->xc_ctx_lock);
> > > - if (commit_lsn)
> > > - *commit_lsn = log->l_cilp->xc_ctx->sequence;
> > > + *commit_lsn = log->l_cilp->xc_ctx->sequence;
> > >
> > > xlog_cil_insert_items(log, log_vector, tp->t_ticket);
> >
> > There's a set of reviewed patches (for 3.3) that change all this
> > code. The null check might still be there, but that needs to be
> > checked.
>
> Which series is that? I mut have to admit I've lost track by now.

The removal of all the old log code.

Cheers,

Dave.
--
Dave Chinner
david@fromorbit.com