Messages in this thread | | | Subject | Re: [PATCH v3 3/4] x86/efi: print size in binary units in efi_print_memmap | From | Andy Shevchenko <> | Date | Tue, 26 Jan 2016 13:59:26 +0200 |
| |
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@linux.intel.com> Intel Finland Oy
| |