aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ext4/mballoc.c
diff options
context:
space:
mode:
authorAneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>2009-01-05 21:34:30 -0500
committerTheodore Ts'o <tytso@mit.edu>2009-01-05 21:34:30 -0500
commit032115fcef837a00336ddf7bda584e89789ea498 (patch)
tree3c36bca32a6590e675146638992ef988a0fbb430 /fs/ext4/mballoc.c
parentfde4d95ad8711c84a36735a17136c45b19746af9 (diff)
ext4: Don't overwrite allocation_context ac_status
We can call ext4_mb_check_limits even after successfully allocating the requested blocks. In that case, make sure we don't overwrite ac_status if it already has the status AC_STATUS_FOUND. This fixes the lockdep warning: ============================================= [ INFO: possible recursive locking detected ] 2.6.28-rc6-autokern1 #1 --------------------------------------------- fsstress/11948 is trying to acquire lock: (&meta_group_info[i]->alloc_sem){----}, at: [<c04d9a49>] ext4_mb_load_buddy+0x9f/0x278 ..... stack backtrace: ..... [<c04db974>] ext4_mb_regular_allocator+0xbb5/0xd44 ..... but task is already holding lock: (&meta_group_info[i]->alloc_sem){----}, at: [<c04d9a49>] ext4_mb_load_buddy+0x9f/0x278 Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu> Cc: stable@kernel.org
Diffstat (limited to 'fs/ext4/mballoc.c')
-rw-r--r--fs/ext4/mballoc.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/ext4/mballoc.c b/fs/ext4/mballoc.c
index 48d606cd740f..6dea637b020a 100644
--- a/fs/ext4/mballoc.c
+++ b/fs/ext4/mballoc.c
@@ -1326,6 +1326,8 @@ static void ext4_mb_check_limits(struct ext4_allocation_context *ac,
1326 struct ext4_free_extent ex; 1326 struct ext4_free_extent ex;
1327 int max; 1327 int max;
1328 1328
1329 if (ac->ac_status == AC_STATUS_FOUND)
1330 return;
1329 /* 1331 /*
1330 * We don't want to scan for a whole year 1332 * We don't want to scan for a whole year
1331 */ 1333 */