* [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h>
@ 2022-11-26 5:09 Thomas Weißschuh
2022-11-26 5:10 ` [PATCH 2/3] powerpc/book3e: " Thomas Weißschuh
` (4 more replies)
0 siblings, 5 replies; 15+ messages in thread
From: Thomas Weißschuh @ 2022-11-26 5:09 UTC (permalink / raw)
To: Luis Chamberlain, Russ Weight, Greg Kroah-Hartman,
Rafael J. Wysocki, linuxppc-dev
Cc: Masahiro Yamada, Thomas Weißschuh, Nicholas Piggin, linux-kernel
utsrelease.h is potentially generated on each build.
By removing this unused include we can get rid of some spurious
recompilations.
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
drivers/base/firmware_loader/firmware.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/base/firmware_loader/firmware.h b/drivers/base/firmware_loader/firmware.h
index fe77e91c38a2..bf549d6500d7 100644
--- a/drivers/base/firmware_loader/firmware.h
+++ b/drivers/base/firmware_loader/firmware.h
@@ -9,8 +9,6 @@
#include <linux/list.h>
#include <linux/completion.h>
-#include <generated/utsrelease.h>
-
/**
* enum fw_opt - options to control firmware loading behaviour
*
base-commit: 0b1dcc2cf55ae6523c6fbd0d741b3ac28c9f4536
--
2.38.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-26 5:09 [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Thomas Weißschuh
@ 2022-11-26 5:10 ` Thomas Weißschuh
2022-11-26 7:36 ` Christophe Leroy
2022-11-26 22:17 ` Masahiro Yamada
2022-11-26 5:10 ` [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h> Thomas Weißschuh
` (3 subsequent siblings)
4 siblings, 2 replies; 15+ messages in thread
From: Thomas Weißschuh @ 2022-11-26 5:10 UTC (permalink / raw)
To: Luis Chamberlain, Russ Weight, Greg Kroah-Hartman,
Rafael J. Wysocki, linuxppc-dev
Cc: Masahiro Yamada, Thomas Weißschuh, Nicholas Piggin, linux-kernel
Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
removed the usage of the define UTS_VERSION but forgot to drop the
include.
Fixes: 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
arch/powerpc/mm/nohash/kaslr_booke.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/powerpc/mm/nohash/kaslr_booke.c b/arch/powerpc/mm/nohash/kaslr_booke.c
index 0d04f9d5da8d..2fb3edafe9ab 100644
--- a/arch/powerpc/mm/nohash/kaslr_booke.c
+++ b/arch/powerpc/mm/nohash/kaslr_booke.c
@@ -19,7 +19,6 @@
#include <asm/cacheflush.h>
#include <asm/kdump.h>
#include <mm/mmu_decl.h>
-#include <generated/utsrelease.h>
struct regions {
unsigned long pa_start;
--
2.38.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h>
2022-11-26 5:09 [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Thomas Weißschuh
2022-11-26 5:10 ` [PATCH 2/3] powerpc/book3e: " Thomas Weißschuh
@ 2022-11-26 5:10 ` Thomas Weißschuh
2022-11-26 22:18 ` Masahiro Yamada
2022-11-26 22:13 ` [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Masahiro Yamada
` (2 subsequent siblings)
4 siblings, 1 reply; 15+ messages in thread
From: Thomas Weißschuh @ 2022-11-26 5:10 UTC (permalink / raw)
To: Luis Chamberlain, Russ Weight, Greg Kroah-Hartman,
Rafael J. Wysocki, linuxppc-dev
Cc: Masahiro Yamada, Thomas Weißschuh, Nicholas Piggin, linux-kernel
Commit 2df8220cc511 ("kbuild: build init/built-in.a just once") moved
the usage of the define UTS_VERSION to the file version-timestamp.c.
version-timestamp.c in turn is included from version.c but already
includes utsversion.h itself properly.
The unneeded include of utsversion.h from version.c can be dropped.
Fixes: 2df8220cc511 ("kbuild: build init/built-in.a just once")
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
init/version.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/init/version.c b/init/version.c
index 01d4ab05f0ba..f117921811b4 100644
--- a/init/version.c
+++ b/init/version.c
@@ -15,7 +15,6 @@
#include <linux/printk.h>
#include <linux/uts.h>
#include <linux/utsname.h>
-#include <generated/utsrelease.h>
#include <linux/proc_ns.h>
static int __init early_hostname(char *arg)
--
2.38.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-26 5:10 ` [PATCH 2/3] powerpc/book3e: " Thomas Weißschuh
@ 2022-11-26 7:36 ` Christophe Leroy
2022-11-26 14:07 ` Thomas Weißschuh
2022-11-26 22:17 ` Masahiro Yamada
1 sibling, 1 reply; 15+ messages in thread
From: Christophe Leroy @ 2022-11-26 7:36 UTC (permalink / raw)
To: Thomas Weißschuh, Luis Chamberlain, Russ Weight,
Greg Kroah-Hartman, Rafael J. Wysocki, linuxppc-dev
Cc: linux-arch, Masahiro Yamada, linux-kernel, Nicholas Piggin
Le 26/11/2022 à 06:10, Thomas Weißschuh a écrit :
> Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
> removed the usage of the define UTS_VERSION but forgot to drop the
> include.
What about:
arch/powerpc/platforms/52xx/efika.c
arch/powerpc/platforms/amigaone/setup.c
arch/powerpc/platforms/chrp/setup.c
arch/powerpc/platforms/powermac/bootx_init.c
I believe you can do a lot more than what you did in your series.
List of files using UTS_VERSION :
$ git grep -l UTS_VERSION
Documentation/kbuild/kbuild.rst
arch/s390/boot/version.c
arch/x86/boot/compressed/kaslr.c
arch/x86/boot/version.c
init/Makefile
init/version-timestamp.c
List of files including generated/utsrelease.h :
$ git grep -l "#include <generated/utsrelease.h>"
Documentation/target/tcm_mod_builder.py
arch/alpha/boot/bootp.c
arch/alpha/boot/bootpz.c
arch/alpha/boot/main.c
arch/powerpc/mm/nohash/kaslr_booke.c
arch/powerpc/platforms/52xx/efika.c
arch/powerpc/platforms/amigaone/setup.c
arch/powerpc/platforms/chrp/setup.c
arch/powerpc/platforms/powermac/bootx_init.c
arch/s390/boot/version.c
arch/x86/boot/compressed/kaslr.c
arch/x86/boot/version.c
crypto/fips.c
drivers/auxdisplay/arm-charlcd.c
drivers/auxdisplay/charlcd.c
drivers/auxdisplay/line-display.c
drivers/base/firmware_loader/firmware.h
drivers/base/firmware_loader/main.c
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c
drivers/gpu/drm/msm/msm_gpu.c
drivers/gpu/drm/vmwgfx/vmwgfx_drv.c
drivers/net/bonding/bonding_priv.h
drivers/net/ethernet/intel/i40e/i40e_main.c
drivers/net/ethernet/intel/ice/ice_main.c
drivers/net/ethernet/intel/ixgbe/ixgbe_fcoe.c
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
drivers/net/ethernet/pensando/ionic/ionic_main.c
drivers/net/ethernet/rocker/rocker_main.c
drivers/net/team/team.c
drivers/nvme/target/admin-cmd.c
drivers/nvme/target/discovery.c
drivers/power/supply/test_power.c
drivers/staging/octeon/ethernet-mdio.c
drivers/target/target_core_configfs.c
drivers/target/tcm_fc/tfc_conf.c
drivers/vhost/scsi.c
drivers/xen/xen-scsiback.c
include/linux/vermagic.h
init/version-timestamp.c
init/version.c
kernel/sys.c
kernel/trace/trace.c
net/ethtool/ioctl.c
net/rxrpc/local_event.c
security/integrity/ima/ima_init.c
Christophe
>
> Fixes: 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
> arch/powerpc/mm/nohash/kaslr_booke.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/powerpc/mm/nohash/kaslr_booke.c b/arch/powerpc/mm/nohash/kaslr_booke.c
> index 0d04f9d5da8d..2fb3edafe9ab 100644
> --- a/arch/powerpc/mm/nohash/kaslr_booke.c
> +++ b/arch/powerpc/mm/nohash/kaslr_booke.c
> @@ -19,7 +19,6 @@
> #include <asm/cacheflush.h>
> #include <asm/kdump.h>
> #include <mm/mmu_decl.h>
> -#include <generated/utsrelease.h>
>
> struct regions {
> unsigned long pa_start;
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-26 7:36 ` Christophe Leroy
@ 2022-11-26 14:07 ` Thomas Weißschuh
2022-11-26 14:54 ` Christophe Leroy
2022-11-27 22:59 ` Michael Ellerman
0 siblings, 2 replies; 15+ messages in thread
From: Thomas Weißschuh @ 2022-11-26 14:07 UTC (permalink / raw)
To: Christophe Leroy
Cc: linux-arch, Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman,
Masahiro Yamada, linux-kernel, Nicholas Piggin, Luis Chamberlain,
linuxppc-dev
On 2022-11-26 07:36+0000, Christophe Leroy wrote:
> Le 26/11/2022 à 06:10, Thomas Weißschuh a écrit :
>> Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
>> removed the usage of the define UTS_VERSION but forgot to drop the
>> include.
>
> What about:
> arch/powerpc/platforms/52xx/efika.c
> arch/powerpc/platforms/amigaone/setup.c
> arch/powerpc/platforms/chrp/setup.c
> arch/powerpc/platforms/powermac/bootx_init.c
>
> I believe you can do a lot more than what you did in your series.
The commit messages are wrong.
They should have said UTS_RELEASE instead of UTS_VERSION.
Could the maintainers fix this up when applying?
I also changed it locally so it will be fixed for v2.
> List of files using UTS_VERSION :
>
> $ git grep -l UTS_VERSION
> [..]
Thomas
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-26 14:07 ` Thomas Weißschuh
@ 2022-11-26 14:54 ` Christophe Leroy
2022-11-27 22:59 ` Michael Ellerman
1 sibling, 0 replies; 15+ messages in thread
From: Christophe Leroy @ 2022-11-26 14:54 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: linux-arch, Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman,
Masahiro Yamada, linux-kernel, Nicholas Piggin, Luis Chamberlain,
linuxppc-dev
Le 26/11/2022 à 15:07, Thomas Weißschuh a écrit :
> On 2022-11-26 07:36+0000, Christophe Leroy wrote:
>> Le 26/11/2022 à 06:10, Thomas Weißschuh a écrit :
>>> Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
>>> removed the usage of the define UTS_VERSION but forgot to drop the
>>> include.
>>
>> What about:
>> arch/powerpc/platforms/52xx/efika.c
>> arch/powerpc/platforms/amigaone/setup.c
>> arch/powerpc/platforms/chrp/setup.c
>> arch/powerpc/platforms/powermac/bootx_init.c
>>
>> I believe you can do a lot more than what you did in your series.
>
> The commit messages are wrong.
> They should have said UTS_RELEASE instead of UTS_VERSION.
Ah, ok. So your series is complete then:
$ git grep -L UTS_RELEASE `git grep -l "#include <generated/utsrelease.h>"`
Documentation/target/tcm_mod_builder.py
arch/powerpc/mm/nohash/kaslr_booke.c
drivers/base/firmware_loader/firmware.h
init/version.c
Reviewed-by: Christophe Leroy <christophe.leroy@csgroup.eu>
>
> Could the maintainers fix this up when applying?
> I also changed it locally so it will be fixed for v2.
>
>> List of files using UTS_VERSION :
>>
>> $ git grep -l UTS_VERSION
>> [..]
>
> Thomas
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h>
2022-11-26 5:09 [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Thomas Weißschuh
2022-11-26 5:10 ` [PATCH 2/3] powerpc/book3e: " Thomas Weißschuh
2022-11-26 5:10 ` [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h> Thomas Weißschuh
@ 2022-11-26 22:13 ` Masahiro Yamada
2022-11-29 1:18 ` Russ Weight
2022-12-08 12:39 ` (subset) " Michael Ellerman
4 siblings, 0 replies; 15+ messages in thread
From: Masahiro Yamada @ 2022-11-26 22:13 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman, linux-kernel,
Nicholas Piggin, Luis Chamberlain, linuxppc-dev
On Sat, Nov 26, 2022 at 2:10 PM Thomas Weißschuh <linux@weissschuh.net> wrote:
>
> utsrelease.h is potentially generated on each build.
> By removing this unused include we can get rid of some spurious
> recompilations.
>
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
Reviewed-by: Masahiro Yamada <masahiroy@kernel.org>
> drivers/base/firmware_loader/firmware.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/base/firmware_loader/firmware.h b/drivers/base/firmware_loader/firmware.h
> index fe77e91c38a2..bf549d6500d7 100644
> --- a/drivers/base/firmware_loader/firmware.h
> +++ b/drivers/base/firmware_loader/firmware.h
> @@ -9,8 +9,6 @@
> #include <linux/list.h>
> #include <linux/completion.h>
>
> -#include <generated/utsrelease.h>
> -
> /**
> * enum fw_opt - options to control firmware loading behaviour
> *
>
> base-commit: 0b1dcc2cf55ae6523c6fbd0d741b3ac28c9f4536
> --
> 2.38.1
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-26 5:10 ` [PATCH 2/3] powerpc/book3e: " Thomas Weißschuh
2022-11-26 7:36 ` Christophe Leroy
@ 2022-11-26 22:17 ` Masahiro Yamada
1 sibling, 0 replies; 15+ messages in thread
From: Masahiro Yamada @ 2022-11-26 22:17 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman, linux-kernel,
Nicholas Piggin, Luis Chamberlain, linuxppc-dev
On Sat, Nov 26, 2022 at 2:10 PM Thomas Weißschuh <linux@weissschuh.net> wrote:
>
> Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
> removed the usage of the define UTS_VERSION but forgot to drop the
> include.
>
> Fixes: 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
Assuming you will fix the commit description,
Reviewed-by: Masahiro Yamada <masahiroy@kernel.org>
> arch/powerpc/mm/nohash/kaslr_booke.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/powerpc/mm/nohash/kaslr_booke.c b/arch/powerpc/mm/nohash/kaslr_booke.c
> index 0d04f9d5da8d..2fb3edafe9ab 100644
> --- a/arch/powerpc/mm/nohash/kaslr_booke.c
> +++ b/arch/powerpc/mm/nohash/kaslr_booke.c
> @@ -19,7 +19,6 @@
> #include <asm/cacheflush.h>
> #include <asm/kdump.h>
> #include <mm/mmu_decl.h>
> -#include <generated/utsrelease.h>
>
> struct regions {
> unsigned long pa_start;
> --
> 2.38.1
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h>
2022-11-26 5:10 ` [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h> Thomas Weißschuh
@ 2022-11-26 22:18 ` Masahiro Yamada
2022-11-29 9:33 ` Masahiro Yamada
0 siblings, 1 reply; 15+ messages in thread
From: Masahiro Yamada @ 2022-11-26 22:18 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman, linux-kernel,
Nicholas Piggin, Luis Chamberlain, linuxppc-dev
On Sat, Nov 26, 2022 at 2:10 PM Thomas Weißschuh <linux@weissschuh.net> wrote:
>
> Commit 2df8220cc511 ("kbuild: build init/built-in.a just once") moved
> the usage of the define UTS_VERSION to the file version-timestamp.c.
With s/UTS_VERSION/UTS_RELEASE/,
Reviewed-by: Masahiro Yamada <masahiroy@kernel.org>
>
> version-timestamp.c in turn is included from version.c but already
> includes utsversion.h itself properly.
>
> The unneeded include of utsversion.h from version.c can be dropped.
>
> Fixes: 2df8220cc511 ("kbuild: build init/built-in.a just once")
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
> init/version.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/init/version.c b/init/version.c
> index 01d4ab05f0ba..f117921811b4 100644
> --- a/init/version.c
> +++ b/init/version.c
> @@ -15,7 +15,6 @@
> #include <linux/printk.h>
> #include <linux/uts.h>
> #include <linux/utsname.h>
> -#include <generated/utsrelease.h>
> #include <linux/proc_ns.h>
>
> static int __init early_hostname(char *arg)
> --
> 2.38.1
>
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-26 14:07 ` Thomas Weißschuh
2022-11-26 14:54 ` Christophe Leroy
@ 2022-11-27 22:59 ` Michael Ellerman
2022-11-29 9:34 ` Masahiro Yamada
1 sibling, 1 reply; 15+ messages in thread
From: Michael Ellerman @ 2022-11-27 22:59 UTC (permalink / raw)
To: Thomas Weißschuh, Christophe Leroy
Cc: linux-arch, Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman,
Masahiro Yamada, linux-kernel, Nicholas Piggin, Luis Chamberlain,
linuxppc-dev
Thomas Weißschuh <linux@weissschuh.net> writes:
> On 2022-11-26 07:36+0000, Christophe Leroy wrote:
>> Le 26/11/2022 à 06:10, Thomas Weißschuh a écrit :
>>> Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
>>> removed the usage of the define UTS_VERSION but forgot to drop the
>>> include.
>>
>> What about:
>> arch/powerpc/platforms/52xx/efika.c
>> arch/powerpc/platforms/amigaone/setup.c
>> arch/powerpc/platforms/chrp/setup.c
>> arch/powerpc/platforms/powermac/bootx_init.c
>>
>> I believe you can do a lot more than what you did in your series.
>
> The commit messages are wrong.
> They should have said UTS_RELEASE instead of UTS_VERSION.
>
> Could the maintainers fix this up when applying?
> I also changed it locally so it will be fixed for v2.
I'll take this patch, but not the others.
cheers
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h>
2022-11-26 5:09 [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Thomas Weißschuh
` (2 preceding siblings ...)
2022-11-26 22:13 ` [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Masahiro Yamada
@ 2022-11-29 1:18 ` Russ Weight
2022-11-29 9:31 ` Masahiro Yamada
2022-12-08 12:39 ` (subset) " Michael Ellerman
4 siblings, 1 reply; 15+ messages in thread
From: Russ Weight @ 2022-11-29 1:18 UTC (permalink / raw)
To: Thomas Weißschuh, Luis Chamberlain, Greg Kroah-Hartman,
Rafael J. Wysocki, linuxppc-dev
Cc: Masahiro Yamada, linux-kernel, Nicholas Piggin
On 11/25/22 21:09, Thomas Weißschuh wrote:
> utsrelease.h is potentially generated on each build.
> By removing this unused include we can get rid of some spurious
> recompilations.
Reviewed-by: Russ Weight <russell.h.weight@intel.com>
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> ---
> drivers/base/firmware_loader/firmware.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/base/firmware_loader/firmware.h b/drivers/base/firmware_loader/firmware.h
> index fe77e91c38a2..bf549d6500d7 100644
> --- a/drivers/base/firmware_loader/firmware.h
> +++ b/drivers/base/firmware_loader/firmware.h
> @@ -9,8 +9,6 @@
> #include <linux/list.h>
> #include <linux/completion.h>
>
> -#include <generated/utsrelease.h>
> -
> /**
> * enum fw_opt - options to control firmware loading behaviour
> *
>
> base-commit: 0b1dcc2cf55ae6523c6fbd0d741b3ac28c9f4536
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h>
2022-11-29 1:18 ` Russ Weight
@ 2022-11-29 9:31 ` Masahiro Yamada
0 siblings, 0 replies; 15+ messages in thread
From: Masahiro Yamada @ 2022-11-29 9:31 UTC (permalink / raw)
To: Russ Weight
Cc: Rafael J. Wysocki, Greg Kroah-Hartman, Thomas Weißschuh,
Nicholas Piggin, linux-kernel, Luis Chamberlain, linuxppc-dev
On Tue, Nov 29, 2022 at 10:18 AM Russ Weight <russell.h.weight@intel.com> wrote:
>
>
>
> On 11/25/22 21:09, Thomas Weißschuh wrote:
> > utsrelease.h is potentially generated on each build.
> > By removing this unused include we can get rid of some spurious
> > recompilations.
> Reviewed-by: Russ Weight <russell.h.weight@intel.com>
> > Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> > ---
> > drivers/base/firmware_loader/firmware.h | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/drivers/base/firmware_loader/firmware.h b/drivers/base/firmware_loader/firmware.h
> > index fe77e91c38a2..bf549d6500d7 100644
> > --- a/drivers/base/firmware_loader/firmware.h
> > +++ b/drivers/base/firmware_loader/firmware.h
> > @@ -9,8 +9,6 @@
> > #include <linux/list.h>
> > #include <linux/completion.h>
> >
> > -#include <generated/utsrelease.h>
> > -
> > /**
> > * enum fw_opt - options to control firmware loading behaviour
> > *
> >
> > base-commit: 0b1dcc2cf55ae6523c6fbd0d741b3ac28c9f4536
>
Applied to linux-kbuild.
Thanks.
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h>
2022-11-26 22:18 ` Masahiro Yamada
@ 2022-11-29 9:33 ` Masahiro Yamada
0 siblings, 0 replies; 15+ messages in thread
From: Masahiro Yamada @ 2022-11-29 9:33 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman, linux-kernel,
Nicholas Piggin, Luis Chamberlain, linuxppc-dev
On Sun, Nov 27, 2022 at 7:18 AM Masahiro Yamada <masahiroy@kernel.org> wrote:
>
> On Sat, Nov 26, 2022 at 2:10 PM Thomas Weißschuh <linux@weissschuh.net> wrote:
> >
> > Commit 2df8220cc511 ("kbuild: build init/built-in.a just once") moved
> > the usage of the define UTS_VERSION to the file version-timestamp.c.
>
> With s/UTS_VERSION/UTS_RELEASE/,
>
>
> Reviewed-by: Masahiro Yamada <masahiroy@kernel.org>
>
> >
> > version-timestamp.c in turn is included from version.c but already
> > includes utsversion.h itself properly.
> >
> > The unneeded include of utsversion.h from version.c can be dropped.
> >
> > Fixes: 2df8220cc511 ("kbuild: build init/built-in.a just once")
> > Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
> > ---
> > init/version.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/init/version.c b/init/version.c
> > index 01d4ab05f0ba..f117921811b4 100644
> > --- a/init/version.c
> > +++ b/init/version.c
> > @@ -15,7 +15,6 @@
> > #include <linux/printk.h>
> > #include <linux/uts.h>
> > #include <linux/utsname.h>
> > -#include <generated/utsrelease.h>
> > #include <linux/proc_ns.h>
> >
> > static int __init early_hostname(char *arg)
> > --
> > 2.38.1
> >
>
>
> --
> Best Regards
> Masahiro Yamada
This patch is so confusing because the subject is also wrong.
I fixed up the commit description:
init/version.c: remove #include <generated/utsrelease.h>
Commit 2df8220cc511 ("kbuild: build init/built-in.a just once") moved
the usage of the define UTS_RELEASE to the file version-timestamp.c.
version-timestamp.c in turn is included from version.c but already
includes utsrelease.h itself properly.
The unneeded include of utsrelease.h from version.c can be dropped.
Fixes: 2df8220cc511 ("kbuild: build init/built-in.a just once")
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Applied to linux-kbuild.
Thanks.
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
2022-11-27 22:59 ` Michael Ellerman
@ 2022-11-29 9:34 ` Masahiro Yamada
0 siblings, 0 replies; 15+ messages in thread
From: Masahiro Yamada @ 2022-11-29 9:34 UTC (permalink / raw)
To: Michael Ellerman
Cc: linux-arch, Russ Weight, Rafael J. Wysocki, Greg Kroah-Hartman,
Thomas Weißschuh, linux-kernel, Luis Chamberlain,
Nicholas Piggin, linuxppc-dev
On Mon, Nov 28, 2022 at 7:59 AM Michael Ellerman <mpe@ellerman.id.au> wrote:
>
> Thomas Weißschuh <linux@weissschuh.net> writes:
> > On 2022-11-26 07:36+0000, Christophe Leroy wrote:
> >> Le 26/11/2022 à 06:10, Thomas Weißschuh a écrit :
> >>> Commit 7ad4bd887d27 ("powerpc/book3e: get rid of #include <generated/compile.h>")
> >>> removed the usage of the define UTS_VERSION but forgot to drop the
> >>> include.
> >>
> >> What about:
> >> arch/powerpc/platforms/52xx/efika.c
> >> arch/powerpc/platforms/amigaone/setup.c
> >> arch/powerpc/platforms/chrp/setup.c
> >> arch/powerpc/platforms/powermac/bootx_init.c
> >>
> >> I believe you can do a lot more than what you did in your series.
> >
> > The commit messages are wrong.
> > They should have said UTS_RELEASE instead of UTS_VERSION.
> >
> > Could the maintainers fix this up when applying?
> > I also changed it locally so it will be fixed for v2.
>
> I'll take this patch, but not the others.
>
> cheers
Okay, I applied 1/3 and 3/3 to the kbuild tree.
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: (subset) [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h>
2022-11-26 5:09 [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Thomas Weißschuh
` (3 preceding siblings ...)
2022-11-29 1:18 ` Russ Weight
@ 2022-12-08 12:39 ` Michael Ellerman
4 siblings, 0 replies; 15+ messages in thread
From: Michael Ellerman @ 2022-12-08 12:39 UTC (permalink / raw)
To: Thomas Weißschuh, Rafael J. Wysocki, Greg Kroah-Hartman,
linuxppc-dev, Russ Weight, Luis Chamberlain
Cc: Masahiro Yamada, linux-kernel, Nicholas Piggin
On Sat, 26 Nov 2022 06:09:59 +0100, Thomas Weißschuh wrote:
> utsrelease.h is potentially generated on each build.
> By removing this unused include we can get rid of some spurious
> recompilations.
>
>
Applied to powerpc/next.
[2/3] powerpc/book3e: remove #include <generated/utsrelease.h>
https://git.kernel.org/powerpc/c/d5090716be6791ada9ee142163a4934c1c147aaa
cheers
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2022-12-08 12:57 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-26 5:09 [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Thomas Weißschuh
2022-11-26 5:10 ` [PATCH 2/3] powerpc/book3e: " Thomas Weißschuh
2022-11-26 7:36 ` Christophe Leroy
2022-11-26 14:07 ` Thomas Weißschuh
2022-11-26 14:54 ` Christophe Leroy
2022-11-27 22:59 ` Michael Ellerman
2022-11-29 9:34 ` Masahiro Yamada
2022-11-26 22:17 ` Masahiro Yamada
2022-11-26 5:10 ` [PATCH 3/3] init/version.c: remove #include <generated/utsversion.h> Thomas Weißschuh
2022-11-26 22:18 ` Masahiro Yamada
2022-11-29 9:33 ` Masahiro Yamada
2022-11-26 22:13 ` [PATCH 1/3] firmware_loader: remove #include <generated/utsrelease.h> Masahiro Yamada
2022-11-29 1:18 ` Russ Weight
2022-11-29 9:31 ` Masahiro Yamada
2022-12-08 12:39 ` (subset) " Michael Ellerman
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).