Messages in this thread Patch in this message |  | | From | Thomas Renninger <> | | Subject | [PATCH 4/4] kernel/module.c: Fix compiler warnings if debug is compiled in | | Date | Thu, 16 Sep 2010 00:11:48 +0200 |
| |
Fixes these warnings: kernel/module.c: In function ‘layout_sections’: kernel/module.c:1776: error: ‘name’ undeclared (first use in this function) kernel/module.c:1776: error: (Each undeclared identifier is reported only once kernel/module.c:1776: error: for each function it appears in.) kernel/module.c: In function ‘move_module’: kernel/module.c:2394: warning: format ‘%lx’ expects type ‘long unsigned int’, but argument 2 has type ‘Elf64_Addr’
if DEBUGP is defined in kernel/module.c
Signed-off-by: Thomas Renninger <trenn@suse.de> CC: Greg KH <gregkh@suse.de> CC: jbaron@redhat.com CC: lkml <linux-kernel@vger.kernel.org> --- kernel/module.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/kernel/module.c b/kernel/module.c index 3912e12..66e4361 100644 --- a/kernel/module.c +++ b/kernel/module.c @@ -1773,7 +1773,6 @@ static void layout_sections(struct module *mod, struct load_info *info) || strstarts(sname, ".init")) continue; s->sh_entsize = get_offset(mod, &mod->core_size, s, i); - DEBUGP("\t%s\n", name); } if (m == 0) mod->core_text_size = mod->core_size; @@ -2390,8 +2389,8 @@ static int move_module(struct module *mod, struct load_info *info) memcpy(dest, (void *)shdr->sh_addr, shdr->sh_size); /* Update sh_addr to point to copy in image. */ shdr->sh_addr = (unsigned long)dest; - DEBUGP("\t0x%lx %s\n", - shdr->sh_addr, info->secstrings + shdr->sh_name); + DEBUGP("\t0x%p %s\n", + (void *)shdr->sh_addr, info->secstrings + shdr->sh_name); } return 0; -- 1.6.0.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |