* XFS update for 3.0-stable
@ 2012-01-26 19:47 bpm
2012-01-26 19:47 ` [PATCH] xfs: fix endian conversion issue in discard code bpm
2012-01-27 0:47 ` XFS update for 3.0-stable Greg KH
0 siblings, 2 replies; 3+ messages in thread
From: bpm @ 2012-01-26 19:47 UTC (permalink / raw)
To: stable, xfs, bpm
Here's a fix for 3.0-stable related to trim support in xfs.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] xfs: fix endian conversion issue in discard code
2012-01-26 19:47 XFS update for 3.0-stable bpm
@ 2012-01-26 19:47 ` bpm
2012-01-27 0:47 ` XFS update for 3.0-stable Greg KH
1 sibling, 0 replies; 3+ messages in thread
From: bpm @ 2012-01-26 19:47 UTC (permalink / raw)
To: stable, xfs, bpm; +Cc: Dave Chinner
From: Dave Chinner <dchinner@redhat.com>
commit b1c770c273a4787069306fc82aab245e9ac72e9d upstream
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.
Note, for 3.0-stable this has been modified to apply to
fs/xfs/linux-2.6/xfs_discard.c instead of fs/xfs/xfs_discard.c. -bpm
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Ben Myers <bpm@sgi.com>
---
fs/xfs/linux-2.6/xfs_discard.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/xfs/linux-2.6/xfs_discard.c b/fs/xfs/linux-2.6/xfs_discard.c
index 244e797..572494f 100644
--- a/fs/xfs/linux-2.6/xfs_discard.c
+++ b/fs/xfs/linux-2.6/xfs_discard.c
@@ -68,7 +68,7 @@ xfs_trim_extents(
* Look up the longest btree in the AGF and start with it.
*/
error = xfs_alloc_lookup_le(cur, 0,
- XFS_BUF_TO_AGF(agbp)->agf_longest, &i);
+ be32_to_cpu(XFS_BUF_TO_AGF(agbp)->agf_longest), &i);
if (error)
goto out_del_cursor;
@@ -84,7 +84,7 @@ xfs_trim_extents(
if (error)
goto out_del_cursor;
XFS_WANT_CORRUPTED_GOTO(i == 1, out_del_cursor);
- ASSERT(flen <= XFS_BUF_TO_AGF(agbp)->agf_longest);
+ ASSERT(flen <= be32_to_cpu(XFS_BUF_TO_AGF(agbp)->agf_longest));
/*
* Too small? Give up.
--
1.7.8.rc4
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: XFS update for 3.0-stable
2012-01-26 19:47 XFS update for 3.0-stable bpm
2012-01-26 19:47 ` [PATCH] xfs: fix endian conversion issue in discard code bpm
@ 2012-01-27 0:47 ` Greg KH
1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2012-01-27 0:47 UTC (permalink / raw)
To: bpm; +Cc: stable, xfs
On Thu, Jan 26, 2012 at 01:47:41PM -0600, bpm@sgi.com wrote:
> Here's a fix for 3.0-stable related to trim support in xfs.
Applied, thanks.
greg k-h
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-01-27 0:48 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-26 19:47 XFS update for 3.0-stable bpm
2012-01-26 19:47 ` [PATCH] xfs: fix endian conversion issue in discard code bpm
2012-01-27 0:47 ` XFS update for 3.0-stable Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox