[PATCH 0/9] implement multi screen clone mode in drm-backend.so

Jason Ekstrand jason at jlekstrand.net
Sun Oct 13 04:53:07 CEST 2013


Xiong,
I'm sorry for taking so long to reply; I meant to do so sooner but haven't
found the time.  First off, I'm glad to see someone implementing output
cloning.  It makes for all sorts of interesting cases that have yet to be
tested.  Also, I plan to implement it in the wayland backend at some point
in the not-so-distant future.  (Yes, it does make sense to put it in the
wayland backend.)

My biggest concern with your current implementation is that you are doing
things in terms of a compositor-wide "clone mode" instead of per output
"this output is a clone of that one".  I think it is reasonable for a
person to want, for instance three outputs A, B, and C where A and B are
"masters" and C is a clone of A.  Making it a compositor-wide flag would
completely rule out this case.

Thanks,
--Jason Ekstrand


On Tue, Sep 17, 2013 at 10:50 PM, Xiong Zhang <xiong.y.zhang at intel.com>wrote:

> the theory for clone mode is: all the outputs share the same fb, when
> primary output
> do modeset and pageflip, clone output do this also.
> I fulfill repaint, VT switch and hotplug function. I don't implement
> dynamical setting
> output's mdoe.
>
>
> Xiong Zhang (9):
>   add multi-screen-mode option to weston.ini
>   find primary output during drm_compositor_create in clone mode
>   one pointer exist in only one output
>   output gl repaint in clone mode
>   deal with VT switch in clone mode
>   hot plug add a output in clone mode
>   maximize and fullscreen support in clone mode
>   hot plug remove a output in clone mode
>   improve hotplug remove a output in clone mode
>
>  src/compositor-drm.c | 390
> +++++++++++++++++++++++++++++++++++++++++++++++----
>  src/compositor.c     |  28 ++++
>  src/compositor.h     |   9 ++
>  src/input.c          |   7 +-
>  src/shell.c          |  10 +-
>  src/zoom.c           |   1 +
>  weston.ini           |   1 +
>  7 files changed, 414 insertions(+), 32 deletions(-)
>
> --
> 1.8.3.2
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131012/f05bcc3d/attachment.html>


More information about the wayland-devel mailing list