Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-09-02 Thread Vineet Gupta
Hi Gilad, On 09/02/2013 02:33 PM, Gilad Ben-Yossef wrote: > Hi, > > On Tue, Aug 27, 2013 at 11:31 AM, Vineet Gupta > wrote: >> Frame pointer on ARC doesn't serve the conventional purpose of stack >> unwinding due to the typical way ABI designates it's usage. > More out of curiosity to understand

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-09-02 Thread Gilad Ben-Yossef
Hi, On Tue, Aug 27, 2013 at 11:31 AM, Vineet Gupta wrote: > > Frame pointer on ARC doesn't serve the conventional purpose of stack > unwinding due to the typical way ABI designates it's usage. More out of curiosity to understand the platform better than actual review - can you explain a little

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-09-02 Thread Vineet Gupta
On 08/30/2013 08:50 PM, Dave Hansen wrote: > On 08/30/2013 12:48 AM, Vineet Gupta wrote: >> If we had ARCH_FRAME_POINTER_UNAVAILABLE (def_bool n), we could potentially >> remove >> ARCH_FRAME_POINTER too: >> The issue is some (sparc, c6x...) which are neither in #1 or #2, and not >> present >>

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-09-02 Thread Vineet Gupta
On 08/30/2013 08:50 PM, Dave Hansen wrote: On 08/30/2013 12:48 AM, Vineet Gupta wrote: If we had ARCH_FRAME_POINTER_UNAVAILABLE (def_bool n), we could potentially remove ARCH_FRAME_POINTER too: The issue is some (sparc, c6x...) which are neither in #1 or #2, and not present in

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-09-02 Thread Gilad Ben-Yossef
Hi, On Tue, Aug 27, 2013 at 11:31 AM, Vineet Gupta vineet.gup...@synopsys.com wrote: Frame pointer on ARC doesn't serve the conventional purpose of stack unwinding due to the typical way ABI designates it's usage. More out of curiosity to understand the platform better than actual review -

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-09-02 Thread Vineet Gupta
Hi Gilad, On 09/02/2013 02:33 PM, Gilad Ben-Yossef wrote: Hi, On Tue, Aug 27, 2013 at 11:31 AM, Vineet Gupta vineet.gup...@synopsys.com wrote: Frame pointer on ARC doesn't serve the conventional purpose of stack unwinding due to the typical way ABI designates it's usage. More out of

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-30 Thread Dave Hansen
On 08/30/2013 12:48 AM, Vineet Gupta wrote: > If we had ARCH_FRAME_POINTER_UNAVAILABLE (def_bool n), we could potentially > remove > ARCH_FRAME_POINTER too: > > 1. arches which explicitly select ARCH_FRAME_POINTER (xtensa, parisc, arm64, > x86, > unicore32, tile) could just drop that select. >

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-30 Thread Vineet Gupta
[+linux-arch and other arch maintainers] On 08/29/2013 08:48 PM, Dave Hansen wrote: > On 08/27/2013 01:31 AM, Vineet Gupta wrote: >> Frame pointer on ARC doesn't serve the conventional purpose of stack >> unwinding due to the typical way ABI designates it's usage. >> Thus it's explicit usage on

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-30 Thread Vineet Gupta
[+linux-arch and other arch maintainers] On 08/29/2013 08:48 PM, Dave Hansen wrote: On 08/27/2013 01:31 AM, Vineet Gupta wrote: Frame pointer on ARC doesn't serve the conventional purpose of stack unwinding due to the typical way ABI designates it's usage. Thus it's explicit usage on ARC is

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-30 Thread Dave Hansen
On 08/30/2013 12:48 AM, Vineet Gupta wrote: If we had ARCH_FRAME_POINTER_UNAVAILABLE (def_bool n), we could potentially remove ARCH_FRAME_POINTER too: 1. arches which explicitly select ARCH_FRAME_POINTER (xtensa, parisc, arm64, x86, unicore32, tile) could just drop that select. 2.

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-29 Thread Vineet Gupta
On 08/29/2013 08:48 PM, Dave Hansen wrote: > > I assume you're sending this my way since getmaintainer.pl has me tagged > I moved a bunch of code in there. :) Indeed :-) > The Kconfig.debug stuff has no real maintainer. It would probably be OK > if you just stick this in your architecture's

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-29 Thread Dave Hansen
On 08/27/2013 01:31 AM, Vineet Gupta wrote: > Frame pointer on ARC doesn't serve the conventional purpose of stack > unwinding due to the typical way ABI designates it's usage. > Thus it's explicit usage on ARC is discouraged (gcc is free to use it, > for some tricky stack frames even if

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-29 Thread Vineet Gupta
Ping ? -Vineet On 08/27/2013 02:01 PM, Vineet Gupta wrote: > Frame pointer on ARC doesn't serve the conventional purpose of stack > unwinding due to the typical way ABI designates it's usage. > Thus it's explicit usage on ARC is discouraged (gcc is free to use it, > for some tricky stack frames

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-29 Thread Vineet Gupta
Ping ? -Vineet On 08/27/2013 02:01 PM, Vineet Gupta wrote: Frame pointer on ARC doesn't serve the conventional purpose of stack unwinding due to the typical way ABI designates it's usage. Thus it's explicit usage on ARC is discouraged (gcc is free to use it, for some tricky stack frames even

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-29 Thread Dave Hansen
On 08/27/2013 01:31 AM, Vineet Gupta wrote: Frame pointer on ARC doesn't serve the conventional purpose of stack unwinding due to the typical way ABI designates it's usage. Thus it's explicit usage on ARC is discouraged (gcc is free to use it, for some tricky stack frames even if

Re: [PATCH] Kconfig.debug: Add FRAME_POINTER anti-dependency for ARC

2013-08-29 Thread Vineet Gupta
On 08/29/2013 08:48 PM, Dave Hansen wrote: I assume you're sending this my way since getmaintainer.pl has me tagged I moved a bunch of code in there. :) Indeed :-) The Kconfig.debug stuff has no real maintainer. It would probably be OK if you just stick this in your architecture's next git