]> git.baikalelectronics.ru Git - kernel.git/commit
virtio: balloon: leak / fill balloon across S4
authorAmit Shah <amit.shah@redhat.com>
Wed, 29 Feb 2012 12:12:51 +0000 (17:42 +0530)
committerRusty Russell <rusty@rustcorp.com.au>
Wed, 29 Feb 2012 22:58:41 +0000 (09:28 +1030)
commit2adbb6e6c77607c5939427bff096e26ea4d59049
treeac0d2afc54f208fce7c91a4f0c3704d95e5d9561
parent25e95a6176830b48db86c8d04c3ac5a96c65e3df
virtio: balloon: leak / fill balloon across S4

commit 90ab08e3ee82bba28fa94198fe7f61d928753515 added support for S4 to
the balloon driver.  The freeze function did nothing to free the pages,
since reclaiming the pages from the host to immediately give them back
(if S4 was successful) seemed wasteful.  Also, if S4 wasn't successful,
the guest would have to re-fill the balloon.  On restore, the pages were
supposed to be marked freed and the free page counters were incremented
to reflect the balloon was totally deflated.

However, this wasn't done right.  The pages that were earlier taken away
from the guest during a balloon inflation operation were just shown as
used pages after a successful restore from S4.  Just a fancy way of
leaking lots of memory.

Instead of trying that, just leak the balloon on freeze and fill it on
restore/thaw paths.  This works properly now.  The optimisation to not
leak can be added later on after a bit of refactoring of the code.

Signed-off-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
drivers/virtio/virtio_balloon.c