On Wed, Dec 07, 2011 at 03:45:10PM +0100, Jiri Denemark wrote: > On Wed, Dec 07, 2011 at 14:30:10 +0000, Daniel P. Berrange wrote: > > While investigating the RPC problems I've noticed that we have had a > > huge performance regression for virDomainGetInfo() due to the change > > which makes virDomainEndJob() save the XML status file every time it > > is called. > > > > Previous to this change, 2000 calls to virDomainGetInfo() took ~2.5 seconds > > > > After this change, 2000 calls to virDomainGetInfo() take 2 *minutes* 45 secs > > > > I don't have an answer right now and it shouldnt block the release, but we > > have todo something to avoid this overhead. > > I think we could change job APIs to only save XML status if async job is > active since we are currently not really interested in other jobs anyway. Ok, that sounds like a reasonable idea. Could you cook up a patch for that. 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