linux-parisc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH palo] ipl: Ensure no GCC builtins replace string functions
@ 2020-05-20 10:30 Jeroen Roovers
  2020-05-20 21:47 ` Helge Deller
  0 siblings, 1 reply; 2+ messages in thread
From: Jeroen Roovers @ 2020-05-20 10:30 UTC (permalink / raw)
  To: linux-parisc; +Cc: Helge Deller, James.Bottomley, Jeroen Roovers

With GCC 10, the ipl is built with the compiler overriding local
definitions of some string functions, causing the ipl to trigger an
HPMC. Fix this by setting the -fno-builtin compiler flag.

Signed-off-by: Jeroen Roovers <jer@gentoo.org>
---
 ipl/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipl/Makefile b/ipl/Makefile
index 913ec66..c1d608b 100644
--- a/ipl/Makefile
+++ b/ipl/Makefile
@@ -39,7 +39,7 @@ endif
 VPATH=../lib:.
 
 AFLAGS	= -I../lib
-CFLAGS	= -DIPL_LOADER -I. -I../lib -I../include -O2 -mdisable-fpregs -Wall -fno-delete-null-pointer-checks
+CFLAGS	= -DIPL_LOADER -I. -I../lib -I../include -O2 -mdisable-fpregs -Wall -fno-delete-null-pointer-checks -fno-builtin
 LDFLAGS  = -N --section-start .init=0x60000 -e '$$START$$'
 
 all:	iplelf
-- 
2.26.2


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

* Re: [PATCH palo] ipl: Ensure no GCC builtins replace string functions
  2020-05-20 10:30 [PATCH palo] ipl: Ensure no GCC builtins replace string functions Jeroen Roovers
@ 2020-05-20 21:47 ` Helge Deller
  0 siblings, 0 replies; 2+ messages in thread
From: Helge Deller @ 2020-05-20 21:47 UTC (permalink / raw)
  To: Jeroen Roovers, linux-parisc; +Cc: James.Bottomley

On 20.05.20 12:30, Jeroen Roovers wrote:
> With GCC 10, the ipl is built with the compiler overriding local
> definitions of some string functions, causing the ipl to trigger an
> HPMC. Fix this by setting the -fno-builtin compiler flag.
>
> Signed-off-by: Jeroen Roovers <jer@gentoo.org>

Thanks Jeroen!

I've applied your patch and am preparing for a palo v2.13 release...

Helge


> ---
>  ipl/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ipl/Makefile b/ipl/Makefile
> index 913ec66..c1d608b 100644
> --- a/ipl/Makefile
> +++ b/ipl/Makefile
> @@ -39,7 +39,7 @@ endif
>  VPATH=../lib:.
>
>  AFLAGS	= -I../lib
> -CFLAGS	= -DIPL_LOADER -I. -I../lib -I../include -O2 -mdisable-fpregs -Wall -fno-delete-null-pointer-checks
> +CFLAGS	= -DIPL_LOADER -I. -I../lib -I../include -O2 -mdisable-fpregs -Wall -fno-delete-null-pointer-checks -fno-builtin
>  LDFLAGS  = -N --section-start .init=0x60000 -e '$$START$$'
>
>  all:	iplelf
>


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

end of thread, other threads:[~2020-05-20 21:48 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-20 10:30 [PATCH palo] ipl: Ensure no GCC builtins replace string functions Jeroen Roovers
2020-05-20 21:47 ` Helge Deller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).