On Thu, Mar 15, 2012 at 05:08:00AM -0400, Christoph Hellwig wrote: > > + xfs_buf_hold(agibp); > > + xfs_buf_relse(agibp); > > This is a very awkward way to write > > xfs_buf_unlock(agibp); /me slaps forehead. -- Dave Chinner david at fromorbit.com