[PATCH] Allow disabling XFree86-DGA, DRI, VidModeExtension extensions

Robert Hooker sarvatt at ubuntu.com
Fri Jul 26 22:53:55 PDT 2013


On Fri, Jul 26, 2013 at 7:53 PM, Alan Coopersmith
<alan.coopersmith at oracle.com> wrote:
> Code to recognize these in extension enable/disable options was wrapped
> in #ifdef XFree86Loader, but that's not defined when building miinitext.c
> since the great module merge of 1.13.  Change to an #ifdef that is defined.
>
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>  mi/miinitext.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> Found while helping a user on #xorg IRC.
>
> Tested by running Xorg -config xorg.conf.no-dga, with this in xorg.conf.no-dga:
> Section "Extensions"
>   Option "XFree86-DGA" "Disable"
> EndSection
>
> Before this fix:
> [1469450.113] (WW) Ignoring unrecognized extension "XFree86-DGA"
>
> After this fix:
> [1469526.870] (**) Extension "XFree86-DGA" is disabled
>
>
> diff --git a/mi/miinitext.c b/mi/miinitext.c
> index 81c663a..dbca9f7 100644
> --- a/mi/miinitext.c
> +++ b/mi/miinitext.c
> @@ -157,7 +157,7 @@ static ExtensionToggle ExtensionToggleList[] = {
>  #ifdef XF86BIGFONT
>      {"XFree86-Bigfont", &noXFree86BigfontExtension},
>  #endif
> -#ifdef XorgLoader
> +#ifdef XORGSERVER
>  #ifdef XFreeXDGA
>      {"XFree86-DGA", &noXFree86DGAExtension},
>  #endif
> --
> 1.7.9.2
>
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel

Summary should be changed to #ifdef XorgLoader instead of #ifdef
XFree86Loader? But still,

Reviewed-by: Robert Hooker <robert.hooker at canonical.com>


More information about the xorg-devel mailing list