[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL v2 2/9] dbus-vmstate: Increase the size of input stream buffer use
From: |
marcandre . lureau |
Subject: |
[PULL v2 2/9] dbus-vmstate: Increase the size of input stream buffer used during load |
Date: |
Thu, 1 Apr 2021 15:55:25 +0400 |
From: Priyankar Jain <priyankar.jain@nutanix.com>
This commit fixes an issue where migration is failing in the load phase
because of a false alarm about data unavailability.
Following is the error received when the amount of data to be transferred
exceeds the default buffer size setup by G_BUFFERED_INPUT_STREAM(4KiB),
even when the maximum data size supported by this backend is 1MiB
(DBUS_VMSTATE_SIZE_LIMIT):
dbus_vmstate_post_load: Invalid vmstate size: 4364
qemu-kvm: error while loading state for instance 0x0 of device
'dbus-vmstate/dbus-vmstate'
This commit sets the size of the input stream buffer used during load to
DBUS_VMSTATE_SIZE_LIMIT which is the maximum amount of data a helper can
send during save phase.
Secondly, this commit makes sure that the input stream buffer is loaded before
checking the size of the data available in it, rectifying the false alarm about
data unavailability.
Fixes: 5010cec2bc87 ("Add dbus-vmstate object")
Signed-off-by: Priyankar Jain <priyankar.jain@nutanix.com>
Message-Id:
<cdaad4718e62bf22fd5e93ef3e252de20da5c17c.1612273156.git.priyankar.jain@nutanix.com>
[ Modified printf format for gsize ]
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
backends/dbus-vmstate.c | 20 +++++++++++++++++---
1 file changed, 17 insertions(+), 3 deletions(-)
diff --git a/backends/dbus-vmstate.c b/backends/dbus-vmstate.c
index 2a0d2e4a31..9cfd758c42 100644
--- a/backends/dbus-vmstate.c
+++ b/backends/dbus-vmstate.c
@@ -204,6 +204,8 @@ static int dbus_vmstate_post_load(void *opaque, int
version_id)
m = g_memory_input_stream_new_from_data(self->data, self->data_size, NULL);
s = g_data_input_stream_new(m);
g_data_input_stream_set_byte_order(s, G_DATA_STREAM_BYTE_ORDER_BIG_ENDIAN);
+ g_buffered_input_stream_set_buffer_size(G_BUFFERED_INPUT_STREAM(s),
+ DBUS_VMSTATE_SIZE_LIMIT);
nelem = g_data_input_stream_read_uint32(s, NULL, &err);
if (err) {
@@ -244,11 +246,23 @@ static int dbus_vmstate_post_load(void *opaque, int
version_id)
}
len = g_data_input_stream_read_uint32(s, NULL, &err);
+ if (len > DBUS_VMSTATE_SIZE_LIMIT) {
+ error_report("%s: Invalid vmstate size: %u", __func__, len);
+ return -1;
+ }
+
+ g_buffered_input_stream_fill(G_BUFFERED_INPUT_STREAM(s), len, NULL,
+ &err);
+ if (err) {
+ goto error;
+ }
+
avail = g_buffered_input_stream_get_available(
G_BUFFERED_INPUT_STREAM(s));
-
- if (len > DBUS_VMSTATE_SIZE_LIMIT || len > avail) {
- error_report("%s: Invalid vmstate size: %u", __func__, len);
+ if (len > avail) {
+ error_report("%s: Not enough data available to load for Id: '%s'. "
+ "Available data size: %zu, Actual vmstate size: %u",
+ __func__, id, avail, len);
return -1;
}
--
2.29.0
- [PULL v2 0/9] For 6.0 patches, marcandre . lureau, 2021/04/01
- [PULL v2 1/9] util: fix use-after-free in module_load_one, marcandre . lureau, 2021/04/01
- [PULL v2 2/9] dbus-vmstate: Increase the size of input stream buffer used during load,
marcandre . lureau <=
- [PULL v2 3/9] docs: simplify each section title, marcandre . lureau, 2021/04/01
- [PULL v2 4/9] yank: Remove dependency on qiochannel, marcandre . lureau, 2021/04/01
- [PULL v2 5/9] yank: Always link full yank code, marcandre . lureau, 2021/04/01
- [PULL v2 6/9] chardev/char.c: Move object_property_try_add_child out of chardev_new, marcandre . lureau, 2021/04/01
- [PULL v2 7/9] chardev/char.c: Always pass id to chardev_new, marcandre . lureau, 2021/04/01
- [PULL v2 8/9] chardev: Fix yank with the chardev-change case, marcandre . lureau, 2021/04/01
- [PULL v2 9/9] tests: Add tests for yank with the chardev-change case, marcandre . lureau, 2021/04/01
- Re: [PULL v2 0/9] For 6.0 patches, Peter Maydell, 2021/04/01