Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1362723

Re: [PATCH 2/2] drm/virtio: send vblank event on plane atomic update

Show all headers | View raw

Hi,

On 21 March 2016 at 19:23, Gustavo Padovan <gustavo@padovan.org> wrote:
> @@ -96,6 +98,11 @@ static void virtio_gpu_plane_atomic_update(struct drm_plane *plane,
>                                       plane->state->crtc_y,
>                                       plane->state->crtc_w,
>                                       plane->state->crtc_h);
> +
> +       spin_lock_irqsave(&crtc->dev->event_lock, flags);
> +       if (crtc->state->event)
> +               drm_crtc_send_vblank_event(crtc, crtc->state->event);
> +       spin_unlock_irqrestore(&crtc->dev->event_lock, flags);

This seems like the wrong place to do it, in that it will generate one
flip event per plane, rather than one per CRTC. So this should
probably be done in the overall atomic_commit hook I think.

Also, without some kind of delay, this means that we'll generate
flip-complete events immediately, which will cause compositors like
Weston to render infinitely fast. It's probably worth looking at what
happened when this came up with Bochs - I'm not sure if we fake a 16ms
delay, or refuse to do async modesets, or what.

Cheers,
Daniel

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 2/2] drm/virtio: send vblank event on plane atomic update Gustavo Padovan <gustavo@padovan.org> - 2016-03-21 20:30 +0100
  Re: [PATCH 2/2] drm/virtio: send vblank event on plane atomic update Daniel Stone <daniel@fooishbar.org> - 2016-03-22 14:20 +0100
  [PATCH v2] drm/virtio: send vblank event after crtc updates Gustavo Padovan <gustavo@padovan.org> - 2016-03-22 21:20 +0100

csiph-web