Linux-parisc archive on lore.kernel.org
 help / color / 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	[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, back to index

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

Linux-parisc archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-parisc/0 linux-parisc/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-parisc linux-parisc/ https://lore.kernel.org/linux-parisc \
		linux-parisc@vger.kernel.org
	public-inbox-index linux-parisc

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-parisc


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git