On Fri, Mar 17, 2017 at 10:46:06AM +0000, David Woodhouse wrote: > On Wed, 2017-03-15 at 18:59 +0900, AKASHI Takahiro wrote: > > > > +pr_info("crashkernel reserved: 0x%016llx - 0x%016llx (%lld MB)\n", > > +crash_base, crash_base + crash_size, crash_size >> 20); > > There's a typo there ? it says MB but you mean MiB. > > Unless you meant crash_size / 1000000 and not crash_size >> 20? Yes and no. This notation is consistent with other places like mem_init() in mm/init.c. Thanks, -Takahiro AKASHI