Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1493259 > unrolled thread

[PATCH v2 1/1] virtio: update balloon size in balloon "probe"

Back to article view | Back to linux.kernel

Contents

  [PATCH v2 1/1] virtio: update balloon size in balloon "probe" "Denis V. Lunev" <den@openvz.org> - 2016-09-29 12:20 +0200
    Re: [PATCH v2 1/1] virtio: update balloon size in balloon "probe" Konstantin Neumoin <kneumoin@virtuozzo.com> - 2016-10-04 12:50 +0200

#1493259 — [PATCH v2 1/1] virtio: update balloon size in balloon "probe"

From: Konstantin Neumoin <kneumoin@virtuozzo.com>

The following commit 'fad7b7b27b6a (virtio_balloon: Use a workqueue
instead of "vballoon" kthread)' has added a regression. Original code with
kthread starts the thread inside probe and checks the necessity to update
balloon inside the thread immediately.

Nowadays the code behaves differently. Work is queued only on the first
command from the host after the negotiation. Thus there is a window
especially at the guest startup or the module reloading when the balloon
size is not updated until the notification from the host.

This patch adds balloon size check at the end of the probe to match
original behaviour.

Signed-off-by: Konstantin Neumoin <kneumoin@virtuozzo.com>
Signed-off-by: Denis V. Lunev <den@openvz.org>
CC: Michael S. Tsirkin <mst@redhat.com>
---
 drivers/virtio/virtio_balloon.c | 2 ++
 1 file changed, 2 insertions(+)

Changes from v1:
- fixed description
- removed update_balloon_size() call

diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c
index 4e7003d..181793f 100644
--- a/drivers/virtio/virtio_balloon.c
+++ b/drivers/virtio/virtio_balloon.c
@@ -577,6 +577,8 @@ static int virtballoon_probe(struct virtio_device *vdev)
 
 	virtio_device_ready(vdev);
 
+	if (towards_target(vb))
+		virtballoon_changed(vdev);
 	return 0;
 
 out_del_vqs:
-- 
2.7.4

[toc] | [next] | [standalone]

#1495354

ping

On 09/29/2016 01:17 PM, Denis V. Lunev wrote:
> From: Konstantin Neumoin <kneumoin@virtuozzo.com>
>
> The following commit 'fad7b7b27b6a (virtio_balloon: Use a workqueue
> instead of "vballoon" kthread)' has added a regression. Original code with
> kthread starts the thread inside probe and checks the necessity to update
> balloon inside the thread immediately.
>
> Nowadays the code behaves differently. Work is queued only on the first
> command from the host after the negotiation. Thus there is a window
> especially at the guest startup or the module reloading when the balloon
> size is not updated until the notification from the host.
>
> This patch adds balloon size check at the end of the probe to match
> original behaviour.
>
> Signed-off-by: Konstantin Neumoin <kneumoin@virtuozzo.com>
> Signed-off-by: Denis V. Lunev <den@openvz.org>
> CC: Michael S. Tsirkin <mst@redhat.com>
> ---
>   drivers/virtio/virtio_balloon.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> Changes from v1:
> - fixed description
> - removed update_balloon_size() call
>
> diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c
> index 4e7003d..181793f 100644
> --- a/drivers/virtio/virtio_balloon.c
> +++ b/drivers/virtio/virtio_balloon.c
> @@ -577,6 +577,8 @@ static int virtballoon_probe(struct virtio_device *vdev)
>   
>   	virtio_device_ready(vdev);
>   
> +	if (towards_target(vb))
> +		virtballoon_changed(vdev);
>   	return 0;
>   
>   out_del_vqs:

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web