Messages in this thread | | | Date | Tue, 14 Jun 2011 16:21:02 +0100 | From | Matthew Garrett <> | Subject | Re: [PATCH] x86, efi: Do not reserve boot services regions within reserved areas |
| |
On Tue, Jun 14, 2011 at 05:03:14PM +0200, Maarten Lankhorst wrote:
> + /* Could not reserve boot area */ > + if (!size) > + continue; > + > + printk(KERN_INFO PFX "Freeing boot area " > + "[0x%llx-0x%llx)\n", start, start+size);
Probably don't need the printk - we could be dumping a few hundred of those.
> - "Kernel-defined memdesc doesn't match the one from EFI!\n"); > + "Kernel-defined memdesc (%lu) " > + "doesn't match the one from EFI(%lu)!\n", > + sizeof(efi_memory_desc_t), memmap.desc_size);
And drop this hunk. It's entirely valid for an implementation to do this, we should just delete the message entirely.
-- Matthew Garrett | mjg59@srcf.ucam.org
| |