[PATCH 1/2] exa: Calling exaMarkSync after UTS is the drivers responsibility.

[Michel Dänzer]

On Fri, 2009-02-06 at 21:16 +0100, Maarten Maathuis wrote:
> ---
>  exa/exa_migration.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/exa/exa_migration.c b/exa/exa_migration.c
> index 19e0f71..8fb6d2f 100644
> --- a/exa/exa_migration.c
> +++ b/exa/exa_migration.c
> @@ -231,7 +231,7 @@ exaCopyDirty(ExaMigrationPtr migrate, RegionPtr pValidDst, RegionPtr pValidSrc,
>  
>      if (access_prepared)
>  	exaFinishAccess(&pPixmap->drawable, fallback_index);
> -    else if (need_sync)
> +    else if (need_sync && sync)
>  	sync (pPixmap->drawable.pScreen);
>  
>      pExaPixmap->offscreen = save_offscreen;
> @@ -276,7 +276,7 @@ exaCopyDirtyToFb (ExaMigrationPtr migrate)
>      exaCopyDirty(migrate, &pExaPixmap->validFB, &pExaPixmap->validSys,
>  		 pExaScr->info->UploadToScreen, pExaPixmap->sys_ptr,
>  		 pExaPixmap->fb_ptr, pExaPixmap->sys_pitch,
> -		 pExaPixmap->fb_pitch, EXA_PREPARE_DEST, exaMarkSync);
> +		 pExaPixmap->fb_pitch, EXA_PREPARE_DEST, NULL);
>  }

Won't this break existing drivers?

What's the problem you're trying to solve with this?


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer