Re: [PATCH v3 3/4] x86/efi: print size in binary units in efi_print_memmap

From: Andy Shevchenko
Date: Tue Jan 26 2016 - 06:59:02 EST


On Tue, 2016-01-26 at 11:50 +0000, Matt Fleming wrote:
> On Mon, 25 Jan, at 08:37:58PM, Elliott, Robert (Persistent Memory)
> wrote:
> >
> > For the UEFI memory map, that was indeed my intention.ÂÂI
> > don't want it silently round to "20 GiB".ÂÂEven rounding
> > to "19.999 GiB" is imprecise.
>
> OK, let's just go with your original patch Robert (minus the @ addr
> bit) since it's pretty small and does what we want for this specific
> case.

However I am against this, but seems reviewers do not leave a chance to
us, I would propose to copy-and-paste table of binary prefixes and use
__ffs64().

--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy