[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option
From: |
Gleb Natapov |
Subject: |
[Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option |
Date: |
Tue, 29 Jul 2008 22:50:48 +0300 |
On Tue, Jul 29, 2008 at 02:22:54PM -0500, Anthony Liguori wrote:
> Gleb Natapov wrote:
>> On Mon, Jul 28, 2008 at 09:37:01AM -0500, Anthony Liguori wrote:
>>
>>> The backdoor interface is deprecated (from a VMware perspective) and
>>> is pretty terrible. I'll go through and do a more thorough review
>>> of the patches Chris posted but one thing I already know I'd like to
>>> see the
>> Review my last submission I linked above then too.
>>
>
> Applying the -uuid support without making use of that uuid anywhere
> isn't very useful.
>
You can read it from monitor, so management software can benefit from
it. And this will reduce my patch backlog :)
>>> UUID plumbed through the SMBIOS tables for x86. That's a requirement
>>> in my mind for adding a -uuid option. I see no harm in also
>>> supporting the backdoor interface but the primary way to expose a
>>> UUID should be SMBIOS.
>>>
>> I am not sure I understand what you mean. Currently SMBIOS tables are
>> built by bochs bios and UUID backdoor is needed to fill in missing info.
>>
>
> But that patch that got pushed into the Bochs BIOS was wrong.
>
> So here's what I'd like to see in order to apply these patches:
>
> 1) A new patch to the Bochs BIOS that used CMOS to pass a UUID (or
> possibly an OF data structure as Blue Swirl suggested--although CMOS is
> safer).
We will run out of space in CMOS quickly if we will use it as qemu->bios
communication channel. Is passing pointer to memory location (Blue Swirl
suggestion) acceptable solution by everyone?
--
Gleb.
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, (continued)
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Gleb Natapov, 2008/07/29
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Jamie Lokier, 2008/07/28
- [Qemu-devel] Re: Re: [PATCH 0/3]: Add UUID command-line option, Sebastian Herbszt, 2008/07/31
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Jamie Lokier, 2008/07/28
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Anthony Liguori, 2008/07/28
- [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Gleb Natapov, 2008/07/28
- [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Anthony Liguori, 2008/07/29
- [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option,
Gleb Natapov <=
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Blue Swirl, 2008/07/30
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Daniel P. Berrange, 2008/07/29
- Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Chris Lalancette, 2008/07/29
Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option, Jamie Lokier, 2008/07/28