Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option and force to select it

From	yalin wang <yalin.wang2010@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option and force to select it
Date	2015-11-10 12:50 +0100
Message-ID	<qtfB0-4wQ-17@gated-at.bofh.it> (permalink)
References	<qsZmz-1yW-41@gated-at.bofh.it> <qtevf-3RD-5@gated-at.bofh.it> <qteYi-4iG-35@gated-at.bofh.it> <qtfrk-4tg-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

> On Nov 10, 2015, at 19:35, Catalin Marinas <catalin.marinas@arm.com> wrote:
> 
> On Tue, Nov 10, 2015 at 07:09:00PM +0800, yalin wang wrote:
>>> On Nov 10, 2015, at 18:37, Catalin Marinas <catalin.marinas@arm.com> wrote:
>>> 
>>> On Mon, Nov 09, 2015 at 10:09:55AM -0800, Yang Shi wrote:
>>>> FRAME_POINTER  is defined in lib/Kconfig.debug, it is unnecessary to redefine
>>>> it in arch/arm64/Kconfig.debug. Actually, the one defined in arm64 directory
>>>> is never used.
>>> 
>>> That's not true since the arm64 definition seems to take precedence.
>>> 
>>>> This adds a dependency on DEBUG_KERNEL for building with frame pointers.
>>> 
>>> It doesn't because arm64 selects ARCH_WANT_FRAME_POINTERS.
>>> 
>>>> ARM64 depends on frame pointer to get correct stack backtrace and need
>>>> FRAME_POINTER kconfig option enabled all the time.
>>>> However, currect implementation makes it could be disabled, so force it
>>>> to be selected by ARM64.
>>>> 
>>>> Signed-off-by: Yang Shi <yang.shi@linaro.org>
>>> 
>>> Patch applied but I changed the commit log slightly. Thanks.
>> i have a question,
>> why FRAME_POINTER  config must be enabled ?
>> and i see ARM arch can  disable this config .
>> if i don’t need stack trace dump and the software release is for 
>> final product , don’t need debug stack trace log .
>> is it possible to disable it for performance reason ?
> 
> If you don't need any stack trace, perf etc., in theory you can disable
> the option. However, the aarch64 gcc compiler always generates it (I'm
> not sure whether the AAPCS mandates it). Anyway, the performance impact
> is very small since there are more general purpose registers available
> in AArch64 already.
> 
i just make a test with -fomit-frame-pointer,  seems gcc can generate code without frame pointer,
like ARM arch.
version:
aarch64-linux-gnu-gcc
gcc version 4.9.2 20140904 (prerelease) (crosstool-NG linaro-1.13.1-4.9-2014.09 - Linaro GCC 4.9-2014.09)

why AARCH64 don’t have frame unwind info just like ARM arch?

Thanks








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

Thread

[V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option and force to select it Yang Shi <yang.shi@linaro.org> - 2015-11-09 19:30 +0100
  Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option  and force to select it Catalin Marinas <catalin.marinas@arm.com> - 2015-11-10 11:40 +0100
    Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option and force to select it yalin wang <yalin.wang2010@gmail.com> - 2015-11-10 12:10 +0100
      Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option  and force to select it Catalin Marinas <catalin.marinas@arm.com> - 2015-11-10 12:40 +0100
        Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option and force to select it yalin wang <yalin.wang2010@gmail.com> - 2015-11-10 12:50 +0100
          Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option  and force to select it Will Deacon <will.deacon@arm.com> - 2015-11-10 13:00 +0100
    Re: [V3 PATCH] arm64: remove redundant FRAME_POINTER kconfig option  and force to select it "Shi, Yang" <yang.shi@linaro.org> - 2015-11-10 18:00 +0100

csiph-web