[PATCH] Add missing GTF modes

2013-02-25 Thread Chris Wilson
A fixed-mode output device like a panel will often only inform of its preferred mode through its EDID. However, the driver will adjust user specified modes for display through use of a panel-fitter allowing greater flexibility in upscaling. This is often used by games to set a low resolution for

Re: xserver release tracker bugs

2013-02-25 Thread Alex Deucher
On Sat, Feb 23, 2013 at 5:59 PM, Dave Airlie airl...@gmail.com wrote: We've never been good about fixing all the bugs on the release tracker, but it appears we've gotten out of the habit of even just reviewing the bugs and moving them forward to the next release when we're done, as most open

Re: [PATCH] os: use libunwind to generate backtraces

2013-02-25 Thread Marcin Slusarz
On Mon, Feb 25, 2013 at 04:17:28PM +1000, Peter Hutterer wrote: On Sat, Feb 23, 2013 at 10:17:20PM +0100, Marcin Slusarz wrote: On Fri, Feb 22, 2013 at 09:02:39AM +1000, Peter Hutterer wrote: On Thu, Feb 21, 2013 at 08:00:07AM +0100, Marcin Ślusarz wrote: 21 lut 2013 00:37, Peter

Re: [PATCH] os: use libunwind to generate backtraces

2013-02-25 Thread Knut Petersen
Thanks a lot for the patch. On an openSuSE 12.2 system libunwind is to old to be recognized by your test (pre-pkg-config version), but fresh code from the git repository solved the problem. Your new patch was not needed for a full build of the xorg tree. After starting

[1.14.0-rc2] xts5 XI tests: backtraces

2013-02-25 Thread Knut Petersen
On an AOpen i915GMm-hfs mobo running openSuSE 12.2 with a full fresh build of todays xorg code (libunwind patch included) the following xts5 XI patches fail miserably: ChangeDeviceControl: [ 45194.049] (EE) [ 45194.049] (EE) Backtrace: [ 45194.051] (EE) 0:

[PATCH] glx: fix uninitialized var in __glXDRIscreenProbe

2013-02-25 Thread Piotr Dziwinski
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=59825 Signed-off-by: Piotr Dziwinski piot...@gmail.com --- glx/glxdri.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/glx/glxdri.c b/glx/glxdri.c index da46468..a997e2f 100644 --- a/glx/glxdri.c +++ b/glx/glxdri.c @@ -971,6 +971,8 @@

Re: [PATCH] os: use libunwind to generate backtraces

2013-02-25 Thread Marcin Slusarz
On Fri, Feb 22, 2013 at 09:02:39AM +1000, Peter Hutterer wrote: On Thu, Feb 21, 2013 at 08:00:07AM +0100, Marcin Ślusarz wrote: 21 lut 2013 00:37, Peter Hutterer peter.hutte...@who-t.net napisał(a): thanks for the changes but after applying this patch locally I do wonder: how did you test

Re: [PATCH] xfree86: bail on misformed acpi strings (#73227)

2013-02-25 Thread Ted Felix
On 02/23/2013 08:43 AM, Julien Cristau wrote: On Tue, Jan 29, 2013 at 16:41:59 +1000, Peter Hutterer wrote: X.Org Bug 73227 http://bugs.freedesktop.org/show_bug.cgi?id=73227 Bug number looks like a typo? Maybe that was supposed to be https://bugs.freedesktop.org/show_bug.cgi?id=55329 56523

[PATCH 0/5] Various GLX fixes

2013-02-25 Thread Adam Jackson
GLX's GetDrawableAttributes request works by simply dumping all the attributes the server knows about to the client, and filtering on the client side. Sadly we only ever bothered to implement enough of that to make texture_from_pixmap work. This series fills in the rest of the bits that GLX 1.3

[PATCH 1/5] glx: Compute number of attributes in GetDrawableAttributes on the fly

2013-02-25 Thread Adam Jackson
This doesn't have any effect yet, but is needed to properly build the reply for pbuffers. Signed-off-by: Adam Jackson a...@redhat.com --- glx/glxcmds.c | 20 +++- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index 574f8ef..09fc719

[PATCH 2/5] glx: Implement GLX_{WIDTH, HEIGHT} in GetDrawableAttributes

2013-02-25 Thread Adam Jackson
Required by GLX 1.4, section 3.3.6, Querying Attributes. Signed-off-by: Adam Jackson a...@redhat.com --- glx/glxcmds.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index 09fc719..53c5eaa 100644 --- a/glx/glxcmds.c +++ b/glx/glxcmds.c @@

[PATCH 3/5] glx: Implement GLX_FBCONFIG_ID in GetDrawableAttributes

2013-02-25 Thread Adam Jackson
Required by GLX 1.4, section 3.3.6, Querying Attributes. Signed-off-by: Adam Jackson a...@redhat.com --- glx/glxcmds.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index 53c5eaa..192c73f 100644 --- a/glx/glxcmds.c +++ b/glx/glxcmds.c @@

[PATCH 4/5] glx: Implement GLX_PRESERVED_CONTENTS drawable attribute

2013-02-25 Thread Adam Jackson
We back pixmaps with pbuffers so they're never actually clobbered, so we're really just recording the value passed in at create time so that you get the same thing when you query. Signed-off-by: Adam Jackson a...@redhat.com --- glx/glxcmds.c | 90

[PATCH 5/5] glx: Handle failure to create the pixmap backing the pbuffer

2013-02-25 Thread Adam Jackson
We happen not to sanitize the width/height we pass to CreatePixmap here, oops. It's not exploitable, but it's certainly a crash, so let's just throw BadAlloc instead. Signed-off-by: Adam Jackson a...@redhat.com --- glx/glxcmds.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [PATCH] os: use libunwind to generate backtraces

2013-02-25 Thread Peter Hutterer
On Mon, Feb 25, 2013 at 07:51:47PM +0100, Marcin Slusarz wrote: On Mon, Feb 25, 2013 at 04:17:28PM +1000, Peter Hutterer wrote: On Sat, Feb 23, 2013 at 10:17:20PM +0100, Marcin Slusarz wrote: On Fri, Feb 22, 2013 at 09:02:39AM +1000, Peter Hutterer wrote: On Thu, Feb 21, 2013 at

[PATCH 1/2] Xi: always return BadMatch for XTest devices ChangeDeviceControl requests

2013-02-25 Thread Peter Hutterer
The only controls that still do something are DEVICE_RESOLUTION and DEVICE_ENABLE. XTest devices have no resolution to change, and they cannot be disabled. So skip the lot, and prevent a crash in the DDX when it's trying to de-reference pInfo-control_proc on device with no pInfo struct.

[PATCH 2/2] Xi: don't ring the device bell if we don't have one

2013-02-25 Thread Peter Hutterer
Test case: xts5/XI/DeviceBell (1/6) Reported-by: Knut Petersen knut_peter...@t-online.de Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Xi/devbell.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Xi/devbell.c b/Xi/devbell.c index c75b94d..d4c9554 100644 ---

Re: [PATCH 4/5] glx: Implement GLX_PRESERVED_CONTENTS drawable attribute

2013-02-25 Thread Eric Anholt
Adam Jackson a...@redhat.com writes: We back pixmaps with pbuffers so they're never actually clobbered, so we're really just recording the value passed in at create time so that you get the same thing when you query. I guess I'm not clear on this one why we wouldn't just always say that we

Re: [PATCH 2/2] Xi: don't ring the device bell if we don't have one

2013-02-25 Thread Knut Petersen
On 26.02.2013 04:35, Peter Hutterer wrote: Test case: xts5/XI/DeviceBell (1/6) Reported-by: Knut Petersen knut_peter...@t-online.de Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Xi/devbell.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Xi/devbell.c