Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744
@ 2023-08-13 17:25 Giulio Benetti
  2023-08-22 20:09 ` Thomas Petazzoni via buildroot
  2023-09-13 15:55 ` Peter Korsgaard
  0 siblings, 2 replies; 4+ messages in thread
From: Giulio Benetti @ 2023-08-13 17:25 UTC (permalink / raw)
  To: buildroot; +Cc: Giulio Benetti

This package is affected by gcc bug 43744 and I have not found a work
around for it(i.e. the common -O0 we use or other), so let's disable it if
gcc has such bug.

Fixes:
http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 package/ramspeed/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/ramspeed/Config.in b/package/ramspeed/Config.in
index b236329a84..bd0438ffe1 100644
--- a/package/ramspeed/Config.in
+++ b/package/ramspeed/Config.in
@@ -1,5 +1,9 @@
+comment "ramspeed needs a toolchain not affected by GCC bug 43744"
+	depends on BR2_TOOLCHAIN_HAS_GCC_BUG_43744
+
 config BR2_PACKAGE_RAMSPEED
 	bool "ramspeed"
+	depends on !BR2_TOOLCHAIN_HAS_GCC_BUG_43744
 	help
 	  RAMspeed is a free open source command line utility
 	  to measure cache and memory performance.
-- 
2.34.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744
  2023-08-13 17:25 [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744 Giulio Benetti
@ 2023-08-22 20:09 ` Thomas Petazzoni via buildroot
  2023-08-22 20:47   ` Giulio Benetti
  2023-09-13 15:55 ` Peter Korsgaard
  1 sibling, 1 reply; 4+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-08-22 20:09 UTC (permalink / raw)
  To: Giulio Benetti; +Cc: buildroot

On Sun, 13 Aug 2023 19:25:24 +0200
Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:

> This package is affected by gcc bug 43744 and I have not found a work
> around for it(i.e. the common -O0 we use or other), so let's disable it if
> gcc has such bug.
> 
> Fixes:
> http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>  package/ramspeed/Config.in | 4 ++++
>  1 file changed, 4 insertions(+)

Applied to master, thanks. FYI, gcc bug 43744 is marked as RESOLVED
FIXED in the gcc bug tracker
(https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43744), so if you don't
reopen it, nobody is ever going to look into it :-/

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744
  2023-08-22 20:09 ` Thomas Petazzoni via buildroot
@ 2023-08-22 20:47   ` Giulio Benetti
  0 siblings, 0 replies; 4+ messages in thread
From: Giulio Benetti @ 2023-08-22 20:47 UTC (permalink / raw)
  To: Thomas Petazzoni; +Cc: buildroot

On 22/08/23 22:09, Thomas Petazzoni via buildroot wrote:
> On Sun, 13 Aug 2023 19:25:24 +0200
> Giulio Benetti <giulio.benetti@benettiengineering.com> wrote:
> 
>> This package is affected by gcc bug 43744 and I have not found a work
>> around for it(i.e. the common -O0 we use or other), so let's disable it if
>> gcc has such bug.
>>
>> Fixes:
>> http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d
>>
>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> ---
>>   package/ramspeed/Config.in | 4 ++++
>>   1 file changed, 4 insertions(+)
> 
> Applied to master, thanks. FYI, gcc bug 43744 is marked as RESOLVED
> FIXED in the gcc bug tracker
> (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43744), so if you don't
> reopen it, nobody is ever going to look into it :-/

Thank you for pointing. Just commented and asked to reopen it.

Best regards
-- 
Giulio Benetti
CEO&CTO@Benetti Engineering sas
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744
  2023-08-13 17:25 [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744 Giulio Benetti
  2023-08-22 20:09 ` Thomas Petazzoni via buildroot
@ 2023-09-13 15:55 ` Peter Korsgaard
  1 sibling, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2023-09-13 15:55 UTC (permalink / raw)
  To: Giulio Benetti; +Cc: buildroot

>>>>> "Giulio" == Giulio Benetti <giulio.benetti@benettiengineering.com> writes:

 > This package is affected by gcc bug 43744 and I have not found a work
 > around for it(i.e. the common -O0 we use or other), so let's disable it if
 > gcc has such bug.

 > Fixes:
 > http://autobuild.buildroot.net/results/ab289769c5fea435934ed260d38e0a4fdd2ba72d

 > Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>

Committed to 2023.02.x and 2023.05.x, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

end of thread, other threads:[~2023-09-13 15:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-13 17:25 [Buildroot] [PATCH] package/ramspeed: disable package if affected from gcc bug 43744 Giulio Benetti
2023-08-22 20:09 ` Thomas Petazzoni via buildroot
2023-08-22 20:47   ` Giulio Benetti
2023-09-13 15:55 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox