[libvirt] [PATCH 0/5] admin: Fix updating of the client limits

Erik Skultety eskultet at redhat.com
Tue Aug 2 13:02:42 UTC 2016


On 30/07/16 15:44, John Ferlan wrote:
> 
> 
> On 07/20/2016 06:53 AM, Erik Skultety wrote:
>> This series slightly refactors the code in virnetserver.c in order to resolve
>> https://bugzilla.redhat.com/show_bug.cgi?id=1357776. The problem resides in
>> updating of the client limits which even though updated, thus "permitting" new
>> connections to be established, do not re-enable polling for the socket file
>> descriptor which results in all new connections still queueing on the socket.
>>
>> Erik Skultety (5):
>>   rpc: virnetserver: Rename ClientSetProcessingControls to
>>     ClientSetLimits
>>   rpc: virnetserver: Move virNetServerCheckLimits which is static up in
>>     the file
>>   rpc: virnetserver: Add code to CheckLimits to handle suspending of
>>     services
>>   admin: rpc: virnetserver: Fix updating of the client limits
>>   rpc: virnetserver: Remove dead code checking the client limits
>>
>>  daemon/admin_server.c  |  4 +--
>>  src/rpc/virnetserver.c | 97 +++++++++++++++++++++++---------------------------
>>  src/rpc/virnetserver.h |  6 ++--
>>  3 files changed, 50 insertions(+), 57 deletions(-)
>>
> 
> ACK series, but consider my note in 5/5
> 

Adjusted and pushed, thanks.

Erik




More information about the libvir-list mailing list