On Fri, Apr 27, 2012 at 12:45:56AM +0530, Amit Shah wrote: > When the balloon module is removed, we deflate the balloon, reclaiming > all the pages that were given to the host. However, we don't update the > config values for the new balloon size, resulting in the host showing > outdated balloon values. > > The size update is done after each leak and fill operation, only the > module removal case was left out. > > Signed-off-by: Amit Shah <amit.shah@xxxxxxxxxx> applied, thanks the rest are 3.5 material imo > --- > drivers/virtio/virtio_balloon.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c > index 6921326..04baad6 100644 > --- a/drivers/virtio/virtio_balloon.c > +++ b/drivers/virtio/virtio_balloon.c > @@ -390,6 +390,7 @@ static void __devexit virtballoon_remove(struct virtio_device *vdev) > /* There might be pages left in the balloon: free them. */ > while (vb->num_pages) > leak_balloon(vb, vb->num_pages); > + update_balloon_size(vb); > > /* Now we reset the device so we can clean up the queues. */ > vdev->config->reset(vdev); > -- > 1.7.7.6 _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization