Re: [PATCH 2/2] DRI2: Add error message when working around driver bug

2010-10-29 Thread Pauli Nieminen
On 28/10/10 21:12 +0200, ext Alex Deucher wrote: On Thu, Oct 28, 2010 at 2:20 PM, Mario Kleiner mario.klei...@tuebingen.mpg.de wrote: On Oct 28, 2010, at 6:02 PM, Jesse Barnes wrote: On Thu, 28 Oct 2010 18:47:09 +0300 Pauli Nieminen ext-pauli.niemi...@nokia.com wrote: Most of what

Re: [PATCH evdev] evdev: added property Evdev Axes Rotation. #27688

2010-10-29 Thread Oliver McFadden
On Fri, 2010-10-29 at 15:37 +1000, ext Peter Hutterer wrote: On Sun, Oct 24, 2010 at 01:45:03PM +0200, Paolo D'Apice wrote: The evdev driver does not allow to set a custom axes rotation as the mousedrv driver does with the option AngleOffset. This option is necessary for some trackballs,

Re: [PATCH 0/3] Disable fop by default in libX11

2010-10-29 Thread Alan Coopersmith
Dan Nicholson wrote: Here's a few patches that allow fop to be used but defaulted to off in libX11. The doc macros have been updated to take an optional argument setting the default that will be used. XORG_WITH_FOP is then passed 'no' from libX11. Pushed the latest version of these patches to

[PATCH] Allow uploading a keymap to a single device

2010-10-29 Thread Dirk Wallenstein
Signed-off-by: Dirk Wallenstein hals...@t-online.de --- This should do the trick. I tested it with two keyboards. xkbcomp.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/xkbcomp.c b/xkbcomp.c index b0df920..a1c6737 100644 --- a/xkbcomp.c +++ b/xkbcomp.c @@ -1037,6

Re: [PATCH] Allow uploading a keymap to a single device

2010-10-29 Thread Dirk Wallenstein
Ups, that's an xkbcomp patch. Sorry, forgot to fill in the label. -- Greetings, Dirk ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH evdev] evdev: added property Evdev Axes Rotation. #27688

2010-10-29 Thread Benjamin Tissoires
Hi, Le 29/10/2010 07:37, Peter Hutterer a écrit : On Sun, Oct 24, 2010 at 01:45:03PM +0200, Paolo D'Apice wrote: The evdev driver does not allow to set a custom axes rotation as the mousedrv driver does with the option AngleOffset. This option is necessary for some trackballs, for example the

Re: [PATCH] Allow uploading a keymap to a single device

2010-10-29 Thread Florian Echtler
On Fri, 2010-10-29 at 09:49 +0200, Dirk Wallenstein wrote: Ups, that's an xkbcomp patch. Sorry, forgot to fill in the label. Ha - thanks! This patch exactly solves my problem described here: http://lists.x.org/archives/xorg/2010-October/051583.html So it seems like it was an xkbcomp issue.

Re: [PATCH 1/2] Replace usage of DamageRegionAppend with DamageDamageRegion to fix reportAfter.

2010-10-29 Thread Michel Dänzer
On Don, 2010-10-28 at 20:46 -0700, Eric Anholt wrote: In all these cases, any rendering implied by this damage has already occurred, and we want to get the damage out to the client. Some of the DamageRegionAppend calls were explicitly telling damage to flush the reportAfter damage out, but

Re: [PATCH] Allow uploading a keymap to a single device

2010-10-29 Thread Julien Cristau
On Fri, Oct 29, 2010 at 09:31:35 +0200, Dirk Wallenstein wrote: Signed-off-by: Dirk Wallenstein hals...@t-online.de --- This should do the trick. I tested it with two keyboards. Should this also be done for the INPUT_XKM case? Cheers, Julien ___

[PATCH xkbcomp v2] Allow uploading a keymap to a single device

2010-10-29 Thread Dirk Wallenstein
Signed-off-by: Dirk Wallenstein hals...@t-online.de --- On Fri, Oct 29, 2010 at 11:06:26AM +0200, Julien Cristau wrote: Should this also be done for the INPUT_XKM case? Oh, of course. Tested, too. The manual page seems to be fulfilled. xkbcomp.c |2 ++ 1 files changed, 2 insertions(+), 0

Re: [PATCH xkbcomp v2] Allow uploading a keymap to a single device

2010-10-29 Thread Julien Cristau
On Fri, Oct 29, 2010 at 11:35:10 +0200, Dirk Wallenstein wrote: Signed-off-by: Dirk Wallenstein hals...@t-online.de Reviewed-by: Julien Cristau jcris...@debian.org --- On Fri, Oct 29, 2010 at 11:06:26AM +0200, Julien Cristau wrote: Should this also be done for the INPUT_XKM case? Oh, of

[PATCH xkbcomp] man: Improve description of device selection option

2010-10-29 Thread Dirk Wallenstein
The device selection option takes effect when loading keymaps, too. Signed-off-by: Dirk Wallenstein hals...@t-online.de --- However, the description can be clarified. I see it separate but I would not mind squashing (I mean, if appropriate) xkbcomp.man |3 ++- 1 files changed, 2

Re: [PATCH 0/4] Improve reporting of resource sizes.

2010-10-29 Thread Tiago Vignatti
On Thu, Oct 28, 2010 at 01:23:40PM +0300, ext Rami Ylim�ki wrote: The motivations of this patchset is to make xrestop and X resource extension report resource sizes more accurately. Currently xrestop is not taking pixmap references from extensions into account. X server pixmap size reporting

Re: [PATCH] shadow: Optimize shadowUpdatePacked(). (#26973)

2010-10-29 Thread Soeren Sandmann
Matt Turner matts...@gmail.com writes: So, do we want this patch? It seems like it does deobfuscate the code a bit, and memcpy is going to be more efficient than byte-by-byte copies. Matt Can I get some Reviewed-bys or Acked-bys? Reviewed-by: Soren Sandmann sandm...@daimi.au.dk

Re: [PATCH] DRI2: Free DRI2 drawable references in DRI2DestroyDrawable

2010-10-29 Thread Rami Ylimäki
From resource management point of view this patch seems correct to me. Reviewed-by: Rami Ylimäki rami.ylim...@vincit.fi On 10/25/2010 04:55 PM, Pauli Nieminen wrote: If client calls DRI2CreateDrawable multiple times for same drawable DRI2 creates multiple references. Multiple references cause

Re: [PATCH 0/3] Disable fop by default in libX11

2010-10-29 Thread Gaetan Nadon
On Fri, 2010-10-29 at 00:23 -0700, Alan Coopersmith wrote: Pushed the latest version of these patches to util-macros libX11. I'll cut one more libX11 RC tomorrow (err, later today now I guess) if the tinderboxes don't light up overnight. I ran a local build and distcheck with no problems.

[PATCH libXt] XtFree() can handle NULL , remove check

2010-10-29 Thread walter harms
From dd37c578f348d9ba3e466bee6458629c5fde03a1 Mon Sep 17 00:00:00 2001 XtFree() can handle NULL , remove check Signed-off-by: walter harms wha...@bfs.de --- src/Varargs.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/Varargs.c b/src/Varargs.c index

Re: RFC: Use libxkbcommon in Xorg

2010-10-29 Thread Tiago Vignatti
Hi Kristian. I tried the library. First impression is good - it works :) I had some comments bellow though. On Fri, Oct 22, 2010 at 04:26:38AM +0200, ext Kristian H�gsberg wrote: At this point libxkbcommon only depends on xproto (for keycodes and a few other defines) and kbproto. it seems

Re: RFC: Use libxkbcommon in Xorg

2010-10-29 Thread ext Tiago Vignatti
On Fri, Oct 29, 2010 at 06:48:43PM +0300, ext Tiago Vignatti wrote: Hi Kristian. I tried the library. First impression is good - it works :) I had some comments bellow though. On Fri, Oct 22, 2010 at 04:26:38AM +0200, ext Kristian H�gsberg wrote: At this point libxkbcommon only

Xv Reput/Reget behaviour changes

2010-10-29 Thread ville . syrjala
I'm trying to eliminate some rather nasty looking on/off blinking that's bothering my video overlays. The xfree86 xv code hooks into ClipNotify, WindowExposures and AdjustFrame and does something a little different in each of them. I tried to decipher the intention of the original code, but even

[PATCH 02/10] xfree86/xv: Eliminate two open coded copies of xf86XVRemovePortFromWindow

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com No functional change. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c | 27 ++- 1 files changed, 2 insertions(+), 25 deletions(-) diff --git a/hw/xfree86/common/xf86xv.c

[PATCH 04/10] xfree86/xv: Allow xf86XVReputOrStopPort() to stop PutVideo/GetVideo ports

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com Modify xf86XVReputOrStopPort() to allow stopping of all types of ports. Will be useful later. No functional change. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c | 33 +++-- 1 files

[PATCH 06/10] xfree86/xv: No need to free composite clip in AdjustFrame

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com Nothing should change in AdjustFrame that would need the composite clip to be recomputed. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git

[PATCH 08/10] xfree86/xv: Factor out the meat of xf86XVAdjustFrame for later reuse

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com No functional change. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c | 66 +++ 1 files changed, 35 insertions(+), 31 deletions(-) diff --git a/hw/xfree86/common/xf86xv.c

[PATCH 09/10] xfree86/modes: Provide a ModeSet hook in ScrnInfo

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com Add a new hook ModeSet that will be called after display mode is changed, or after the display has been panned. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86str.h |4 +++- hw/xfree86/modes/xf86Crtc.c |6 ++

[PATCH 05/10] xfree86/xv: Change the behaviour of ClipNotify to reput instead of stop

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com When ClipNotify gets called for a visible window, reput instead of stopping the port. This eliminates nasty overlay flickering that happens during clip changes. If the window is invisible or if ReputImage isn't supported stop and remove the port from

[PATCH 03/10] xfree86/xv: Factor out the meat of xf86XVWindowExposures for later reuse

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com No fucntional change. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c | 53 +++ 1 files changed, 28 insertions(+), 25 deletions(-) diff --git a/hw/xfree86/common/xf86xv.c

[PATCH 07/10] xfree86/xv: Change the behaviour of AdjustFrame to reput everything

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com Also reput PutVideo/GetVideo ports in AdjustFrame. This makes the overlay track the screen panning instead of staying stationary in the wrong place. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c | 32

[PATCH 01/10] xfree86/xv: Remove useless NULL check from ClipNotify

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com WinPriv-PortRec should never be NULL as WinPriv itself would be removed from the list when the port is removed from the window. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c |3 --- 1 files changed, 0

[PATCH 10/10] xfree86/xv: Provide a ModeSet hook

2010-10-29 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@nokia.com Reput all ports when the display mode or panning has been changed by RandR code. This makes the overlays appear in the correct position on the screen. Signed-off-by: Ville Syrjälä ville.syrj...@nokia.com --- hw/xfree86/common/xf86xv.c | 24

Re: [PATCH 1/2] Replace usage of DamageRegionAppend with DamageDamageRegion to fix reportAfter.

2010-10-29 Thread Eric Anholt
On Fri, 29 Oct 2010 10:46:00 +0200, Michel Dänzer mic...@daenzer.net wrote: On Don, 2010-10-28 at 20:46 -0700, Eric Anholt wrote: In all these cases, any rendering implied by this damage has already occurred, and we want to get the damage out to the client. Some of the DamageRegionAppend

Re: [PATCH] DRI2: Free DRI2 drawable references in DRI2DestroyDrawable

2010-10-29 Thread Kristian Høgsberg
On Mon, Oct 25, 2010 at 9:55 AM, Pauli Nieminen ext-pauli.niemi...@nokia.com wrote: If client calls DRI2CreateDrawable multiple times for same drawable DRI2 creates multiple references. Multiple references cause DRI2 send multiple invalidate events for same client. Yup, there is a leak, but I

Re: [PATCH 1/4] dix: Provide means to report exact sizes of resources.

2010-10-29 Thread Alan Coopersmith
Rami Ylimäki wrote: On 10/28/2010 04:48 PM, Alan Coopersmith wrote: +/** + * Get the function used to calculate resource size. Extensions and + * drivers need to be able to determine the current size calculation + * function if they want to wrap or override it. + * + * @param[in] type

[PATCH] provide handling for the horizontal component of a scrollball input

2010-10-29 Thread Jesse Adkins
From: Iain Hibbert plu...@rya-online.net --- src/bsd_mouse.c | 11 ++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/src/bsd_mouse.c b/src/bsd_mouse.c index 83054eb..d16d621 100644 --- a/src/bsd_mouse.c +++ b/src/bsd_mouse.c @@ -53,12 +53,15 @@ #define