The patch titled Subject: x86/efi: print size and base in binary units in efi_print_memmap has been removed from the -mm tree. Its filename was x86-efi-print-size-and-base-in-binary-units-in-efi_print_memmap.patch This patch was dropped because an alternative patch was merged ------------------------------------------------------ From: Robert Elliott <elliott@xxxxxxx> Subject: x86/efi: print size and base in binary units in efi_print_memmap Print the base in the best-fit B, KiB, MiB, etc. units rather than always MiB. This avoids rounding, which can be misleading. Use proper IEC binary units (KiB, MiB, etc.) rather than misuse SI decimal units (KB, MB, etc.). old: efi: mem61: [Persistent Memory | | | | | | | |WB|WT|WC|UC] range=[0x0000000880000000-0x0000000c7fffffff) (16384MB) new: efi: mem61: [Persistent Memory | | | | | | | |WB|WT|WC|UC] range=[0x0000000880000000-0x0000000c7fffffff] (16 GiB) Signed-off-by: Robert Elliott <elliott@xxxxxxx> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Cc: Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> Cc: Ingo Molnar <mingo@xxxxxxxxxx> Cc: "H. Peter Anvin" <hpa@xxxxxxxxx> Cc: Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- arch/x86/platform/efi/efi.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-) diff -puN arch/x86/platform/efi/efi.c~x86-efi-print-size-and-base-in-binary-units-in-efi_print_memmap arch/x86/platform/efi/efi.c --- a/arch/x86/platform/efi/efi.c~x86-efi-print-size-and-base-in-binary-units-in-efi_print_memmap +++ a/arch/x86/platform/efi/efi.c @@ -35,10 +35,12 @@ #include <linux/efi.h> #include <linux/efi-bgrt.h> #include <linux/export.h> +#include <linux/bitops.h> #include <linux/bootmem.h> #include <linux/slab.h> #include <linux/memblock.h> #include <linux/spinlock.h> +#include <linux/string_helpers.h> #include <linux/uaccess.h> #include <linux/time.h> #include <linux/io.h> @@ -117,6 +119,14 @@ void efi_get_time(struct timespec *now) now->tv_nsec = 0; } +static char * __init efi_size_format(char *buf, size_t size, u64 bytes) +{ + unsigned long i = bytes ? __ffs64(bytes) / 10 : 0; + + snprintf(buf, size, "%llu %s", bytes >> (i * 10), string_units_2[i]); + return buf; +} + void __init efi_find_mirror(void) { void *p; @@ -225,21 +235,20 @@ int __init efi_memblock_x86_reserve_rang void __init efi_print_memmap(void) { #ifdef EFI_DEBUG - efi_memory_desc_t *md; void *p; int i; for (p = memmap.map, i = 0; p < memmap.map_end; p += memmap.desc_size, i++) { - char buf[64]; + efi_memory_desc_t *md = p; + u64 size = md->num_pages << EFI_PAGE_SHIFT; + char buf[64], buf3[32]; - md = p; - pr_info("mem%02u: %s range=[0x%016llx-0x%016llx) (%lluMB)\n", + pr_info("mem%02u: %s range=[0x%016llx-0x%016llx] (%s)\n", i, efi_md_typeattr_format(buf, sizeof(buf), md), - md->phys_addr, - md->phys_addr + (md->num_pages << EFI_PAGE_SHIFT), - (md->num_pages >> (20 - EFI_PAGE_SHIFT))); + md->phys_addr, md->phys_addr + size, + efi_size_format(buf3, sizeof(buf3), size)); } #endif /* EFI_DEBUG */ } _ Patches currently in -mm which might be from elliott@xxxxxxx are -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html