[PATCH] xfree86: go further with "legacy" PCI bus probe for non-seat0 seats (#66851)

Laércio Benedito Sivali de Sousa lbsousajr at gmail.com
Mon Sep 16 12:58:30 PDT 2013


Just found that this patch has been already discussed some time ago (before
my subscrption to the list). Latest Dave Airlie opinion was:

"this one looks simple, but looks can be deceiving, and I've gotten bitten
by applying patches in this area without through testing on a few
platforms!"

http://www.mail-archive.com/xorg-devel@lists.x.org/msg36239.html

CANTATE DOMINO CANTICUM NOVUM
QUIA MIRABILIA FECIT

Laércio


2013/9/6 Laércio de Sousa <lbsousajr at gmail.com>

> This fixes a bug that prevents X.Org from starting properly on,
> e.g., NVIDIA cards with proprietary drivers, when "-seat" option
> is passed with an argument different from "seat0".
>
> Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=66851
>
> Signed-off-by: Laércio de Sousa <lbsousajr at gmail.com>
> ---
>  hw/xfree86/common/xf86Bus.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86Bus.c b/hw/xfree86/common/xf86Bus.c
> index e101537..91af72f 100644
> --- a/hw/xfree86/common/xf86Bus.c
> +++ b/hw/xfree86/common/xf86Bus.c
> @@ -81,7 +81,7 @@ xf86CallDriverProbe(DriverPtr drv, Bool detect_only)
>      if (drv->platformProbe != NULL) {
>          foundScreen = xf86platformProbeDev(drv);
>      }
> -    if (ServerIsNotSeat0())
> +    if (ServerIsNotSeat0() && foundScreen)
>          return foundScreen;
>  #endif
>
> @@ -201,7 +201,7 @@ xf86BusProbe(void)
>  {
>  #ifdef XSERVER_PLATFORM_BUS
>      xf86platformProbe();
> -    if (ServerIsNotSeat0())
> +    if (ServerIsNotSeat0() && xf86_num_platform_devices > 0)
>          return;
>  #endif
>  #ifdef XSERVER_LIBPCIACCESS
> --
> 1.8.1.4
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-devel/attachments/20130916/527b6934/attachment.html>


More information about the xorg-devel mailing list