x86, efi: Make efi_memblock_x86_reserve_range more readable
authorBorislav Petkov <bp@suse.de>
Fri, 1 Mar 2013 16:07:44 +0000 (17:07 +0100)
committerMatt Fleming <matt.fleming@intel.com>
Mon, 4 Mar 2013 15:23:56 +0000 (15:23 +0000)
So basically this function copies EFI memmap stuff from boot_params into
the EFI memmap descriptor and reserves memory for it. Make it much more
readable.

Signed-off-by: Borislav Petkov <bp@suse.de>
Cc: Matthew Garret <mjg59@srcf.ucam.org>
Signed-off-by: Matt Fleming <matt.fleming@intel.com>
arch/x86/platform/efi/efi.c

index e075e474245b64e1ddceadb32c3304db67c226f4..430cd784a0de8ea3e1886576e4605b761df4ab76 100644 (file)
@@ -350,24 +350,25 @@ static void __init do_add_efi_memmap(void)
 
 int __init efi_memblock_x86_reserve_range(void)
 {
+       struct efi_info *e = &boot_params.efi_info;
        unsigned long pmap;
 
 #ifdef CONFIG_X86_32
        /* Can't handle data above 4GB at this time */
-       if (boot_params.efi_info.efi_memmap_hi) {
+       if (e->efi_memmap_hi) {
                pr_err("Memory map is above 4GB, disabling EFI.\n");
                return -EINVAL;
        }
-       pmap = boot_params.efi_info.efi_memmap;
+       pmap =  e->efi_memmap;
 #else
-       pmap = (boot_params.efi_info.efi_memmap |
-               ((__u64)boot_params.efi_info.efi_memmap_hi<<32));
+       pmap = (e->efi_memmap | ((__u64)e->efi_memmap_hi << 32));
 #endif
-       memmap.phys_map = (void *)pmap;
-       memmap.nr_map = boot_params.efi_info.efi_memmap_size /
-               boot_params.efi_info.efi_memdesc_size;
-       memmap.desc_version = boot_params.efi_info.efi_memdesc_version;
-       memmap.desc_size = boot_params.efi_info.efi_memdesc_size;
+       memmap.phys_map         = (void *)pmap;
+       memmap.nr_map           = e->efi_memmap_size /
+                                 e->efi_memdesc_size;
+       memmap.desc_size        = e->efi_memdesc_size;
+       memmap.desc_version     = e->efi_memdesc_version;
+
        memblock_reserve(pmap, memmap.nr_map * memmap.desc_size);
 
        return 0;