Re: [PATCH v5 0/7] xen: Add EFI support

From: Matt Fleming
Date: Wed Jun 18 2014 - 10:15:51 EST


On Fri, 13 Jun, at 07:00:16PM, Daniel Kiper wrote:
> Hey,
>
> This patch series adds EFI support for Xen dom0 guests.
> It is based on Jan Beulich and Tang Liang work. I was
> trying to take into account all previous comments,
> however, if I missed something sorry for that.
>
> Daniel
>
> arch/x86/kernel/setup.c | 4 +-
> arch/x86/platform/efi/efi.c | 77 ++++++++++-------
> arch/x86/xen/enlighten.c | 24 ++++++
> drivers/firmware/efi/efi.c | 26 +++---
> drivers/xen/Kconfig | 4 +
> drivers/xen/Makefile | 3 +
> drivers/xen/efi.c | 374 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> include/linux/efi.h | 3 +-
> include/xen/interface/platform.h | 123 +++++++++++++++++++++++++++
> 9 files changed, 595 insertions(+), 43 deletions(-)

FWIW I'm much happier with this version than v4. There's nothing that
jumps out at me as being obviously wrong. Just a few minor cleanups
needed.

Which tree is this intended to go through? I'm more than happy to take
it through the EFI tree, particularly since it touches
include/linux/efi.h and drivers/firmware/efi/efi.c which is the core EFI
interface for all platforms.

But if it's going through the Xen tree or something, I can supply my
Acked-by's (once the aforementioned minor cleanups are taken care of).

--
Matt Fleming, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/