[libvirt] [PATCH 1/2] conf: Provide error on undefined iothreadsched entry
John Ferlan
jferlan at redhat.com
Mon Aug 15 14:55:33 UTC 2016
When commit id '6dfb4507' refactored where the iothreadsched data was
stored, the error message for when the virDomainIOThreadIDFind failed
to find an iothreadid ("iothreadsched attribute 'iothreads' uses
undefined iothread ids") was lost. This led to the possibility that
someone would try to use it, but receive the generic message "An error
occurred, but the cause is unknown".
This patch adds the error message back so that someone will know that
they have an invalid configuration.
Signed-off-by: John Ferlan <jferlan at redhat.com>
---
src/conf/domain_conf.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 82876f3..9037304 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -15490,8 +15490,12 @@ virDomainDefGetIOThreadSched(virDomainDefPtr def,
{
virDomainIOThreadIDDefPtr iothrinfo;
- if (!(iothrinfo = virDomainIOThreadIDFind(def, iothread)))
+ if (!(iothrinfo = virDomainIOThreadIDFind(def, iothread))) {
+ virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
+ _("Cannot find 'iothread' : %u"),
+ iothread);
return NULL;
+ }
return &iothrinfo->sched;
}
--
2.7.4
More information about the libvir-list
mailing list