Re: [PATCH 0/9] Slightly rework our streams

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

 



On 20.04.2016 16:04, John Ferlan wrote:
> 
> 
> On 04/15/2016 09:51 AM, Michal Privoznik wrote:
>> This is not the big patch set that enables sparse streams. Not
>> just yet. I'm merely sending first few patches that prepare the
>> environment for that. These can, however, go in independent of
>> sparse streams.
>>
>> Michal Privoznik (9):
>>   daemonClientStream: Use unsigned int to store stream @serial
>>   daemon stream: Prefer bool over unsigned int var:1
>>   daemon stream: Convert @tx in daemonClientStream to bool
>>   daemon stream: Remove useless empty lines from header file
>>   virNetClientCallDispatchStream: Update comment
>>   daemonStreamHandleRead: Rework to follow our coding pattern
>>   Revert "rpc: Fix slow volume download (virsh vol-download)"
>>   virnetclientstream: Process stream messages later
>>   virStream{Recv,Send}All: Increase client buffer
>>
>>  daemon/stream.c               | 116 +++++++++++++++----------------
>>  daemon/stream.h               |   2 -
>>  src/libvirt-stream.c          |   5 +-
>>  src/rpc/virnetclient.c        |   6 +-
>>  src/rpc/virnetclientstream.c  | 158 +++++++++++++++---------------------------
>>  src/rpc/virnetserverprogram.c |  12 ++--
>>  src/rpc/virnetserverprogram.h |   4 +-
>>  7 files changed, 129 insertions(+), 174 deletions(-)
>>
> 
> I'm assuming for patch 2-3, the memory for the int -> bool conversions
> is all "local" to the client side...
> 
> I've made comments on a few patches... I'm not an expert in this space
> by any stretch - hopefully Martin can look at patch 8 (and since it goes
> w/ 7 - they're a matched set.
> 
> ACK for at at least 1-6... I'm OK with 7&8 (although I had a double
> check type question in 8 regarding the while loop and usage of 'want' as
> a loop control along with the st->rx that could have been Queue'd.
> 
> I'm OK with patch 9 as well - I think it's fine - a bit more memory for
> a bit less back and forth.  Standard tradeoff to me ;-). Perhaps someone
> else may have agita over it though, so lets just be sure no one does
> before pushing that one.


Thank you, I've pushed 1-6 for now as I wait for more reviews on the rest.

Michal

--
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]