Re: [PATCH v3 1/1] os: added the -upstart option for Upstart's signaling method.

2010-07-14 Thread Peter Hutterer
On Wed, Jul 14, 2010 at 07:49:39AM +0300, Oliver McFadden wrote: This is very similar to the RunFromSmartParent (implicit) option, except we do not send the signal to our parent process, but our own process instead, and that signal is SIGSTOP, not SIGUSR1. Upstart will detect this, realize

[PATCH 1/1] os: added the -sigstop option for Upstart (or equivalent) startup.

2010-07-14 Thread Oliver McFadden
This is very similar to the RunFromSmartParent (implicit) option, except we do not send the signal to our parent process, but our own process instead, and that signal is SIGSTOP, not SIGUSR1. Upstart or a similar equivalent program will detect this, realize that we are ready to accept clients

RE: [Xorg-driver-geode] Rendering in geode

2010-07-14 Thread Jonathan Morton
On Wed, 2010-07-14 at 09:29 +0800, Huang, FrankR wrote: I have some understanding about your explanation. Is that to say if the source pixel is out of the source image, the modulus operation will be used to get the source pixel to to rendered, not for the rendering region. Take my example,

Re: [PATCH] int10: Remove the vm86 and stub backends

2010-07-14 Thread Tiago Vignatti
Hi! On Tue, Jul 13, 2010 at 11:32:54PM +0200, ext Adam Jackson wrote: vm86 has been defaulted off since 1.6, and is still a terrible idea to actually use. Time to say goodbye. My empirical evidences say that we can't do this. I had different behaviour running some systems with x86emu and

Re: [PATCH] VGA arbiter: No need for arbitration around CreateGC.

2010-07-14 Thread Tiago Vignatti
On Wed, Jul 14, 2010 at 12:35:12AM +0200, ext Jamey Sharp wrote: None of XAA, EXA, or UXA do any hardware access during CreateGC, so they don't need VGA arbitration. I haven't found any open source drivers that hook CreateGC, so they're safe. I'd be surprised if any driver directly hooks

[RFC] dri2: Expose API for drivers to set drawable swap limit.

2010-07-14 Thread Pauli Nieminen
This allows ddx to set swap_limit if there is variable number of buffers for drawable. This allows ddx driver to select triple buffering to avoid problems if frame rate is close to VSYNC rate. While driver may want to keep composite swaps double buffered to save memory. Signed-off-by: Pauli

Re: [PATCH] VGA arbiter: No need for arbitration around CreateGC.

2010-07-14 Thread Vignatti Tiago (Nokia-MS/Helsinki)
On Wed, Jul 14, 2010 at 03:19:11PM +0200, Vignatti Tiago (Nokia-MS/Helsinki) wrote: On Wed, Jul 14, 2010 at 12:35:12AM +0200, ext Jamey Sharp wrote: None of XAA, EXA, or UXA do any hardware access during CreateGC, so they don't need VGA arbitration. I haven't found any open source drivers

Re: [PATCH 2/2] Delete Colormap-devPriv. It isn't used any more.

2010-07-14 Thread Tiago Vignatti
On Wed, Jul 14, 2010 at 12:37:08AM +0200, ext Jamey Sharp wrote: Signed-off-by: Jamey Sharp ja...@minilop.net --- include/colormapst.h |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/include/colormapst.h b/include/colormapst.h index c4e24a6..bb79c86 100644 ---

[PULL libXi] Random patches fixing memory leaks, uninitialized variables and display locking

2010-07-14 Thread Pauli Nieminen
Hi, I have has a few libXi patches lying around. First 5 patches were reviewed by Julien Cristau but the last one didn't get any review. Pauli The following changes since commit d36447a69789b7acfd54b625caa299780480: Matt Dew (1): specs: convert input .ms specs from xorg-docs

Re: [PATCH] VGA arbiter: No need for arbitration around CreateGC.

2010-07-14 Thread Jamey Sharp
Thanks for reviewing! (And thanks to Alex and Jeremy for this round of reviews too!) I don't quite understand your comments on this patch, though, Tiago. On Wed, Jul 14, 2010 at 6:25 AM, Vignatti Tiago (Nokia-MS/Helsinki) tiago.vigna...@nokia.com wrote: On Wed, Jul 14, 2010 at 03:19:11PM +0200,

Re: [PATCH] VGA arbiter: No need for arbitration around CreateGC.

2010-07-14 Thread Vignatti Tiago (Nokia-MS/Helsinki)
Hey Jamey! On Wed, Jul 14, 2010 at 04:25:39PM +0200, ext Jamey Sharp wrote: On Wed, Jul 14, 2010 at 6:25 AM, Vignatti Tiago (Nokia-MS/Helsinki) tiago.vigna...@nokia.com wrote: On Wed, Jul 14, 2010 at 03:19:11PM +0200, Vignatti Tiago (Nokia-MS/Helsinki) wrote: as discussed already in

[PATCH] miPolyGlyphBlt: Delete redundant ValidateGC.

2010-07-14 Thread Jamey Sharp
The caller is required to have validated the GC for the drawable before calling any ops, including PolyGlyphBlt. Signed-off-by: Jamey Sharp ja...@minilop.net --- mi/miglblt.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/mi/miglblt.c b/mi/miglblt.c index

Re: [PATCH] Remove xcalibrate and tslib support

2010-07-14 Thread Tiago Vignatti
Hi Timothy! Sorry to reply this thread so late. On Thu, Jul 08, 2010 at 08:34:20PM +0200, ext Timothy Meade wrote: I would rather move forward not backwards and getting XF86 to a usable memory footprint and startup time would certainly be moving forward (unacceptible startup time being any

Re: [PATCH] VGA arbiter: No need for arbitration around CreateGC.

2010-07-14 Thread Jamey Sharp
On Wed, Jul 14, 2010 at 7:44 AM, Vignatti Tiago (Nokia-MS/Helsinki) tiago.vigna...@nokia.com wrote: On Wed, Jul 14, 2010 at 04:25:39PM +0200, ext Jamey Sharp wrote: First, I don't understand this use of signed-off-by. Did you mean reviewed-by? So I considered myself involved in the

Re: [RFC] dri2: Expose API for drivers to set drawable swap limit.

2010-07-14 Thread Jesse Barnes
On Wed, 14 Jul 2010 16:20:40 +0300 Pauli Nieminen ext-pauli.niemi...@nokia.com wrote: This allows ddx to set swap_limit if there is variable number of buffers for drawable. This allows ddx driver to select triple buffering to avoid problems if frame rate is close to VSYNC rate. While driver

Re: [PATCH] miPolyGlyphBlt: Delete redundant ValidateGC.

2010-07-14 Thread Keith Packard
On Wed, 14 Jul 2010 07:57:11 -0700, Jamey Sharp ja...@minilop.net wrote: The caller is required to have validated the GC for the drawable before calling any ops, including PolyGlyphBlt. Reviewed-by: Keith Packard kei...@keithp.com -- keith.pack...@intel.com pgpGNfHKDQ1kI.pgp Description:

[PATCH 2/2] rootless: ROOTLESS_GLOBAL_COORDS is always set, so unifdef it.

2010-07-14 Thread Jamey Sharp
Signed-off-by: Jamey Sharp ja...@minilop.net Cc: Jeremy Huddleston jerem...@apple.com Cc: Jon TURNEY jon.tur...@dronecode.org.uk --- miext/rootless/rootlessConfig.h |2 -- miext/rootless/README.txt | 15 ++- miext/rootless/rootless.h |5 +

[PATCH 1/2] rootless: ROOTLESS_TRACK_DAMAGE is never set, so unifdef it.

2010-07-14 Thread Jamey Sharp
Signed-off-by: Jamey Sharp ja...@minilop.net Cc: Jeremy Huddleston jerem...@apple.com Cc: Jon TURNEY jon.tur...@dronecode.org.uk --- miext/rootless/rootlessConfig.h |2 -- hw/xwin/winscrinit.c|2 -- miext/rootless/README.txt | 19 ++-

Re: [PATCH 2/2] rootless: ROOTLESS_GLOBAL_COORDS is always set, so unifdef it.

2010-07-14 Thread Jeremy Huddleston
Reviewed-by: Jeremy Huddleston jerem...@apple.com On Jul 14, 2010, at 11:44, Jamey Sharp wrote: Signed-off-by: Jamey Sharp ja...@minilop.net Cc: Jeremy Huddleston jerem...@apple.com Cc: Jon TURNEY jon.tur...@dronecode.org.uk --- miext/rootless/rootlessConfig.h |2 --

Re: [PATCH 1/2] rootless: ROOTLESS_TRACK_DAMAGE is never set, so unifdef it.

2010-07-14 Thread Jeremy Huddleston
Reviewed-by: Jeremy Huddleston jerem...@apple.com On Jul 14, 2010, at 11:44, Jamey Sharp wrote: Signed-off-by: Jamey Sharp ja...@minilop.net Cc: Jeremy Huddleston jerem...@apple.com Cc: Jon TURNEY jon.tur...@dronecode.org.uk --- miext/rootless/rootlessConfig.h |2 -- hw/xwin/winscrinit.c

[PATCH] rootless: Delete dead remnants of rootless acceleration.

2010-07-14 Thread Jamey Sharp
Jeremy deleted rootlessAccelInit's implementation in 2008 in commit 587c010a1cd733fded4d49dc339df0634bda8be6. Delete its prototype and the remaining commented-out call to it. It still makes sense for the rootless GC ops to relax the planemask, but that's independent of the size of the operation,

Re: [PATCH] rootless: Delete dead remnants of rootless acceleration.

2010-07-14 Thread Jeremy Huddleston
Reviewed-by: Jeremy Huddleston jerem...@apple.com On Jul 14, 2010, at 13:49, Jamey Sharp wrote: Jeremy deleted rootlessAccelInit's implementation in 2008 in commit 587c010a1cd733fded4d49dc339df0634bda8be6. Delete its prototype and the remaining commented-out call to it. It still makes sense

xwin sets ROOTLESS_PROTECT_ALPHA to NO: bug?

2010-07-14 Thread Jamey Sharp
The generic rootless layer can be built to preserve an alpha channel for depth 24 images stored at 32 bits per pixel. It does this if rootlessConfig.h defines ROOTLESS_PROTECT_ALPHA; it doesn't matter what it's defined to. When the server is built on Windows, ROOTLESS_PROTECT_ALPHA is defined to

Current 1.9 release blocking bugs

2010-07-14 Thread Keith Packard
Here's the current list of bugs blocking the 1.9 release: Summary: A couple of rendering issues remaining that need Xlib test cases, and then a handful of input bugs. Otherwise, I think we're in good shape. Details: * 3040: Bad interaction between xinerama and

[PATCH] xkb: use GetMaster instead of dev-u.master.

2010-07-14 Thread Peter Hutterer
Devices that are both pointers and keyboards are not affected by keyboard changes as their master device is a master pointer, not a master keyboard. Use GetMaster() instead to ensure devices that are attached to the paired master pointer device will still be update. Signed-off-by: Peter Hutterer