Re: [PATCH 00/10] Backported some patches from refactory branches (5th Nov) second

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

 



On Thu, Nov 5, 2015 at 3:42 PM, Fabiano Fidêncio <fidencio@xxxxxxxxxx> wrote:
> On Thu, Nov 5, 2015 at 2:23 PM, Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
>> This patchset supersed last patchset.
>>
>> Changes:
>> - removed merged patches;
>> - added some patches to the set.
>>
>> Alon's patch is removed from patchset, there is no reason to keep
>> sending it all the time as already on the ML not changed.
>>
>> Looks like today I added too few patches.
>>
>> Marc-André Lureau (10):
>>   server: remove dispatching creation of worker channels
>
> As you said, Christophe is going to split this one, right?
>
>>   worker: move delta computation
>
> Was previously ACKED.
>
>>   worker: move shadow_new() and container_new()
>
> There is a problem with this patch, already pointed out in the first
> round of reviews Today.
>
>>   worker: rename red_pipe_add_drawable* to dcc_add_drawable*
>>   worker: remove unused WORKER_TO_DCC macro
>
> These two are the ones you split, right? So, they need review.
>
>>   worker: move remove worker from WORKER_FOREACH_DCC_SAFE
>
> Already ACKED. But I'd like to ask you to change the commit log "move
> remove worker ... " is a bit confusing ...
>
>>   worker: move some compress stats to display
>>   worker: use RED_CHANNEL_CLIENT for dcc
>>   worker: use more RED_CHANNEL_CLIENT
>>   worker: use more DCC_TO_WORKER
>
> These for patches are new and they need review, right?
>
>>
>>  server/dispatcher.c         |   6 +-
>>  server/display-channel.c    | 100 ++++++++++
>>  server/display-channel.h    |   4 +
>>  server/red_dispatcher.c     |  74 +++-----
>>  server/red_dispatcher.h     |  36 +---
>>  server/red_worker.c         | 445 ++++++++++++++------------------------------
>>  server/red_worker.h         |   2 +
>>  server/spice_server_utils.h |   1 -
>>  server/tree.c               |  39 ++++
>>  server/tree.h               |   9 +
>>  10 files changed, 321 insertions(+), 395 deletions(-)
>>
>> --
>> 2.4.3
>>
>> _______________________________________________
>> Spice-devel mailing list
>> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
>> http://lists.freedesktop.org/mailman/listinfo/spice-devel

And somehow I managed to only send it to Frediano. :-\
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]