From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:46309) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RYz31-0008Nd-SQ for qemu-devel@nongnu.org; Fri, 09 Dec 2011 06:55:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RYz30-0000qZ-Fc for qemu-devel@nongnu.org; Fri, 09 Dec 2011 06:55:27 -0500 Received: from mx1.redhat.com ([209.132.183.28]:45864) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RYz30-0000qN-5p for qemu-devel@nongnu.org; Fri, 09 Dec 2011 06:55:26 -0500 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id pB9BtPNB008245 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Fri, 9 Dec 2011 06:55:25 -0500 Date: Fri, 9 Dec 2011 11:55:21 +0000 From: "Daniel P. Berrange" Message-ID: <20111209115521.GB25940@redhat.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH 0/3] balloon: error if guest driver is not initialised Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amit Shah Cc: Luiz Capitulino , qemu list , Markus Armbruster On Fri, Dec 09, 2011 at 05:19:35PM +0530, Amit Shah wrote: > Hello, > > These patches make qemu generate an error on failure in setting a > balloon value: > > (qemu) balloon 400 > Device 'balloon' has not been activated > > This can happen when the guest hasn't yet loaded a balloon driver. This is a pretty significant change in behaviour/semantics of the balloon driver interface. eg, when libvirt starts a guest, it launches QEMU paused, and immediately sets a balloon target, which may be lower than the memory limit, and then starts CPUS to let the guest boot. The guest will utilize this target when it loads the balloon driver during bootup. With this proposed change, libvirt will immediately get an error, and have to guess when the balloon driver might be loaded by the guest, in order to set the limit post-boot. > Previously, such a ballooning attempt didn't produce an error, but the > virtio-balloon driver remembered the value nevertheless, causing a > module load in the guest using the attempted balloon value > originally. This may not be the desired course of action. This is *exactly* desired behaviour. Please don't change this. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|