[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] balloon: error if guest driver is not initi
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] balloon: error if guest driver is not initialised |
Date: |
Fri, 9 Dec 2011 17:32:33 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On (Fri) 09 Dec 2011 [11:55:21], Daniel P. Berrange wrote:
> 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.
OK, thanks.
I'll retract this patch series.
Amit
- [Qemu-devel] [PATCH 0/3] balloon: error if guest driver is not initialised, Amit Shah, 2011/12/09
- [Qemu-devel] [PATCH 1/3] balloon: Make functions return 0 on OK, -1 on error., Amit Shah, 2011/12/09
- [Qemu-devel] [PATCH 2/3] balloon: report error if ballooning operation fails, Amit Shah, 2011/12/09
- [Qemu-devel] [PATCH 3/3] virtio-balloon: report error if balloon driver in guest not available, Amit Shah, 2011/12/09
- Re: [Qemu-devel] [PATCH 0/3] balloon: error if guest driver is not initialised, Daniel P. Berrange, 2011/12/09
- Re: [Qemu-devel] [PATCH 0/3] balloon: error if guest driver is not initialised,
Amit Shah <=