lkml.org 
[lkml]   [2011]   [Jan]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] Support BE8 mode kernel modules relocation
On Mon, Jan 17, 2011 at 04:42:07PM +0800, Stanley.Miao wrote:
> The code section in BE8 kernel modules is in little-endian while data
> section is in big-endian. When reading code from memory in the relocation
> procedure, these instructions are read according to big-endian, so they
> need to be inverted before writing to memory and after reading from memory.
>
> Signed-off-by: Stanley.Miao <stanley.miao@windriver.com>
> ---
> arch/arm/Makefile | 1 +
> arch/arm/include/asm/io.h | 12 ++++++++++
> arch/arm/kernel/module.c | 51 ++++++++++++++++++++++++--------------------
> 3 files changed, 41 insertions(+), 23 deletions(-)
>
> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
> index 3eec793..03e7761 100644
> --- a/arch/arm/Makefile
> +++ b/arch/arm/Makefile
> @@ -13,6 +13,7 @@
> LDFLAGS_vmlinux :=-p --no-undefined -X
> ifeq ($(CONFIG_CPU_ENDIAN_BE8),y)
> LDFLAGS_vmlinux += --be8
> +LDFLAGS_MODULE += --be8
> endif
>
> OBJCOPYFLAGS :=-O binary -R .note -R .note.gnu.build-id -R .comment -S
> diff --git a/arch/arm/include/asm/io.h b/arch/arm/include/asm/io.h
> index 815efa2..2f088c5 100644
> --- a/arch/arm/include/asm/io.h
> +++ b/arch/arm/include/asm/io.h
> @@ -233,6 +233,18 @@ extern void _memset_io(volatile void __iomem *, int, size_t);
>
> #endif /* __mem_pci */
>
> +#ifdef CONFIG_CPU_ENDIAN_BE8
> +#define read_instr32(c) __swab32(*(u32 *)c)
> +#define read_instr16(c) __swab16(*(u16 *)c)
> +#define write_instr32(v,a) (*(u32 *)(a) = __swab32((__force __u32)(v)))
> +#define write_instr16(v,a) (*(u16 *)(a) = __swab16((__force __u16)(v)))
> +#else
> +#define read_instr32(c) (*(u32 *)c)
> +#define read_instr16(c) (*(u16 *)c)
> +#define write_instr32(v,a) (*(u32 *)(a) = (v))
> +#define write_instr16(v,a) (*(u16 *)(a) = (v))
> +#endif
> +

It makes no sense to put these in asm/io.h


\
 
 \ /
  Last update: 2011-01-17 09:55    [from the cache]
©2003-2011 Jasper Spaans