From: Gleb Natapov <gleb@qumranet.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 0/6] Add UUID command-line option
Date: Sun, 24 Aug 2008 16:05:10 +0300 [thread overview]
Message-ID: <20080824130510.GC6192@minantech.com> (raw)
In-Reply-To: <f43fc5580808240545u23472f09v58269a3ecc586c09@mail.gmail.com>
On Sun, Aug 24, 2008 at 03:45:51PM +0300, Blue Swirl wrote:
> > It is hard enough to change interface between qemu and bochs BIOS. How
> > complex it will be to simultaneously change interface for several boot
> > loaders.
>
> This is because no Qemu developer has commit rights to Bochs, and
> Bochs releases too infrequently. With commit rights, synchronizing a
> commit would not be too difficult. And this should be needed only if
> the common interface changes incompatibly.
>
Is this situation going to change? Are Qemu developers has commit rights
to other boot loader?
BTW we are going to use qemu<->bios channel not only for UUID and cpu
frequency, but for number of other things. Adding ACPI tables provided
by user for instance (this is needed for Vista OEM installation).
> > > UUID is not architecture specific, so it should use the
> > > main structure instead if the architecture specific substructure
> > > (nvram_arch*). Adding UUID to unused fields will not break anything.
> >
> > Most info in ohwcfg_v3_t are not needed (or can be obtained by other
> > means) by PC BIOS, so there is no point in coping the whole structure
> > into BIOS. Of cause BIOS don't have to copy entire ohwcfg_v3_t, but
>
> For example, Bochs seems to use i440fx registers to determine the
> available physical memory. This could be changed to use the
> configuration structure instead. It's a matter of taste, but I would
> find this an improvement.
If by Bochs you mean Bochs BIOS then this is not the case. There are
standard CMOS locations where amount of available memory is specified.
Qemu initialize those locations and BIOS reads them. I don't think this is
going to change since Bochs BIOS is used not only by Qemu. If only Bochs
will move to firmware interface too...
>
> > access only required fields by reading only specific offsets, but then the
> > interface will be exactly like the one I proposed with only difference
> > that instead of specifying magic value (like 1 for reading UUID in my
> > patch series), BIOS will have to specify magic offset (like 0xE0).
>
> There is no need for a magic offset, ohwcfg_v3_t is designed to be
> included even from asm.
>
The thing is I don't want to copy the whole ROM into BIOS during boot.
As far as I see on other platforms ROM is memory mapped, so no copying
is required and this was you first proposal, but if we are going to use
port IO to access ROM then it will either have to be copied into main
memory and then accessed, or only required fields will have to be
copied and that mean coping data from magic offsets.
--
Gleb.
next prev parent reply other threads:[~2008-08-24 13:05 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-24 11:32 [Qemu-devel] [PATCH 0/6] Add UUID command-line option Gleb Natapov
2008-08-24 11:33 ` [Qemu-devel] [PATCH 1/6] Use IO port for qemu<->guest BIOS communication Gleb Natapov
2008-08-24 17:39 ` Blue Swirl
2008-08-25 9:48 ` Gleb Natapov
2008-08-24 11:33 ` [Qemu-devel] [PATCH 2/6] Add -uuid command line option Gleb Natapov
2008-08-24 11:33 ` [Qemu-devel] [PATCH 3/6] Add "info uuid" command to monitor Gleb Natapov
2008-08-24 17:43 ` Blue Swirl
2008-08-25 9:46 ` Gleb Natapov
2008-08-24 11:33 ` [Qemu-devel] [PATCH 4/6] Use libuuid if available Gleb Natapov
2008-08-24 17:50 ` Blue Swirl
2008-08-25 9:44 ` Gleb Natapov
2008-08-24 11:33 ` [Qemu-devel] [PATCH 5/6] Add UUID to BIOS configuration info Gleb Natapov
2008-08-24 11:33 ` [Qemu-devel] [PATCH 6/6] Pass cpu speed into SM BIOS Gleb Natapov
2008-08-24 18:14 ` Blue Swirl
2008-08-25 9:44 ` Gleb Natapov
2008-08-24 12:01 ` [Qemu-devel] [PATCH 0/6] Add UUID command-line option Blue Swirl
2008-08-24 12:24 ` Gleb Natapov
2008-08-24 12:45 ` Blue Swirl
2008-08-24 13:05 ` Gleb Natapov [this message]
2008-08-24 13:35 ` Blue Swirl
2008-08-24 14:11 ` Gleb Natapov
2008-08-24 17:17 ` Blue Swirl
2008-08-24 19:23 ` Kevin O'Connor
2008-08-24 13:09 ` Avi Kivity
2008-08-24 13:43 ` Blue Swirl
2008-08-24 13:55 ` Avi Kivity
2008-08-24 16:56 ` Blue Swirl
2008-08-25 9:32 ` Avi Kivity
2008-08-25 14:22 ` Anthony Liguori
2008-08-24 18:50 ` Kevin O'Connor
2008-08-25 9:29 ` Avi Kivity
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=20080824130510.GC6192@minantech.com \
--to=gleb@qumranet.com \
--cc=qemu-devel@nongnu.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).