From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=47093 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OdhMr-0004IH-H4 for qemu-devel@nongnu.org; Tue, 27 Jul 2010 06:26:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OdhMq-0000n1-5y for qemu-devel@nongnu.org; Tue, 27 Jul 2010 06:26:37 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49854) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OdhMp-0000mo-UL for qemu-devel@nongnu.org; Tue, 27 Jul 2010 06:26:36 -0400 Date: Tue, 27 Jul 2010 11:26:32 +0100 From: "Daniel P. Berrange" Subject: Re: [Qemu-devel] [PATCH] move 'unsafe' to end of caching modes in help Message-ID: <20100727102632.GI12387@redhat.com> References: <4C4704FC020000480009AB6E@sinclair.provo.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4C4704FC020000480009AB6E@sinclair.provo.novell.com> Reply-To: "Daniel P. Berrange" List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Bruce Rogers Cc: qemu-devel@nongnu.org On Wed, Jul 21, 2010 at 02:32:28PM -0600, Bruce Rogers wrote: > Libvirt parses qemu help output to determine qemu features. In particular > it probes for the following: "cache=writethrough|writeback|none". The > addition of the unsafe cache mode was inserted within this string, as > opposed to being added to the end, which impacted libvirt's probe. > Unbreak libvirt by keeping the existing cache modes intact and add > unsafe to the end. We have inverted the check we made so that instead of doing a positive check for the new syntax, we do a negative check for the old syntax, avoiding this problem. Of couse any existing deployed livirt will still be broken with latest QEMU, but the previous QEMU version string change mean that is already the case for any libvirt < 0.8.2 Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|