On Thu, Oct 15, 2015 at 16:43:55 -0400, John Ferlan wrote:
[...]
---
src/conf/domain_conf.c | 39 +++++++++++++++++++++++++++++++--------
1 file changed, 31 insertions(+), 8 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 70b2afc..3e15dcc 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
[...]
@@ -2341,19 +2344,39 @@
virDomainIOThreadIDDefArrayInit(virDomainDefPtr def)
if (def->iothreads == def->niothreadids)
return 0;
- while (def->niothreadids != def->iothreads) {
- if (!virDomainIOThreadIDFind(def, iothread_id)) {
- virDomainIOThreadIDDefPtr iothrid;
+ /* iothread's are numbered starting at 1, account for that */
+ thrmap = virBitmapNew(def->iothreads + 1);
You want to check whether the allocation succeeded ...
+ virBitmapSetAll(thrmap);
... or this will crash.
- if (!(iothrid = virDomainIOThreadIDAdd(def, iothread_id)))
- goto error;
- iothrid->autofill = true;
ACK,
Peter