All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: Juergen Gross <jgross@suse.com>,
	The development of GNU GRUB <grub-devel@gnu.org>,
	Daniel Kiper <daniel.kiper@oracle.com>,
	xen-devel <xen-devel@lists.xenproject.org>
Subject: Re: [Xen-devel] Xen PVH support in grub2
Date: Tue, 3 Oct 2017 09:56:58 +0100	[thread overview]
Message-ID: <20171003085658.4jftgr3jumck566v@MacBook-Pro-de-Roger.local> (raw)
In-Reply-To: <2d6e2b3a-4750-98a1-85a0-10b690a1b060@oracle.com>

On Fri, Sep 29, 2017 at 05:22:25PM +0000, Boris Ostrovsky wrote:
> On 09/29/2017 01:07 PM, Roger Pau Monné wrote:
> > On Fri, Sep 29, 2017 at 05:02:48PM +0000, Boris Ostrovsky wrote:
> >> On 09/29/2017 11:51 AM, Roger Pau Monné wrote:
> >>> On Fri, Sep 29, 2017 at 03:33:58PM +0000, Juergen Gross wrote:
> >>>> On 29/09/17 17:24, Roger Pau Monné wrote:
> >>>>> On Fri, Sep 29, 2017 at 02:46:53PM +0000, Juergen Gross wrote:
> >>>>> Then, I also wonder whether it would make sense for this grub to load
> >>>>> the kernel using the PVH entry point or the native entry point. Would
> >>>>> it be possible to boot a Linux kernel up to the point where cpuid can
> >>>>> be used inside of a PVH container?
> >>>> I don't think today's Linux allows that. This has been discussed
> >>>> very thoroughly at the time Boris added PVH V2 support to the kernel.
> >>> OK, I'm not going to insist on that, but my plans for FreeBSD is to
> >>> make the native entry point capable of booting inside of a PVH
> >>> container up to the point where cpuid (or whatever method) can be used
> >>> to detect the environment.
> >>>
> >>> Do you recall what's preventing the native entry point from booting
> >>> inside of a PVH container? If certain emulated devices not present are
> >>> needed at early boot we could look into either replacing them with
> >>> other options available inside of a PVH container, or as a last resort
> >>> making them available on a PVH container.
> >> Very much IIRC one of the reasons was the fact that zeropage
> >> (bootparams) needed to be properly formatted. And the hypercall page
> >> needs to be set up.
> > But in this case bootparams is going to be setup by grub, so it should
> > be fine (just like it's done on bare metal).
> 
> Yes, I think so.
> 
> >
> > Couldn't the hypercall page be setup at some point during early boot?
> > Not sure if setting it up at the same point HVM does would be fine for
> > PVH?
> 
> Probably --- I think the only reason we set it early is because we need
> to call XENMEM_memory_map to set bootparams. One other thing I noticed
> is that we need to set acpi_irq_model before hypervisor is discovered
> (can't remember why) but I suppose this can be worked around.
> 
> Having said that --- since for direct boot we still need to go via
> PVH-specific entry point I am not sure how much we will gain by having
> grub avoid it.

Being able to boot inside of a PVH container using the native entry
point would prevent us from having to add PVH loader specific code to
each firmware we plan to support in PVH mode.

If Linux must be started using the PVH entry point in order to run
inside of a PVH container, it means we would have to add a PVH loader
to OVFM and grub at least.

OTOH if Linux is capable of booting from the native entry point inside
of a PVH container, we would only have to port OVMF and grub in order
to work inside of a PVH container, leaving the rest of the logic
untouched.

Thanks, Roger.


  parent reply	other threads:[~2017-10-03  8:57 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-29 14:46 Xen PVH support in grub2 Juergen Gross
2017-09-29 15:07 ` George Dunlap
2017-09-29 15:07   ` [Xen-devel] " George Dunlap
2017-09-29 15:24 ` Roger Pau Monné
2017-09-29 15:24   ` [Xen-devel] " Roger Pau Monné
2017-09-29 15:33   ` Juergen Gross
2017-09-29 15:51     ` Roger Pau Monné
2017-09-29 15:51       ` [Xen-devel] " Roger Pau Monné
2017-09-29 17:02       ` Boris Ostrovsky
2017-09-29 17:02         ` [Xen-devel] " Boris Ostrovsky
2017-09-29 17:07         ` Roger Pau Monné
2017-09-29 17:07           ` [Xen-devel] " Roger Pau Monné
2017-09-29 17:22           ` Boris Ostrovsky
2017-09-29 17:22             ` [Xen-devel] " Boris Ostrovsky
2017-10-03  8:56             ` Roger Pau Monné
2017-10-03  8:56             ` Roger Pau Monné [this message]
2017-10-03 12:47               ` [Xen-devel] " Boris Ostrovsky
2017-10-03 12:47               ` Boris Ostrovsky
2017-11-03 12:00       ` Juergen Gross
2017-11-03 12:00         ` [Xen-devel] " Juergen Gross
2017-11-03 12:17         ` Roger Pau Monné
2017-11-03 12:17         ` [Xen-devel] " Roger Pau Monné
2017-11-03 12:50           ` Juergen Gross
2017-11-03 12:50           ` [Xen-devel] " Juergen Gross
2017-11-03 14:07             ` Roger Pau Monné
2017-11-03 14:24               ` Juergen Gross
2017-11-03 14:36                 ` Boris Ostrovsky
2017-11-03 14:36                   ` [Xen-devel] " Boris Ostrovsky
2017-11-03 14:59                   ` Juergen Gross
2017-11-03 15:10                     ` Boris Ostrovsky
2017-11-03 15:27                       ` Juergen Gross
2017-11-03 18:05                         ` Juergen Gross
2017-11-03 18:19                           ` Boris Ostrovsky
2017-11-03 18:19                           ` [Xen-devel] " Boris Ostrovsky
2017-11-03 18:23                             ` Juergen Gross
2017-11-03 18:35                               ` Boris Ostrovsky
2017-11-03 18:40                                 ` Juergen Gross
2017-11-03 19:00                                   ` Boris Ostrovsky
2017-11-03 19:00                                   ` [Xen-devel] " Boris Ostrovsky
2017-11-06  7:16                                     ` Juergen Gross
2017-11-06  7:16                                     ` [Xen-devel] " Juergen Gross
2017-11-06 14:51                                       ` Boris Ostrovsky
2017-11-06 14:51                                       ` [Xen-devel] " Boris Ostrovsky
2017-11-06 15:05                                         ` Juergen Gross
2017-11-06 16:42                                           ` Boris Ostrovsky
2017-11-07  7:42                                             ` Juergen Gross
2017-11-07  7:42                                             ` [Xen-devel] " Juergen Gross
2017-11-07 16:10                                               ` Boris Ostrovsky
2017-11-07 16:10                                               ` [Xen-devel] " Boris Ostrovsky
2017-11-09  5:01                                                 ` Konrad Rzeszutek Wilk
2017-11-09  5:01                                                 ` [Xen-devel] " Konrad Rzeszutek Wilk
2017-11-06 16:42                                           ` Boris Ostrovsky
2017-11-06 15:05                                         ` Juergen Gross
2017-11-03 18:40                                 ` Juergen Gross
2017-11-03 18:35                               ` Boris Ostrovsky
2017-11-03 18:37                               ` Roger Pau Monné
2017-11-03 18:37                                 ` [Xen-devel] " Roger Pau Monné
2017-11-03 18:47                                 ` Juergen Gross
2017-11-03 18:47                                 ` Juergen Gross
2017-11-03 18:23                             ` Juergen Gross
2017-11-03 18:05                         ` Juergen Gross
2017-11-03 15:27                       ` Juergen Gross
2017-11-03 15:10                     ` Boris Ostrovsky
2017-11-03 14:59                   ` Juergen Gross
2017-11-03 14:24               ` Juergen Gross
2017-11-03 14:07             ` Roger Pau Monné
2017-11-06 11:36           ` [Xen-devel] " Juergen Gross
2017-11-07 14:49             ` Juergen Gross
2017-11-07 14:49             ` [Xen-devel] " Juergen Gross
2017-11-06 11:36           ` Juergen Gross
2017-09-29 15:33   ` Juergen Gross
2017-09-29 15:34   ` George Dunlap
2017-09-29 15:34     ` [Xen-devel] " George Dunlap

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=20171003085658.4jftgr3jumck566v@MacBook-Pro-de-Roger.local \
    --to=roger.pau@citrix.com \
    --cc=boris.ostrovsky@oracle.com \
    --cc=daniel.kiper@oracle.com \
    --cc=grub-devel@gnu.org \
    --cc=jgross@suse.com \
    --cc=xen-devel@lists.xenproject.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.