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

Jason Ekstrand jason at jlekstrand.net
Wed Oct 23 04:17:42 CEST 2013


Xiong,
I'm not sure that we need to go that far right now.  I'm fine for now with
just a config option.  My primary concern is that you should be able to
have three outputs A, B, and C where A and B are masters and C is cloned to
A.  In your original implementation it looked to me like it was only
possible to have one master and if you had three outputs then they would
all have to show the same thing.
--Jason Ekstrand

On Tue, Oct 22, 2013 at 9:06 PM, Zhang, Xiong Y <xiong.y.zhang at intel.com>wrote:

>  So you want to adjust output’s work mode (clone or extend) at runtime.***
> *
>
> Is this correct a new protocol should be added ?****
>
> ** **
>
> thanks****
>
> *From:* Jason Ekstrand [mailto:jason at jlekstrand.net]
> *Sent:* Wednesday, October 23, 2013 9:52 AM
>
> *To:* Zhang, Xiong Y
> *Cc:* wayland-devel at lists.freedesktop.org
> *Subject:* Re: [PATCH 0/9] implement multi screen clone mode in
> drm-backend.so****
>
>  ** **
>
> Xiong,****
>
> A "clone" option would be one way to configure it.  I just don't want to
> be tied to a single master output.****
>
> --Jason****
>
> ** **
>
> On Tue, Oct 22, 2013 at 8:46 PM, Zhang, Xiong Y <xiong.y.zhang at intel.com>
> wrote:****
>
> Jason,****
>
> You mean that we can add a “clone” config option in output section of
> Weston.ini file. ****
>
> The “clone” config option decide whether this output is a slave output and
> which master output   this slave output clone from .****
>
> Is my understand correct ?****
>
>  ****
>
> thanks****
>
> *From:* Jason Ekstrand [mailto:jason at jlekstrand.net]
> *Sent:* Sunday, October 13, 2013 10:53 AM
> *To:* Zhang, Xiong Y
> *Cc:* wayland-devel at lists.freedesktop.org
> *Subject:* Re: [PATCH 0/9] implement multi screen clone mode in
> drm-backend.so****
>
>  ****
>
> 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/20131022/66869509/attachment.html>


More information about the wayland-devel mailing list