On 07/26/2013 01:13 AM, 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(-)
Looks like a better location to me. You may still want to wait for Dan's review, but you have my ACK, and I think this is probably safe for 1.1.1 as a bug-fix. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org