Re: [PATCH resent] configure: introduce --{enable, disable}-fontserver

2010-05-21 Thread Keith Packard
On Wed,  5 May 2010 17:14:41 +0300, Tiago Vignatti tiago.vigna...@nokia.com 
wrote:

 Keith, no one took this one. Care to pull it?

I though I had replied suggesting that this should depend on what support was
provided in the system libXfont instead of having the server be
configured separately.

-- 
keith.pack...@intel.com


pgpNXMpNGiXY9.pgp
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH resent] configure: introduce --{enable, disable}-fontserver

2010-05-21 Thread Vignatti Tiago (Nokia-D/Helsinki)
On Fri, May 21, 2010 at 04:54:01PM +0200, ext Keith Packard wrote:
 On Wed,  5 May 2010 17:14:41 +0300, Tiago Vignatti tiago.vigna...@nokia.com 
 wrote:
 
  Keith, no one took this one. Care to pull it?
 
 I though I had replied suggesting that this should depend on what support was
 provided in the system libXfont instead of having the server be
 configured separately.

yeah, adjust .pc from libXfont seems to solve the issue. I'm just lazily
hanging this work for some days because I don't know exactly the trick to do
so.


Maybe some autoconf ninja on the list knows?

Tiago
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel


[PATCH resent] configure: introduce --{enable, disable}-fontserver

2010-05-05 Thread Tiago Vignatti
Some system might not want font servers support. This patch is sort of revert
of f3b29efc.

In my machine, I see 20kB of RSS being saved in libXfont mapped in Xorg
process when I disabled font server support and other kind of fonts in the
library. I used something like this: --disable-pcfformat --disable-bdfformat
--disable-snfformat --disable-freetype --disable-fc

The default library built was taking:
   textdata bss dec hex filename
 26174344721536  267751   415e7 ./lib/libXfont.so

and with these flags, it jumps to:
   textdata bss dec hex filename
 15768424201188  161292   2760c ./lib/libXfont.so

Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com
---
Keith, no one took this one. Care to pull it?

 configure.ac|5 +
 dix/dixfonts.c  |2 ++
 include/dix-config.h.in |3 +++
 3 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/configure.ac b/configure.ac
index 1a4dc9a..ecdcbe1 100644
--- a/configure.ac
+++ b/configure.ac
@@ -621,6 +621,7 @@ AC_ARG_ENABLE(xcalibrate, 
AS_HELP_STRING([--enable-xcalibrate], [Build XCali
 AC_ARG_ENABLE(tslib,  AS_HELP_STRING([--enable-tslib], [Build kdrive 
tslib touchscreen support (default: disabled)]), [TSLIB=$enableval], [TSLIB=no])
 AC_ARG_ENABLE(dbe,AS_HELP_STRING([--disable-dbe], [Build DBE 
extension (default: enabled)]), [DBE=$enableval], [DBE=yes])
 AC_ARG_ENABLE(xf86bigfont,AS_HELP_STRING([--disable-xf86bigfont], [Build 
XF86 Big Font extension (default: disabled)]), [XF86BIGFONT=$enableval], 
[XF86BIGFONT=no])
+AC_ARG_ENABLE(fontserver, AS_HELP_STRING([--disable-fontserver], [Build 
support for font servers (default: enabled)]), [FONTSERVER=$enableval], 
[FONTSERVER=yes])
 AC_ARG_ENABLE(dpms,   AS_HELP_STRING([--disable-dpms], [Build DPMS 
extension (default: enabled)]), [DPMSExtension=$enableval], [DPMSExtension=yes])
 AC_ARG_ENABLE(config-udev,AS_HELP_STRING([--enable-config-udev], [Build 
udev support (default: no)]), [CONFIG_UDEV=$enableval], [CONFIG_UDEV=no])
 AC_ARG_ENABLE(config-dbus,AS_HELP_STRING([--enable-config-dbus], [Build 
D-BUS API support (default: no)]), [CONFIG_DBUS_API=$enableval], 
[CONFIG_DBUS_API=no])
@@ -1096,6 +1097,10 @@ else
 fi
 AM_CONDITIONAL(XCALIBRATE, [test x$XCALIBRATE = xyes])
 
+if test x$FONTSERVER = xyes; then
+AC_DEFINE(FONTSERVER, 1, [Use font servers])
+fi
+
 AC_DEFINE(RENDER, 1, [Support RENDER extension])
 RENDER_LIB='$(top_builddir)/render/librender.la'
 RENDER_INC='-I$(top_srcdir)/render'
diff --git a/dix/dixfonts.c b/dix/dixfonts.c
index 329318d..de386d4 100644
--- a/dix/dixfonts.c
+++ b/dix/dixfonts.c
@@ -1927,7 +1927,9 @@ InitFonts (void)
 
 BuiltinRegisterFpeFunctions();
 FontFileRegisterFpeFunctions();
+#ifdef FONTSERVER
 fs_register_fpe_functions();
+#endif
 }
 
 int
diff --git a/include/dix-config.h.in b/include/dix-config.h.in
index b8bb0e2..e6e4beb 100644
--- a/include/dix-config.h.in
+++ b/include/dix-config.h.in
@@ -21,6 +21,9 @@
 /* Default font path */
 #undef COMPILEDDEFAULTFONTPATH
 
+/* Use font servers */
+#undef FONTSERVER
+
 /* Miscellaneous server configuration files path */
 #undef SERVER_MISC_CONFIG_PATH
 
-- 
1.6.0.4

___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel


Re: [PATCH resent] configure: introduce --{enable, disable}-fontserver

2010-05-05 Thread Keith Packard
On Wed,  5 May 2010 17:14:41 +0300, Tiago Vignatti tiago.vigna...@nokia.com 
wrote:

 Keith, no one took this one. Care to pull it?

This seems like a reasonable change, but it seems like it requires a
custom build of libXfont, right? In which case, we should be fixing that
library and having the X server auto-detect whether fontserver support
is available there, perhaps using a variable in the .pc file.

-- 
keith.pack...@intel.com


pgp1wpRo8Ebvb.pgp
Description: PGP signature
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel