All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARC: extable: Enable sorting at build time
@ 2013-11-15  6:45 Vineet Gupta
  2013-11-19 11:08 ` Vineet Gupta
  2013-11-19 17:21 ` David Daney
  0 siblings, 2 replies; 5+ messages in thread
From: Vineet Gupta @ 2013-11-15  6:45 UTC (permalink / raw)
  To: linux-kernel; +Cc: Vineet Gupta, David Daney, Michal Marek, Francois Bedard

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
---
 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:
-- 
1.8.1.2


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

* Re: [PATCH] ARC: extable: Enable sorting at build time
  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-19 17:21 ` David Daney
  1 sibling, 1 reply; 5+ messages in thread
From: Vineet Gupta @ 2013-11-19 11:08 UTC (permalink / raw)
  To: linux-kernel; +Cc: David Daney, Michal Marek, Francois Bedard

Ping ? people OK if I take this via arc tree (after baking it in linux-next for
some time).

-Vineet

On 11/15/2013 12:15 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
> ---
>  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:


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

* Re: [PATCH] ARC: extable: Enable sorting at build time
  2013-11-19 11:08 ` Vineet Gupta
@ 2013-11-19 15:00   ` Michal Marek
  2013-11-20  4:53     ` Vineet Gupta
  0 siblings, 1 reply; 5+ messages in thread
From: Michal Marek @ 2013-11-19 15:00 UTC (permalink / raw)
  To: Vineet Gupta, linux-kernel; +Cc: David Daney, Francois Bedard

Dne 19.11.2013 12:08, Vineet Gupta napsal(a):
> Ping ? people OK if I take this via arc tree (after baking it in linux-next for
> some time).

It makes more sense to add it to the arc tree than kbuild. If that's
what you are asking?

Thanks,
Michal

> 
> -Vineet
> 
> On 11/15/2013 12:15 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
>> ---
>>  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:
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 


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

* Re: [PATCH] ARC: extable: Enable sorting at build time
  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 17:21 ` David Daney
  1 sibling, 0 replies; 5+ messages in thread
From: David Daney @ 2013-11-19 17:21 UTC (permalink / raw)
  To: Vineet Gupta; +Cc: linux-kernel, David Daney, Michal Marek, Francois Bedard

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:
>


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

* Re: [PATCH] ARC: extable: Enable sorting at build time
  2013-11-19 15:00   ` Michal Marek
@ 2013-11-20  4:53     ` Vineet Gupta
  0 siblings, 0 replies; 5+ messages in thread
From: Vineet Gupta @ 2013-11-20  4:53 UTC (permalink / raw)
  To: Michal Marek, linux-kernel; +Cc: David Daney, Francois Bedard

On 11/19/2013 08:30 PM, Michal Marek wrote:
> Dne 19.11.2013 12:08, Vineet Gupta napsal(a):
>> Ping ? people OK if I take this via arc tree (after baking it in linux-next for
>> some time).
> It makes more sense to add it to the arc tree than kbuild. If that's
> what you are asking?

Yep, that's what I was asking (and fishing for a ACK or two which I now have
thanks to David).
Patch added to ARC for-curr so it gets a decent -next exposure before sending over
to Linus in 3.13 cycle.

Thx,
-Vineet

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

end of thread, other threads:[~2013-11-20  4:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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 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.