From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KNkmV-0006KO-O9 for qemu-devel@nongnu.org; Tue, 29 Jul 2008 04:42:08 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KNkmT-0006Ip-PD for qemu-devel@nongnu.org; Tue, 29 Jul 2008 04:42:06 -0400 Received: from [199.232.76.173] (port=36786 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KNkmT-0006IV-BG for qemu-devel@nongnu.org; Tue, 29 Jul 2008 04:42:05 -0400 Received: from mx1.redhat.com ([66.187.233.31]:50352) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1KNkMh-0000Mp-La for qemu-devel@nongnu.org; Tue, 29 Jul 2008 04:15:27 -0400 Date: Tue, 29 Jul 2008 09:15:25 +0100 From: "Daniel P. Berrange" Subject: Re: [Qemu-devel] Re: [PATCH 0/3]: Add UUID command-line option Message-ID: <20080729081525.GI32498@redhat.com> References: <488DC8B2.1070009@redhat.com> <20080728141515.GJ3196@minantech.com> <488DD98D.5010907@codemonkey.ws> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <488DD98D.5010907@codemonkey.ws> Reply-To: "Daniel P. Berrange" , qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Gleb Natapov , Chris Lalancette On Mon, Jul 28, 2008 at 09:37:01AM -0500, Anthony Liguori wrote: > Gleb Natapov wrote: > >Hi Chris, > > > >On Mon, Jul 28, 2008 at 03:25:06PM +0200, Chris Lalancette wrote: > > > >>All, > >> Attached is a three part patch series to implement a "-uuid" > >> command-line > >>option to qemu. The original work for this was all done by Gleb Natapov; > >>since > >>there hadn't seemed to be any activity on this since June, I picked up his > >>patches and ran with them (original posting: > >>http://www.archivum.info/qemu-devel@nongnu.org/2008-06/msg00167.html). > >>These > >>patches are basically the same as his, just cleaned up according to > >>Anthony > >>Liguori's feedback. > >> > >Actually there was activity on this. I addressed Anthony's feedback and > >reposted the patch series (I think even twice). Here is the last one: > >http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00448.html > >http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00442.html > >http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00443.html > >http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00444.html > >http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00445.html > >http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00446.html > > > >Fabrice said that he doesn't like vmware backdoor interface though > >(http://lists.gnu.org/archive/html/qemu-devel/2008-06/msg00487.html) > >I asked what interface he wants to use for qemu/guest communication, but > >he never replied to this :( > > > > 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 > 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'd really welcome addition of SMBIOS support too - for x86 the UUID in the SMBIOS tables, is the most effective way for a guest admin to find their machine identify in a format that can be correlated with the host machine by external admin tools. Xen has code for doing SMBIOS tables that could be useful to anyone wanting to work on this. Though it won't be a straight port since they integrate it with the HVM loader firmware, the general data table setup code ought to be reusable, only leaving the BIOS integration part. Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|