<div dir="ltr"><div class="gmail_extra">Xiong,<br></div><div class="gmail_extra">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.<br>
</div><div class="gmail_extra">--Jason Ekstrand<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 22, 2013 at 9:06 PM, Zhang, Xiong Y <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">
<div link="blue" vlink="purple" lang="ZH-CN">
<div>
<p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">So you want to adjust output’s work mode (clone or extend) at runtime.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">Is this correct a new protocol should be added ?<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">thanks<u></u><u></u></span></p>
<div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" lang="EN-US">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" lang="EN-US"> Jason Ekstrand [mailto:<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>]
<br>
<b>Sent:</b> Wednesday, October 23, 2013 9:52 AM</span></p><div><div class="h5"><br>
<b>To:</b> Zhang, Xiong Y<br>
<b>Cc:</b> <a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedesktop.org</a><br>
<b>Subject:</b> Re: [PATCH 0/9] implement multi screen clone mode in drm-backend.so<u></u><u></u></div></div><p></p>
</div><div><div class="h5">
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<div>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US">Xiong,<u></u><u></u></span></p>
</div>
<p class="MsoNormal"><span lang="EN-US">A "clone" option would be one way to configure it. I just don't want to be tied to a single master output.<u></u><u></u></span></p>
</div>
<p class="MsoNormal"><span lang="EN-US">--Jason<u></u><u></u></span></p>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span lang="EN-US"><u></u> <u></u></span></p>
<div>
<p class="MsoNormal"><span lang="EN-US">On Tue, Oct 22, 2013 at 8:46 PM, Zhang, Xiong Y <<a href="mailto:xiong.y.zhang@intel.com" target="_blank">xiong.y.zhang@intel.com</a>> wrote:<u></u><u></u></span></p>
<div>
<div>
<p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">Jason,</span><span lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal" style="text-indent:9.0pt">
<span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">You mean that we can add a “clone” config option in output section of Weston.ini file.
</span><span lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal" style="text-indent:9.0pt">
<span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">The “clone” config option decide whether this output is a slave output and which master output this slave output clone from .</span><span lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal" style="text-indent:9.0pt">
<span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">Is my understand correct ?</span><span lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal" style="text-indent:9.0pt">
<span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US"> </span><span lang="EN-US"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.5pt;font-family:"Calibri","sans-serif";color:#1f497d" lang="EN-US">thanks</span><span lang="EN-US"><u></u><u></u></span></p>
<div style="border:none;border-top:solid #b5c4df 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" lang="EN-US">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" lang="EN-US"> Jason
Ekstrand [mailto:<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>]
<br>
<b>Sent:</b> Sunday, October 13, 2013 10:53 AM<br>
<b>To:</b> Zhang, Xiong Y<br>
<b>Cc:</b> <a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedesktop.org</a><br>
<b>Subject:</b> Re: [PATCH 0/9] implement multi screen clone mode in drm-backend.so</span><span lang="EN-US"><u></u><u></u></span></p>
</div>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US"> <u></u><u></u></span></p>
<div>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US">Xiong,<u></u><u></u></span></p>
</div>
<p class="MsoNormal"><span lang="EN-US">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.)<u></u><u></u></span></p>
</div>
<div>
<p class="MsoNormal"><span lang="EN-US"> <u></u><u></u></span></p>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span lang="EN-US">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.<u></u><u></u></span></p>
</div>
<div>
<p class="MsoNormal"><span lang="EN-US">Thanks,<u></u><u></u></span></p>
</div>
<div>
<p class="MsoNormal"><span lang="EN-US">--Jason Ekstrand<u></u><u></u></span></p>
</div>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><span lang="EN-US"> <u></u><u></u></span></p>
<div>
<p class="MsoNormal"><span lang="EN-US">On Tue, Sep 17, 2013 at 10:50 PM, Xiong Zhang <<a href="mailto:xiong.y.zhang@intel.com" target="_blank">xiong.y.zhang@intel.com</a>> wrote:<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">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 style="color:#888888"><br>
--<br>
1.8.3.2<br>
<br>
_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">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></span><u></u><u></u></span></p>
</div>
<p class="MsoNormal"><span lang="EN-US"> <u></u><u></u></span></p>
</div>
</div>
</div>
</div>
</div>
</div>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div></div></div>
</div>
</blockquote></div><br></div></div>