Daniel P. Berrange wrote:
On Fri, May 11, 2007 at 11:15:29AM +0100, Richard W.M. Jones wrote:Daniel Veillard wrote:Yes it does, so that third patch is wrong. In fact xenHypervisorInit already produces errors for all paths except the one where opening the HV connection fails.On Thu, May 10, 2007 at 05:24:00PM +0100, Richard W.M. Jones wrote:* Add some extra generated files to .cvsignore * Remove an unused header in qemud/uuid.c* Log an error if xenHypervisorOpen fails because xenHypervisorInit has failed.just wondering about the last one, would that add a message if trying as non-root ?Just make reporting the HV initialization error message conditional on getuid() == 0. We always want to see the error if root, and never if non-root.
That's not good. With the call to virXenError as is in the patch, it would hide other important errors. If we move the error to the part where it can't open /proc/xen/privcmd, then people would get that error when they didn't have Xen installed (more of a feature than an error).
OK to commit the first two?Yes.
About to commit. Rich. -- Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/ Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street,Windsor, Berkshire, SL4 1TE, United Kingdom. Registered in England and Wales
under Company Registration No. 03798903
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature