[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 00/18] x86: multiboot2 protocol support
From: |
Ian Campbell |
Subject: |
Re: [PATCH 00/18] x86: multiboot2 protocol support |
Date: |
Tue, 3 Mar 2015 12:39:45 +0000 |
On Tue, 2015-03-03 at 13:36 +0100, Daniel Kiper wrote:
> On Tue, Mar 03, 2015 at 12:10:35PM +0000, Ian Campbell wrote:
> > On Fri, 2015-01-30 at 18:54 +0100, Daniel Kiper wrote:
> > > xen/arch/x86/Makefile | 17 ++--
> > > xen/arch/x86/boot/Makefile | 3 +-
> > > xen/arch/x86/boot/head.S | 291
> > > ++++++++++++++++++++++++++++++++++++++++++++++++++----
> > > xen/arch/x86/boot/reloc.c | 219
> > > ++++++++++++++++++++++++++++++++++-------
> > > xen/arch/x86/dmi_scan.c | 4 +-
> > > xen/arch/x86/domain_page.c | 2 +-
> > > xen/arch/x86/e820.c | 29 ++----
> > > xen/arch/x86/efi/Makefile | 12 +--
> > > xen/arch/x86/efi/efi-boot.h | 66 +++++++++++--
> > > xen/arch/x86/efi/stub.c | 41 --------
> > > xen/arch/x86/mpparse.c | 4 +-
> > > xen/arch/x86/setup.c | 30 +++---
> > > xen/arch/x86/shutdown.c | 3 +-
> > > xen/arch/x86/time.c | 2 +-
> > > xen/arch/x86/x86_64/asm-offsets.c | 8 ++
> > > xen/arch/x86/xen.lds.S | 2 -
> > > xen/common/efi/boot.c | 441
> > > +++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------
> > > xen/common/efi/runtime.c | 11 ++-
> > > xen/drivers/acpi/osl.c | 2 +-
> > > xen/include/xen/efi.h | 6 +-
> > > xen/include/xen/multiboot2.h | 169 ++++++++++++++++++++++++++++++++
> > > 21 files changed, 1018 insertions(+), 344 deletions(-)
> >
> > From the diffstat I'm not sure: is there any ARM impact I should worry
> > about reviewing? Any patches in particular my feedback is needed on?
>
> There are not any functional changes in ARM here. There are some small
> changes which are needed to make new x86 stuff coexist with ARM.
> Additionally, I thought that you are still interested in x86 things.
Not totally disinterested in x86 things, but not especially interested
in x86/EFI/multiboot unless I have to be.
> Am I right? If you do not have a time right now ignore this patch series.
> New thing will appear in 3-4 weeks.
OK, thanks.
> Or if you completely not interested in
> that stuff just drop me a line and I will drop you from distribution list.
If you could just CC me on the bits which impact ARM (or generic code)
that would be ok, but otherwise I can cope with ignoring irrelevant
parts of a series.
Thanks,
Ian.