tree 5db3e7934507218698e81b2b22bed18e19fec262
parent e51a94766131914e7de2ab01b6eef90d6f2b37c7
author Anton Blanchard <[EMAIL PROTECTED]> Tue Apr 12 08:25:11 2005
committer Linus Torvalds <[EMAIL PROTECTED]> Tue Apr 12 08:25:11 2005
[PATCH] ppc64: remove -fno-omit-frame-pointer
During some code inspection using gcc 4.0 I noticed a stack frame was being
created for a number of functions that didnt require it. For example:
c0000000000df944 <._spin_unlock>:
c0000000000df944: fb e1 ff f0 std r31,-16(r1)
c0000000000df948: f8 21 ff c1 stdu r1,-64(r1)
c0000000000df94c: 7c 3f 0b 78 mr r31,r1
c0000000000df950: 7c 20 04 ac lwsync
c0000000000df954: e8 21 00 00 ld r1,0(r1)
c0000000000df958: 38 00 00 00 li r0,0
c0000000000df95c: 90 03 00 00 stw r0,0(r3)
c0000000000df960: eb e1 ff f0 ld r31,-16(r1)
c0000000000df964: 4e 80 00 20 blr
It turns out we are adding -fno-omit-frame-pointer to ppc64 which is
causing the above behaviour. Removing that flag results in much better
code:
c0000000000d5b30 <._spin_unlock>:
c0000000000d5b30: 7c 20 04 ac lwsync
c0000000000d5b34: 38 00 00 00 li r0,0
c0000000000d5b38: 90 03 00 00 stw r0,0(r3)
c0000000000d5b3c: 4e 80 00 20 blr
We dont require a frame pointer to debug on ppc64, so remove it.
Signed-off-by: Anton Blanchard <[EMAIL PROTECTED]>
Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
Kconfig | 4 ----
1 files changed, 4 deletions(-)
Index: arch/ppc64/Kconfig
===================================================================
--- cf3ada1d0edc3c7b9e0955455056a0048a864710/arch/ppc64/Kconfig (mode:100644
sha1:0d2cb92200f58ec01b5a192ea8c6d309547be086)
+++ 5db3e7934507218698e81b2b22bed18e19fec262/arch/ppc64/Kconfig (mode:100644
sha1:f18fee1fba3824f11fdb815529cab80bdfb0b662)
@@ -40,10 +40,6 @@
bool
default y
-config FRAME_POINTER
- bool
- default y
-
# We optimistically allocate largepages from the VM, so make the limit
# large enough (16MB). This badly named config option is actually
# max order + 1
-
To unsubscribe from this list: send the line "unsubscribe bk-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
