All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Daney <ddaney@caviumnetworks.com>
To: Vineet Gupta <Vineet.Gupta1@synopsys.com>
Cc: <linux-kernel@vger.kernel.org>,
	David Daney <david.daney@cavium.com>,
	Michal Marek <mmarek@suse.cz>,
	Francois Bedard <Francois.Bedard@synopsys.com>
Subject: Re: [PATCH] ARC: extable: Enable sorting at build time
Date: Tue, 19 Nov 2013 09:21:22 -0800	[thread overview]
Message-ID: <528B9E12.7060108@caviumnetworks.com> (raw)
In-Reply-To: <1384497942-26810-1-git-send-email-vgupta@synopsys.com>

On 11/14/2013 10:45 PM, Vineet Gupta wrote:
> Avoids wasting cycles at boot specially on slower simulators
>
> Signed-off-by: Vineet Gupta <vgupta@synopsys.com>
> Cc: David Daney <david.daney@cavium.com>
> Cc: Michal Marek <mmarek@suse.cz>
> Cc: Francois Bedard <fbedard@synopsys.com>
> Cc: linux-kernel@vger.kernel.org

I haven't tested it, but this looks sane.

Acked-by: David Daney <david.daney@cavium.com>

I have no opinion about the path this should take towards Linus' tree.

> ---
>   arch/arc/Kconfig      | 1 +
>   scripts/sortextable.c | 5 +++++
>   2 files changed, 6 insertions(+)
>
> diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig
> index 91dbb2757afd..080580216301 100644
> --- a/arch/arc/Kconfig
> +++ b/arch/arc/Kconfig
> @@ -8,6 +8,7 @@
>
>   config ARC
>   	def_bool y
> +	select BUILDTIME_EXTABLE_SORT
>   	select CLONE_BACKWARDS
>   	# ARC Busybox based initramfs absolutely relies on DEVTMPFS for /dev
>   	select DEVTMPFS if !INITRAMFS_SOURCE=""
> diff --git a/scripts/sortextable.c b/scripts/sortextable.c
> index 7c2310c5b996..e3fb1c36ed0f 100644
> --- a/scripts/sortextable.c
> +++ b/scripts/sortextable.c
> @@ -31,6 +31,10 @@
>   #include <tools/be_byteshift.h>
>   #include <tools/le_byteshift.h>
>
> +#ifndef EM_ARCOMPACT
> +#define EM_ARCOMPACT	93
> +#endif
> +
>   #ifndef EM_AARCH64
>   #define EM_AARCH64	183
>   #endif
> @@ -244,6 +248,7 @@ do_file(char const *const fname)
>   	case EM_S390:
>   		custom_sort = sort_relative_table;
>   		break;
> +	case EM_ARCOMPACT:
>   	case EM_ARM:
>   	case EM_AARCH64:
>   	case EM_MIPS:
>


      parent reply	other threads:[~2013-11-19 17:21 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-15  6:45 [PATCH] ARC: extable: Enable sorting at build time Vineet Gupta
2013-11-19 11:08 ` Vineet Gupta
2013-11-19 15:00   ` Michal Marek
2013-11-20  4:53     ` Vineet Gupta
2013-11-19 17:21 ` David Daney [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=528B9E12.7060108@caviumnetworks.com \
    --to=ddaney@caviumnetworks.com \
    --cc=Francois.Bedard@synopsys.com \
    --cc=Vineet.Gupta1@synopsys.com \
    --cc=david.daney@cavium.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmarek@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.