Dave Chinner | 1 Feb 2012 22:13

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

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 <at> fromorbit.com

_______________________________________________
xfs mailing list
xfs <at> oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs


Gmane