diff options
author | Chris Mason <chris.mason@oracle.com> | 2012-02-28 18:42:44 +0100 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2012-03-03 13:42:35 +0100 |
commit | a175423c831ea582c06784d1e172d2ce1d79923a (patch) | |
tree | 742ca97279da69c981008620a3d2f0c48acdf6f4 /fs | |
parent | btrfs: fix locking issues in find_parent_nodes() (diff) | |
download | linux-a175423c831ea582c06784d1e172d2ce1d79923a.tar.xz linux-a175423c831ea582c06784d1e172d2ce1d79923a.zip |
Btrfs: fix casting error in scrub reada code
The reada code from scrub was casting down a u64 to
an unsigned long so it could insert it into a radix tree.
What it really wanted to do was cast down the result of a shift, instead
of casting down the u64. The bug resulted in trying to insert our
reada struct into the wrong place, which caused soft lockups and other
problems.
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/reada.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/btrfs/reada.c b/fs/btrfs/reada.c index 2373b39a132b..22db04550f6a 100644 --- a/fs/btrfs/reada.c +++ b/fs/btrfs/reada.c @@ -305,7 +305,7 @@ again: spin_lock(&fs_info->reada_lock); ret = radix_tree_insert(&dev->reada_zones, - (unsigned long)zone->end >> PAGE_CACHE_SHIFT, + (unsigned long)(zone->end >> PAGE_CACHE_SHIFT), zone); spin_unlock(&fs_info->reada_lock); |