efi/libstub/x86: Remove pointless zeroing of apm_bios_info
authorArd Biesheuvel <ardb@kernel.org>
Mon, 10 Feb 2020 16:02:30 +0000 (17:02 +0100)
committerArd Biesheuvel <ardb@kernel.org>
Sat, 22 Feb 2020 22:37:37 +0000 (23:37 +0100)
We have some code in the EFI stub entry point that takes the address
of the apm_bios_info struct in the newly allocated and zeroed out
boot_params structure, only to zero it out again. This is pointless
so remove it.

Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
arch/x86/boot/compressed/eboot.c

index 32423e83ba8f9214ecba16fd80a050a10ea9cd54..4745a1ee79536049af5916de5d187dd70900719e 100644 (file)
@@ -358,7 +358,6 @@ efi_status_t __efiapi efi_pe_entry(efi_handle_t handle,
                                   efi_system_table_t *sys_table_arg)
 {
        struct boot_params *boot_params;
-       struct apm_bios_info *bi;
        struct setup_header *hdr;
        efi_loaded_image_t *image;
        efi_guid_t proto = LOADED_IMAGE_PROTOCOL_GUID;
@@ -389,7 +388,6 @@ efi_status_t __efiapi efi_pe_entry(efi_handle_t handle,
        memset(boot_params, 0x0, 0x4000);
 
        hdr = &boot_params->hdr;
-       bi = &boot_params->apm_bios_info;
 
        /* Copy the second sector to boot_params */
        memcpy(&hdr->jump, image->image_base + 512, 512);
@@ -416,9 +414,6 @@ efi_status_t __efiapi efi_pe_entry(efi_handle_t handle,
        hdr->ramdisk_image = 0;
        hdr->ramdisk_size = 0;
 
-       /* Clear APM BIOS info */
-       memset(bi, 0, sizeof(*bi));
-
        status = efi_parse_options(cmdline_ptr);
        if (status != EFI_SUCCESS)
                goto fail2;