Daniel Veillard wrote:
On Wed, Oct 31, 2007 at 05:20:01PM +0900, Saori Fukuta wrote:
On Tue, 30 Oct 2007 16:54:36 -0400 Daniel Veillard wrote:
The associated patch compiles, but I have not yet tested it, it's
basically how I would expect to finish the NUMA work, but it certainly need
debug and testing. I will look at this tomorrow, but I welcome feedback :-)
sounds good to me, I tested with your patch, and I have four fixes for it.
I tested with the latest code, and found a problem with specifying a
value in the cpuset in XML that is greater than maxcpu-1. The attached
patch corrects this behavior, but the issue remains that an out of range
cpu will prevent the domain from starting. This is the way xm create
works today.
In talking with Daniel about this, he made the point that it is not
necessarily desirable for the failure of a tuning parameter (e.g.,
config specifies cpu 5 and the domain is now being started on a 4 cpu
machine) to cause domain creation to fail. He mentioned phones ringing
in the middle of the night somewhere for support when the domain fails
to start. I think he has a point. Wouldn't it make more sense to start
the domain(s) at perhaps suboptimal performance, than to have everything
shut down until someone can come and figure out the problem?
It would seem better if the domain started (with no cpu affinity) and a
warning was posted. This might also argue for separation of domain
config and tuning parameters. Or at least for 2 flavors of create? One
that fails if tuning parameters cannot be activated, and one that doesn't.
--
Elizabeth Kon (Beth)
IBM Linux Technology Center
Open Hypervisor Team
email: eak@xxxxxxxxxx
Index: src/xml.c
===================================================================
RCS file: /data/cvs/libvirt/src/xml.c,v
retrieving revision 1.97
diff -u -r1.97 xml.c
--- src/xml.c 31 Oct 2007 09:39:13 -0000 1.97
+++ src/xml.c 2 Nov 2007 15:20:44 -0000
@@ -126,7 +126,7 @@
while ((*cur >= '0') && (*cur <= '9')) {
ret = ret * 10 + (*cur - '0');
- if (ret > maxcpu)
+ if (ret >= maxcpu)
return (-1);
cur++;
}
@@ -1647,6 +1647,8 @@
}
}
free(cpuset);
+ if (res < 0)
+ goto error;
} else {
virXMLError(conn, VIR_ERR_NO_MEMORY, xmldesc, 0);
}
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list