[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <52B46CC7.7060903@gmail.com>
Date: Fri, 20 Dec 2013 08:13:59 -0800
From: David Daney <ddaney.cavm@...il.com>
To: Michal Simek <michal.simek@...inx.com>
CC: linux-kernel@...r.kernel.org, monstr@...str.eu,
David Daney <david.daney@...ium.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will.deacon@....com>,
microblaze-uclinux@...e.uq.edu.au
Subject: Re: [PATCH] microblaze: extable: Sort the exception table at build
time
On 12/20/2013 01:29 AM, Michal Simek wrote:
> Sort the exception table at build-time rather than during boot.
>
> Microblaze is the same case as AARCH64 that's why
> EM_MICROBLAZE conditional check was added to allow
> cross-compilation on machines which are not running
> the latest libc-dev.
>
> Inspired by AARCH64 commit:
> "arm64: extable: sort the exception table at build time"
> (sha1: adace89562c7a9645b8dc84f6e1ac7ba8756094e)
>
> Signed-off-by: Michal Simek <michal.simek@...inx.com>
This seems plausible, I didn't test it, but this is how it is supposed
to work, so...
Acked-by: David Daney <david.daney@...ium.com>
> ---
>
> arch/microblaze/Kconfig | 1 +
> scripts/sortextable.c | 5 +++++
> 2 files changed, 6 insertions(+)
>
> diff --git a/arch/microblaze/Kconfig b/arch/microblaze/Kconfig
> index e23cccd..8d581ab 100644
> --- a/arch/microblaze/Kconfig
> +++ b/arch/microblaze/Kconfig
> @@ -30,6 +30,7 @@ config MICROBLAZE
> select MODULES_USE_ELF_RELA
> select CLONE_BACKWARDS3
> select CLKSRC_OF
> + select BUILDTIME_EXTABLE_SORT
>
> config SWAP
> def_bool n
> diff --git a/scripts/sortextable.c b/scripts/sortextable.c
> index 7941fbd..cc49062 100644
> --- a/scripts/sortextable.c
> +++ b/scripts/sortextable.c
> @@ -39,6 +39,10 @@
> #define EM_AARCH64 183
> #endif
>
> +#ifndef EM_MICROBLAZE
> +#define EM_MICROBLAZE 189
> +#endif
> +
> static int fd_map; /* File descriptor for file being modified. */
> static int mmap_failed; /* Boolean flag. */
> static void *ehdr_curr; /* current ElfXX_Ehdr * for resource cleanup */
> @@ -275,6 +279,7 @@ do_file(char const *const fname)
> case EM_ARCOMPACT:
> case EM_ARM:
> case EM_AARCH64:
> + case EM_MICROBLAZE:
> case EM_MIPS:
> break;
> } /* end switch */
> --
> 1.8.2.3
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists