[PATCH weston] clients/dmabuf-intel: use three buffers

[Pekka Paalanen]

On Mon, 13 Jun 2016 08:14:12 -0500
Yong Bakos <junk at humanoriented.com> wrote:

> On Jun 13, 2016, at 6:17 AM, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> > 
> > From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> > 
> > Use three buffers like simple-dmabuf-v4l instead of just two.
> > 
> > This is required, because when a frame callback arrives, the just
> > committed buffer is only on its way to the screen, while the previous
> > buffer is still being scanned out. It will take for the page flip to
> > complete, before the previous buffer is release. However, we want to be
> > able to repaint already at the frame callback, so three buffers can be
> > necessary.
> > 
> > This patch fixes weston-simple-dmabuf-intel to not abort with "Both
> > buffers busy at redraw()." when hardware overlays are used and the
> > surface gets directly scanned out.
> > 
> > Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>  
> 
> While I could not test this, it's a straightforward change, and it
> does touch all cases in simple-dmabuf-intel.
> 
> Reviewed-by: Yong Bakos <ybakos at humanoriented.com>

Thanks for the review, pushed:
   be112d4..d56b94a  master -> master


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160617/e3a5a2ce/attachment.sig>