[PATCH] configure: dix: support a variable number of screen structures
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
> 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