[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <200711292052.lATKqmNQ006825@imap1.linux-foundation.org>
Date: Thu, 29 Nov 2007 12:52:48 -0800
From: akpm@...ux-foundation.org
To: cmm@...ibm.com
Cc: tytso@....edu, linux-ext4@...r.kernel.org,
akpm@...ux-foundation.org, sandeen@...hat.com
Subject: [patch 1/8] ext4: fix MB_DEBUG format warnings
From: Eric Sandeen <sandeen@...hat.com>
(applies at end of current ext4 patch series)
Fix a slew of compile warnings from mismatched format specifiers
when MB_DEBUG is turned on.
Signed-off-by: Eric Sandeen <sandeen@...hat.com>
Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
---
fs/ext4/mballoc.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff -puN fs/ext4/mballoc.c~ext4-fix-mb_debug-format-warnings fs/ext4/mballoc.c
--- a/fs/ext4/mballoc.c~ext4-fix-mb_debug-format-warnings
+++ a/fs/ext4/mballoc.c
@@ -954,7 +954,7 @@ static int ext4_mb_init_cache(struct pag
get_bh(bh[i]);
bh[i]->b_end_io = end_buffer_read_sync;
submit_bh(READ, bh[i]);
- mb_debug("read bitmap for group %u\n", first_group + i);
+ mb_debug("read bitmap for group %lu\n", first_group + i);
}
/* wait for I/O completion */
@@ -2670,7 +2670,7 @@ static void ext4_mb_free_committed_block
if (md == NULL)
break;
- mb_debug("gonna free %u blocks in group %u (0x%p):",
+ mb_debug("gonna free %u blocks in group %lu (0x%p):",
md->num, md->group, md);
err = ext4_mb_load_buddy(sb, md->group, &e4b);
@@ -3029,7 +3029,7 @@ static void ext4_mb_normalize_group_requ
BUG_ON(lg == NULL);
ac->ac_g_ex.fe_len = EXT4_SB(sb)->s_mb_group_prealloc;
- mb_debug("#%u: goal %u blocks for locality group\n",
+ mb_debug("#%u: goal %lu blocks for locality group\n",
current->pid, ac->ac_g_ex.fe_len);
}
@@ -3255,7 +3255,7 @@ static void ext4_mb_use_inode_pa(struct
BUG_ON(pa->pa_free < len);
pa->pa_free -= len;
- mb_debug("use %lu/%lu from inode pa %p\n", start, len, pa);
+ mb_debug("use %llu/%lu from inode pa %p\n", start, len, pa);
}
/*
@@ -3282,7 +3282,7 @@ static void ext4_mb_use_group_pa(struct
* pa and think that it have enought free blocks if we
* don't update pa_free here right ?
*/
- mb_debug("use %lu/%lu from group pa %p\n", pa->pa_lstart-len, len, pa);
+ mb_debug("use %u/%u from group pa %p\n", pa->pa_lstart-len, len, pa);
}
/*
@@ -3513,7 +3513,7 @@ static int ext4_mb_new_inode_pa(struct e
pa->pa_deleted = 0;
pa->pa_linear = 0;
- mb_debug("new inode pa %p: %lu/%lu for %lu\n", pa,
+ mb_debug("new inode pa %p: %llu/%u for %u\n", pa,
pa->pa_pstart, pa->pa_len, pa->pa_lstart);
ext4_mb_use_inode_pa(ac, pa);
@@ -3569,7 +3569,7 @@ static int ext4_mb_new_group_pa(struct e
pa->pa_deleted = 0;
pa->pa_linear = 1;
- mb_debug("new group pa %p: %lu/%lu for %lu\n", pa,
+ mb_debug("new group pa %p: %llu/%u for %u\n", pa,
pa->pa_pstart, pa->pa_len, pa->pa_lstart);
ext4_mb_use_group_pa(ac, pa);
@@ -4072,8 +4072,8 @@ static int ext4_mb_initialize_context(st
* locality group. this is a policy, actually */
ext4_mb_group_or_file(ac);
- mb_debug("init ac: %u blocks @ %llu, goal %llu, flags %x, 2^%d, "
- "left: %llu/%llu, right %llu/%llu to %swritable\n",
+ mb_debug("init ac: %u blocks @ %u, goal %u, flags %x, 2^%d, "
+ "left: %u/%u, right %u/%u to %swritable\n",
(unsigned) ar->len, (unsigned) ar->logical,
(unsigned) ar->goal, ac->ac_flags, ac->ac_2order,
(unsigned) ar->lleft, (unsigned) ar->pleft,
@@ -4288,7 +4288,7 @@ static int ext4_mb_free_metadata(handle_
page_cache_get(e4b->bd_bitmap_page);
db->bb_md_cur = md;
db->bb_tid = handle->h_transaction->t_tid;
- mb_debug("new md 0x%p for group %u\n",
+ mb_debug("new md 0x%p for group %lu\n",
md, md->group);
} else {
kfree(md);
_
-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists