On Thu 15-03-12 05:08:00, Christoph Hellwig wrote: > > + xfs_buf_hold(agibp); > > + xfs_buf_relse(agibp); > > This is a very awkward way to write > > xfs_buf_unlock(agibp); Hmm, looking into the functions you are right. I'll change this. Honza -- Jan Kara <jack at suse.cz> SUSE Labs, CR