All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] MIPS: init module specific mips_hi16_list to NULL
@ 2012-08-10 10:33 Tony Wu
  2012-08-10 10:38 ` John Crispin
  0 siblings, 1 reply; 2+ messages in thread
From: Tony Wu @ 2012-08-10 10:33 UTC (permalink / raw)
  To: ralf, linux-mips

MIPS: init module specific mips_hi16_list to NULL

mips_hi16_list was moved from global to mod_arch_specific. While
global, it was bss initialized automatically, when moved to
mod_arch_specific, we have to do the zero initialization.

Signed-off-by: Tony Wu <tung7970@gmail.com>
---
 arch/mips/kernel/module.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/mips/kernel/module.c b/arch/mips/kernel/module.c
index 8b29976..8e1fb80 100644
--- a/arch/mips/kernel/module.c
+++ b/arch/mips/kernel/module.c
@@ -276,6 +276,7 @@ int apply_relocate(Elf_Shdr *sechdrs, const char *strtab,
 	pr_debug("Applying relocate section %u to %u\n", relsec,
 	       sechdrs[relsec].sh_info);
 
+	me->arch.r_mips_hi16_list = NULL;
 	for (i = 0; i < sechdrs[relsec].sh_size / sizeof(*rel); i++) {
 		/* This is where to make the change */
 		location = (void *)sechdrs[sechdrs[relsec].sh_info].sh_addr
-- 
1.7.4.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] MIPS: init module specific mips_hi16_list to NULL
  2012-08-10 10:33 [PATCH] MIPS: init module specific mips_hi16_list to NULL Tony Wu
@ 2012-08-10 10:38 ` John Crispin
  0 siblings, 0 replies; 2+ messages in thread
From: John Crispin @ 2012-08-10 10:38 UTC (permalink / raw)
  To: linux-mips

On 10/08/12 12:33, Tony Wu wrote:
> MIPS: init module specific mips_hi16_list to NULL
> 
> mips_hi16_list was moved from global to mod_arch_specific. While
> global, it was bss initialized automatically, when moved to
> mod_arch_specific, we have to do the zero initialization.
> 
> Signed-off-by: Tony Wu <tung7970@gmail.com>


Hi Tony,

can you reference the offending commit that caused the regression inside
the patches description please ...

John

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-08-10 10:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-10 10:33 [PATCH] MIPS: init module specific mips_hi16_list to NULL Tony Wu
2012-08-10 10:38 ` John Crispin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.