On 10/07/2016 06:58 AM, Daniel P. Berrange wrote:
On Fri, Oct 07, 2016 at 06:47:04AM -0400, John Ferlan wrote:
> Add VIR_DOMAIN_BLOCK_IOTUNE*LENGTH and VIR_DOMAIN_TUNABLE_BLKDEV*LENGTH
> parameters and their descriptions.
>
> Signed-off-by: John Ferlan <jferlan(a)redhat.com>
> ---
> Changes | 2 +-
> Virt.xs | 12 ++++++++++++
> lib/Sys/Virt/Domain.pm | 25 +++++++++++++++++++++++++
> 3 files changed, 38 insertions(+), 1 deletion(-)
>
> diff --git a/Changes b/Changes
> index 4985100..8d5683a 100644
> --- a/Changes
> +++ b/Changes
> @@ -2,7 +2,7 @@ Revision history for perl module Sys::Virt
>
> 2.4.0 2016-00-00
>
> - - XXX
> + - Add constants for the new block iotune parameters for length
>
> 2.3.0 2016-10-06
>
> diff --git a/Virt.xs b/Virt.xs
> index b4ca049..88ed722 100644
> --- a/Virt.xs
> +++ b/Virt.xs
> @@ -8275,6 +8275,12 @@ BOOT:
> REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_READ_IOPS_SEC_MAX,
BLOCK_IOTUNE_READ_IOPS_SEC_MAX);
> REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_WRITE_IOPS_SEC_MAX,
BLOCK_IOTUNE_WRITE_IOPS_SEC_MAX);
> REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_SIZE_IOPS_SEC,
BLOCK_IOTUNE_SIZE_IOPS_SEC);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_TOTAL_BYTES_SEC_MAX_LENGTH,
BLOCK_IOTUNE_TOTAL_BYTES_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_READ_BYTES_SEC_MAX_LENGTH,
BLOCK_IOTUNE_READ_BYTES_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_WRITE_BYTES_SEC_MAX_LENGTH,
BLOCK_IOTUNE_WRITE_BYTES_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_TOTAL_IOPS_SEC_MAX_LENGTH,
BLOCK_IOTUNE_TOTAL_IOPS_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_READ_IOPS_SEC_MAX_LENGTH,
BLOCK_IOTUNE_READ_IOPS_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_BLOCK_IOTUNE_WRITE_IOPS_SEC_MAX_LENGTH,
BLOCK_IOTUNE_WRITE_IOPS_SEC_MAX_LENGTH);
>
>
> REGISTER_CONSTANT(VIR_DOMAIN_BLOCK_RESIZE_BYTES, BLOCK_RESIZE_BYTES);
> @@ -8484,6 +8490,12 @@ BOOT:
> REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_TOTAL_BYTES_SEC_MAX,
TUNABLE_BLKDEV_TOTAL_BYTES_SEC_MAX);
> REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_TOTAL_IOPS_SEC_MAX,
TUNABLE_BLKDEV_TOTAL_IOPS_SEC_MAX);
> REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_SIZE_IOPS_SEC,
TUNABLE_BLKDEV_SIZE_IOPS_SEC);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_READ_BYTES_SEC_MAX_LENGTH,
TUNABLE_BLKDEV_READ_BYTES_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_READ_IOPS_SEC_MAX_LENGTH,
TUNABLE_BLKDEV_READ_IOPS_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_WRITE_BYTES_SEC_MAX_LENGTH,
TUNABLE_BLKDEV_WRITE_BYTES_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_WRITE_IOPS_SEC_MAX_LENGTH,
TUNABLE_BLKDEV_WRITE_IOPS_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_TOTAL_BYTES_SEC_MAX_LENGTH,
TUNABLE_BLKDEV_TOTAL_BYTES_SEC_MAX_LENGTH);
> + REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_BLKDEV_TOTAL_IOPS_SEC_MAX_LENGTH,
TUNABLE_BLKDEV_TOTAL_IOPS_SEC_MAX_LENGTH);
> REGISTER_CONSTANT_STR(VIR_DOMAIN_TUNABLE_CPU_IOTHREADSPIN,
TUNABLE_IOTHREADSPIN);
You missed docs these this set of constants, so i'm suprised
make test didn't complain
Oh right - forgot the second hunk... They're just a copy of the BLOCK
tunable anyway, so I copied and renamed the params to match the other
TUNABLE_BLKDEV defs in lib/Sys/Virt/Domain.pm:
diff --git a/lib/Sys/Virt/Domain.pm b/lib/Sys/Virt/Domain.pm
index 8bf4115..b2488a6 100644
--- a/lib/Sys/Virt/Domain.pm
+++ b/lib/Sys/Virt/Domain.pm
@@ -4297,6 +4297,31 @@ Maximum write throughput in I/O operations per sec
The maximum I/O operations per second
+=item Sys::Virt::Domain::TUNABLE_BLKDEV_TOTAL_BYTES_SEC_MAX_LENGTH
+
+The duration in seconds allowed for maximum total bytes processed per
second.
+
+=item Sys::Virt::Domain::TUNABLE_BLKDEV_READ_BYTES_SEC_MAX_LENGTH
+
+The duration in seconds allowed for maximum bytes read per second.
+
+=item Sys::Virt::Domain::TUNABLE_BLKDEV_WRITE_BYTES_SEC_MAX_LENGTH
+
+The duration in seconds allowed for maximum bytes written per second.
+
+=item Sys::Virt::Domain::TUNABLE_BLKDEV_TOTAL_IOPS_SEC_MAX_LENGTH
+
+The duration in seconds allowed for maximum total I/O operations processed
+per second.
+
+=item Sys::Virt::Domain::TUNABLE_BLKDEV_READ_IOPS_SEC_MAX_LENGTH
+
+The duration in seconds allowed for maximum I/O operations read per second.
+
+=item Sys::Virt::Domain::TUNABLE_BLKDEV_WRITE_IOPS_SEC_MAX_LENGTH
+
+The duration in seconds allowed for maximum I/O operations written per
second.
+
John