[Spice-devel] [PATCH xf86-video-qxl] Provide an implementation for put_image in dfps mode.

[Christophe Fergeau]

On Wed, Dec 02, 2015 at 02:14:32PM -0600, Jeremy White wrote:
> This prevents the fallback from calling prepare_access
> against the whole screen, which in turn keeps us from
> transmitting the whole screen more than necessary.
> 
> Signed-off-by: Jeremy White <jwhite at codeweavers.com>
> ---
>  src/dfps.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
> 
> diff --git a/src/dfps.c b/src/dfps.c
> index 568ba5e..f113f27 100644
> --- a/src/dfps.c
> +++ b/src/dfps.c
> @@ -270,15 +270,22 @@ static Bool dfps_put_image (PixmapPtr dest, int x, int y, int w, int h,
>                 char *src, int src_pitch)
>  {
>      dfps_info_t *info;
> +    FbBits *dst;
> +    FbStride dst_stride;
> +    int dst_bpp;
>  
>      if (!(info = dfps_get_info (dest)))
>          return FALSE;
>  
>      dfps_update_box(&info->updated_region, x, x + w, y, y + h);
>  
> -    /* We can avoid doing the put image ourselves, as the uxa driver
> -       will fall back and do it for us if we return false */
> -    return FALSE;
> +    fbPrepareAccess(dest);

Wondering if this should be uxa_prepare_access() rather than
fbPrepareAccess? Not familiar at all with all of this, so I don't really
know...

Acked-by: Christophe Fergeau <cfergeau at redhat.com>

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20151211/2802b9f9/attachment.sig>