[PATCH intel] make MAXSCREENS run-time configurable

Alan Coopersmith alan.coopersmith at oracle.com
Thu Apr 8 11:03:19 PDT 2010


Shouldn't the driver patches be choosing between the old and new ways
with #ifdef MAXSCREENS or #ifdef MAXSCREENSALLOC_FATAL to allow them
to continue to work with stable releases while 1.9 is in development?

	-alan-

Tiago Vignatti wrote:
> Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
> ---
>  src/i810_dga.c |    4 +++-
>  1 files changed, 3 insertions(+), 1 deletions(-)
> 
> diff --git a/src/i810_dga.c b/src/i810_dga.c
> index 3f53057..72808ab 100644
> --- a/src/i810_dga.c
> +++ b/src/i810_dga.c
> @@ -140,7 +140,7 @@ I810DGAInit(ScreenPtr pScreen)
>     return DGAInit(pScreen, &I810DGAFuncs, modes, num);
>  }
>  
> -static DisplayModePtr I810SavedDGAModes[MAXSCREENS];
> +static DisplayModePtr *I810SavedDGAModes;
>  
>  static Bool
>  I810_SetMode(ScrnInfoPtr pScrn, DGAModePtr pMode)
> @@ -150,6 +150,8 @@ I810_SetMode(ScrnInfoPtr pScrn, DGAModePtr pMode)
>  
>     MARKER();
>  
> +   MAXSCREENSALLOC_FATAL(I810SavedDGAModes);
> +
>     if (!pMode) {			/* restore the original mode */
>        DPRINTF(PFX, "Restoring original mode (from DGA mode)\n");
>        if (pI810->DGAactive) {


-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list