<div dir="ltr"><div><div>Xiong,<br></div>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.)<br>
</div><div><br></div><div>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.<br>
<br></div><div>Thanks,<br></div><div>--Jason Ekstrand<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Sep 17, 2013 at 10:50 PM, Xiong Zhang <span dir="ltr"><<a href="mailto:xiong.y.zhang@intel.com" target="_blank">xiong.y.zhang@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">the theory for clone mode is: all the outputs share the same fb, when primary output<br>
do modeset and pageflip, clone output do this also.<br>
I fulfill repaint, VT switch and hotplug function. I don't implement dynamical setting<br>
output's mdoe.<br>
<br>
<br>
Xiong Zhang (9):<br>
  add multi-screen-mode option to weston.ini<br>
  find primary output during drm_compositor_create in clone mode<br>
  one pointer exist in only one output<br>
  output gl repaint in clone mode<br>
  deal with VT switch in clone mode<br>
  hot plug add a output in clone mode<br>
  maximize and fullscreen support in clone mode<br>
  hot plug remove a output in clone mode<br>
  improve hotplug remove a output in clone mode<br>
<br>
 src/compositor-drm.c | 390 +++++++++++++++++++++++++++++++++++++++++++++++----<br>
 src/compositor.c     |  28 ++++<br>
 src/compositor.h     |   9 ++<br>
 src/input.c          |   7 +-<br>
 src/shell.c          |  10 +-<br>
 src/zoom.c           |   1 +<br>
 weston.ini           |   1 +<br>
 7 files changed, 414 insertions(+), 32 deletions(-)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.8.3.2<br>
<br>
_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</font></span></blockquote></div><br></div>