From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39958) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1akuCm-0007ij-JL for qemu-devel@nongnu.org; Tue, 29 Mar 2016 10:01:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1akuCg-000858-ER for qemu-devel@nongnu.org; Tue, 29 Mar 2016 10:01:12 -0400 Received: from mailhub.sw.ru ([195.214.232.25]:21664 helo=relay.sw.ru) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1akuCg-00081t-1a for qemu-devel@nongnu.org; Tue, 29 Mar 2016 10:01:06 -0400 From: "Denis V. Lunev" Date: Tue, 29 Mar 2016 17:00:49 +0300 Message-Id: <1459260049-18231-1-git-send-email-den@openvz.org> Subject: [Qemu-devel] [PATCH 1/1] virtio-balloon: reset the statistic timer to load device List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: "Denis V. Lunev" , "Michael S. Tsirkin" , rkagan@virtuozzo.com, Pavel Butsykin From: Pavel Butsykin If before loading snapshot we had set the timer of statistics, then after applying snapshot the expiry time would be irrelevant for the restored state of the virtual clocks. A simple fix is just to restart the timer after loading snapshot. For the user it may look like a long delay of statistics update after switch to the snapshot. Signed-off-by: Pavel Butsykin Reviewed-by: Roman Kagan Signed-off-by: Denis V. Lunev CC: Michael S. Tsirkin --- hw/virtio/virtio-balloon.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c index 22ad25c..c74101e 100644 --- a/hw/virtio/virtio-balloon.c +++ b/hw/virtio/virtio-balloon.c @@ -426,6 +426,10 @@ static int virtio_balloon_load_device(VirtIODevice *vdev, QEMUFile *f, s->num_pages = qemu_get_be32(f); s->actual = qemu_get_be32(f); + + if (balloon_stats_enabled(s)) { + balloon_stats_change_timer(s, s->stats_poll_interval); + } return 0; } -- 2.1.4