[PATCH] configure: dix: support a variable number of screen structures

Jon TURNEY jon.turney at dronecode.org.uk
Fri Mar 12 04:30:25 PST 2010


On 10/03/2010 18:22, Tiago Vignatti wrote:
> Some environments don't require a big and fixed number (MAXSCREENS) of the
> screen structures (ScreenInfo and bunch of others). We could improve the
> memory usage by setting a smaller number when the number of physical screens
> is known.
>
> This patch implements this feature, adjusting MAXSCREENS in autoconf time.
>
> I can see two or three pages of RSS saved in my machine when
> -with-maxscreens=1.
>
> Signed-off-by: Tiago Vignatti <tiago.vignatti-xNZwKgViW5gAvxtiuMwx3w at public.gmane.org>
> ---
> I'm not sure if I should declare it inside dix-config.h though. Is it okay to
> set there?
>
> Also, I'm afraid I might be breaking other DDXs. Jeremy, Jon Turney and
> others, can you please check this? Thanks.

Doesn't seem to break anything in XWin for me, and we already check for and 
disallow attempts to configure more than MAXSCREENS screen, so:

Tested-by: Jon TURNEY <jon.turney at dronecode.org.uk>


More information about the xorg-devel mailing list