]> git.proxmox.com Git - mirror_ubuntu-zesty-kernel.git/commit
virtio_balloon: init 1st buffer in stats vq
authorLadi Prosek <lprosek@redhat.com>
Thu, 23 Mar 2017 07:04:18 +0000 (08:04 +0100)
committerTim Gardner <tim.gardner@canonical.com>
Mon, 3 Apr 2017 09:05:51 +0000 (10:05 +0100)
commitbe82994d7d8ac0a1f6f17e4ffc0a9dce12677ab6
tree467f8a59d5c20e2b0966b44f649b09198bafe162
parent10fa66bd0679bb4865a40e2d83ba949d4a3e8543
virtio_balloon: init 1st buffer in stats vq

BugLink: http://bugs.launchpad.net/bugs/1678930
commit fc8653228c8588a120f6b5dad6983b7b61ff669e upstream.

When init_vqs runs, virtio_balloon.stats is either uninitialized or
contains stale values. The host updates its state with garbage data
because it has no way of knowing that this is just a marker buffer
used for signaling.

This patch updates the stats before pushing the initial buffer.

Alternative fixes:
* Push an empty buffer in init_vqs. Not easily done with the current
  virtio implementation and violates the spec "Driver MUST supply the
  same subset of statistics in all buffers submitted to the statsq".
* Push a buffer with invalid tags in init_vqs. Violates the same
  spec clause, plus "invalid tag" is not really defined.

Note: the spec says:
When using the legacy interface, the device SHOULD ignore all values in
the first buffer in the statsq supplied by the driver after device
initialization. Note: Historically, drivers supplied an uninitialized
buffer in the first buffer.

Unfortunately QEMU does not seem to implement the recommendation
even for the legacy interface.

Signed-off-by: Ladi Prosek <lprosek@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
drivers/virtio/virtio_balloon.c