[PATCH 11/11] glx: Implement protocol for glXCreateContextAttribsARB

2012-04-13 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick Reviewed-by: Dave Airlie --- glx/createcontext.c | 213 ++- 1 files changed, 212 insertions(+), 1 deletions(-) diff --git a/glx/createcontext.c b/glx/createcontext.c index 9c1eab3..025c423 100644 -

[PATCH 09/11] glx: Use one function to add a context to all global tables

2012-04-13 Thread Ian Romanick
From: Ian Romanick Instead of having separate __glXAddContextToList and AddResource functions, just have one function that does both steps. Signed-off-by: Ian Romanick Reviewed-by: Jesse Barnes --- glx/glxcmds.c |6 ++ glx/glxext.c | 11 +-- glx/glxext.h |2 +- 3 files

[PATCH 10/11] glx: Make several functions available outside the glxcmds.c compilation unit

2012-04-13 Thread Ian Romanick
From: Ian Romanick validGlxScreen, validGlxFBConfig, validGlxContext, and __glXdirectContextCreate will soon be used by createcontext.c. Signed-off-by: Ian Romanick Reviewed-by: Dave Airlie --- glx/glxcmds.c|8 glx/glxcontext.h | 13 + 2 files changed, 17 insert

[PATCH 08/11] glx: Initialize remaining context fields

2012-04-13 Thread Ian Romanick
From: Ian Romanick There is no reason to assume the screen's context allocated initialized these fields, so don't. Signed-off-by: Ian Romanick Reviewed-by: Jesse Barnes --- glx/glxcmds.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds

[PATCH 07/11] glx: Initialize all context fields together

2012-04-13 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick Reviewed-by: Jesse Barnes --- glx/glxcmds.c | 26 +- 1 files changed, 9 insertions(+), 17 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index f754104..81e1b7c 100644 --- a/glx/glxcmds.c +++ b/glx/glxcmds.c @@ -

[PATCH 06/11] glx: Implement GLX SetClientInfoARB protocol

2012-04-13 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick Reviewed-by: Dave Airlie --- glx/clientinfo.c | 49 +++-- 1 files changed, 47 insertions(+), 2 deletions(-) diff --git a/glx/clientinfo.c b/glx/clientinfo.c index 15bbf15..b26ac1a 100644 --- a/glx/cli

[PATCH 05/11] glx: Optionally call DRI2 createContextAttribs from __glXDRIscreenCreateContext

2012-04-13 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick --- glx/glxdri2.c | 156 +++-- 1 files changed, 151 insertions(+), 5 deletions(-) diff --git a/glx/glxdri2.c b/glx/glxdri2.c index 656b577..ba09cbd 100644 --- a/glx/glxdri2.c +++ b/glx/glxdri2.c

[PATCH 04/11] glx: Add tracking for GLX_ARB_create_context and GLX_ARB_create_context_profile

2012-04-13 Thread Ian Romanick
From: Ian Romanick The formatting of the extension data table was mangled by indent. This patch also undoes that and adds approripate control comments so that indent won't do it again. Signed-off-by: Ian Romanick Reviewed-by: Dave Airlie --- glx/extension_string.c | 34 +++--

[PATCH 03/11] glx: Extend __GLXscreen::createContext to take attributes

2012-04-13 Thread Ian Romanick
From: Ian Romanick The attributes will be used for glXCreateContextAttribsARB additions in follow-on patches. Signed-off-by: Ian Romanick Reviewed-by: Jesse Barnes --- glx/glxcmds.c | 12 ++-- glx/glxdri.c | 10 +- glx/glxdri2.c |5 - glx/glxdriswra

[PATCH 02/11] glx: Don't track GLClientmajorVersion or GLClientminorVersion

2012-04-13 Thread Ian Romanick
From: Ian Romanick Nothing uses these fields anywhere in the server. Signed-off-by: Ian Romanick Reviewed-by: Jesse Barnes --- glx/glxcmds.c |2 -- glx/glxext.c|2 -- glx/glxserver.h |2 -- 3 files changed, 0 insertions(+), 6 deletions(-) diff --git a/glx/glxcmds.c b/glx/gl

[PATCH 01/11] glx: Fix mishandling of shared contexts

2012-04-13 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick Reviewed-by: Eric Anholt --- glx/glxcmds.c | 22 +++--- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/glx/glxcmds.c b/glx/glxcmds.c index 24fed8f..6fa10fe 100644 --- a/glx/glxcmds.c +++ b/glx/glxcmds.c @@ -251,

[PATCH 00/11] Implement GLX_ARB_create_context

2012-04-13 Thread Ian Romanick
This is mostly a re-send of my previous patch series. I have incorporated all (I think?) of the previous review comments, and I have (mostly) applied the new indent rules. I have tested with all of the piglit GLX_ARB_create_context tests. There are two failures. One is a case of returning the wr

Re: [PATCH:xf86-video-modesetting] Include on platforms that need it for alloca

2012-04-13 Thread Alan Coopersmith
I'd be happier with that as well, but that's up to Dave I guess. -alan- On 04/13/12 11:22 AM, Jeremy Huddleston wrote: > Can we please instead fix this by removing the alloca() calls? > > --Jeremy > > """ > BUGS > alloca() is machine and compiler dependent; its use is discouraged.

Re: [PATCH:xf86-video-modesetting] Include on platforms that need it for alloca

2012-04-13 Thread Jeremy Huddleston
Can we please instead fix this by removing the alloca() calls? --Jeremy """ BUGS alloca() is machine and compiler dependent; its use is discouraged. alloca() is slightly unsafe because it cannot ensure that the pointer returned points to a valid and usable block of memory. The a

Re: [PATCH v2 00/14] Fix signal-unsafe logging

2012-04-13 Thread Jeremy Huddleston
On Apr 13, 2012, at 10:10 AM, Alan Coopersmith wrote: > On 04/13/12 09:33 AM, Chase Douglas wrote: >>> The X server must be using SigIO for some really good reasons, what are >>> they? >> >> To reduce input visual latency. Signal handlers are run immediately when >> the signal occurs. The pro

Re: [PATCH v2 00/14] Fix signal-unsafe logging

2012-04-13 Thread Alan Coopersmith
On 04/13/12 09:33 AM, Chase Douglas wrote: >> The X server must be using SigIO for some really good reasons, what are they? > > To reduce input visual latency. Signal handlers are run immediately when > the signal occurs. The process is interrupted no matter where it is, > including when it is 500

Re: [PATCH v2 00/14] Fix signal-unsafe logging

2012-04-13 Thread Chase Douglas
On 04/13/2012 02:22 AM, Michal Suchanek wrote: > On 13 April 2012 03:09, Peter Hutterer wrote: >> typedef union { >> uint32_t ui; >> int32_t i; >> char *c; >> const char *str; >> void *ptr; >> } log_param_t; >> >> /* Log a message using only signal safe functions. */ >>

Re: [PATCH v2 00/14] Fix signal-unsafe logging

2012-04-13 Thread Chase Douglas
On 04/13/2012 01:34 AM, Daniel Kurtz wrote: > On Fri, Apr 13, 2012 at 9:09 AM, Peter Hutterer > wrote: >> On Wed, Apr 11, 2012 at 06:33:26PM -0700, Chase Douglas wrote: >>> On 04/11/2012 04:53 PM, Peter Hutterer wrote: On Mon, Apr 09, 2012 at 11:17:26AM -0700, Chase Douglas wrote: > This

Re: [PATCH] Fix coasting speed

2012-04-13 Thread Pierre Lulé
Cool! Here is the tag : Signed-off-by: Pierre Lulé 2012/4/13 Peter Hutterer > > On Wed, Mar 28, 2012 at 12:13:30AM +0200, Pierre Lulé wrote: > > Oops, left a typo in previous patch. Here's the correction. > > > > Fixes a bug introduced in commit 2603ad69b997c999404ecc441e0d64ea2cc22018 > > (Us

Re: [PATCH synaptics] support inverted scroll direction

2012-04-13 Thread Michal Suchanek
On 13 April 2012 05:50, Peter Hutterer wrote: > On Wed, Apr 11, 2012 at 10:00:56PM -0400, Alyssa Hung wrote: >> This patch adds two options to the synaptics driver: InvertEdgeScroll >> and InvertTwoFingerScroll. >> >> When enabled, they will cause the scroll direction to be inverted when >> edge s

Re: [PATCH v2 00/14] Fix signal-unsafe logging

2012-04-13 Thread Michal Suchanek
On 13 April 2012 03:09, Peter Hutterer wrote: > On Wed, Apr 11, 2012 at 06:33:26PM -0700, Chase Douglas wrote: >> On 04/11/2012 04:53 PM, Peter Hutterer wrote: >> > On Mon, Apr 09, 2012 at 11:17:26AM -0700, Chase Douglas wrote: >> >> This new patch set includes a few fixes to the first and many mo

Re: [PATCH v2 00/14] Fix signal-unsafe logging

2012-04-13 Thread Daniel Kurtz
On Fri, Apr 13, 2012 at 9:09 AM, Peter Hutterer wrote: > On Wed, Apr 11, 2012 at 06:33:26PM -0700, Chase Douglas wrote: >> On 04/11/2012 04:53 PM, Peter Hutterer wrote: >> > On Mon, Apr 09, 2012 at 11:17:26AM -0700, Chase Douglas wrote: >> >> This new patch set includes a few fixes to the first an

Re: [PATCH v2] xfree86: after VT switching back, only enable previously enabled devices

2012-04-13 Thread Daniel Stone
On 13 April 2012 03:27, Chase Douglas wrote: > Hmmm.. I'm not crazy about putting a single-function quirk inside an > external record, but I can't think of anything better. > > Reviewed-by: Chase Douglas Reviewed-by: Daniel Stone ___ xorg-devel@lists.