No subject


Mon May 2 16:12:28 PDT 2011


worker->surfaces  should have the required state.

So, I wouldn't apply it until we prove it is necessary (not the other
way around).

> ---
> =A0server/red_worker.c | =A0 15 ++++++++-------
> =A01 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/server/red_worker.c b/server/red_worker.c
> index 399ebea..550bda9 100644
> --- a/server/red_worker.c
> +++ b/server/red_worker.c
> @@ -10156,6 +10156,8 @@ static inline void handle_dev_update(RedWorker *w=
orker)
> =A0 =A0 uint32_t num_dirty_rects;
> =A0 =A0 uint32_t surface_id;
> =A0 =A0 uint32_t clear_dirty_region;
> + =A0 =A0RingItem *link;
> + =A0 =A0Surfaces *surfaces;
>
> =A0 =A0 receive_data(worker->channel, &surface_id, sizeof(uint32_t));
> =A0 =A0 receive_data(worker->channel, &rect, sizeof(SpiceRect *));
> @@ -10170,18 +10172,17 @@ static inline void handle_dev_update(RedWorker =
*worker)
> =A0 =A0 validate_surface(&worker->surfaces, surface_id);
> =A0 =A0 red_update_area(worker, rect, surface_id);
>
> - =A0 =A0surface =3D &worker->surfaces.surfaces[surface_id];
> - =A0 =A0region_ret_rects(&surface->draw_dirty_region, dirty_rects, num_d=
irty_rects);
> -
> - =A0 =A0if (clear_dirty_region) {
> - =A0 =A0 =A0 =A0region_clear(&surface->draw_dirty_region);
> + =A0 =A0SURFACES_FOREACH(link, surfaces, worker) {
> + =A0 =A0 =A0 =A0surface =3D &surfaces->surfaces[surface_id];
> + =A0 =A0 =A0 =A0region_ret_rects(&surface->draw_dirty_region, dirty_rect=
s, num_dirty_rects);
> + =A0 =A0 =A0 =A0if (clear_dirty_region) {
> + =A0 =A0 =A0 =A0 =A0 =A0region_clear(&surface->draw_dirty_region);
> + =A0 =A0 =A0 =A0}
> =A0 =A0 }
> -
> =A0 =A0 message =3D RED_WORKER_MESSAGE_READY;
> =A0 =A0 write_message(worker->channel, &message);
> =A0}
>
> -
> =A0static inline void handle_dev_add_memslot(RedWorker *worker)
> =A0{
> =A0 =A0 RedWorkerMessage message;
> --
> 1.7.4.4
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>



--=20
Marc-Andr=E9 Lureau


More information about the Spice-devel mailing list