Re: [PATCH] dri2: Fix authentication for v8 clients (v2).

2012-06-28 Thread Chris Wilson
On Thu, 28 Jun 2012 12:07:11 +1000, Christopher James Halse Rogers christopher.halse.rog...@canonical.com wrote: The legacy logic was embarassingly wrong; AuthMagic should return errno, so returning FALSE only when AuthMagic returns nonzero is exactly wrong. v2: Match drmAuthMagic by

Re: [PATCH] dri2: Fix authentication for v8 clients (v2).

2012-06-28 Thread Michel Dänzer
On Don, 2012-06-28 at 12:07 +1000, Christopher James Halse Rogers wrote: The legacy logic was embarassingly wrong; AuthMagic should return errno, so returning FALSE only when AuthMagic returns nonzero is exactly wrong. v2: Match drmAuthMagic by returning -EINVAL rather than EINVAL Fix

Re: [PATCH 2/2] randr: add provider object (v3.1)

2012-06-28 Thread Dave Airlie
  /* Event selection bits */   #define RRScreenChangeNotifyMask  (1L 0)   /* V1.2 additions */   #define RRCrtcChangeNotifyMask     (1L 1)   #define RROutputChangeNotifyMask    (1L 2)   #define RROutputPropertyNotifyMask  (1L 3) +#define RRProviderPropertyNotifyMask        (1L 4) Do

[PATCH] randr: add provider object (v4)

2012-06-28 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com A provider object represents a GPU or virtual device that provides rendering or output services to the X server. This is the first rev of a protocol to enumerate providers devices, set their roles, and provide generic properties based on output properties for

Re: [PATCH] dri2: Fix authentication for v8 clients (v2).

2012-06-28 Thread Knut Petersen
: The legacy logic was embarassingly wrong; AuthMagic should return errno, so returning FALSE only when AuthMagic returns nonzero is exactly wrong. v2: Match drmAuthMagic by returning -EINVAL rather than EINVAL Fix trailing whitespace ACK. Tested-by: Knut Petersen

[PATCH] randr: add provider object (v5)

2012-06-28 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com A provider object represents a GPU or virtual device that provides rendering or output services to the X server. This is the first rev of a protocol to enumerate providers devices, set their roles, and provide generic properties based on output properties for

Re: [PATCH 2/2] randr: add provider object (v3.1)

2012-06-28 Thread Dave Airlie
On Thu, Jun 28, 2012 at 10:36 AM, Dave Airlie airl...@gmail.com wrote:   /* Event selection bits */   #define RRScreenChangeNotifyMask  (1L 0)   /* V1.2 additions */   #define RRCrtcChangeNotifyMask     (1L 1)   #define RROutputChangeNotifyMask    (1L 2)   #define

[PATCH] randr: add provider object (v5.1)

2012-06-28 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com A provider object represents a GPU or virtual device that provides rendering or output services to the X server. This is the first rev of a protocol to enumerate providers devices, set their roles, and provide generic properties based on output properties for

[PATCH] glx: Pass GLX drawable ID to DRI2CreateDrawable().

2012-06-28 Thread Michel Dänzer
From: Michel Dänzer michel.daen...@amd.com Otherwise the DRI2Drawable may retain references to the destroyed __GLXDRIdrawable, leading to use after free in __glXDRIinvalidateBuffers(). Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50019 Signed-off-by: Michel Dänzer

Re: [PATCH] glx: Pass GLX drawable ID to DRI2CreateDrawable().

2012-06-28 Thread Chris Wilson
On Thu, 28 Jun 2012 13:39:29 +0200, Michel Dänzer mic...@daenzer.net wrote: From: Michel Dänzer michel.daen...@amd.com Otherwise the DRI2Drawable may retain references to the destroyed __GLXDRIdrawable, leading to use after free in __glXDRIinvalidateBuffers(). Bugzilla:

[PATCH v2] Look for ModuleData only in appropriate library

2012-06-28 Thread Michal Srb
LoaderSymbol calls dlsym with RTLD_DEFAULT pseudo handle making it search in every loaded library. In addition glibc adds NODELETE flag to the library containing the symbol. It's used in doLoadModule to locate modulenameModuleData symbol, the module's library gets the flag and is kept in memory

Re: [PATCH] randr: first check pScrPriv before using the pointer at RRFirstOutput

2012-06-28 Thread Keith Packard
Ricardo Salveti de Araujo ricardo.salv...@linaro.org writes: Fix a seg fault in case pScrPriv is NULL at ProcRRGetScreenInfo, which later calls RRFirstOutput. Merged, along two more similar cases and some warning cleanups. 3ef3ce0..855003c master - master -- keith.pack...@intel.com

Re: [PULL] misc fixes

2012-06-28 Thread Keith Packard
Peter Hutterer peter.hutte...@who-t.net writes: Peter Hutterer (4): dix: if the scroll valuator reaches INT_MAX, reset to 0 os: print newline after printing display name AC_SUBST the GLX_SYS_LIBS include: document _XkbErrCode2 macros Sergei Trofimovich (1):

Re: [PULL] signal-safe logging and input ABI 18

2012-06-28 Thread Keith Packard
Peter Hutterer peter.hutte...@who-t.net writes: Yay for 'make check' today! Print backtrace in a signal-safe manner This causes the 'input' test to fail: $ ./input Testing double to FP1616/FP3232 conversions [dix] EventToCore: Not implemented yet [dix] EventToCore: Not implemented yet

Re: [PULL] hw/xwin/glx fixes

2012-06-28 Thread Keith Packard
Jon TURNEY jon.tur...@dronecode.org.uk writes: Jon TURNEY (5): hw/xwin/glx: Create a new dispatch table rather than modifying the existing one hw/xwin/glx: Fix glxLogExtensions to handle a null string without crashing hw/xwin/glx: Blacklist 'GDI generic' GL renderer

Re: [PATCH] dri2: Fix authentication for v8 clients (v2).

2012-06-28 Thread Keith Packard
Christopher James Halse Rogers christopher.halse.rog...@canonical.com writes: The legacy logic was embarassingly wrong; AuthMagic should return errno, so returning FALSE only when AuthMagic returns nonzero is exactly wrong. v2: Match drmAuthMagic by returning -EINVAL rather than EINVAL

Re: [PATCH] dix/render: consolidate window format matching code.

2012-06-28 Thread Keith Packard
Dave Airlie airl...@gmail.com writes: From: Dave Airlie airl...@redhat.com This code existed in 3 different forms, perhaps it should be consolidated. Reviewed-by: Keith Packard kei...@keithp.com Signed-off-by: Dave Airlie airl...@redhat.com Merged. a7b97b0..957bf95 master - master --

Re: [PATCH 1/4] shadowfb calls GetGlyphs() so list libXfont as a dependency

2012-06-28 Thread Alan Coopersmith
On 06/25/12 01:24 AM, Yaakov (Cygwin/X) wrote: On Sun, 2012-06-24 at 09:31 -0700, Alan Coopersmith wrote: So replace the original [PATCH 1/4] with the following which adds GetGlyphs() to dixfonts.c, making all platforms export it from the same binary, instead of differing based on how

Re: [PATCH 2/2] randr: add provider object (v3.1)

2012-06-28 Thread Aaron Plattner
On 06/28/2012 03:21 AM, Dave Airlie wrote: On Thu, Jun 28, 2012 at 10:36 AM, Dave Airlie airl...@gmail.com wrote: /* Event selection bits */ #define RRScreenChangeNotifyMask (1L 0) /* V1.2 additions */ #define RRCrtcChangeNotifyMask (1L 1) #define RROutputChangeNotifyMask

Re: [PATCH] randr: add provider object (v5.1)

2012-06-28 Thread Aaron Plattner
On 06/28/2012 03:25 AM, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com A provider object represents a GPU or virtual device that provides rendering or output services to the X server. This is the first rev of a protocol to enumerate providers devices, set their roles, and provide

Re: [PATCH 2/2] randr: add provider object (v3.1)

2012-06-28 Thread Keith Packard
Aaron Plattner aplatt...@nvidia.com writes: On 06/28/2012 03:21 AM, Dave Airlie wrote: Keith, thoughts? Well, RRScreenChangeNotify is supposed to signal any 'screen configuration changes', which presumably includes adding or removing crtcs or outputs... However, something a bit simpler, like

Re: [PATCH 1/4] shadowfb calls GetGlyphs() so list libXfont as a dependency

2012-06-28 Thread Yaakov (Cygwin/X)
On Thu, 2012-06-28 at 15:15 -0700, Alan Coopersmith wrote: Okay, how about we make the compiler think they have the same name (so we don't have to go fix all the callers right away) while letting the linker use a new name so there's no clash? Try #3 attached. How does the saying go, third

Re: [PULL] signal-safe logging and input ABI 18

2012-06-28 Thread Peter Hutterer
On Thu, Jun 28, 2012 at 10:35:00AM -0700, Keith Packard wrote: Peter Hutterer peter.hutte...@who-t.net writes: Yay for 'make check' today! Print backtrace in a signal-safe manner This causes the 'input' test to fail: $ ./input Testing double to FP1616/FP3232 conversions [dix]

[PATCH 1/2] os: fix vararg length calculation

2012-06-28 Thread Peter Hutterer
Make %u and %x sizeof(unsigned int), %p sizeof(void*). This is printf behaviour and we can't guarantee that void* is uint64_t anyway. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- This needs to be squashed into: Add LogMessageVerbSigSafe() for logging messages while in signal

[PATCH 2/2] os: print offset as unsigned int, not long unsigned int

2012-06-28 Thread Peter Hutterer
pnprintf() takes unsigned int for %u Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- To be squashed into Print backtrace in a signal-safe manner os/backtrace.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/os/backtrace.c b/os/backtrace.c index