On Fri, 22 Apr 2016 15:19:04 +0200
David Fort <[email protected]> wrote:

> Set an update delay that is a multiple of the output frequency, or we may do
> incorrect weston_output_finish_frame calls.
> ---
>  src/compositor-rdp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/compositor-rdp.c b/src/compositor-rdp.c
> index 9274674..4cbc1c3 100644
> --- a/src/compositor-rdp.c
> +++ b/src/compositor-rdp.c
> @@ -378,7 +378,7 @@ rdp_output_repaint(struct weston_output *output_base, 
> pixman_region32_t *damage)
>       pixman_region32_subtract(&ec->primary_plane.damage,
>                                &ec->primary_plane.damage, damage);
>  
> -     wl_event_source_timer_update(output->finish_frame_timer, 16);
> +     wl_event_source_timer_update(output->finish_frame_timer, 10);
>       return 0;
>  }
>  

Hi,

I'm worried that there are more things wrong. This does not look like a
fix, it just improves the odds of not failing if any.

So, you have a 100 Hz output update frequency on RDP?

What actually determines the frequency if not exactly the timer you
change here?

In any case, patches 1-4 are:
Acked-by: Pekka Paalanen <[email protected]>


Thanks,
pq

Attachment: pgp0TmNLYdy_3.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to