[patch] add module_arch_cleanup() and improve module debugging output

[David Mosberger]

Rusty,

The patch below updates the other platforms with
module_arch_cleanup(). Also, I added more debug output to
kernel/module.c since I found it useful to be able to see the final
section layout.

--david

diff -Nru a/kernel/module.c b/kernel/module.c
--- a/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -909,6 +909,9 @@
list_del(&mod->list);
spin_unlock_irq(&modlist_lock);

+ /* Arch-specific cleanup. */
+ module_arch_cleanup(mod);
+
/* Module unload stuff */
module_unload_free(mod);

@@ -1241,6 +1244,7 @@
mod->module_init = ptr;

/* Transfer each section which specifies SHF_ALLOC */
+ DEBUGP("final section addresses:\n");
for (i = 0; i < hdr->e_shnum; i++) {
void *dest;

@@ -1258,6 +1262,7 @@
sechdrs[i].sh_size);
/* Update sh_addr to point to copy in image. */
sechdrs[i].sh_addr = (unsigned long)dest;
+ DEBUGP("\t0x%lx %s\n", sechdrs[i].sh_addr, secstrings + sechdrs[i].sh_name);
}
/* Module has been moved. */
mod = (void *)sechdrs[modindex].sh_addr;
diff -Nru a/include/linux/moduleloader.h b/include/linux/moduleloader.h
--- a/include/linux/moduleloader.h Wed Apr 9 14:49:49 2003
+++ b/include/linux/moduleloader.h Wed Apr 9 14:49:49 2003
@@ -41,4 +41,7 @@
const Elf_Shdr *sechdrs,
struct module *mod);

+/* Any cleanup needed when module leaves. */
+void module_arch_cleanup(struct module *mod);
+
#endif
diff -Nru a/arch/alpha/kernel/module.c b/arch/alpha/kernel/module.c
--- a/arch/alpha/kernel/module.c Wed Apr 9 14:49:48 2003
+++ b/arch/alpha/kernel/module.c Wed Apr 9 14:49:48 2003
@@ -300,3 +300,8 @@
{
return 0;
}
+
+void
+module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/arm/kernel/module.c b/arch/arm/kernel/module.c
--- a/arch/arm/kernel/module.c Wed Apr 9 14:49:48 2003
+++ b/arch/arm/kernel/module.c Wed Apr 9 14:49:48 2003
@@ -159,3 +159,8 @@
{
return 0;
}
+
+void
+module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/i386/kernel/module.c b/arch/i386/kernel/module.c
--- a/arch/i386/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/arch/i386/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -110,3 +110,7 @@
{
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/parisc/kernel/module.c b/arch/parisc/kernel/module.c
--- a/arch/parisc/kernel/module.c Wed Apr 9 14:49:50 2003
+++ b/arch/parisc/kernel/module.c Wed Apr 9 14:49:50 2003
@@ -568,3 +568,7 @@
#endif
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/ppc/kernel/module.c b/arch/ppc/kernel/module.c
--- a/arch/ppc/kernel/module.c Wed Apr 9 14:49:48 2003
+++ b/arch/ppc/kernel/module.c Wed Apr 9 14:49:48 2003
@@ -269,3 +269,7 @@
{
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/ppc64/kernel/module.c b/arch/ppc64/kernel/module.c
--- a/arch/ppc64/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/arch/ppc64/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -386,3 +386,7 @@
me->extable.entry + me->extable.num_entries);
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/s390/kernel/module.c b/arch/s390/kernel/module.c
--- a/arch/s390/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/arch/s390/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -348,3 +348,7 @@
kfree(me->arch.syminfo);
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/s390x/kernel/module.c b/arch/s390x/kernel/module.c
--- a/arch/s390x/kernel/module.c Wed Apr 9 14:49:50 2003
+++ b/arch/s390x/kernel/module.c Wed Apr 9 14:49:50 2003
@@ -374,3 +374,7 @@
kfree(me->arch.syminfo);
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/sparc/kernel/module.c b/arch/sparc/kernel/module.c
--- a/arch/sparc/kernel/module.c Wed Apr 9 14:49:48 2003
+++ b/arch/sparc/kernel/module.c Wed Apr 9 14:49:48 2003
@@ -145,3 +145,7 @@
{
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/sparc64/kernel/module.c b/arch/sparc64/kernel/module.c
--- a/arch/sparc64/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/arch/sparc64/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -273,3 +273,7 @@
{
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/v850/kernel/module.c b/arch/v850/kernel/module.c
--- a/arch/v850/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/arch/v850/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -230,3 +230,8 @@

return 0;
}
+
+void
+module_arch_cleanup(struct module *mod)
+{
+}
diff -Nru a/arch/x86_64/kernel/module.c b/arch/x86_64/kernel/module.c
--- a/arch/x86_64/kernel/module.c Wed Apr 9 14:49:49 2003
+++ b/arch/x86_64/kernel/module.c Wed Apr 9 14:49:49 2003
@@ -231,3 +231,7 @@
{
return 0;
}
+
+void module_arch_cleanup(struct module *mod)
+{
+}
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

[Rusty Russell]

In message <200304092154....@napali.hpl.hp.com> you write:
> Rusty,
>
> The patch below updates the other platforms with
> module_arch_cleanup(). Also, I added more debug output to
> kernel/module.c since I found it useful to be able to see the final
> section layout.

Yep, looks great.

Linus, please apply if you haven't already.

Thanks,
Rusty.
--
Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

Name: module_arch_cleanup hook
Author: David Mosberger
Status: Trivial

D: The patch below updates the other platforms with
D: module_arch_cleanup(). Also, I added more debug output to
D: kernel/module.c since I found it useful to be able to see the final
D: section layout.