* [Buildroot] [PATCH 0/2] package/chicken: pass TARGET_CFLAGS to the build system
@ 2024-08-11 0:00 Max Filippov
2024-08-11 0:00 ` [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture" Max Filippov
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Max Filippov @ 2024-08-11 0:00 UTC (permalink / raw)
To: buildroot; +Cc: Woodrow Douglass, Thomas Petazzoni
Hello,
this series reverts xtensa-specific patch applied to the chicken package
and instead passes TARGET_CFLAGS to the chicken build system.
Max Filippov (2):
Revert "package/chicken: move literals into text section for xtensa
architecture"
package/chicken: pass TARGET_CFLAGS to the build system
.../0001-xtensa-text-section-literals.patch | 30 -------------------
package/chicken/chicken.mk | 15 ++--------
2 files changed, 3 insertions(+), 42 deletions(-)
delete mode 100644 package/chicken/0001-xtensa-text-section-literals.patch
--
2.39.2
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture"
2024-08-11 0:00 [Buildroot] [PATCH 0/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
@ 2024-08-11 0:00 ` Max Filippov
2024-08-12 12:39 ` Woody Douglass via buildroot
2024-08-11 0:00 ` [Buildroot] [PATCH 2/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
2024-08-12 20:23 ` [Buildroot] [PATCH 0/2] " Thomas Petazzoni via buildroot
2 siblings, 1 reply; 7+ messages in thread
From: Max Filippov @ 2024-08-11 0:00 UTC (permalink / raw)
To: buildroot; +Cc: Woodrow Douglass, Thomas Petazzoni
This reverts commit 86e51fd5fe2810bbf833b48c4914dcd2de01bdbe.
Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
---
.../0001-xtensa-text-section-literals.patch | 30 -------------------
package/chicken/chicken.mk | 14 ++-------
2 files changed, 2 insertions(+), 42 deletions(-)
delete mode 100644 package/chicken/0001-xtensa-text-section-literals.patch
diff --git a/package/chicken/0001-xtensa-text-section-literals.patch b/package/chicken/0001-xtensa-text-section-literals.patch
deleted file mode 100644
index b93aaf9e218d..000000000000
--- a/package/chicken/0001-xtensa-text-section-literals.patch
+++ /dev/null
@@ -1,30 +0,0 @@
-From e8d177f4d4e1c6bc41787ea1621e323ff4ad493e Mon Sep 17 00:00:00 2001
-From: Woodrow Douglass <wdouglass@carnegierobotics.com>
-Date: Mon, 22 Jul 2024 08:50:37 -0400
-Subject: [PATCH] Move literals into text section for xtensa architecture
-
-Upstream: https://bugs.call-cc.org/ticket/1840
-Signed-off-by: Woodrow Douglass <wdouglass@carnegierobotics.com>
----
- Makefile.linux | 5 +++++
- 1 file changed, 5 insertions(+)
-
-diff --git a/Makefile.linux b/Makefile.linux
-index b628cda4..e97c2b94 100644
---- a/Makefile.linux
-+++ b/Makefile.linux
-@@ -52,6 +52,11 @@ USES_SONAME = yes
-
- # architectures
-
-+ifeq ($(ARCH),xtensa)
-+C_COMPILER_OPTIONS += -mtext-section-literals
-+LINKER_OPTIONS += -mtext-section-literals
-+endif
-+
- ifeq ($(ARCH),x32)
- C_COMPILER_OPTIONS += -mx32
- LINKER_OPTIONS += -mx32
---
-2.39.2
-
diff --git a/package/chicken/chicken.mk b/package/chicken/chicken.mk
index 0520280134ab..2667f7823d8a 100644
--- a/package/chicken/chicken.mk
+++ b/package/chicken/chicken.mk
@@ -11,19 +11,9 @@ CHICKEN_LICENSE_FILES = LICENSE
CHICKEN_CPE_ID_VENDOR = call-cc
CHICKEN_INSTALL_STAGING = YES
-# Chicken only uses the "arch" variable for some special-case compile
-# arguments If it's empty, it tries to detect the arch host Filter out
-# values that have an effect, or pass "unused" here
-ifeq ($(NORMALIZED_ARCH),x86_64)
-CHICKEN_ARCH = x86-64
-else ifeq ($(NORMALIZED_ARCH),xtensa)
-CHICKEN_ARCH = xtensa
-else
-CHICKEN_ARCH = unused
-endif
-
+# If ARCH is not set, it attempts to autodiscover. But it is anyway not used.
CHICKEN_MAKE_OPTS = \
- ARCH="$(CHICKEN_ARCH)" \
+ ARCH=unused \
C_COMPILER="$(TARGET_CC)" \
CXX_COMPILER="$(TARGET_CXX)" \
PREFIX=/usr \
--
2.39.2
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH 2/2] package/chicken: pass TARGET_CFLAGS to the build system
2024-08-11 0:00 [Buildroot] [PATCH 0/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
2024-08-11 0:00 ` [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture" Max Filippov
@ 2024-08-11 0:00 ` Max Filippov
2024-08-12 20:23 ` [Buildroot] [PATCH 0/2] " Thomas Petazzoni via buildroot
2 siblings, 0 replies; 7+ messages in thread
From: Max Filippov @ 2024-08-11 0:00 UTC (permalink / raw)
To: buildroot; +Cc: Woodrow Douglass, Thomas Petazzoni
chicken build system expects non-standard environment variables with
compiler names and compiler options. Pass buildroot TARGET_CFLAGS in
chicken C_COMPILER_OPTIMIZATION_OPTIONS.
Fixes: http://autobuild.buildroot.net/results/ce36170d67c4702f34bee025d8a46e5269c4573e/
Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
---
package/chicken/chicken.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/chicken/chicken.mk b/package/chicken/chicken.mk
index 2667f7823d8a..0000fc635ed5 100644
--- a/package/chicken/chicken.mk
+++ b/package/chicken/chicken.mk
@@ -16,6 +16,7 @@ CHICKEN_MAKE_OPTS = \
ARCH=unused \
C_COMPILER="$(TARGET_CC)" \
CXX_COMPILER="$(TARGET_CXX)" \
+ C_COMPILER_OPTIMIZATION_OPTIONS="$(TARGET_CFLAGS)" \
PREFIX=/usr \
PLATFORM=linux \
LIBRARIAN="$(TARGET_AR)"
--
2.39.2
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture"
2024-08-11 0:00 ` [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture" Max Filippov
@ 2024-08-12 12:39 ` Woody Douglass via buildroot
2024-08-12 18:29 ` Max Filippov
0 siblings, 1 reply; 7+ messages in thread
From: Woody Douglass via buildroot @ 2024-08-12 12:39 UTC (permalink / raw)
To: Max Filippov, buildroot@buildroot.org; +Cc: Thomas Petazzoni
On 8/10/24 20:00, Max Filippov wrote:
> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
>
>
> This reverts commit 86e51fd5fe2810bbf833b48c4914dcd2de01bdbe.
> Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
> ---
> .../0001-xtensa-text-section-literals.patch | 30 -------------------
> package/chicken/chicken.mk | 14 ++-------
> 2 files changed, 2 insertions(+), 42 deletions(-)
> delete mode 100644 package/chicken/0001-xtensa-text-section-literals.patch
>
> diff --git a/package/chicken/0001-xtensa-text-section-literals.patch b/package/chicken/0001-xtensa-text-section-literals.patch
> deleted file mode 100644
> index b93aaf9e218d..000000000000
> --- a/package/chicken/0001-xtensa-text-section-literals.patch
> +++ /dev/null
> @@ -1,30 +0,0 @@
> -From e8d177f4d4e1c6bc41787ea1621e323ff4ad493e Mon Sep 17 00:00:00 2001
> -From: Woodrow Douglass <wdouglass@carnegierobotics.com>
> -Date: Mon, 22 Jul 2024 08:50:37 -0400
> -Subject: [PATCH] Move literals into text section for xtensa architecture
> -
> -Upstream: https://bugs.call-cc.org/ticket/1840
> -Signed-off-by: Woodrow Douglass <wdouglass@carnegierobotics.com>
> ----
> - Makefile.linux | 5 +++++
> - 1 file changed, 5 insertions(+)
> -
> -diff --git a/Makefile.linux b/Makefile.linux
> -index b628cda4..e97c2b94 100644
> ---- a/Makefile.linux
> -+++ b/Makefile.linux
> -@@ -52,6 +52,11 @@ USES_SONAME = yes
> -
> - # architectures
> -
> -+ifeq ($(ARCH),xtensa)
> -+C_COMPILER_OPTIONS += -mtext-section-literals
> -+LINKER_OPTIONS += -mtext-section-literals
> -+endif
> -+
> - ifeq ($(ARCH),x32)
> - C_COMPILER_OPTIONS += -mx32
> - LINKER_OPTIONS += -mx32
> ---
> -2.39.2
> -
> diff --git a/package/chicken/chicken.mk b/package/chicken/chicken.mk
> index 0520280134ab..2667f7823d8a 100644
> --- a/package/chicken/chicken.mk
> +++ b/package/chicken/chicken.mk
> @@ -11,19 +11,9 @@ CHICKEN_LICENSE_FILES = LICENSE
> CHICKEN_CPE_ID_VENDOR = call-cc
> CHICKEN_INSTALL_STAGING = YES
>
> -# Chicken only uses the "arch" variable for some special-case compile
> -# arguments If it's empty, it tries to detect the arch host Filter out
> -# values that have an effect, or pass "unused" here
> -ifeq ($(NORMALIZED_ARCH),x86_64)
> -CHICKEN_ARCH = x86-64
changing the spelling to `x86-64` here is a good idea i think, because
chicken does add some flags. Other then that, i agree that the
C_COMPILER_OPTIMIZATION_OPTIONS approach is better here
> -else ifeq ($(NORMALIZED_ARCH),xtensa)
> -CHICKEN_ARCH = xtensa
> -else
> -CHICKEN_ARCH = unused
> -endif
> -
> +# If ARCH is not set, it attempts to autodiscover. But it is anyway not used.
> CHICKEN_MAKE_OPTS = \
> - ARCH="$(CHICKEN_ARCH)" \
> + ARCH=unused \
> C_COMPILER="$(TARGET_CC)" \
> CXX_COMPILER="$(TARGET_CXX)" \
> PREFIX=/usr \
> --
> 2.39.2
>
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture"
2024-08-12 12:39 ` Woody Douglass via buildroot
@ 2024-08-12 18:29 ` Max Filippov
2024-08-12 18:53 ` Woody Douglass via buildroot
0 siblings, 1 reply; 7+ messages in thread
From: Max Filippov @ 2024-08-12 18:29 UTC (permalink / raw)
To: Woody Douglass; +Cc: Thomas Petazzoni, buildroot@buildroot.org
On Mon, Aug 12, 2024 at 5:39 AM Woody Douglass
<wdouglass@carnegierobotics.com> wrote:
> On 8/10/24 20:00, Max Filippov wrote:
> > CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
> >
> >
> > This reverts commit 86e51fd5fe2810bbf833b48c4914dcd2de01bdbe.
> > Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
> > ---
> > .../0001-xtensa-text-section-literals.patch | 30 -------------------
> > package/chicken/chicken.mk | 14 ++-------
> > 2 files changed, 2 insertions(+), 42 deletions(-)
> > delete mode 100644 package/chicken/0001-xtensa-text-section-literals.patch
> >
> > diff --git a/package/chicken/0001-xtensa-text-section-literals.patch b/package/chicken/0001-xtensa-text-section-literals.patch
> > deleted file mode 100644
> > index b93aaf9e218d..000000000000
> > --- a/package/chicken/0001-xtensa-text-section-literals.patch
> > +++ /dev/null
> > @@ -1,30 +0,0 @@
> > -From e8d177f4d4e1c6bc41787ea1621e323ff4ad493e Mon Sep 17 00:00:00 2001
> > -From: Woodrow Douglass <wdouglass@carnegierobotics.com>
> > -Date: Mon, 22 Jul 2024 08:50:37 -0400
> > -Subject: [PATCH] Move literals into text section for xtensa architecture
> > -
> > -Upstream: https://bugs.call-cc.org/ticket/1840
> > -Signed-off-by: Woodrow Douglass <wdouglass@carnegierobotics.com>
> > ----
> > - Makefile.linux | 5 +++++
> > - 1 file changed, 5 insertions(+)
> > -
> > -diff --git a/Makefile.linux b/Makefile.linux
> > -index b628cda4..e97c2b94 100644
> > ---- a/Makefile.linux
> > -+++ b/Makefile.linux
> > -@@ -52,6 +52,11 @@ USES_SONAME = yes
> > -
> > - # architectures
> > -
> > -+ifeq ($(ARCH),xtensa)
> > -+C_COMPILER_OPTIONS += -mtext-section-literals
> > -+LINKER_OPTIONS += -mtext-section-literals
> > -+endif
> > -+
> > - ifeq ($(ARCH),x32)
> > - C_COMPILER_OPTIONS += -mx32
> > - LINKER_OPTIONS += -mx32
> > ---
> > -2.39.2
> > -
> > diff --git a/package/chicken/chicken.mk b/package/chicken/chicken.mk
> > index 0520280134ab..2667f7823d8a 100644
> > --- a/package/chicken/chicken.mk
> > +++ b/package/chicken/chicken.mk
> > @@ -11,19 +11,9 @@ CHICKEN_LICENSE_FILES = LICENSE
> > CHICKEN_CPE_ID_VENDOR = call-cc
> > CHICKEN_INSTALL_STAGING = YES
> >
> > -# Chicken only uses the "arch" variable for some special-case compile
> > -# arguments If it's empty, it tries to detect the arch host Filter out
> > -# values that have an effect, or pass "unused" here
> > -ifeq ($(NORMALIZED_ARCH),x86_64)
> > -CHICKEN_ARCH = x86-64
>
> changing the spelling to `x86-64` here is a good idea i think, because
> chicken does add some flags. Other then that, i agree that the
> C_COMPILER_OPTIMIZATION_OPTIONS approach is better here
I believe that it should be done in a separate change as it has
nothing to do with the topic of 'move literals into text section for
xtensa architecture'.
--
Thanks.
-- Max
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture"
2024-08-12 18:29 ` Max Filippov
@ 2024-08-12 18:53 ` Woody Douglass via buildroot
0 siblings, 0 replies; 7+ messages in thread
From: Woody Douglass via buildroot @ 2024-08-12 18:53 UTC (permalink / raw)
To: Max Filippov; +Cc: Thomas Petazzoni, buildroot@buildroot.org
On 8/12/24 14:29, Max Filippov wrote:
> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
>
>
> On Mon, Aug 12, 2024 at 5:39 AM Woody Douglass
> <wdouglass@carnegierobotics.com> wrote:
>> On 8/10/24 20:00, Max Filippov wrote:
>>> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender.
>>>
>>>
>>> This reverts commit 86e51fd5fe2810bbf833b48c4914dcd2de01bdbe.
>>> Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
>>> ---
>>> .../0001-xtensa-text-section-literals.patch | 30 -------------------
>>> package/chicken/chicken.mk | 14 ++-------
>>> 2 files changed, 2 insertions(+), 42 deletions(-)
>>> delete mode 100644 package/chicken/0001-xtensa-text-section-literals.patch
>>>
>>> diff --git a/package/chicken/0001-xtensa-text-section-literals.patch b/package/chicken/0001-xtensa-text-section-literals.patch
>>> deleted file mode 100644
>>> index b93aaf9e218d..000000000000
>>> --- a/package/chicken/0001-xtensa-text-section-literals.patch
>>> +++ /dev/null
>>> @@ -1,30 +0,0 @@
>>> -From e8d177f4d4e1c6bc41787ea1621e323ff4ad493e Mon Sep 17 00:00:00 2001
>>> -From: Woodrow Douglass <wdouglass@carnegierobotics.com>
>>> -Date: Mon, 22 Jul 2024 08:50:37 -0400
>>> -Subject: [PATCH] Move literals into text section for xtensa architecture
>>> -
>>> -Upstream: https://bugs.call-cc.org/ticket/1840
>>> -Signed-off-by: Woodrow Douglass <wdouglass@carnegierobotics.com>
>>> ----
>>> - Makefile.linux | 5 +++++
>>> - 1 file changed, 5 insertions(+)
>>> -
>>> -diff --git a/Makefile.linux b/Makefile.linux
>>> -index b628cda4..e97c2b94 100644
>>> ---- a/Makefile.linux
>>> -+++ b/Makefile.linux
>>> -@@ -52,6 +52,11 @@ USES_SONAME = yes
>>> -
>>> - # architectures
>>> -
>>> -+ifeq ($(ARCH),xtensa)
>>> -+C_COMPILER_OPTIONS += -mtext-section-literals
>>> -+LINKER_OPTIONS += -mtext-section-literals
>>> -+endif
>>> -+
>>> - ifeq ($(ARCH),x32)
>>> - C_COMPILER_OPTIONS += -mx32
>>> - LINKER_OPTIONS += -mx32
>>> ---
>>> -2.39.2
>>> -
>>> diff --git a/package/chicken/chicken.mk b/package/chicken/chicken.mk
>>> index 0520280134ab..2667f7823d8a 100644
>>> --- a/package/chicken/chicken.mk
>>> +++ b/package/chicken/chicken.mk
>>> @@ -11,19 +11,9 @@ CHICKEN_LICENSE_FILES = LICENSE
>>> CHICKEN_CPE_ID_VENDOR = call-cc
>>> CHICKEN_INSTALL_STAGING = YES
>>>
>>> -# Chicken only uses the "arch" variable for some special-case compile
>>> -# arguments If it's empty, it tries to detect the arch host Filter out
>>> -# values that have an effect, or pass "unused" here
>>> -ifeq ($(NORMALIZED_ARCH),x86_64)
>>> -CHICKEN_ARCH = x86-64
>> changing the spelling to `x86-64` here is a good idea i think, because
>> chicken does add some flags. Other then that, i agree that the
>> C_COMPILER_OPTIMIZATION_OPTIONS approach is better here
> I believe that it should be done in a separate change as it has
> nothing to do with the topic of 'move literals into text section for
> xtensa architecture'.
>
> --
> Thanks.
> -- Max
Sorry, this email was sent by mistake -- the `git send-email` workflow
is very new for me.
-Woody Douglass
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 0/2] package/chicken: pass TARGET_CFLAGS to the build system
2024-08-11 0:00 [Buildroot] [PATCH 0/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
2024-08-11 0:00 ` [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture" Max Filippov
2024-08-11 0:00 ` [Buildroot] [PATCH 2/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
@ 2024-08-12 20:23 ` Thomas Petazzoni via buildroot
2 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni via buildroot @ 2024-08-12 20:23 UTC (permalink / raw)
To: Max Filippov; +Cc: Woodrow Douglass, buildroot
On Sat, 10 Aug 2024 17:00:27 -0700
Max Filippov <jcmvbkbc@gmail.com> wrote:
> Max Filippov (2):
> Revert "package/chicken: move literals into text section for xtensa
> architecture"
> package/chicken: pass TARGET_CFLAGS to the build system
Thanks, both applied to master!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2024-08-12 20:23 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-11 0:00 [Buildroot] [PATCH 0/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
2024-08-11 0:00 ` [Buildroot] [PATCH 1/2] Revert "package/chicken: move literals into text section for xtensa architecture" Max Filippov
2024-08-12 12:39 ` Woody Douglass via buildroot
2024-08-12 18:29 ` Max Filippov
2024-08-12 18:53 ` Woody Douglass via buildroot
2024-08-11 0:00 ` [Buildroot] [PATCH 2/2] package/chicken: pass TARGET_CFLAGS to the build system Max Filippov
2024-08-12 20:23 ` [Buildroot] [PATCH 0/2] " Thomas Petazzoni via buildroot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox