[PATCH 1/4] libxfs: handle read errors in libxfs_trans_read_buf

Alex Elder aelder at sgi.com
Wed Sep 21 15:13:26 CDT 2011


On Wed, 2011-09-21 at 15:49 -0400, Christoph Hellwig wrote:
> 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.
> 

You're right.	-Alex




More information about the xfs mailing list