On Fri, Jul 26, 2013 at 09:13:43AM +0200, Michal Privoznik wrote: > Currently, even if max_client limit is hit, we accept() incoming > connection request, but close it immediately. This has disadvantage of > not using listen() queue. We should accept() only those clients we > know we can serve and let all other wait in the (limited) queue. > --- > > diff to v1: > -moved logic from virNetServerDispatchCheck to virNetServerAddClient > > src/rpc/virnetserver.c | 33 ++++++++++++++++++++++++++++----- > 1 file changed, 28 insertions(+), 5 deletions(-) ACK, I like this version more. NB, I think we must wait till post-1.1.1 to push this, since it is really a new feature and so not suitable for freeze policy. 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