On Wed, Sep 21, 2011 at 01:45:07PM -0500, Alex Elder wrote: > Probably wouldn't hurt to initialize b_fsprivate2 even > if the transaction pointer is null. Looks good though. It would - given that another transaction could be set in there. Fairly unlikely in userspace, but quite possible in the kernel. _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs