public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: mark.rutland@arm.com (Mark Rutland)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters
Date: Fri, 11 Sep 2015 17:25:59 +0100	[thread overview]
Message-ID: <20150911162559.GA8726@leverpostej> (raw)
In-Reply-To: <20150911124643.GB4530@olila.local.net-space.pl>

On Fri, Sep 11, 2015 at 01:46:43PM +0100, Daniel Kiper wrote:
> On Thu, Sep 10, 2015 at 05:23:02PM +0100, Mark Rutland wrote:
> > > > C) When you could go:
> > > >
> > > >    DT -> Discover Xen -> Xen-specific stuff -> Xen-specific EFI/ACPI discovery
> > >
> > > I take you mean discovering Xen with the usual Xen hypervisor node on
> > > device tree. I think that C) is a good option actually. I like it. Not
> > > sure why we didn't think about this earlier. Is there anything EFI or
> > > ACPI which is needed before Xen support is discovered by
> > > arch/arm64/kernel/setup.c:setup_arch -> xen_early_init()?
> >
> > Currently lots (including the memory map). With the stuff to support
> > SPCR, the ACPI discovery would be moved before xen_early_init().
> >
> > > If not, we could just go for this. A lot of complexity would go away.
> >
> > I suspect this would still be fairly complex, but would at least prevent
> > the Xen-specific EFI handling from adversely affecting the native case.
> >
> > > > D) If you want to be generic:
> > > >    EFI -> EFI application -> EFI tables -> ACPI tables -> Xen-specific stuff
> > > >           \------------------------------------------/
> > > > 	   (virtualize these, provide shims to Dom0, but handle
> > > > 	    everything in Xen itself)
> > >
> > > I think that this is good in theory but could turn out to be a lot of
> > > work in practice. We could probably virtualize the RuntimeServices but
> > > the BootServices are troublesome.
> >
> > What's troublesome with the boot services?
> >
> > What can't be simulated?
> 
> How do you want to access bare metal EFI boot services from dom0 if they
> were shutdown long time ago before loading dom0 image?

I don't want to.

I asked "What can't be simulated?" because I assumed everything
necessary/mandatory could be simulated without needinng access to any
real EFI boot services.

As far as I can see all that's necessary is to provide a compatible
interface.

> What do you need from EFI boot services in dom0?

The ability to call ExitBootServices() and SetVirtualAddressMap() on a
_virtual_ address map for _virtual_ services provided by the hypervisor.
A console so that I can log things early on.

Mark.

  parent reply	other threads:[~2015-09-11 16:25 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-10  8:41 [PATCH] efi/libstub/fdt: Standardize the names of EFI stub parameters Shannon Zhao
2015-09-10  9:52 ` Mark Rutland
2015-09-10 10:19   ` Stefano Stabellini
2015-09-10 11:24     ` Mark Rutland
2015-09-10 11:37       ` Stefano Stabellini
2015-09-10 12:15         ` Mark Rutland
2015-09-10 12:58           ` [Xen-devel] " Ian Campbell
2015-09-10 13:08             ` Jan Beulich
2015-09-10 13:30               ` Ian Campbell
2015-09-10 13:52           ` Stefano Stabellini
2015-09-10 14:13             ` Leif Lindholm
2015-09-10 14:49             ` Mark Rutland
2015-09-10 16:10               ` Stefano Stabellini
2015-09-10 16:23                 ` Mark Rutland
2015-09-11 12:46                   ` Daniel Kiper
2015-09-11 13:14                     ` Stefano Stabellini
2015-09-11 13:30                       ` Ard Biesheuvel
2015-09-11 15:45                         ` Daniel Kiper
2015-09-14  8:42                           ` Shannon Zhao
2015-09-14  9:09                             ` Ard Biesheuvel
2015-09-14  9:31                               ` Shannon Zhao
2015-09-14  9:36                                 ` Ard Biesheuvel
2015-09-14 10:39                                   ` [Xen-devel] " Jan Beulich
2015-09-14 11:16                                     ` Ard Biesheuvel
2015-09-14 11:34                                       ` Jan Beulich
2015-09-11 16:33                         ` Mark Rutland
2015-09-11 16:25                     ` Mark Rutland [this message]
2015-09-12 11:36                       ` Daniel Kiper
2015-09-14  9:25                         ` Mark Rutland
2015-09-14  9:43                           ` Ard Biesheuvel
2015-09-14  9:57                             ` Ian Campbell
2015-09-14 10:02                               ` Ard Biesheuvel
2015-09-14 10:25                                 ` Ian Campbell
2015-09-14 12:28                             ` Daniel Kiper
2015-09-14 13:09                               ` Ard Biesheuvel
2015-09-14 13:57                                 ` Daniel Kiper
2015-09-14  9:47                           ` Stefano Stabellini
2015-09-14 12:19                           ` Daniel Kiper
2015-09-11 11:00                 ` Ian Campbell
2015-09-10 12:55         ` Jan Beulich
2015-09-10 14:53           ` Mark Rutland
2015-09-10 15:06             ` Jan Beulich
2015-09-11 16:36               ` Mark Rutland
2015-09-10 11:32 ` Andrew Turner
2015-09-10 11:48   ` Julien Grall
2015-09-10 12:05     ` Roger Pau Monné
2015-09-10 12:53       ` [Xen-devel] " Julien Grall
2015-09-17 11:43 ` Shannon Zhao

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150911162559.GA8726@leverpostej \
    --to=mark.rutland@arm.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox