Re: [PATCH 1/5] dix: make MAXSCREENS run-time configurable

2010-04-09 Thread Tiago Vignatti
On Thu, Apr 08, 2010 at 11:23:21PM +0200, ext Keith Packard wrote: Yeah, for things which are created after the ScreenRec is allocated, it makes sense to just allocate them right in the ScreenRec. For things which are in the xfree86 DDX and are allocated before the ScreenRec, those can be

[PATCH 1/5] dix: make MAXSCREENS run-time configurable

2010-04-08 Thread Tiago Vignatti
Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com --- Xext/panoramiX.c | 12 +- Xext/panoramiX.h |2 +- Xext/panoramiXprocs.c | 121 +++-- Xext/panoramiXsrv.h |2 +- Xext/xvdisp.c |8 +++- dix/events.c

Re: [PATCH 1/5] dix: make MAXSCREENS run-time configurable

2010-04-08 Thread Aaron Plattner
On Thu, Apr 08, 2010 at 08:06:33AM -0700, Tiago Vignatti wrote: diff --git a/Xext/panoramiX.c b/Xext/panoramiX.c index cfeba01..7d1b849 100644 --- a/Xext/panoramiX.c +++ b/Xext/panoramiX.c @@ -122,7 +122,7 @@ typedef struct { CloseScreenProcPtr CloseScreen; } PanoramiXScreenRec,

Re: [PATCH 1/5] dix: make MAXSCREENS run-time configurable

2010-04-08 Thread Keith Packard
On Thu, 8 Apr 2010 12:24:41 -0700, Aaron Plattner aplatt...@nvidia.com wrote: Since you're breaking the ABI already, could we move this sort of dynamically-allocated global array into the pScreen or pScrnInfo instead, either as just a plain old struct member or as a DevPrivate hook? That