<div dir="ltr">Hans, could you please consider this other patch from Oleg Samarin, too? It's important in order to set up multiseat properly when the secondary graphics devices are not KMS-compliant (e.g. NVIDIA/AMD graphics cards with proprietary drivers), and the display manager doesn't allow passing a custom "-config" option to X server (e.g. GDM).<div>

<br></div><div><a href="http://lists.x.org/archives/xorg-devel/2013-September/037946.html">http://lists.x.org/archives/xorg-devel/2013-September/037946.html</a><br><div><br></div><div>It introduces a new MatchSeat key for sections "Device", "Screen", and "ServerLayout" in xorg.conf. The most common case is having a single file e.g. /etc/X11/xorg.conf.d/90-seat.conf of kind</div>

<div><br></div><div>Section "Device"</div><div>    Identifier "card1"</div><div>    Driver "nvidia"</div><div>    MatchSeat "seat-1"</div><div>EndSection</div><div><br></div><div>If one needs to include also a "Screen" section, both "Device" and "Screen" sections must have a MatchSeat key with the same value. Here's an example from my real setup:</div>

<div><br></div><div><div>Section "Device"</div><div>    Identifier "videocard_sm501"</div><div>    BusID "PCI:4:1:0"</div><div>    Driver "siliconmotion"</div><div>    Option "PanelSize" "1360x768"</div>

<div>    MatchSeat "seat-thinnet"</div><div>EndSection</div><div><br></div><div>Section "Screen"</div><div>    Identifier "screen_sm501"</div><div>    Device "videocard_sm501"</div>

<div>    DefaultDepth 16</div><div>    MatchSeat "seat-thinnet"</div><div>EndSection</div><div><br></div><div>In one of my previous tests, putting a MatchSeat entry in "ServerLayout" section made X server segfault. I'll repeat my tests soon.</div>

<div><br></div><div>Thanks in advance!</div></div></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">2013/12/12 Hans de Goede <span dir="ltr"><<a href="mailto:hdegoede@redhat.com" target="_blank">hdegoede@redhat.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi,<div class="im"><br>
<br>
On 12/12/2013 05:22 PM, Laércio de Sousa wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Updated patch following Hans de Goede's advice.<br>
</blockquote>
<br></div>
Thanks, added to my tree with the 2 elements of the test swapped<br>
as discussed. I've also changed the test from:<br>
<br>
if (!((flags & HW_SKIP_CONSOLE) || ServerIsNotSeat0()))<br>
<br>
to:<br>
<br>
if (!(flags & HW_SKIP_CONSOLE) && !ServerIsNotSeat0()))<br>
<br>
Because the amount of parenthesis in the former was making<br>
it hard on the eyes IMHO.<br>
<br>
You can find it here for now:<br>
<a href="http://cgit.freedesktop.org/~jwrdegoede/xserver" target="_blank">http://cgit.freedesktop.org/~<u></u>jwrdegoede/xserver</a><br>
<br>
After the 1.15 release I'll send a pull request for this to<br>
get added into xserver git master.<br>
<br>
Regards,<br>
<br>
Hans<br>
</blockquote></div><br></div>