|
 |
77609c |
From 91465d5e7cb407604fd8697587a52aafe0040ad7 Mon Sep 17 00:00:00 2001
|
|
|
77609c |
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
|
|
|
77609c |
Date: Tue, 14 Sep 2021 14:17:16 +0100
|
|
|
77609c |
Subject: [PATCH 1/2] virtio-balloon: Fix page-poison subsection name
|
|
|
77609c |
MIME-Version: 1.0
|
|
|
77609c |
Content-Type: text/plain; charset=UTF-8
|
|
|
77609c |
Content-Transfer-Encoding: 8bit
|
|
|
77609c |
|
|
|
77609c |
RH-Author: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
77609c |
RH-MergeRequest: 46: virtio-balloon: Fix page-poison subsection name
|
|
|
77609c |
RH-Commit: [1/1] b5907c252dae636b4b145a6f13fbed4bf5fad9d2 (dagrh/c-9-s-qemu-kvm)
|
|
|
77609c |
RH-Bugzilla: 1984401
|
|
|
77609c |
RH-Acked-by: Philippe Mathieu-Daudé <philmd@redhat.com>
|
|
|
77609c |
RH-Acked-by: David Hildenbrand <david@redhat.com>
|
|
|
77609c |
RH-Acked-by: Cornelia Huck <cohuck@redhat.com>
|
|
|
77609c |
|
|
|
77609c |
The subsection name for page-poison was typo'd as:
|
|
|
77609c |
|
|
|
77609c |
vitio-balloon-device/page-poison
|
|
|
77609c |
|
|
|
77609c |
Note the missing 'r' in virtio.
|
|
|
77609c |
|
|
|
77609c |
When we have a machine type that enables page poison, and the guest
|
|
|
77609c |
enables it (which needs a new kernel), things fail rather unpredictably.
|
|
|
77609c |
|
|
|
77609c |
The fallout from this is that most of the other subsections fail to
|
|
|
77609c |
load, including things like the feature bits in the device, one
|
|
|
77609c |
possible fallout is that the physical addresses of the queues
|
|
|
77609c |
then get aligned differently and we fail with an error about
|
|
|
77609c |
last_avail_idx being wrong.
|
|
|
77609c |
It's not obvious to me why this doesn't produce a more obvious failure,
|
|
|
77609c |
but virtio's vmstate loading is a bit open-coded.
|
|
|
77609c |
|
|
|
77609c |
Fixes: 7483cbbaf82 ("virtio-balloon: Implement support for page poison reporting feature")
|
|
|
77609c |
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1984401
|
|
|
77609c |
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
|
|
|
77609c |
Message-Id: <20210914131716.102851-1-dgilbert@redhat.com>
|
|
|
77609c |
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
77609c |
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
|
|
|
77609c |
Reviewed-by: David Hildenbrand <david@redhat.com>
|
|
|
77609c |
(cherry picked from commit 243a9284a989a38e32ceb3990eb795f5cf6f3be0)
|
|
|
77609c |
---
|
|
|
77609c |
hw/virtio/virtio-balloon.c | 2 +-
|
|
|
77609c |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
77609c |
|
|
|
77609c |
diff --git a/hw/virtio/virtio-balloon.c b/hw/virtio/virtio-balloon.c
|
|
|
77609c |
index 4b5d9e5e50..ced070d64f 100644
|
|
|
77609c |
--- a/hw/virtio/virtio-balloon.c
|
|
|
77609c |
+++ b/hw/virtio/virtio-balloon.c
|
|
|
77609c |
@@ -850,7 +850,7 @@ static const VMStateDescription vmstate_virtio_balloon_free_page_hint = {
|
|
|
77609c |
};
|
|
|
77609c |
|
|
|
77609c |
static const VMStateDescription vmstate_virtio_balloon_page_poison = {
|
|
|
77609c |
- .name = "vitio-balloon-device/page-poison",
|
|
|
77609c |
+ .name = "virtio-balloon-device/page-poison",
|
|
|
77609c |
.version_id = 1,
|
|
|
77609c |
.minimum_version_id = 1,
|
|
|
77609c |
.needed = virtio_balloon_page_poison_support,
|
|
|
77609c |
--
|
|
|
77609c |
2.27.0
|
|
|
77609c |
|