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 <[email protected]>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 > [email protected] > http://lists.freedesktop.org/mailman/listinfo/wayland-devel >
_______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
