[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 --- 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 6acc81d..daac60c 100644 -

[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 --- This needs to be squashed into: Add LogMessageVerbSigSafe() for logging messages while in signal context Fixes the test/input

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 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:

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, thi

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

2012-06-28 Thread Keith Packard
Aaron Plattner 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 a global event tha

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 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 base

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 wrote: /* Event selection bits */ #define RRScreenChangeNotifyMask (1L << 0) /* V1.2 additions */ #define RRCrtcChangeNotifyMask (1L << 1) #define RROutputChangeNotifyMask(1L << 2)

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 l

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

2012-06-28 Thread Keith Packard
Dave Airlie writes: > From: Dave Airlie > > This code existed in 3 different forms, perhaps it should be > consolidated. > > Reviewed-by: Keith Packard > Signed-off-by: Dave Airlie Merged. a7b97b0..957bf95 master -> master -- keith.pack...@intel.com pgpLhr07EwU3N.pgp Description: PGP

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

2012-06-28 Thread Keith Packard
Christopher James Halse Rogers 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 > Fix trailing whitespace Merged.

Re: [PULL] hw/xwin/glx fixes

2012-06-28 Thread Keith Packard
Jon TURNEY 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 > hw/xwin/glx: Upda

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

2012-06-28 Thread Keith Packard
Peter Hutterer 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 [dix] EventToCore: Not

Re: [PULL] misc fixes

2012-06-28 Thread Keith Packard
Peter Hutterer 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): > test/.gitignore: a

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

2012-06-28 Thread Keith Packard
Ricardo Salveti de Araujo 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 pgpXsJsMLsHmq.pgp Descripti

[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 ModuleData symbol, the module's library gets the flag and is kept in memory even after i

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 wrote: > From: Michel Dänzer > > 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 >

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

2012-06-28 Thread Michel Dänzer
From: Michel Dänzer 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 --- glx/glxdri2.c |2 +- 1 file chang

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

2012-06-28 Thread Dave Airlie
From: Dave Airlie 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 them v1.1: fix ty

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 wrote: >>>   /* Event selection bits */ >>>   #define RRScreenChangeNotifyMask  (1L << 0) >>>   /* V1.2 additions */ >>>   #define RRCrtcChangeNotifyMask     (1L << 1) >>>   #define RROutputChangeNotifyMask    (1L << 2) >>>   #define RROutputPropertyNo

[PATCH] randr: add provider object (v5)

2012-06-28 Thread Dave Airlie
From: Dave Airlie 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 them v1.1: fix ty

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 (v4)

2012-06-28 Thread Dave Airlie
From: Dave Airlie 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 them v1.1: fix ty

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

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 > F

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 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 >