* [Buildroot] [PATCH 1/1] package/esp-hosted: disable -Werror
@ 2024-03-05 17:19 Fabrice Fontaine
2024-03-05 17:24 ` Giulio Benetti
2024-03-06 15:52 ` Peter Korsgaard
0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2024-03-05 17:19 UTC (permalink / raw)
To: buildroot; +Cc: Giulio Benetti, Fabrice Fontaine
Disable -Werror to fix the following build failure raised at least since
commit a382a7d55432918d2a1e47ae4cbe7ed50fa17a08:
In function ‘destroy_cmd_wq’,
inlined from ‘esp_commands_teardown’ at /home/autobuild/autobuild/instance-4/output-1/build/esp-hosted-ce3c50a33fa4bc562a1b6cbcee292c1ae0b0a404/esp_hosted_ng/host/esp_cmd.c:1467:2:
./include/linux/workqueue.h:639:9: error: call to ‘__warn_flushing_systemwide_wq’ declared with attribute warning: Please avoid flushing system-wide workqueues. [-Werror=attribute-warning]
639 | __warn_flushing_systemwide_wq(); \
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/autobuild/autobuild/instance-4/output-1/build/esp-hosted-ce3c50a33fa4bc562a1b6cbcee292c1ae0b0a404/esp_hosted_ng/host/esp_cmd.c:408:17: note: in expansion of macro ‘flush_scheduled_work’
408 | flush_scheduled_work();
| ^~~~~~~~~~~~~~~~~~~~
Fixes:
- http://autobuild.buildroot.org/results/6ac7a4601938d3296ed1657c06f8cdf433757d73
- http://autobuild.buildroot.org/results/7997cc8a67645a6e1cf4e24d172c6feae459dcfb
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/esp-hosted/esp-hosted.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/esp-hosted/esp-hosted.mk b/package/esp-hosted/esp-hosted.mk
index cc61e39b36..02de3dbc52 100644
--- a/package/esp-hosted/esp-hosted.mk
+++ b/package/esp-hosted/esp-hosted.mk
@@ -12,6 +12,7 @@ ESP_HOSTED_LICENSE_FILE = LICENSE
ESP_HOSTED_MODULE_SUBDIRS = esp_hosted_ng/host
define ESP_HOSTED_LINUX_CONFIG_FIXUPS
+ $(call KCONFIG_DISABLE_OPT,CONFIG_WERROR)
$(call KCONFIG_ENABLE_OPT,CONFIG_NET)
$(call KCONFIG_ENABLE_OPT,CONFIG_WIRELESS)
$(call KCONFIG_ENABLE_OPT,CONFIG_CFG80211)
--
2.43.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/esp-hosted: disable -Werror
2024-03-05 17:19 [Buildroot] [PATCH 1/1] package/esp-hosted: disable -Werror Fabrice Fontaine
@ 2024-03-05 17:24 ` Giulio Benetti
2024-03-06 15:52 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Giulio Benetti @ 2024-03-05 17:24 UTC (permalink / raw)
To: Fabrice Fontaine; +Cc: buildroot
Hello Fabrice,
> Il giorno 5 mar 2024, alle ore 18:19, Fabrice Fontaine <fontaine.fabrice@gmail.com> ha scritto:
>
> Disable -Werror to fix the following build failure raised at least since
> commit a382a7d55432918d2a1e47ae4cbe7ed50fa17a08:
>
> In function ‘destroy_cmd_wq’,
> inlined from ‘esp_commands_teardown’ at /home/autobuild/autobuild/instance-4/output-1/build/esp-hosted-ce3c50a33fa4bc562a1b6cbcee292c1ae0b0a404/esp_hosted_ng/host/esp_cmd.c:1467:2:
> ./include/linux/workqueue.h:639:9: error: call to ‘__warn_flushing_systemwide_wq’ declared with attribute warning: Please avoid flushing system-wide workqueues. [-Werror=attribute-warning]
> 639 | __warn_flushing_systemwide_wq(); \
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> /home/autobuild/autobuild/instance-4/output-1/build/esp-hosted-ce3c50a33fa4bc562a1b6cbcee292c1ae0b0a404/esp_hosted_ng/host/esp_cmd.c:408:17: note: in expansion of macro ‘flush_scheduled_work’
> 408 | flush_scheduled_work();
> | ^~~~~~~~~~~~~~~~~~~~
>
> Fixes:
> - http://autobuild.buildroot.org/results/6ac7a4601938d3296ed1657c06f8cdf433757d73
> - http://autobuild.buildroot.org/results/7997cc8a67645a6e1cf4e24d172c6feae459dcfb
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Thank you for the patch.
Reviewed-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
Best regards
Giulio
> ---
> package/esp-hosted/esp-hosted.mk | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/esp-hosted/esp-hosted.mk b/package/esp-hosted/esp-hosted.mk
> index cc61e39b36..02de3dbc52 100644
> --- a/package/esp-hosted/esp-hosted.mk
> +++ b/package/esp-hosted/esp-hosted.mk
> @@ -12,6 +12,7 @@ ESP_HOSTED_LICENSE_FILE = LICENSE
> ESP_HOSTED_MODULE_SUBDIRS = esp_hosted_ng/host
>
> define ESP_HOSTED_LINUX_CONFIG_FIXUPS
> + $(call KCONFIG_DISABLE_OPT,CONFIG_WERROR)
> $(call KCONFIG_ENABLE_OPT,CONFIG_NET)
> $(call KCONFIG_ENABLE_OPT,CONFIG_WIRELESS)
> $(call KCONFIG_ENABLE_OPT,CONFIG_CFG80211)
> --
> 2.43.0
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/esp-hosted: disable -Werror
2024-03-05 17:19 [Buildroot] [PATCH 1/1] package/esp-hosted: disable -Werror Fabrice Fontaine
2024-03-05 17:24 ` Giulio Benetti
@ 2024-03-06 15:52 ` Peter Korsgaard
1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2024-03-06 15:52 UTC (permalink / raw)
To: Fabrice Fontaine; +Cc: Giulio Benetti, buildroot
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
> Disable -Werror to fix the following build failure raised at least since
> commit a382a7d55432918d2a1e47ae4cbe7ed50fa17a08:
> In function ‘destroy_cmd_wq’,
> inlined from ‘esp_commands_teardown’ at /home/autobuild/autobuild/instance-4/output-1/build/esp-hosted-ce3c50a33fa4bc562a1b6cbcee292c1ae0b0a404/esp_hosted_ng/host/esp_cmd.c:1467:2:
> ./include/linux/workqueue.h:639:9: error: call to ‘__warn_flushing_systemwide_wq’ declared with attribute warning: Please avoid flushing system-wide workqueues. [-Werror=attribute-warning]
> 639 | __warn_flushing_systemwide_wq(); \
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> /home/autobuild/autobuild/instance-4/output-1/build/esp-hosted-ce3c50a33fa4bc562a1b6cbcee292c1ae0b0a404/esp_hosted_ng/host/esp_cmd.c:408:17:
> note: in expansion of macro ‘flush_scheduled_work’
> 408 | flush_scheduled_work();
> | ^~~~~~~~~~~~~~~~~~~~
> Fixes:
> - http://autobuild.buildroot.org/results/6ac7a4601938d3296ed1657c06f8cdf433757d73
> - http://autobuild.buildroot.org/results/7997cc8a67645a6e1cf4e24d172c6feae459dcfb
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> package/esp-hosted/esp-hosted.mk | 1 +
> 1 file changed, 1 insertion(+)
> diff --git a/package/esp-hosted/esp-hosted.mk b/package/esp-hosted/esp-hosted.mk
> index cc61e39b36..02de3dbc52 100644
> --- a/package/esp-hosted/esp-hosted.mk
> +++ b/package/esp-hosted/esp-hosted.mk
> @@ -12,6 +12,7 @@ ESP_HOSTED_LICENSE_FILE = LICENSE
> ESP_HOSTED_MODULE_SUBDIRS = esp_hosted_ng/host
> define ESP_HOSTED_LINUX_CONFIG_FIXUPS
> + $(call KCONFIG_DISABLE_OPT,CONFIG_WERROR)
I wonder if we shouldn't unconditionally disable CONFIG_WERROR in
linux.mk, similar to how we do for a number of other packages - The
Linux build could fail with warnings if a too old/new toolchain is used.
I doubt a lot of real defconfigs want to use CONFIG_WERROR, the problem
is most likely only for the arch defconfigs:
git grep CONFIG_WERROR=y
arch/x86/configs/i386_defconfig:CONFIG_WERROR=y
arch/x86/configs/x86_64_defconfig:CONFIG_WERROR=y
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-03-06 15:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-05 17:19 [Buildroot] [PATCH 1/1] package/esp-hosted: disable -Werror Fabrice Fontaine
2024-03-05 17:24 ` Giulio Benetti
2024-03-06 15:52 ` Peter Korsgaard
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.