External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-22 05:35:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

API: Remove internal impl detail from VIR_DOMAIN_IOTHREAD_THREAD_POOL_{MIN,MAX} description

Browse Source 
When introducing VIR_DOMAIN_IOTHREAD_THREAD_POOL_MIN and
VIR_DOMAIN_IOTHREAD_THREAD_POOL_MAX typed parameters, I've made a
shortcut. Since at the monitor level these two are set in two
separate calls and minimum has to be always smaller than maximum
(or equal to it), it may happen that one of the values we want to
set violates this restriction. So I've put a little note in the
public API description warning users about this.

However, the proper solution is to have a logic that checks the
current values and based on that set either minimum or maximum
value first. But until we get there, remove that note from the
public API before it gets released.

Related: https://gitlab.com/libvirt/libvirt/-/issues/339
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Michal Privoznik 2022-06-30 08:32:49 +02:00
parent 7989edd742
commit 0ba3870406
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
include/libvirt/libvirt-domain.h
View File

@ -2513,9 +2513,7 @@ int virDomainDelIOThread(virDomainPtr domain,
* *
* Sets the lower bound for thread pool size. A value of -1 disables this bound * Sets the lower bound for thread pool size. A value of -1 disables this bound
* leaving hypervisor use its default value, though this value is not accepted * leaving hypervisor use its default value, though this value is not accepted
* for running domains. Due to internal implementation it's recommended to set * for running domains. Accepted type is VIR_TYPED_PARAM_INT.
* VIR_DOMAIN_IOTHREAD_THREAD_POOL_MIN and VIR_DOMAIN_IOTHREAD_THREAD_POOL_MAX
* separately. Accepted type is VIR_TYPED_PARAM_INT.
* *
* Since: 8.5.0 * Since: 8.5.0
*/ */
@ -2527,9 +2525,7 @@ int virDomainDelIOThread(virDomainPtr domain,
* Sets the upper bound for thread pool size. A value of -1 disables this bound * Sets the upper bound for thread pool size. A value of -1 disables this bound
* leaving hypervisor use its default value, though this value is not accepted * leaving hypervisor use its default value, though this value is not accepted
* for running domains. Since the upper band has to be equal to or greater than * for running domains. Since the upper band has to be equal to or greater than
* lower bound value of 0 is not accepted. Due to internal implementation it's * lower bound value of 0 is not accepted. Accepted type is
* recommended to set VIR_DOMAIN_IOTHREAD_THREAD_POOL_MIN and
* VIR_DOMAIN_IOTHREAD_THREAD_POOL_MAX separately. Accepted type is
* VIR_TYPED_PARAM_INT. * VIR_TYPED_PARAM_INT.
* *
* Since: 8.5.0 * Since: 8.5.0