grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
From: Daniel Kiper <daniel.kiper@oracle.com>
To: Juergen Gross <jgross@suse.com>
Cc: The development of GNU GRUB <grub-devel@gnu.org>,
	phcoder@gmail.com, xen-devel@lists.xen.org
Subject: Re: [PATCH 0/8] xen: add pvh guest support
Date: Tue, 9 Oct 2018 14:54:27 +0200	[thread overview]
Message-ID: <20181009125427.GA4472@olila.i.net-space.pl> (raw)
In-Reply-To: <99628ba5-f300-0d0a-0484-c357b38058c7@suse.com>

On Tue, Oct 09, 2018 at 11:35:28AM +0200, Juergen Gross wrote:
> On 15/02/2018 13:02, Daniel Kiper wrote:
> > Hi Juergen,
> >
> > Sorry for huge delay. It looks that I am recovering slowly and
> > probably I will have more time for reviews.
> >
> > On Wed, Nov 29, 2017 at 02:46:42PM +0100, Juergen Gross wrote:
> >> This patch series adds support for booting Linux as PVH guest.
> >>
> >> Similar to i386/xen and x86_64/xen platforms the new i386/xenpvh
> >> platform grub is booted as a standalone image directly by Xen.
> >>
> >> For booting Linux kernel it is using the standard linux kernel
> >> loader. The only modification of the linux loader is to pass the
> >> ACPI RSDP address via boot parameters to the kernel, as that table
> >> might not be located at the usual physical address just below 1MB.
> >
> > AIUI PVH is quite generic idea and can be implemented by different
> > virtualization platforms. IIRC Maran Wilson works on PVH for KVM.
> > So, would not it make more sense to have platform independent GRUB2
> > PVH code and then on top of that build Xen and KVM support? Could
> > you do that?
>
> Finally I have some time to work on this series again.

Great! I was a bit afraid that it will never happen...

> I don't think this makes sense now for the following reasons:
>
> - Maran's series is stalled
> - The main work is hypervisor specific and I have no idea right now
>   what would be needed for KVM
> - Maran's series was targeted for a quick boot of the kernel right
>   from qemu without any other boot loader

OK, so, let's leave it aside.

I have seen new patchset. Thanks a lot! I will take a look at it this
or next week.

Daniel


      reply	other threads:[~2018-10-09 12:55 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-29 13:46 [PATCH 0/8] xen: add pvh guest support Juergen Gross
2017-11-29 13:46 ` [PATCH 1/8] xen: add some xen headers Juergen Gross
2018-02-15 11:12   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 2/8] loader/linux: support passing rsdp address via boot params Juergen Gross
2018-02-15 11:17   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 3/8] xen: carve out grant tab initialization into dedicated function Juergen Gross
2018-02-15 11:26   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 4/8] xen: add xen pvh guest support to grub-core Juergen Gross
2018-02-15 11:31   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 5/8] xenpvh: add build runes for grub-core Juergen Gross
2018-02-15 11:39   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 6/8] xenpvh: support building a standalone image Juergen Gross
2018-02-15 11:47   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 7/8] xenpvh: support grub-install for xenpvh Juergen Gross
2018-02-15 11:49   ` Daniel Kiper
2017-11-29 13:46 ` [PATCH 8/8] xenpvh: add support to configure Juergen Gross
2018-02-15 11:51   ` Daniel Kiper
2017-11-30 21:03 ` [PATCH 0/8] xen: add pvh guest support Daniel Kiper
2017-12-01  5:37   ` Juergen Gross
2017-12-01 11:12     ` Daniel Kiper
2017-12-14 11:19       ` Daniel Kiper
2017-12-14 11:26         ` Juergen Gross
2017-12-14 11:32           ` Daniel Kiper
2017-12-14 11:44             ` Juergen Gross
2018-01-29 12:15               ` Daniel Kiper
2018-01-29 13:33                 ` Juergen Gross
2018-02-15 12:02 ` Daniel Kiper
2018-10-09  9:35   ` Juergen Gross
2018-10-09 12:54     ` Daniel Kiper [this message]

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=20181009125427.GA4472@olila.i.net-space.pl \
    --to=daniel.kiper@oracle.com \
    --cc=grub-devel@gnu.org \
    --cc=jgross@suse.com \
    --cc=phcoder@gmail.com \
    --cc=xen-devel@lists.xen.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;
as well as URLs for NNTP newsgroup(s).