Re: [PATCH xf86-video-qxl] Fix drawable mm_time with KMS

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

 



Hey,

On Fri, Jun 19, 2015 at 02:05:10PM +0200, Javier Celaya wrote:
> When KMS is enabled, the mm_time value of a QXLDrawable is undefined.
> This results in severe synchronization problems. This patch sets it to
> zero to force the spice-server to use the local clock.
> ---
>  src/qxl_surface.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/qxl_surface.c b/src/qxl_surface.c
> index c5e76e0..95d34db 100644
> --- a/src/qxl_surface.c
> +++ b/src/qxl_surface.c
> @@ -86,6 +86,7 @@ make_drawable (qxl_screen_t *qxl, qxl_surface_t *surf, uint8_t type,
>      
>      if (!qxl->kms_enabled)
>          drawable->mm_time = qxl->rom->mm_clock;
> +    else drawable->mm_time = 0;

With what spice-server version have you been having this issue? Since
commit c541d7e29dc0053e6, mm_time should be unconditionnally set
server-side. Is there anything causing qxl->rom->mm_clock not to work
with older spice-server and kms? In other words, why not remove the if
(!qxl->kms_enabled) check?

Christophe

Attachment: pgpYSHtXG6CFs.pgp
Description: PGP signature

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