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@xxxxxxx> SUSE Labs, CR _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs