From 66bfc7cc61ca0f6ca8d57811024565d2c89f47f6 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Tue, 4 Oct 2016 07:07:20 -0400 Subject: [PATCH] remote: Increase bound limit for virDomainGetBlockIoTune We are about to add 6 new values to fetch. This will put us over the current limit of 16 (we're at 13 now). Once there are more than 16 parameters, this will affect existing clients that attempt to fetch blockiotune config values for the domain from the remote host since the server side has no mechanism to determine whether the capability for the emulator exists and thus would attempt to return all known values from the persistentDef. If attempting to fetch the blockiotune values from a running domain, the code will check the emulator capabilities and set maxparams (in qemuDomainGetBlockIoTune) appropriately. On the client side of the remote connection, it uses this constant in xdr_remote_domain_get_block_io_tune_ret and virTypedParamsDeserialize calls, so if a remote server returns more than 16 parameters, then the client will fail with "Unable to decode message payload". Signed-off-by: John Ferlan --- src/remote/remote_protocol.x | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/remote/remote_protocol.x b/src/remote/remote_protocol.x index 8a8e26303a..e8382dc51d 100644 --- a/src/remote/remote_protocol.x +++ b/src/remote/remote_protocol.x @@ -104,7 +104,7 @@ const REMOTE_DOMAIN_BLKIO_PARAMETERS_MAX = 16; const REMOTE_DOMAIN_MEMORY_PARAMETERS_MAX = 16; /* Upper limit on list of blockio tuning parameters. */ -const REMOTE_DOMAIN_BLOCK_IO_TUNE_PARAMETERS_MAX = 16; +const REMOTE_DOMAIN_BLOCK_IO_TUNE_PARAMETERS_MAX = 32; /* Upper limit on list of numa parameters. */ const REMOTE_DOMAIN_NUMA_PARAMETERS_MAX = 16; -- GitLab