diff options
author | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2010-02-26 02:29:17 -0500 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2010-02-26 02:29:17 -0500 |
commit | 3d98ffbffb16f2a1569b83cb78db0b5100e6c937 (patch) | |
tree | 3604899600b8ce7c95d67b9299e1fb6b91005773 /arch/powerpc | |
parent | 874f2f997dbe041a6c6e509dae8656ed9022d65d (diff) |
powerpc: Fix lwsync feature fixup vs. modules on 64-bit
Anton's commit enabling the use of the lwsync fixup mechanism on 64-bit
breaks modules. The lwsync fixup section uses .long instead of the
FTR_ENTRY_OFFSET macro used by other fixups sections, and thus will
generate 32-bit relocations that our module loader cannot resolve.
This changes it to use the same type as other feature sections.
Note however that we might want to consider using 32-bit for all the
feature fixup offsets and add support for R_PPC_REL32 to module_64.c
instead as that would reduce the size of the kernel image. I'll leave
that as an exercise for the reader for now...
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'arch/powerpc')
-rw-r--r-- | arch/powerpc/include/asm/feature-fixups.h | 2 | ||||
-rw-r--r-- | arch/powerpc/lib/feature-fixups.c | 3 |
2 files changed, 3 insertions, 2 deletions
diff --git a/arch/powerpc/include/asm/feature-fixups.h b/arch/powerpc/include/asm/feature-fixups.h index cbd4dfa4bce2..96a7d067fbb2 100644 --- a/arch/powerpc/include/asm/feature-fixups.h +++ b/arch/powerpc/include/asm/feature-fixups.h | |||
@@ -165,7 +165,7 @@ label##2: \ | |||
165 | .pushsection sect,"a"; \ | 165 | .pushsection sect,"a"; \ |
166 | .align 2; \ | 166 | .align 2; \ |
167 | label##3: \ | 167 | label##3: \ |
168 | .long label##1b-label##3b; \ | 168 | FTR_ENTRY_OFFSET label##1b-label##3b; \ |
169 | .popsection; | 169 | .popsection; |
170 | 170 | ||
171 | #endif /* __ASM_POWERPC_FEATURE_FIXUPS_H */ | 171 | #endif /* __ASM_POWERPC_FEATURE_FIXUPS_H */ |
diff --git a/arch/powerpc/lib/feature-fixups.c b/arch/powerpc/lib/feature-fixups.c index 4dee652dfcad..e640175b65ae 100644 --- a/arch/powerpc/lib/feature-fixups.c +++ b/arch/powerpc/lib/feature-fixups.c | |||
@@ -112,7 +112,8 @@ void do_feature_fixups(unsigned long value, void *fixup_start, void *fixup_end) | |||
112 | 112 | ||
113 | void do_lwsync_fixups(unsigned long value, void *fixup_start, void *fixup_end) | 113 | void do_lwsync_fixups(unsigned long value, void *fixup_start, void *fixup_end) |
114 | { | 114 | { |
115 | int *start, *end, *dest; | 115 | long *start, *end; |
116 | unsigned int *dest; | ||
116 | 117 | ||
117 | if (!(value & CPU_FTR_LWSYNC)) | 118 | if (!(value & CPU_FTR_LWSYNC)) |
118 | return ; | 119 | return ; |