EXT: [PATCH weston v10 1/1] weston: support clone mode on DRM-frontend

Pekka Paalanen ppaalanen at gmail.com
Fri Jun 29 13:16:07 UTC 2018


On Fri, 29 Jun 2018 10:08:17 +0000
"Ray, Ian (GE Healthcare)" <ian.ray at ge.com> wrote:

> > On 6 Jun 2018, at 14.30, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> > 
> > From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> > 
> > Add a new output section key "same-as" for configuring clone mode. An
> > output marked "same-as" another output will be configured identically to
> > the other output.
> > 
> > The current implementation supports only CRTC sharing for clone mode.
> > Independent CRTC clone mode cannot be supported until output layout
> > logic is moved from libweston into the frontend and libweston's damage
> > tracking issues stemming from overlapping outputs are solved.
> > 
> > Quite a lot of infrastructure is needed to properly configure clone
> > mode. The implemented logic allows easy addition of independent CRTC
> > clone mode once libweston supports it. The idea is that wet_layoutput is
> > the item to be laid out and all weston_outputs a wet_layoutput
> > contains show exactly the same area of the desktop.
> > 
> > The configuration logic attempts to automatically fall back to creating
> > more weston_outputs when all heads do not work under the same
> > weston_output. For now, the fallback path ends with an error message.
> > 
> > Enabling a weston_output is bit complicated, because one needs to first
> > collect all relevant heads, try to attach them all to the weston_output,
> > and then back up head by head until enabling the weston_output succeeds.
> > A new weston_output is created for the left-over heads and the process
> > is repeated.
> > 
> > CRTC-sharing clone mode is the most efficient clone mode, offering
> > synchronized scanout timings, but it is not always supported by
> > hardware.
> >   
> 
> Reviewed-by: Ian Ray <ian.ray at ge.com>

Pushed:
   6a699b1a..90718170  master -> master


Thanks,
pq

> > v10:
> > - rebased trivial conflicts in man page
> > - switch to gitlab issue URL
> > 
> > v9:
> > - replace weston_compositor_set_heads_changed_cb() with
> >  weston_compositor_add_heads_changed_listener()
> > - remove workaround in simple_head_enable()
> > 
> > v6:
> > - Add man-page note about cms-colord.
> > - Don't create an output just to turn it off.
> > 
> > Fixes: https://gitlab.freedesktop.org/wayland/weston/issues/22
> > 
> > Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> > Acked-by: Derek Foreman <derekf at osg.samsung.com>
> > Acked-by: Daniel Stone <daniels at collabora.com>
> > ---
> > compositor/main.c  | 492 ++++++++++++++++++++++++++++++++++++++++++++++++++---
> > man/weston-drm.man |  12 ++
> > 2 files changed, 484 insertions(+), 20 deletions(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20180629/6932852c/attachment.sig>


More information about the wayland-devel mailing list