Re: [PATCH 5/7] qemu: propagate iotune settings to all disks in the group

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 08, 2020 at 09:49:29AM +0300, Nikolay Shirokovskiy wrote:
> Currently upon successfull call to qemu's implementation of
> virDomainSetBlockIoTune iotune settings are changed only for the
> disk given in API if the disk is in iotune group while we need
> to change the settings for all disks in the group.
> 
> Signed-off-by: Nikolay Shirokovskiy <nshirokovskiy@xxxxxxxxxxxxx>
> ---
>  src/qemu/qemu_driver.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)

I'd suggest re-ordering so that this patch is *before* the
patch 3 which forbids the inconsistent disk settings.

> 
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index f1ee25aebd..5713266329 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -19102,6 +19102,29 @@ qemuDomainSetBlockIoTuneDefaults(virDomainBlockIoTuneInfoPtr newinfo,
>  }
>  
>  
> +static void
> +qemuDomainSetGroupBlockIoTune(virDomainDefPtr def,
> +                              virDomainBlockIoTuneInfoPtr iotune)
> +{
> +    size_t i;
> +
> +    if (!iotune->group_name)
> +        return;
> +
> +    for (i = 0; i < def->ndisks; i++) {
> +        virDomainDiskDefPtr d = def->disks[i];
> +
> +        if (STREQ_NULLABLE(d->blkdeviotune.group_name, iotune->group_name)) {
> +            char *tmp;
> +
> +            tmp = d->blkdeviotune.group_name;

nitpick - assign tmp at time of declaration.

> +            d->blkdeviotune = *iotune;
> +            d->blkdeviotune.group_name = tmp;
> +        }
> +    }
> +}

Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux