[PULL to discuss] Remove kdrive, Xnest, and Xvfb

Jeremy Huddleston jeremyhu at apple.com
Mon Mar 26 19:46:58 PDT 2012


On Mar 26, 2012, at 6:01 PM, Alan Coopersmith <alan.coopersmith at oracle.com> wrote:

> On 03/26/12 04:13 PM, Jeremy Huddleston wrote:
>> These need to die.  This removes 30K lines of code from xorg-server.  It must be good!
>> 
>> Most functionality of these servers can be provide by Xorg with either the nested or dummy video driver.  If someone really misses functionality, we should fix that deficiency in hw/xfree86, xf86-video-dummy, or xf86-video-nested.  Also, there's nothing stopping anyone from using older server versions if they still need these DDXs.
> 
> The giant blocker from my point of view is that by just deleting them, you've
> made it impossible for non-root users to run them, since Xorg only reads config
> files from system directories when run as a root user.

Yes, and this is certainly something we should address this cycle.  There's no reason that Xorg needs to be suid root when using drivers like dummy, nested, vnc, ...

> At the very minimum we should ship a couple simple configs such as (guessing at
> contents here, someone would need to test and refine):
> 
> /usr/lib/X11/Xorg-vfb.conf:
> ------------------------------
> Section "Device"
>        Identifier  "vfb0"
>        Driver      "dummy"
> EndSection
> 
> Section "ServerFlags"
> 	Option "AutoAddDevices" "false"
> EndSection
> ------------------------------
> 
> 
> /usr/lib/X11/Xorg-nested.conf:
> ------------------------------
> Section "Device"
>        Identifier  "Screen0"
>        Driver      "nested"
> EndSection
> 
> Section "ServerFlags"
> 	Option "AutoAddDevices" "false"
> EndSection
> 
> Section "InputDevice"
>    Identifier     "Mouse0"
>    Driver         "nested"
> EndSection
> 
> Section "InputDevice"
>    Identifier     "Keyboard0"
>    Driver         "nested"
> EndSection

That's pretty much what I'm shipping in the current XQuartz betas for this purpose.




More information about the xorg-devel mailing list