Re: [PATCH v8 07/18] remote: New APIs for ThrottleGroup lifecycle management

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

 



On Wed, Feb 19, 2025 at 22:27:11 +0530, Harikumar Rajkumar wrote:
> From: Chun Feng Wu <danielwuwy@xxxxxxx>
> 
> Defined new public APIs:
> * virDomainSetThrottleGroup to add or update throttlegroup within specific domain,
>   it will be referenced by throttlefilter later in disk to do limits
> * virDomainGetThrottleGroup to get throttlegroup info, old-style is discarded
>   (APIs to query first for the number of parameters and then give it a
>   reasonably-sized pointer), instead, the new approach is adopted that
>   API returns allocated array of fields and number of fileds that are in it.
> * virDomainDelThrottleGroup to delete throttlegroup, it fails if this throttlegroup
>   is still referenced by some throttlefilter
> 
> Signed-off-by: Chun Feng Wu <danielwuwy@xxxxxxx>
> 
> * Reimplement getter API to fetch data from XML.
> * Apply suggested coding style changes.
> * Update of code documentation comments.
> * Update the version to 11.1.0.
> 
> Signed-off-by: Harikumar Rajkumar <harirajkumar230@xxxxxxxxx>
> ---

[...]

> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
> index 30a9f806f0..df7481c144 100644
> --- a/src/libvirt_private.syms
> +++ b/src/libvirt_private.syms
> @@ -680,6 +680,14 @@ virDomainTaintMessageTypeFromString;
>  virDomainTaintMessageTypeToString;
>  virDomainTaintTypeFromString;
>  virDomainTaintTypeToString;
> +virDomainThrottleFilterDefClear;
> +virDomainThrottleFilterFind;
> +virDomainThrottleGroupAdd;
> +virDomainThrottleGroupByName;
> +virDomainThrottleGroupDefCopy;
> +virDomainThrottleGroupDefFree;
> +virDomainThrottleGroupDel;
> +virDomainThrottleGroupUpdate;
>  virDomainTimerModeTypeFromString;
>  virDomainTimerModeTypeToString;
>  virDomainTimerNameTypeFromString;

This also doesn't belong into this patch.



[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