* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC @ 2018-04-02 9:18 Alexey Brodkin 2018-04-03 20:17 ` Tom Rini 2018-04-04 13:18 ` Tom Rini 0 siblings, 2 replies; 9+ messages in thread From: Alexey Brodkin @ 2018-04-02 9:18 UTC (permalink / raw) To: u-boot With the most recent tools for ARC (arc-2017.09) in case of "naked" function compiler throws a warning: ---------------------------------->8----------------------------- board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target } ^ ---------------------------------->8----------------------------- That happens because the compiler doesn't handle "naked" functions as a special case where stack calculation shouldn't be done. But for now until this is fixed in GCC to get clean buildman output we're disabling stack-usage check for ARC. See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html for more background. Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> Cc: Masahiro Yamada <yamada.masahiro@socionext.com> Cc: Tom Rini <trini@konsulko.com> --- Makefile | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Makefile b/Makefile index 5fa14789d99f..c80b902afdce 100644 --- a/Makefile +++ b/Makefile @@ -600,9 +600,13 @@ KBUILD_CFLAGS += -g KBUILD_AFLAGS += -g # Report stack usage if supported +# ARC tools based on GCC 7.1 has an issue with stack usage +# with naked functions, see commit message for more details +ifndef CONFIG_ARC ifeq ($(shell $(CONFIG_SHELL) $(srctree)/scripts/gcc-stack-usage.sh $(CC)),y) KBUILD_CFLAGS += -fstack-usage endif +endif KBUILD_CFLAGS += $(call cc-option,-Wno-format-nonliteral) -- 2.14.3 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-02 9:18 [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC Alexey Brodkin @ 2018-04-03 20:17 ` Tom Rini 2018-04-03 20:21 ` Alexey Brodkin 2018-04-04 13:18 ` Tom Rini 1 sibling, 1 reply; 9+ messages in thread From: Tom Rini @ 2018-04-03 20:17 UTC (permalink / raw) To: u-boot On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > With the most recent tools for ARC (arc-2017.09) in case of > "naked" function compiler throws a warning: > ---------------------------------->8----------------------------- > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > } > ^ > ---------------------------------->8----------------------------- > > That happens because the compiler doesn't handle "naked" functions > as a special case where stack calculation shouldn't be done. > > But for now until this is fixed in GCC to get clean buildman output > we're disabling stack-usage check for ARC. > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > for more background. > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > Cc: Tom Rini <trini@konsulko.com> Erm, this patch isn't working for me. I take the current ARC PR, and apply this, and I still see the warning showing up. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180403/e13d7e53/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-03 20:17 ` Tom Rini @ 2018-04-03 20:21 ` Alexey Brodkin 2018-04-03 20:25 ` Tom Rini 0 siblings, 1 reply; 9+ messages in thread From: Alexey Brodkin @ 2018-04-03 20:21 UTC (permalink / raw) To: u-boot Hi Tom, On Tue, 2018-04-03 at 16:17 -0400, Tom Rini wrote: > On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > > > With the most recent tools for ARC (arc-2017.09) in case of > > "naked" function compiler throws a warning: > > ---------------------------------->8----------------------------- > > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > > } > > ^ > > ---------------------------------->8----------------------------- > > > > That happens because the compiler doesn't handle "naked" functions > > as a special case where stack calculation shouldn't be done. > > > > But for now until this is fixed in GCC to get clean buildman output > > we're disabling stack-usage check for ARC. > > > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > > for more background. > > > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > > Cc: Tom Rini <trini@konsulko.com> > > Erm, this patch isn't working for me. I take the current ARC PR, and > apply this, and I still see the warning showing up. How do you see an issue? Could you please make sure arc-2017.09 tools are used but not arc-2016.09? See my change for TravisCI in the same PR. -Alexey ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-03 20:21 ` Alexey Brodkin @ 2018-04-03 20:25 ` Tom Rini 2018-04-03 20:27 ` Alexey Brodkin 0 siblings, 1 reply; 9+ messages in thread From: Tom Rini @ 2018-04-03 20:25 UTC (permalink / raw) To: u-boot On Tue, Apr 03, 2018 at 08:21:06PM +0000, Alexey Brodkin wrote: > Hi Tom, > > On Tue, 2018-04-03 at 16:17 -0400, Tom Rini wrote: > > On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > > > > > With the most recent tools for ARC (arc-2017.09) in case of > > > "naked" function compiler throws a warning: > > > ---------------------------------->8----------------------------- > > > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > > > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > > > } > > > ^ > > > ---------------------------------->8----------------------------- > > > > > > That happens because the compiler doesn't handle "naked" functions > > > as a special case where stack calculation shouldn't be done. > > > > > > But for now until this is fixed in GCC to get clean buildman output > > > we're disabling stack-usage check for ARC. > > > > > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > > > for more background. > > > > > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > > > Cc: Tom Rini <trini@konsulko.com> > > > > Erm, this patch isn't working for me. I take the current ARC PR, and > > apply this, and I still see the warning showing up. > > How do you see an issue? > Could you please make sure arc-2017.09 tools are used but not arc-2016.09? > See my change for TravisCI in the same PR. ... but the patch here is disabling -fstack-usage altogether for ARC, yes? -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180403/04ee407f/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-03 20:25 ` Tom Rini @ 2018-04-03 20:27 ` Alexey Brodkin 2018-04-03 20:49 ` Tom Rini 0 siblings, 1 reply; 9+ messages in thread From: Alexey Brodkin @ 2018-04-03 20:27 UTC (permalink / raw) To: u-boot Hi Tom, On Tue, 2018-04-03 at 16:25 -0400, Tom Rini wrote: > On Tue, Apr 03, 2018 at 08:21:06PM +0000, Alexey Brodkin wrote: > > Hi Tom, > > > > On Tue, 2018-04-03 at 16:17 -0400, Tom Rini wrote: > > > On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > > > > > > > With the most recent tools for ARC (arc-2017.09) in case of > > > > "naked" function compiler throws a warning: > > > > ---------------------------------->8----------------------------- > > > > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > > > > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > > > > } > > > > ^ > > > > ---------------------------------->8----------------------------- > > > > > > > > That happens because the compiler doesn't handle "naked" functions > > > > as a special case where stack calculation shouldn't be done. > > > > > > > > But for now until this is fixed in GCC to get clean buildman output > > > > we're disabling stack-usage check for ARC. > > > > > > > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > > > > for more background. > > > > > > > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > Cc: Tom Rini <trini@konsulko.com> > > > > > > Erm, this patch isn't working for me. I take the current ARC PR, and > > > apply this, and I still see the warning showing up. > > > > How do you see an issue? > > Could you please make sure arc-2017.09 tools are used but not arc-2016.09? > > See my change for TravisCI in the same PR. > > ... but the patch here is disabling -fstack-usage altogether for ARC, > yes? Right, but that's a temporary "patch" for the problem in hand. Once GCC 8.x gets released we'll just switch to it and that problem shouldn't bother us any longer. -Alexey ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-03 20:27 ` Alexey Brodkin @ 2018-04-03 20:49 ` Tom Rini 2018-04-04 8:51 ` Alexey Brodkin 0 siblings, 1 reply; 9+ messages in thread From: Tom Rini @ 2018-04-03 20:49 UTC (permalink / raw) To: u-boot On Tue, Apr 03, 2018 at 08:27:20PM +0000, Alexey Brodkin wrote: > Hi Tom, > > On Tue, 2018-04-03 at 16:25 -0400, Tom Rini wrote: > > On Tue, Apr 03, 2018 at 08:21:06PM +0000, Alexey Brodkin wrote: > > > Hi Tom, > > > > > > On Tue, 2018-04-03 at 16:17 -0400, Tom Rini wrote: > > > > On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > > > > > > > > > With the most recent tools for ARC (arc-2017.09) in case of > > > > > "naked" function compiler throws a warning: > > > > > ---------------------------------->8----------------------------- > > > > > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > > > > > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > > > > > } > > > > > ^ > > > > > ---------------------------------->8----------------------------- > > > > > > > > > > That happens because the compiler doesn't handle "naked" functions > > > > > as a special case where stack calculation shouldn't be done. > > > > > > > > > > But for now until this is fixed in GCC to get clean buildman output > > > > > we're disabling stack-usage check for ARC. > > > > > > > > > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > > > > > for more background. > > > > > > > > > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > > > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > > Cc: Tom Rini <trini@konsulko.com> > > > > > > > > Erm, this patch isn't working for me. I take the current ARC PR, and > > > > apply this, and I still see the warning showing up. > > > > > > How do you see an issue? > > > Could you please make sure arc-2017.09 tools are used but not arc-2016.09? > > > See my change for TravisCI in the same PR. > > > > ... but the patch here is disabling -fstack-usage altogether for ARC, > > yes? > > Right, but that's a temporary "patch" for the problem in hand. > Once GCC 8.x gets released we'll just switch to it and that problem shouldn't > bother us any longer. OK. But this patch doesn't work and we're trying -fstack-usage is the problem I see :) -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180403/ef3e7c3d/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-03 20:49 ` Tom Rini @ 2018-04-04 8:51 ` Alexey Brodkin 2018-04-04 13:14 ` Tom Rini 0 siblings, 1 reply; 9+ messages in thread From: Alexey Brodkin @ 2018-04-04 8:51 UTC (permalink / raw) To: u-boot Hi Tom, On Tue, 2018-04-03 at 16:49 -0400, Tom Rini wrote: > On Tue, Apr 03, 2018 at 08:27:20PM +0000, Alexey Brodkin wrote: > > Hi Tom, > > > > On Tue, 2018-04-03 at 16:25 -0400, Tom Rini wrote: > > > On Tue, Apr 03, 2018 at 08:21:06PM +0000, Alexey Brodkin wrote: > > > > Hi Tom, > > > > > > > > On Tue, 2018-04-03 at 16:17 -0400, Tom Rini wrote: > > > > > On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > > > > > > > > > > > With the most recent tools for ARC (arc-2017.09) in case of > > > > > > "naked" function compiler throws a warning: > > > > > > ---------------------------------->8----------------------------- > > > > > > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > > > > > > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > > > > > > } > > > > > > ^ > > > > > > ---------------------------------->8----------------------------- > > > > > > > > > > > > That happens because the compiler doesn't handle "naked" functions > > > > > > as a special case where stack calculation shouldn't be done. > > > > > > > > > > > > But for now until this is fixed in GCC to get clean buildman output > > > > > > we're disabling stack-usage check for ARC. > > > > > > > > > > > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > > > > > > for more background. > > > > > > > > > > > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > > > > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > > > Cc: Tom Rini <trini@konsulko.com> > > > > > > > > > > Erm, this patch isn't working for me. I take the current ARC PR, and > > > > > apply this, and I still see the warning showing up. > > > > > > > > How do you see an issue? > > > > Could you please make sure arc-2017.09 tools are used but not arc-2016.09? > > > > See my change for TravisCI in the same PR. > > > > > > ... but the patch here is disabling -fstack-usage altogether for ARC, > > > yes? > > > > Right, but that's a temporary "patch" for the problem in hand. > > Once GCC 8.x gets released we'll just switch to it and that problem shouldn't > > bother us any longer. > > OK. But this patch doesn't work and we're trying -fstack-usage is the > problem I see :) Well probably I'm missing something :) Here's a TravisCI job for ARC that passes perfectly fine: https://travis-ci.org/abrodkin/u-boot/jobs/362026822 Stuff that is built there is here: https://github.com/abrodkin/u-boot/commits/staging-arc Basically it's: * v2018.05-rc1 + * My last PR (tags/arc-for-2018.05) + * [This] "disable stack-usage" patch ------------------------>8------------------------ # git log --oneline 7917ac883723 Makefile: Disable stack-usage check for ARC 9fe2b77b313b ARC: HSDK: Enable SPI flash support 11007a25b835 ARC: HSDK: Add platform-specific commands 416c03d2f09f ARC: Bump ARC tools used in TravisCI to the most recent release arc-2017.09 94f247d0c145 ARC: AXS10x: DTS: Remove unused interrupt properties 645b5afbb821 (tag: v2018.05-rc1, mainline/master) Prepare v2018.05-rc1 ------------------------>8------------------------ -Alexey ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-04 8:51 ` Alexey Brodkin @ 2018-04-04 13:14 ` Tom Rini 0 siblings, 0 replies; 9+ messages in thread From: Tom Rini @ 2018-04-04 13:14 UTC (permalink / raw) To: u-boot On Wed, Apr 04, 2018 at 08:51:16AM +0000, Alexey Brodkin wrote: > Hi Tom, > > On Tue, 2018-04-03 at 16:49 -0400, Tom Rini wrote: > > On Tue, Apr 03, 2018 at 08:27:20PM +0000, Alexey Brodkin wrote: > > > Hi Tom, > > > > > > On Tue, 2018-04-03 at 16:25 -0400, Tom Rini wrote: > > > > On Tue, Apr 03, 2018 at 08:21:06PM +0000, Alexey Brodkin wrote: > > > > > Hi Tom, > > > > > > > > > > On Tue, 2018-04-03 at 16:17 -0400, Tom Rini wrote: > > > > > > On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > > > > > > > > > > > > > With the most recent tools for ARC (arc-2017.09) in case of > > > > > > > "naked" function compiler throws a warning: > > > > > > > ---------------------------------->8----------------------------- > > > > > > > board/synopsys/hsdk/hsdk.c: In function 'hsdk_core_init_f': > > > > > > > board/synopsys/hsdk/hsdk.c:345:1: warning: stack usage computation not supported for this target > > > > > > > } > > > > > > > ^ > > > > > > > ---------------------------------->8----------------------------- > > > > > > > > > > > > > > That happens because the compiler doesn't handle "naked" functions > > > > > > > as a special case where stack calculation shouldn't be done. > > > > > > > > > > > > > > But for now until this is fixed in GCC to get clean buildman output > > > > > > > we're disabling stack-usage check for ARC. > > > > > > > > > > > > > > See https://lists.denx.de/pipermail/u-boot/2018-April/324455.html > > > > > > > for more background. > > > > > > > > > > > > > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > > > > > > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > > > > > > > Cc: Tom Rini <trini@konsulko.com> > > > > > > > > > > > > Erm, this patch isn't working for me. I take the current ARC PR, and > > > > > > apply this, and I still see the warning showing up. > > > > > > > > > > How do you see an issue? > > > > > Could you please make sure arc-2017.09 tools are used but not arc-2016.09? > > > > > See my change for TravisCI in the same PR. > > > > > > > > ... but the patch here is disabling -fstack-usage altogether for ARC, > > > > yes? > > > > > > Right, but that's a temporary "patch" for the problem in hand. > > > Once GCC 8.x gets released we'll just switch to it and that problem shouldn't > > > bother us any longer. > > > > OK. But this patch doesn't work and we're trying -fstack-usage is the > > problem I see :) > > Well probably I'm missing something :) > > Here's a TravisCI job for ARC that passes perfectly fine: > https://travis-ci.org/abrodkin/u-boot/jobs/362026822 > > Stuff that is built there is here: > https://github.com/abrodkin/u-boot/commits/staging-arc Yes, OK, moving to 2017.09 also is fine here, so I'm just going to accept it and move on. Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180404/17fe1d2a/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC 2018-04-02 9:18 [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC Alexey Brodkin 2018-04-03 20:17 ` Tom Rini @ 2018-04-04 13:18 ` Tom Rini 1 sibling, 0 replies; 9+ messages in thread From: Tom Rini @ 2018-04-04 13:18 UTC (permalink / raw) To: u-boot On Mon, Apr 02, 2018 at 12:18:02PM +0300, Alexey Brodkin wrote: > With the most recent tools for ARC (arc-2017.09) in case of > "naked" function compiler throws a warning: Applied to u-boot/master, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180404/403f601b/attachment.sig> ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-04-04 13:18 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-04-02 9:18 [U-Boot] [PATCH] Makefile: Disable stack-usage check for ARC Alexey Brodkin 2018-04-03 20:17 ` Tom Rini 2018-04-03 20:21 ` Alexey Brodkin 2018-04-03 20:25 ` Tom Rini 2018-04-03 20:27 ` Alexey Brodkin 2018-04-03 20:49 ` Tom Rini 2018-04-04 8:51 ` Alexey Brodkin 2018-04-04 13:14 ` Tom Rini 2018-04-04 13:18 ` Tom Rini
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox