On Mon, Dec 30, 2013 at 07:23:00PM +0200, Pavel Fux wrote: > Hi, > > I am talking about *Bug > 987088*<https://bugzilla.redhat.com/show_bug.cgi?id=987088>, > I have added a patch and a description of the fix, this is a copy of my > comment on the bug: > > the default code behavior is wait for 3 seconds and if the socket is not > opened yet, print this error and terminate. > > the code is in src/qemu/qemu_monitor.c in function qemuMonitorOpenUnix. > > In 2009 there was a patch that added the original 3 seconds retry, the > patch can be found here: > > http://www.redhat.com/archives/libvir-list/2009-July/msg00335.html > > I have added a patch with this solution: > > the default behavior stays the same, but a user can add a configuration > variable to qemu.conf and change the timeout value. > > every system needs a different value according to their system > configuration but anyway 3 seconds is not suitable for all cases. I think this is an inappropriate configuration setting. If we need a longer timeout then we should just use a longer timeout not require an admin to wait to see if ther box is broken and then try to set a config param to fix it. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list