<div dir="ltr"><div>Hi Oleg!</div><div><br></div><div>You'll need to take care with ServerLayout section. I've detected a small problem with your patch in a very particular situation:</div><div><br></div><div>Consider a machine with two seats, seat0 and seat1. Suppose you only have a file /etc/X11/xorg.conf.d/90-seat.conf with the following structure:</div>
<div><br></div><div>Section "Device"</div><div> Identifier "device0"</div><div> (...)</div><div> MatchSeat "seat1"</div><div>EndSection</div><div><br></div><div><div>Section "Screen"</div>
<div> Identifier "screen0"</div><div> Device "device0"</div><div> (...)</div><div> MatchSeat "seat1"</div><div>EndSection</div></div><div><br></div><div><div>Section "ServerLayout"</div>
<div> Identifier "layout0"</div><div> Screen "screen0"</div><div> (...)</div><div> MatchSeat "seat1"</div><div>EndSection</div></div><div><br></div><div>In this situation, the seat0 X server will fail to find the default layout and will crash. If you remove the ServerLayout section from file, this problem disappears.</div>
<div><br></div><div>I would suggest also another improvement for your patch: could you modify it so that you can get rid of those MatchSeat repetitions? For example, if you have something like</div><div><br></div><div><div>
Section "Device"</div><div> Identifier "device0"</div><div> (...)</div><div> MatchSeat "seat1"</div><div>EndSection</div><div><br></div><div><div>Section "Screen"</div><div>
Identifier "screen0"</div><div> Device "device0"</div><div> (...)</div><div>EndSection</div></div></div><div><br></div><div>then seat0 X server should ignore screen "screen0", since it links to device "device0", which already has a MatchSeat entry.</div>
<div><br></div><div>Regards,</div></div><div class="gmail_extra"><br clear="all"><div><div dir="ltr">CANTATE DOMINO CANTICUM NOVUM<div>QUIA MIRABILIA FECIT</div><div><br></div><div>Laércio</div></div></div>
<br><br><div class="gmail_quote">2014-03-28 8:39 GMT-03:00 Laércio de Sousa <span dir="ltr"><<a href="mailto:lbsousajr@gmail.com" target="_blank">lbsousajr@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2014-03-27 17:05 GMT-03:00 Oleg Samarin <span dir="ltr"><<a href="mailto:osamarin68@gmail.com" target="_blank">osamarin68@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="overflow:hidden"><div><div class="h5">Hello!<br>
<br>
Sometimes ago I made a patch that allows to configure Devices, Screens<br>
and Layouts belonging to the different seats in a multi-seat<br>
environment. It is useful for the setting up with several videocards<br>
using propritery drivers (I've tested with Nvidia).<br>
<br>
Because my patch has not been sesolved upstream, I resend it again<br>
adopted to the current master.</div></div><div><div><img src="http://images/cleardot.gif"></div></div></div></blockquote></div><br>Acked-by: Laércio de Sousa <<a href="mailto:lbsousajr@gmail.com" target="_blank">lbsousajr@gmail.com</a>></div>
<div class="gmail_extra">Tested-by: Laércio de Sousa <<a href="mailto:lbsousajr@gmail.com" target="_blank">lbsousajr@gmail.com</a>><br><br clear="all"><div><div dir="ltr">CANTATE DOMINO CANTICUM NOVUM<div>QUIA MIRABILIA FECIT</div>
<span class="HOEnZb"><font color="#888888">
<div><br></div><div>Laércio</div></font></span></div></div>
</div></div>
</blockquote></div><br></div>