[PATCH 6/7] xfs: replace xfs_mod_incore_sb_batched

Christoph Hellwig hch at infradead.org
Thu Feb 5 08:19:54 CST 2015


On Thu, Feb 05, 2015 at 09:10:44AM -0500, Brian Foster wrote:
> >  	/* apply remaining deltas */
> > +	spin_lock(&mp->m_sb_lock);
> >  	if (rtxdelta) {
> > -		error = xfs_mod_frextents(mp, rtxdelta);
> > +		error = xfs_sb_mod64(&mp->m_sb.sb_frextents, rtxdelta);
> 
> Any reason why we don't continue to use the xfs_mod_frextents() function
> introduced in the previous patch? Seems like we should be consistent one
> way or the other.

We're already under the sb_lock and would need another lock roundtrip to
avoid a deadlock in that case.  But mit might be worth to simply make
the real time extent counter a percpu one to behave similar to the other
counters that can be manipulated outside of the transaction commit code.



More information about the xfs mailing list