Help: XF86DGAGetVideo:failed to map video memory. I used libXxf86dga-1.1.1 on Ubuntu 10.04 on intel GMA 4500MHD(Intel GM45 chipset)

2011-10-11 Thread Focus . Luo
Hi all, 1, Does the intel GM45 chipset support DGA(direct graphic access)? 2, If it supports, why I used the function XF86DGAGetVideo(dis, DefaultScreen(dis), addr, width, bank, ram); and It returen XF86DGAGetVideo:failed to map video memory. But I used the function XF86DGAQueryDirectVideo,

questions about GEM buffer object

2011-10-11 Thread Wang Baisheng
Hi All: I have some confused for the GEM buffer object. There must be one GEM buffer object for the framebuffer to be displayed on screen, right ? And for every window, there is also one GEM buffer object, right ? If so, my question is: Is the GEM buffer object for the window (not direct

[PULL RESEND] more prep work for screen crossing fixes, inverted button emulation fix

2011-10-11 Thread Peter Hutterer
added a few more prep work for the screen crossing fixes on top. Commit 32b289e46cc2d5ec32ff0c4ba5bbfbf602afb388 The following changes since commit 6378d0233d21088b6429755627b4253859892c72: Merge remote-tracking branch 'herrb/master' (2011-10-03 13:56:06 -0700) are available in the git

Re: [PATCH xserver] check for elevated privileges not uid=0

2011-10-11 Thread Tormod Volden
On Mon, Oct 10, 2011 at 9:04 PM, Antoine Martin wrote: And another blooper, sorry for the spam. Correct patch attached. Need sleep, will pick this up tomorrow. Subject: [PATCH xserver] check for elevated privileges not uid=0 (V3) Couldn't find any reference to it, so I just removed

[PATCH] Fix drain_console unregistration

2011-10-11 Thread Tomáš Trnka
Bug introduced by 9dca441670d261a9a9fb6108960ed48f3d58fb7f xfree86: add a hook to replace the new console handler. console_handler was not being set, making the server eat up CPU spinning in WaitForSomething selecting consoleFd over and over again, every time trying to unregister drain_console

[PATCH V4] xserver: check for elevated privileges not uid=0

2011-10-11 Thread Antoine Martin
This allows us to run the server as a normal user whilst still being able to use the -modulepath, -logfile and -config switches We define a xf86PrivsElevated which will do the checks and cache the result in case it is called more than once. Also renamed the paths #defines to match their new

Re: Help: XF86DGAGetVideo:failed to map video memory. I used libXxf86dga-1.1.1 on Ubuntu 10.04 on intel GMA 4500MHD(Intel GM45 chipset)

2011-10-11 Thread Corbin Simpson
On Mon, Oct 10, 2011 at 6:17 PM, Focus.Luo luoyanq...@sjtu.org wrote: Hi all, 1, Does the intel GM45 chipset support DGA(direct graphic access)? 2, If it supports, why I used the function XF86DGAGetVideo(dis, DefaultScreen(dis), addr, width, bank, ram); and It returen XF86DGAGetVideo:failed

valgrind errors in X server

2011-10-11 Thread Michal Suchanek
Hello, While valgrinding the X server to track a double free in one of my patches I found the errors below. I run Xorg +extension GLX +extension RANDR +extension RENDER -logfile /scratch/xdummy.log -config ~hramrach/xdummy.conf :1 ==3355== Conditional jump or move depends on uninitialised

Re: [PATCH V4] xserver: check for elevated privileges not uid=0

2011-10-11 Thread Michal Suchanek
Hello, I added some debug prints and it looks ok. Tested-by: Michal Suchanek hramr...@centrum.cz BTW it might need some explanation why the server was dropping privileges in this message: + if (seteuid(oldeuid) != 0) { +FatalError(Failed to drop privileges. Exiting\n);

[PATCH] Export include/input.h:GetMaster() function

2011-10-11 Thread Adam Tkac
Hello all, attached patch exports the GetMaster() function, it is needed by TigerVNC's libvnc.so module. Please merge it also to 1.11 release, thank you! Comments are welcomed. Regards, Adam From 1cf3957095fdfae8ca3382c48e502d30df8ffeff Mon Sep 17 00:00:00 2001 From: Adam Tkac at...@redhat.com

Re: [PATCH/libX11] Return name instead of value in XGetIMValues() and XSetIMValues()

2011-10-11 Thread Yann Droneaud
Le lundi 10 octobre 2011 à 14:00 -0700, Jeremy Huddleston a écrit : The src changes look right, but I'm wondering if this is a documentation bug rather than an implementation bug. Returning the value is not really useful to diagnose an error, a program would have to manage a conversion table

Re: [PATCH/libX11] Return name instead of value in XGetIMValues() and XSetIMValues()

2011-10-11 Thread Yann Droneaud
Le lundi 10 octobre 2011 à 14:00 -0700, Jeremy Huddleston a écrit : The src changes look right, but I'm wondering if this is a documentation bug rather than an implementation bug. Is anyone relying on the current (undocumented) behavior? XGetICValues() and XSetICValues() returns the name

[PATCH] Return name instead of False in XSetICValues()

2011-10-11 Thread Yann Droneaud
In case of error, XSetICValues() must return the first argument that failed to be set. But in some error paths, it returns False, which is converted to NULL, so the function returns OK in case of error. Signed-off-by: Yann Droneaud y...@droneaud.fr --- modules/im/ximcp/imRm.c |6 +++--- 1

Re: valgrind errors in X server

2011-10-11 Thread Jeremy Huddleston
On Oct 11, 2011, at 02:52, Michal Suchanek wrote: Hello, While valgrinding the X server to track a double free in one of my patches I found the errors below. I run Xorg +extension GLX +extension RANDR +extension RENDER -logfile /scratch/xdummy.log -config ~hramrach/xdummy.conf :1

Re: [PATCH/libX11] Return name instead of value in XGetIMValues() and XSetIMValues()

2011-10-11 Thread Jeremy Huddleston
Ok, thanks for addressing my concerns. Reviewed-by: Jeremy Huddleston jerem...@apple.com On Oct 11, 2011, at 06:08, Yann Droneaud wrote: Le lundi 10 octobre 2011 à 14:00 -0700, Jeremy Huddleston a écrit : The src changes look right, but I'm wondering if this is a documentation bug rather

Re: [PATCH] XIM: Make Xim handle NEED_SYNC_REPLY flag

2011-10-11 Thread Jeremy Huddleston
Choe, I'm a bit confused here. This looks like an older version of your patch from: https://bugs.freedesktop.org/attachment.cgi?id=13405 Bernd provided an updated version https://bugs.freedesktop.org/attachment.cgi?id=21093 which took into account changes in libX11

Re: [PATCH libpciaccess 2/2 v3] linux: Implement map_legacy

2011-10-11 Thread Javier Pello
On Fri, 2011-10-07 at 10:18:20 -0700, Jeremy Huddleston wrote: +char name[PATH_MAX]; +int flags = O_RDONLY; +int prot = PROT_READ; +int fd; +int ret=0; + +if (map_flags PCI_DEV_MAP_FLAG_WRITABLE) { + flags |= O_WRONLY; + prot |= PROT_WRITE; +} This

Taking a screenshot with translucent windows

2011-10-11 Thread Anthony Petrov
Hello, Our application needs to be able to take screenshots programmatically. Historically, we used an xwd-like approach for this - basically, drawing the entire windows hierarchy starting from the root window into a pixmap. However, on modern systems with translucency effects enabled, the

[PATCH v2] NEED_SYNC_REPLY flag should be in Xim not in Xic

2011-10-11 Thread Jeremy Huddleston
From: Choe Hwanjin choe.hwan...@gmail.com Because the focused Xic can be changed before sending sync reply. After focused Xic changed, the new Xic doesn't have NEED_SYNC_REPLY flag enabled, so libX11 doesn't send XIM_SYNC_REPLY packet. This patch adds sync reply flag to Xim and removes sync

[PATCH] Add option to use DPI calculated from EDID

2011-10-11 Thread Michal Suchanek
Hello, this is a patch that fixes a long-standing issue with the X server. The X server discards EDID information and sets DPI always to 96. This is a regression since 1.7 introduced in fff00d . There is no option to use the information obtained from EDID. Thanks Michal From

Re: [PATCH] Zero-fill holes in the buffer rather than leaving memory uninitialized.

2011-10-11 Thread Jeremy Huddleston
You are missing this in src/SMlibint.h: #include string.h string.h includes the prototype for memset(). Also, please add a Signed-off-by: line to your commit message along with my: Reviewed-by: Jeremy Huddleston jerem...@apple.com Once the above are fixed, please resend to the list, and I'll

Re: [PATCH] Add new compose sequences

2011-10-11 Thread Jeremy Huddleston
+Multi_key F i: ffi Ufb03 # LATIN SMALL LIGATURE FFI +Multi_key F l: ffl Ufb04 # LATIN SMALL LIGATURE FFL Why are those not f f i and f f l instead? On Oct 9, 2011, at 23:45, Marko Myllynen wrote: This patch adds few new compose sequences to the en_US

Re: [PATCH] Add option to use DPI calculated from EDID

2011-10-11 Thread Alex Deucher
On Tue, Oct 11, 2011 at 1:16 PM, Michal Suchanek hramr...@centrum.cz wrote: Hello, this is a patch that fixes a long-standing issue with the X server. The X server discards EDID information and sets DPI always to 96. This is a regression since 1.7 introduced in fff00d . There is no option

Re: [PATCH libpciaccess 2/2 v3] linux: Implement map_legacy

2011-10-11 Thread Jeremy Huddleston
Thanks Javier! Pushed. On Oct 11, 2011, at 00:46, Javier Pello wrote: On Fri, 2011-10-07 at 10:18:20 -0700, Jeremy Huddleston wrote: +char name[PATH_MAX]; +int flags = O_RDONLY; +int prot = PROT_READ; +int fd; +int ret=0; + +if (map_flags

Re: valgrind errors in X server

2011-10-11 Thread Peter Harris
On 2011-10-11 05:52, Michal Suchanek wrote: Hello, While valgrinding the X server to track a double free in one of my patches I found the errors below. ==3355== Conditional jump or move depends on uninitialised value(s) ==3355==at 0x6DCE290: inflateReset2 (in /usr/lib/libz.so.1.2.3.4)

Re: [PATCH] Add option to use DPI calculated from EDID

2011-10-11 Thread Matt Dew
On 10/11/2011 11:25 AM, Alex Deucher wrote: On Tue, Oct 11, 2011 at 1:16 PM, Michal Suchanekhramr...@centrum.cz wrote: Hello, this is a patch that fixes a long-standing issue with the X server. The X server discards EDID information and sets DPI always to 96. This is a regression since 1.7

Re: [PATCH] Add new compose sequences

2011-10-11 Thread James Cloos
JH == Jeremy Huddleston jerem...@apple.com writes: JH +Multi_key F i: ffi Ufb03 # LATIN SMALL LIGATURE FFI JH +Multi_key F l: ffl Ufb04 # LATIN SMALL LIGATURE FFL JH Why are those not f f i and f f l instead? I presume he used F in lieu of ff because ff leads to

Re: [PATCH] Add new compose sequences

2011-10-11 Thread Jeremy Huddleston
On Oct 11, 2011, at 11:47, James Cloos wrote: JH == Jeremy Huddleston jerem...@apple.com writes: JH +Multi_key F i : ffi Ufb03 # LATIN SMALL LIGATURE FFI JH +Multi_key F l : ffl Ufb04 # LATIN SMALL LIGATURE FFL JH Why are those not f f i and

Re: [PATCH] Add option to use DPI calculated from EDID

2011-10-11 Thread Michal Suchanek
On 11 October 2011 20:20, Matt Dew mar...@osource.org wrote: On 10/11/2011 11:25 AM, Alex Deucher wrote: On Tue, Oct 11, 2011 at 1:16 PM, Michal Suchanekhramr...@centrum.cz  wrote: Hello, this is a patch that fixes a long-standing issue with the X server. The X server discards EDID

Re: valgrind errors in X server

2011-10-11 Thread Michal Suchanek
On 11 October 2011 20:08, Peter Harris phar...@opentext.com wrote: On 2011-10-11 05:52, Michal Suchanek wrote: Hello, While valgrinding the X server to track a double free in one of my patches I found the errors below. ==3355== Conditional jump or move depends on uninitialised value(s)

Re: [PATCH] os: Remove Error()

2011-10-11 Thread Peter Hutterer
On Mon, Oct 10, 2011 at 08:54:54PM -0700, Jeremy Huddleston wrote: Signed-off-by: Jeremy Huddleston jerem...@apple.com Reviewed-by: Peter Hutterer peter.hutte...@who-t.net Cheers, Peter --- doc/Xserver-spec.xml | 10 +- include/os.h |1 - os/backtrace.c |

Re: PATCH document -background none option

2011-10-11 Thread Alan Coopersmith
On 10/ 7/11 05:41 AM, Michal Suchanek wrote: On 6 October 2011 23:39, Alan Coopersmithalan.coopersm...@oracle.com wrote: On 10/ 6/11 07:54 AM, Michal Suchanek wrote: this patch documents the option introduced in commit 8976e97. +++ b/hw/xfree86/man/Xorg.man Since the option is in the

[PULL ping/updated] BUS cleanup and pot pouri

2011-10-11 Thread Jeremy Huddleston
This is an updated [PULL] request for the BUS cleanup changes I've been wrangling as well as various misc bugfixes. As noted earlier, this [PULL] does *NOT* remove the xf86MapVidMem API as was done in my earlier set. Instead, this just deprecates the functionality that will be removed

Re: [PATCH v2] NEED_SYNC_REPLY flag should be in Xim not in Xic

2011-10-11 Thread Jeremy Huddleston
On Oct 11, 2011, at 18:34, Choe Hwanjin wrote: I've made my patch again against libX11 git master 22ba43d198dcca86c88eb15a56fc7d8fc47c422e. This patch already has changes that revert commit 1a1a42a3ca1dfaf42f1094936b71c140fc030fcb. I don't see any significant differences between your patch

Re: [PATCH] os: Remove Error()

2011-10-11 Thread Alan Coopersmith
On 10/10/11 08:54 PM, Jeremy Huddleston wrote: Signed-off-by: Jeremy Huddlestonjerem...@apple.com --- doc/Xserver-spec.xml | 10 +- include/os.h |1 - os/backtrace.c |5 +++-- os/log.c | 14 -- os/xprintf.c |5 +++-- 5

1.11.2 RC1 on Friday

2011-10-11 Thread Jeremy Huddleston
Please send any merge requests or nominations by Thursday for inclusion in 1.11.2 RC1. Thanks, Jeremy ___ 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] Export include/input.h:GetMaster() function

2011-10-11 Thread Peter Hutterer
On Tue, Oct 11, 2011 at 03:12:56PM +0200, Adam Tkac wrote: attached patch exports the GetMaster() function, it is needed by TigerVNC's libvnc.so module. Please merge it also to 1.11 release, thank you! where/how do you need this? afaict this function hasn't been exported since it was added

Re: [PATCH v2] NEED_SYNC_REPLY flag should be in Xim not in Xic

2011-10-11 Thread Choe Hwanjin
I've made my patch again against libX11 git master 22ba43d198dcca86c88eb15a56fc7d8fc47c422e. This patch already has changes that revert commit 1a1a42a3ca1dfaf42f1094936b71c140fc030fcb. I don't see any significant differences between your patch and mine. And your patch have some problem. See