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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]