All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Kiper <daniel.kiper@oracle.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: xen-devel <xen-devel@lists.xen.org>,
	keir@xen.org, ian.campbell@citrix.com, konrad.wilk@oracle.com,
	stefano.stabellini@eu.citrix.com
Subject: Re: New Xen boot infrastructure proposal
Date: Wed, 22 May 2013 17:01:04 +0200	[thread overview]
Message-ID: <20130522150104.GE25607@debian70-amd64.local.net-space.pl> (raw)
In-Reply-To: <519CF35902000078000D8264@nat28.tlf.novell.com>

On Wed, May 22, 2013 at 03:33:29PM +0100, Jan Beulich wrote:
> >>> On 22.05.13 at 16:09, Daniel Kiper <daniel.kiper@oracle.com> wrote:

[...]

> > I think that passing info about system via many not "linked" variables
> > is not the best idea. It works in that way today because multiboot
> > structure is not extensible. That is why I think we should find new
> > solution. Our new custom build structure which contains only stuff
> > required by Xen looks good. All members are easliy accessible from C.
> > It could be easliy extended (if we need it just add new member) because
> > it would not be linked with specific boot protocol.
>
> Why does  it matter whether the MBI structure is extensible?

If we stick to current MBI I am not able to pass (in sensible way),
from preloader to __start_xen(), e.g. ACPI and EFI stuff from multiboot2
protocol. That is why I think we should build our own struct which is not
linked with any boot protocol. That way we could avoid similar problems
in the future.

> Rather than copying everything around a number of times, we
> can as well use is where it lives naturally. The only requirement

This is the best approach if we would have one type of boot protocol.
It is not true if we would like to support more then one.

> is that all information be accessible - whether in one strcture,
> two, or a dozen doesn't matter.

I agree that this is not a must but I prefer to have "all in one" :-)))...
We could do that cleanups (by the way) if we decide what to do with above issues.

Daniel

  reply	other threads:[~2013-05-22 15:01 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-21 10:36 New Xen boot infrastructure proposal Daniel Kiper
2013-05-21 11:39 ` Stefano Stabellini
2013-05-21 12:57   ` Daniel Kiper
2013-05-21 12:03 ` Jan Beulich
2013-05-22 14:09   ` Daniel Kiper
2013-05-22 14:33     ` Jan Beulich
2013-05-22 15:01       ` Daniel Kiper [this message]
2013-05-22 15:16         ` Jan Beulich
2013-05-22 16:47           ` Konrad Rzeszutek Wilk
2013-05-22 16:56             ` Keir Fraser
2013-05-23  6:37             ` Jan Beulich
2013-05-21 12:43 ` David Vrabel
2013-05-22 14:19   ` Daniel Kiper
2013-05-21 12:52 ` Ian Campbell
2013-05-22 14:27   ` Daniel Kiper
2013-05-22 14:35     ` Jan Beulich
2013-05-22 15:09     ` Ian Campbell
2013-05-22 15:25       ` Ian Campbell
2013-05-22 15:34         ` Daniel Kiper
2013-05-22 15:41           ` Ian Campbell
2013-05-22 16:19             ` Daniel Kiper
2013-05-23 13:33               ` Ian Campbell
2013-05-21 13:24 ` Keir Fraser
2013-05-22 14:43   ` Daniel Kiper
2013-05-22 15:10     ` Jan Beulich
2013-05-22 15:59       ` Daniel Kiper
2013-05-22 16:40         ` Keir Fraser

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=20130522150104.GE25607@debian70-amd64.local.net-space.pl \
    --to=daniel.kiper@oracle.com \
    --cc=JBeulich@suse.com \
    --cc=ian.campbell@citrix.com \
    --cc=keir@xen.org \
    --cc=konrad.wilk@oracle.com \
    --cc=stefano.stabellini@eu.citrix.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 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.