Re: [PATCH v2] xf86: Fix non-PCI configuration-less setups

2012-11-08 Thread Thierry Reding
On Wed, Oct 17, 2012 at 12:06:47PM +0200, Thierry Reding wrote: For non-PCI video devices, such as those found on many ARM embedded systems, the X server currently requires the BusID option to specify the full path to the DRM device's sysfs node in order to properly match it against the probed

[PATCH xf86-video-modesetting 1/2] build: Put m4 macros in m4 subdirectory

2012-11-08 Thread Thierry Reding
This silences a warning from libtoolize when running the autogen.sh script. Signed-off-by: Thierry Reding thierry.red...@avionic-design.de --- Makefile.am | 2 ++ autogen.sh | 1 + configure.ac | 1 + 3 files changed, 4 insertions(+) diff --git a/Makefile.am b/Makefile.am index

[PATCH xf86-video-modesetting 2/2] Remove call to miInitializeBackingStore()

2012-11-08 Thread Thierry Reding
Recent versions of the X server no longer provide this function, which has been obsolete for over 2 years now. Signed-off-by: Thierry Reding thierry.red...@avionic-design.de --- src/driver.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/driver.c b/src/driver.c index 38a1c09..4866585

[PATCH 0/7] Various DIX patches

2012-11-08 Thread Jon TURNEY
Various DIX patches Patches #2, #4 and #7 touch Xquartz to adjust for these changes, and I have tried to ensure that they do the right thing, but I am not able to test that Xquartz builds with those changes. Jon TURNEY (7): os: Remove any old logfile before trying to write to it Allow DDX

[PATCH 1/7] os: Remove any old logfile before trying to write to it

2012-11-08 Thread Jon TURNEY
If we are not backing up logfiles, remove the old logfile before trying to write a new logfile, as otherwise the operation may fail if the previous logfile was created by a different user. This change is useful when: - The DDX doesn't use the logfile backup mechanism (i.e. not Xorg) - The DDX is

[PATCH 2/7] Allow DDX to provide a main()

2012-11-08 Thread Jon TURNEY
XQuartz already conditionally renames main() as dix_main() so it can provide it's own main(). This isn't ideal as it prevents libdix built this way from being useful with any other DDX. So instead, always name that function dix_main(), and provide a stub main() which just calls it where needed.

[PATCH 3/7] glx: Synchronize Xserver glx/rensize.c with mesa src/glx/compsize.c

2012-11-08 Thread Jon TURNEY
https://bugs.freedesktop.org/show_bug.cgi?id=30102 Signed-off-by: Jon TURNEY jon.tur...@dronecode.org.uk Reviewed-by: Colin Harrison colin.harri...@virgin.net --- glx/rensize.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/glx/rensize.c b/glx/rensize.c index

[PATCH 4/7] Move pseudoramiX code where it can be shared between Xwin and Xquartz

2012-11-08 Thread Jon TURNEY
Move pseudoramiX code to a separate top-level directory. Link Xwin and Xquartz with libPseudoramiX I'm not sure moving this to a top-level directory is appropriate, but I'm not sure where else it fits. Future work: pseudoramiX can probably be consolidated with the rrxinerama code (which I think

[PATCH 5/7] Build dpmsstubs.c once as a convenience library, rather than once for each DDX which wants to use it

2012-11-08 Thread Jon TURNEY
Signed-off-by: Jon TURNEY jon.tur...@dronecode.org.uk --- Xext/Makefile.am |4 +++- hw/vfb/Makefile.am |4 ++-- hw/xnest/Makefile.am |4 ++-- hw/xwin/Makefile.am |4 ++-- test/Makefile.am |2 +- 5 files changed, 10 insertions(+), 8 deletions(-) diff --git

[PATCH 6/7] Build Xi/stubs.c once as a convenience library, rather than once for each DDX which wants to use it

2012-11-08 Thread Jon TURNEY
Signed-off-by: Jon TURNEY jon.tur...@dronecode.org.uk --- Xi/Makefile.am |5 +++-- hw/vfb/Makefile.am |4 ++-- hw/xnest/Makefile.am |4 ++-- hw/xwin/Makefile.am |4 ++-- test/Makefile.am |4 ++-- 5 files changed, 11 insertions(+), 10 deletions(-) diff --git

[PATCH 7/7] Add a screen PaintWindow wrapper

2012-11-08 Thread Jon TURNEY
Revisit b3415187e92960cbff784108b5a3a8d130dc34c5, which shoves some rootless specific code directly into miPaintWindow. Unfortunately as written, this means that dix mi/libmi (when built with ROOTLESS defined) must be linked into a ddx which links with miext/rootless/librootless, which makes it

[PATCH 1/4] xfree86: Remove some unused variables

2012-11-08 Thread Thierry Reding
GCC complains about these variables being set but not used, so they can just as well be removed. Signed-off-by: Thierry Reding thierry.red...@avionic-design.de --- hw/xfree86/common/xf86Helper.c | 4 +--- hw/xfree86/modes/xf86RandR12.c | 14 -- 2 files changed, 1 insertion(+), 17

[PATCH 2/4] xfree86: Override physical output size only if EDID is available

2012-11-08 Thread Thierry Reding
The output's physical dimensions may be set by the driver, which is the case at least for xf86-video-modesetting. It makes no sense to overwrite it with 0 if EDID is unavailable because it will cause the only source of this information to be lost. Signed-off-by: Thierry Reding

[PATCH 4/4] HACK: xfree86: modes: Recompute DPI when the screen size changes

2012-11-08 Thread Thierry Reding
The X server start up with the assumption that the display has 96 DPI unless configured otherwise. However under the proper circumstances, it can compute the correct value from the information provided by output resources. This commit adds this computation directly in xf86RandR12ScreenSetSize()

[PATCH 3/4] xfree86: Use physical dimensions provided by output

2012-11-08 Thread Thierry Reding
If an output provides its physical dimensions, they should be used in preference to computing them on the assumption that the display has a default DPI setting. Signed-off-by: Thierry Reding thierry.red...@avionic-design.de --- hw/xfree86/modes/xf86RandR12.c | 12 +--- 1 file changed, 9

Re: [PATCH 1/7] os: Remove any old logfile before trying to write to it

2012-11-08 Thread walter harms
Am 08.11.2012 14:41, schrieb Jon TURNEY: If we are not backing up logfiles, remove the old logfile before trying to write a new logfile, as otherwise the operation may fail if the previous logfile was created by a different user. what are the permissions of that file ? normally you can

Re: [PATCH 1/7] os: Remove any old logfile before trying to write to it

2012-11-08 Thread Ángel González
On 08/11/12 18:21, walter harms wrote: Am 08.11.2012 14:41, schrieb Jon TURNEY: If we are not backing up logfiles, remove the old logfile before trying to write a new logfile, as otherwise the operation may fail if the previous logfile was created by a different user. what are the

Possible regression with GLX+Xvfb after 5f5bbbe?

2012-11-08 Thread Raphael Kubo da Costa
Hey there, everyone. I've recently had to test some code which relies on GLX using Xvfb on xorg-server 1.13.0, but it looks like it is not loaded anymore after commit 5f5bbbe543f65c48ecbb5cce80116a86ca3fbe86. It turns out that this commit removed a duplicate code path in miinitext.c which was

Re: [PATCH 1/4] xfree86: Remove some unused variables

2012-11-08 Thread Aaron Plattner
These all seem fine to me. For the series, Reviewed-by: Aaron Plattner aplatt...@nvidia.com -- Aaron On 11/08/2012 05:50 AM, Thierry Reding wrote: GCC complains about these variables being set but not used, so they can just as well be removed. Signed-off-by: Thierry Reding

Re: [PATCH xorg-gtest 01/11] Drop .gz tarballs, bz2 is enough

2012-11-08 Thread Chase Douglas
On Mon, Oct 29, 2012 at 6:38 PM, Peter Hutterer peter.hutte...@who-t.netwrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 59fd496..e3afde3 100644 ---

Re: [PATCH 0/3] Fix pointer emulation on non-ownership windows

2012-11-08 Thread Chase Douglas
On Thu, Nov 1, 2012 at 10:17 PM, Peter Hutterer peter.hutte...@who-t.netwrote: https://bugs.freedesktop.org/show_bug.cgi?id=56557 Pointer emulation on the root window does not work correctly. We currently only deliver to window owners, but no client owns the root window. The real bug here

Re: [PATCH xorg-gtest] Expand on the default 'failed to open connection to display' error

2012-11-08 Thread Chase Douglas
On Mon, Nov 5, 2012 at 5:28 PM, Peter Hutterer peter.hutte...@who-t.netwrote: Virtually everyone trying the tests the first time will run into this issue since we cannot check if the ABI for dummy and whatever else is needed actually matches the server (well, we have a test for that, but

Re: [PATCH libXi] man: add generation of missing man pages for XIGrabTouchBegin

2012-11-08 Thread Chase Douglas
On Tue, Nov 6, 2012 at 2:52 PM, Peter Hutterer peter.hutte...@who-t.netwrote: The man page itself already contained the description, but it was missing from NAME so the shadow man pages were not generated. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- man/Makefile.am | 6

Re: [PATCH xorg-gtest 2/2] xserver: install default X error handler

2012-11-08 Thread Chase Douglas
On Tue, Nov 6, 2012 at 7:57 PM, Peter Hutterer peter.hutte...@who-t.netwrote: Xlib's default error handler prints the error and calls exit(1). Tests that accidentally trigger an error thus quit without cleaning up properly. Install a default error handler that prints the basic info and

Re: [PATCH xorg-gtest 1/2] gtest: add gtest-spi.h header

2012-11-08 Thread Chase Douglas
On Tue, Nov 6, 2012 at 7:57 PM, Peter Hutterer peter.hutte...@who-t.netwrote: This header is for self-testing, specifically EXPECT_FATAL_FAILURE Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- gtest/include/Makefile.am | 2 +- gtest/include/gtest/gtest-spi.h | 232

Re: [PATCH v2] xf86: Fix non-PCI configuration-less setups

2012-11-08 Thread Dave Airlie
On Wed, Oct 17, 2012 at 8:06 PM, Thierry Reding thierry.red...@avionic-design.de wrote: For non-PCI video devices, such as those found on many ARM embedded systems, the X server currently requires the BusID option to specify the full path to the DRM device's sysfs node in order to properly

Re: [PATCH xorg-gtest 01/11] Drop .gz tarballs, bz2 is enough

2012-11-08 Thread Daniel Stone
On 30 October 2012 12:38, Peter Hutterer peter.hutte...@who-t.net wrote: --- a/configure.ac +++ b/configure.ac @@ -8,7 +8,7 @@ AC_CONFIG_SRCDIR([Makefile.am]) AC_CONFIG_MACRO_DIR([m4]) # Initialize Automake -AM_INIT_AUTOMAKE([foreign dist-bzip2]) +AM_INIT_AUTOMAKE([foreign dist-bzip2

Re: [PATCH xorg-gtest 01/11] Drop .gz tarballs, bz2 is enough

2012-11-08 Thread Peter Hutterer
On Fri, Nov 09, 2012 at 01:44:16PM +1100, Daniel Stone wrote: On 30 October 2012 12:38, Peter Hutterer peter.hutte...@who-t.net wrote: --- a/configure.ac +++ b/configure.ac @@ -8,7 +8,7 @@ AC_CONFIG_SRCDIR([Makefile.am]) AC_CONFIG_MACRO_DIR([m4]) # Initialize Automake

Re: [PATCH xorg-gtest 2/2] xserver: install default X error handler

2012-11-08 Thread Peter Hutterer
On Thu, Nov 08, 2012 at 05:46:12PM -0800, Chase Douglas wrote: On Tue, Nov 6, 2012 at 7:57 PM, Peter Hutterer peter.hutte...@who-t.netwrote: Xlib's default error handler prints the error and calls exit(1). Tests that accidentally trigger an error thus quit without cleaning up properly.