[PATCH 0/5 xserver] make MAXSCREENS run-time configurable

Tiago Vignatti tiago.vignatti at nokia.com
Thu Apr 8 08:06:32 PDT 2010


Yay, the RFC from last week became a patch series now!

The most notable points compared with RFC is that by default uses MAXSCREENS as
16 (the same as the current implementation without patch). Besides, we have now
patches for all DDXes that uses MAXSCREENS and some xorg drivers either.

The series weakly depends on two other patches that I sent for review
recently:
    Death to Multibuffer extension:
    http://lists.x.org/archives/xorg-devel/2010-April/006972.html

and
    xvfb: process command line arguments afterwards in InitOutput
    http://lists.x.org/archives/xorg-devel/2010-April/007050.html


The testing coverage plan:

- xnest, xvfb were tested by myself and ran okay. But of course, reviewers
  there are requested.

- xwin has been tested by Jon Turney - actually himself made the work there
  (thanks a lot Jon!), so would be nice someone else test either.

- both xdarwin and kdrive are not using MAXSCREENS. Even though, would be nice
  to get a tested-by from these just to be sure nothing is broken in dix.

- dmx has already this work there and we just need to do a clean up to use the
  same functions I placed misc.h. But meh, that's a next task.

- drivers: we need testers for intel. I tested my self nv and mga. ati uses
  DiDGA, which doesn't need a patch there. Some good soul also would create
  patches for other drivers.


Please, review it. Thanks

PS: someone brave enough can give a try on this series + xvfb + MBE Death all
here: git://anongit.freedesktop.org/~vignatti/xserver[xvfb-fix_AND_maxscreen]

           Tiago



Jon TURNEY (1):
  xwin: make MAXSCREENS run-time configurable

Tiago Vignatti (4):
  dix: make MAXSCREENS run-time configurable
  xfree86: make MAXSCREENS run-time configurable
  xvfb: make MAXSCREENS run-time configurable
  xnest: make MAXSCREENS run-time configurable

 Xext/panoramiX.c                      |   12 +-
 Xext/panoramiX.h                      |    2 +-
 Xext/panoramiXprocs.c                 |  121 +++++--
 Xext/panoramiXsrv.h                   |    2 +-
 Xext/xvdisp.c                         |    8 +-
 dix/events.c                          |    1 +
 dix/globals.c                         |    7 +-
 dix/main.c                            |   47 +++
 dix/window.c                          |    2 +-
 fb/fbcmap.c                           |   11 +-
 hw/vfb/InitOutput.c                   |   10 +-
 hw/xfree86/common/xf86Cursor.c        |   67 ++--
 hw/xfree86/common/xf86xv.c            |    3 +-
 hw/xfree86/dixmods/extmod/xf86dga2.c  |    4 +-
 hw/xfree86/os-support/bsd/arm_video.c |    5 +-
 hw/xnest/Args.c                       |    5 +-
 hw/xnest/Color.c                      |    6 +-
 hw/xnest/Init.c                       |    3 +
 hw/xnest/Screen.c                     |    4 +-
 hw/xnest/Screen.h                     |    4 +-
 hw/xwin/InitOutput.c                  |   19 +-
 hw/xwin/win.h                         |    8 +-
 hw/xwin/winglobals.c                  |    3 +-
 hw/xwin/winprocarg.c                  |  633 +++++++++------------------------
 hw/xwin/winvalargs.c                  |    2 +-
 include/cursor.h                      |    2 +-
 include/dixstruct.h                   |    1 -
 include/globals.h                     |    9 +-
 include/inputstr.h                    |    2 +-
 include/misc.h                        |   44 +++-
 include/scrnintstr.h                  |    2 +-
 include/windowstr.h                   |    2 +-
 mi/micmap.c                           |    4 +-
 mi/micmap.h                           |    2 +-
 os/utils.c                            |   14 +
 render/render.c                       |   10 +-
 36 files changed, 506 insertions(+), 575 deletions(-)



More information about the xorg-devel mailing list