On 11/07/14 08:41, Matt Fleming wrote:
i.e. that the memory used for alignment was not explicitly
allocated. If the EFI boot stub instead allocates space for the
aligned kernel using AllocatePages() (and allocates enough space for
the whole of init_size), then the problem I described does not
exist.
Right, this shouldn't be a problem because we do in fact allocate space
using the EFI boottime services in efi_relocate_kernel(), taking the
alignment into account, and then perform the kernel image copy.
I still think your change makes sense, I'm just inclined to delete the
paragraph referring to the corruption bug (which we've established
doesn't exist).
Since the patch itself is so trivial, I'm happy for you to commit it as
your own, with whatever commit message you think makes most sense.
Michael
--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html