This is a note to let you know that I've just added the patch titled x86/boot: Drop redundant code setting the root device to the 6.1-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: x86-boot-drop-redundant-code-setting-the-root-device.patch and it can be found in the queue-6.1 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From stable+bounces-40243-greg=kroah.com@xxxxxxxxxxxxxxx Fri Apr 19 10:11:52 2024 From: Ard Biesheuvel <ardb+git@xxxxxxxxxx> Date: Fri, 19 Apr 2024 10:11:13 +0200 Subject: x86/boot: Drop redundant code setting the root device To: stable@xxxxxxxxxxxxxxx Message-ID: <20240419081105.3817596-32-ardb+git@xxxxxxxxxx> From: Ard Biesheuvel <ardb@xxxxxxxxxx> [ Commit 7448e8e5d15a3c4df649bf6d6d460f78396f7e1e upstream ] The root device defaults to 0,0 and is no longer configurable at build time [0], so there is no need for the build tool to ever write to this field. [0] 079f85e624189292 ("x86, build: Do not set the root_dev field in bzImage") This change has no impact on the resulting bzImage binary. Signed-off-by: Ard Biesheuvel <ardb@xxxxxxxxxx> Signed-off-by: Ingo Molnar <mingo@xxxxxxxxxx> Link: https://lore.kernel.org/r/20230912090051.4014114-23-ardb@xxxxxxxxxx Signed-off-by: Ard Biesheuvel <ardb@xxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- arch/x86/boot/header.S | 2 +- arch/x86/boot/tools/build.c | 7 ------- 2 files changed, 1 insertion(+), 8 deletions(-) --- a/arch/x86/boot/header.S +++ b/arch/x86/boot/header.S @@ -235,7 +235,7 @@ root_flags: .word ROOT_RDONLY syssize: .long 0 /* Filled in by build.c */ ram_size: .word 0 /* Obsolete */ vid_mode: .word SVGA_MODE -root_dev: .word 0 /* Filled in by build.c */ +root_dev: .word 0 /* Default to major/minor 0/0 */ boot_flag: .word 0xAA55 # offset 512, entry point --- a/arch/x86/boot/tools/build.c +++ b/arch/x86/boot/tools/build.c @@ -40,10 +40,6 @@ typedef unsigned char u8; typedef unsigned short u16; typedef unsigned int u32; -#define DEFAULT_MAJOR_ROOT 0 -#define DEFAULT_MINOR_ROOT 0 -#define DEFAULT_ROOT_DEV (DEFAULT_MAJOR_ROOT << 8 | DEFAULT_MINOR_ROOT) - /* Minimal number of setup sectors */ #define SETUP_SECT_MIN 5 #define SETUP_SECT_MAX 64 @@ -399,9 +395,6 @@ int main(int argc, char ** argv) update_pecoff_setup_and_reloc(i); - /* Set the default root device */ - put_unaligned_le16(DEFAULT_ROOT_DEV, &buf[508]); - /* Open and stat the kernel file */ fd = open(argv[2], O_RDONLY); if (fd < 0) Patches currently in stable-queue which might be from kroah.com@xxxxxxxxxxxxxxx are queue-6.1/x86-boot-drop-pe-coff-.reloc-section.patch queue-6.1/x86-sme-move-early-sme-kernel-encryption-handling-into-.head.text.patch queue-6.1/x86-boot-omit-compression-buffer-from-pe-coff-image-memory-footprint.patch queue-6.1/x86-mm-remove-p-d_page_mask-and-p-d_page_size-macros.patch queue-6.1/x86-efi-drop-alignment-flags-from-pe-section-headers.patch queue-6.1/x86-boot-increase-section-and-file-alignment-to-4k-512.patch queue-6.1/x86-head-64-add-missing-__head-annotation-to-startup_64_load_idt.patch queue-6.1/x86-boot-define-setup-size-in-linker-script.patch queue-6.1/x86-boot-split-off-pe-coff-.data-section.patch queue-6.1/x86-boot-derive-file-size-from-_edata-symbol.patch queue-6.1/x86-boot-construct-pe-coff-.text-section-from-assembler.patch queue-6.1/x86-boot-grab-kernel_info-offset-from-zoffset-header-directly.patch queue-6.1/x86-sev-move-early-startup-code-into-.head.text-section.patch queue-6.1/x86-boot-remove-the-bugger-off-message.patch queue-6.1/x86-head-64-move-the-__head-definition-to-asm-init.h.patch queue-6.1/x86-boot-drop-references-to-startup_64.patch queue-6.1/x86-efistub-use-1-1-file-memory-mapping-for-pe-coff-.compat-section.patch queue-6.1/x86-efi-drop-efi-stub-.bss-from-.data-section.patch queue-6.1/x86-efi-disregard-setup-header-of-loaded-image.patch queue-6.1/x86-efistub-remap-kernel-text-read-only-before-dropping-nx-attribute.patch queue-6.1/x86-efistub-reinstate-soft-limit-for-initrd-loading.patch queue-6.1/x86-boot-set-efi-handover-offset-directly-in-header-asm.patch queue-6.1/x86-boot-drop-redundant-code-setting-the-root-device.patch