* [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD
@ 2025-09-15 7:11 Geert Uytterhoeven
2025-09-15 8:14 ` David Disseldorp
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Geert Uytterhoeven @ 2025-09-15 7:11 UTC (permalink / raw)
To: Andrew Morton, David Disseldorp, Martin Wilck, Alexander Viro,
Christian Brauner, Jan Kara
Cc: Askar Safin, linux-fsdevel, linux-kernel, Geert Uytterhoeven
INITRAMFS_PRESERVE_MTIME is only used in init/initramfs.c and
init/initramfs_test.c. Hence add a dependency on BLK_DEV_INITRD, to
prevent asking the user about this feature when configuring a kernel
without initramfs support.
Fixes: 1274aea127b2e8c9 ("initramfs: add INITRAMFS_PRESERVE_MTIME Kconfig option")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
init/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/init/Kconfig b/init/Kconfig
index e3eb63eadc8757a1..c0c61206499e6bd5 100644
--- a/init/Kconfig
+++ b/init/Kconfig
@@ -1501,6 +1501,7 @@ config BOOT_CONFIG_EMBED_FILE
config INITRAMFS_PRESERVE_MTIME
bool "Preserve cpio archive mtimes in initramfs"
+ depends on BLK_DEV_INITRD
default y
help
Each entry in an initramfs cpio archive carries an mtime value. When
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD
2025-09-15 7:11 [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD Geert Uytterhoeven
@ 2025-09-15 8:14 ` David Disseldorp
2025-09-15 11:39 ` Martin Wilck
2025-09-15 13:02 ` Christian Brauner
2 siblings, 0 replies; 4+ messages in thread
From: David Disseldorp @ 2025-09-15 8:14 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Andrew Morton, Martin Wilck, Alexander Viro, Christian Brauner,
Jan Kara, Askar Safin, linux-fsdevel, linux-kernel
On Mon, 15 Sep 2025 09:11:05 +0200, Geert Uytterhoeven wrote:
> INITRAMFS_PRESERVE_MTIME is only used in init/initramfs.c and
> init/initramfs_test.c. Hence add a dependency on BLK_DEV_INITRD, to
> prevent asking the user about this feature when configuring a kernel
> without initramfs support.
>
> Fixes: 1274aea127b2e8c9 ("initramfs: add INITRAMFS_PRESERVE_MTIME Kconfig option")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Makes sense, thanks.
Reviewed-by: David Disseldorp <ddiss@suse.de>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD
2025-09-15 7:11 [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD Geert Uytterhoeven
2025-09-15 8:14 ` David Disseldorp
@ 2025-09-15 11:39 ` Martin Wilck
2025-09-15 13:02 ` Christian Brauner
2 siblings, 0 replies; 4+ messages in thread
From: Martin Wilck @ 2025-09-15 11:39 UTC (permalink / raw)
To: Geert Uytterhoeven, Andrew Morton, David Disseldorp,
Alexander Viro, Christian Brauner, Jan Kara
Cc: Askar Safin, linux-fsdevel, linux-kernel
On Mon, 2025-09-15 at 09:11 +0200, Geert Uytterhoeven wrote:
> INITRAMFS_PRESERVE_MTIME is only used in init/initramfs.c and
> init/initramfs_test.c. Hence add a dependency on BLK_DEV_INITRD, to
> prevent asking the user about this feature when configuring a kernel
> without initramfs support.
>
> Fixes: 1274aea127b2e8c9 ("initramfs: add INITRAMFS_PRESERVE_MTIME
> Kconfig option")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Martin Wilck <mwilck@suse.com>
> ---
> init/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/init/Kconfig b/init/Kconfig
> index e3eb63eadc8757a1..c0c61206499e6bd5 100644
> --- a/init/Kconfig
> +++ b/init/Kconfig
> @@ -1501,6 +1501,7 @@ config BOOT_CONFIG_EMBED_FILE
>
> config INITRAMFS_PRESERVE_MTIME
> bool "Preserve cpio archive mtimes in initramfs"
> + depends on BLK_DEV_INITRD
> default y
> help
> Each entry in an initramfs cpio archive carries an mtime
> value. When
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD
2025-09-15 7:11 [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD Geert Uytterhoeven
2025-09-15 8:14 ` David Disseldorp
2025-09-15 11:39 ` Martin Wilck
@ 2025-09-15 13:02 ` Christian Brauner
2 siblings, 0 replies; 4+ messages in thread
From: Christian Brauner @ 2025-09-15 13:02 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Christian Brauner, Askar Safin, linux-fsdevel, linux-kernel,
Andrew Morton, David Disseldorp, Martin Wilck, Alexander Viro,
Jan Kara
On Mon, 15 Sep 2025 09:11:05 +0200, Geert Uytterhoeven wrote:
> INITRAMFS_PRESERVE_MTIME is only used in init/initramfs.c and
> init/initramfs_test.c. Hence add a dependency on BLK_DEV_INITRD, to
> prevent asking the user about this feature when configuring a kernel
> without initramfs support.
>
>
Applied to the vfs-6.18.misc branch of the vfs/vfs.git tree.
Patches in the vfs-6.18.misc branch should appear in linux-next soon.
Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.
It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.
Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs-6.18.misc
[1/1] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD
https://git.kernel.org/vfs/vfs/c/74792608606a
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-09-15 13:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-15 7:11 [PATCH] init: INITRAMFS_PRESERVE_MTIME should depend on BLK_DEV_INITRD Geert Uytterhoeven
2025-09-15 8:14 ` David Disseldorp
2025-09-15 11:39 ` Martin Wilck
2025-09-15 13:02 ` Christian Brauner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox