[PATCH 2/2] xfs: fix endian conversion issue in discard code

Ben Myers bpm at sgi.com
Fri Jan 6 09:38:35 CST 2012


On Fri, Jan 06, 2012 at 10:12:54AM -0500, Christoph Hellwig wrote:
> On Tue, Dec 20, 2011 at 10:36:38PM -0600, Ben Myers wrote:
> > On Wed, Dec 21, 2011 at 11:07:42AM +1100, Dave Chinner wrote:
> > > From: Dave Chinner <dchinner at redhat.com>
> > > 
> > > When finding the longest extent in an AG, we read the value directly
> > > out of the AGF buffer without endian conversion. This will give an
> > > incorrect length, resulting in FITRIM operations potentially not
> > > trimming everything that it should.
> > 
> > Looks good to me.
> > 
> > > Signed-off-by: Dave Chinner <dchinner at redhat.com>
> > 
> > Reviewed-by: Ben Myers <bpm at sgi.com>
> 
> This is a pretty serious thing, can you send it to Linus as soon as
> he takes the current pull request?

yep.




More information about the xfs mailing list