Re: How to edit Drawable/XdbeBackBuffer data?...

2012-08-21 Thread Aaron Plattner
On 08/21/2012 03:56 PM, Lee Fallat wrote: Alright, well I've decided to take another approach. I'm XGetImage()ing the root window, then I darken my pixels, then, I go to store the data in a Drawable via XPutImage, but this Drawable isn't retaining the data, it's just white when I XCopyArea()

Re: Is it possible to force a monitor to stay in the server layout?

2012-12-21 Thread Aaron Plattner
On 12/21/12 06:29, Sven Berkvens-Matthijsse wrote: Hello, I can't seem to find an answer to my question anywhere on the web. Either my use case is weird, or I'm not searching correctly :-) I have X.org server 1.11.3 running on Ubuntu. I'm using the radeon driver with a Radeon HD6450. I have

Re: X display locking

2013-03-07 Thread Aaron Plattner
On 03/07/2013 06:31 AM, Torsten Jager wrote: Hello! What is the proper usage of XLockDisplay () / XUnlockDisplay () when an application has multiple threads using * normal Xlib functions * Xitk functions * libGL and/or * libvdpau ? XLockDisplay / XUnlockDisplay is only required

[ANNOUNCE] xrandr 1.4.3

2014-08-02 Thread Aaron Plattner
- --verbose is specified, adds the ability for the -x and -y switches to be used to undo previous reflections, and adds the missing --brightness option to the - --help usage summary. Aaron Plattner (1): xrandr 1.4.3 Connor Behan (2): Remove duplicate printing of the axis Allow -x

Re: using uinput connect X to proprietary (TCP based) keyboard endpoint

2014-10-23 Thread Aaron Plattner
On 10/16/2014 08:12 AM, Peter Hutterer wrote: On Thu, Oct 16, 2014 at 04:48:37AM +0200, arne.ad...@t-online.de wrote: Hi, I am trying to integrate a proprietary keyboard, sending linux scancodes via TCP. My idea is to use uinput to forward the received keycodes to locally running applications

Re: how can I identify which video card on a mutli-card linux box a display belongs to

2014-11-21 Thread Aaron Plattner
On 11/10/2014 12:56 AM, Chris Wilson wrote: On Mon, Nov 10, 2014 at 04:31:59PM +0800, Zhang Fan wrote: Hi all, I'm developing a video matrix system for Linux. Typically multiple vendors' video card and/or multiple cards of same vendor/model might be used in one system. I can list the display

Re: combining Render's Composite + SHM's PutImage

2015-04-16 Thread Aaron Plattner
On 04/13/2015 02:33 AM, Nigel Tao wrote: On Fri, Apr 10, 2015 at 10:23 PM, Nigel Tao nigel...@golang.org wrote: Even where SHM CreatePixmap works, I can only seem to create a depth-24 pixmap, which defeats the purpose of alpha-blending if the shared-memory image's alpha channel is implicitly

Re: libxrandr XRRGetCrtcInfo() misinterpretation by nouveau or NVIDIA?

2015-06-19 Thread Aaron Plattner
On 06/19/2015 03:49 AM, Thomas Richter wrote: Am 19.06.2015 um 12:38 schrieb Chris Wilson: Why is it that in the presence of randr1.3 XRRGetCrtcInfo() returns semantically(!) different information than XRRGetCrtcInfo() with randr1.2 only? In the former case, it returns the entire panning

Re: xf86-video-dummy: resize to exact resolution

2016-06-15 Thread Aaron Plattner
On 06/14/2016 03:53 PM, Erik Jensen wrote: > When using Xorg with the dummy driver to host a virtual session for > remote access, it is often desirable to resize the session to exactly > match the resolution of the client. While the dummy driver supports > switching between predefined resolutions,

Re: XCompositeNameWindowPixmap vs using window directly

2016-02-10 Thread Aaron Plattner
On 02/09/2016 12:48 PM, adlo wrote: > I am writing a window switcher application using GTK, Cairo, and Xlib that > shows previews of windows. > > Assuming that I don't want to save the window's image for later use, what are > the advantages of using XCompositeNameWindowPixmap () compared to

[ANNOUNCE] xrandr 1.5.0

2016-02-23 Thread Aaron Plattner
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 xrandr is a command line interface to the X11 Resize, Rotate, and Reflect (RandR) extension. This release adds support for the new monitor objects added in RandR 1.5, and fixes a few bugs. Aaron Plattner (2): Split verbose mode printing

Re: Diagnosing first vs subsequent performance

2016-01-20 Thread Aaron Plattner
My guess is that each X server you start is switching to its own VT. Since you're running Xorg by itself, there are initially no clients connected. When you run an application such as glxinfo that exits immediately, or kill your copy of glxgears, it causes the server to reset, which makes it

Re: Corrupted XImage retrieved from a Window area

2016-08-02 Thread Aaron Plattner
I think I found the problem and sent a patch: https://lists.x.org/archives/xorg-devel/2016-August/050544.html https://patchwork.freedesktop.org/patch/102574/ On 08/01/2016 07:44 AM, Fabien Lelaquais wrote: Thansk again to you, Thomas, and Carsten. I suspect I still was not clear enough. My

Re: Getting help debugging X client crashes

2016-08-09 Thread Aaron Plattner
On 08/08/2016 03:01 AM, Michael Titke wrote: > On 05/08/2016 20:17, Matt Lauria wrote: >> >> Can someone direct me where to get help tracking down a bug in X? >> >> >> >> I’ve built a GUI (python3.4 using tkinter running gnome on RHEL 6.8) >> which crashes daily with a: >> >> >> >> X Error of

Re: RandR 1.5 Monitors: "No monitor named '...'"

2016-09-14 Thread Aaron Plattner
On 09/11/2016 08:29 PM, Nathan Schulte wrote: > I'm using X.org w/ Debian Sid: > >> nmschulte@desmas-l:~$ Xorg -version >> >> X.Org X Server 1.18.4 >> Release Date: 2016-07-19 >> X Protocol Version 11, Revision 0 >> Build Operating System: Linux 3.16.0-4-amd64 x86_64 Debian >> Current Operating

Re: XOpenDisplay call sequence

2016-11-29 Thread Aaron Plattner
On 11/28/2016 01:31 PM, Krzywicki, Alan wrote: > > So if I follow the XOpenDisplay sequence up the stack I see > xcb_connect() / _/xcb_open/_abstract( ) trying to open > “/tmp/.X11-unix/X0” with protocol set to 0. On one system it > eventually calls select(), on another it uses poll() instead,

Re: RandR 1.5 "Monitors" and splitting a single physical display

2017-03-15 Thread Aaron Plattner
-Stream Transport disabled). You're not going to be able to rotate one monitor with the existing protocol. Rotation happens at the crtc, not the output or monitor. > I ran into issues trying to do setup multiple Monitors for an Output, > and sent a mail to the list in September 2016; Aaro

Re: X running on target is picking drivers from wrong directory

2017-07-19 Thread Aaron Plattner
On 07/18/2017 11:49 PM, abhijit wrote: > Hi, > > I am trying to build xserver 1.17.1 for imx57 based board. But cross > compiled executable is trying to pick libraries and drivers from NFS > root file system path. > > Following is the configuration command, > > ./configure

Re: Xorg VRAM leak because of Qt/OpenGL Application

2018-07-02 Thread Aaron Plattner
On 07/02/2018 09:22 AM, Dennis Clarke wrote: > On 07/01/2018 10:11 PM, Mathieu Westphal wrote: >> Hello list, >> >> I am working on a complex Qt/OpenGL Application. >> Xorg starts leaking in VRAM when i'm using the application and never >> release the memory, until I restart X of course. >> >> $

Re: xorg.0.log repetitive entries

2017-10-20 Thread Aaron Plattner
On 10/20/2017 09:30 AM, Adam Jackson wrote: On Thu, 2017-10-19 at 20:29 -0300, sawb...@gmx.net wrote: For example, these lines accounting for the mode setting of each one of my three monitors: (II) NVIDIA(0): Setting mode "CRT-0:1280x1024" (II) NVIDIA(1): Setting mode "DFP-0:1280x1024" (II)

Re: xorg.conf Settings Being Overridden By nvidia-auto-select

2018-01-31 Thread Aaron Plattner
Hi Junk, This indicates that some X client (probably your desktop environment) is overriding the mode set in xorg.conf with its own. -- Aaron On 01/31/2018 12:52 PM, Junk Moody wrote: I'm having trouble with nvidia-auto-select apparently overriding my settings in xorg.conf. I used

Re: xorg disable GPU acceleration

2018-10-22 Thread Aaron Plattner
On 10/14/18 11:37 PM, Łukasz Maśko wrote: > Dnia sobota, 13 października 2018 09:22:16 Иван Талалаев pisze: > [...] >> Do you know how to disable xorg hardware acceleration? > > If you don't need hardware acceleration, maybe you could use the generic VESA > driver instead of the NVidia one? >

Re: Xorg.O.log (WW) message

2018-09-26 Thread Aaron Plattner
On 9/24/18 3:08 PM, sawb...@gmx.net wrote: Hello: I have this message in my Xorg.O.log: [code] [34.298] (WW) Unresolved symbol: fbGetGCPrivateKey [/code] I'm running Devuan ASCII with two NVidia caards for three monitors and using proprietary drivers. What version of the X server are

Re: what does "+"(preferred) means in xrandr?

2018-11-26 Thread Aaron Plattner
"preferred" generally means that the Extended Display Information Data (EDID) from the monitor indicates that that mode most closely matches the native timings of the display. All modes listed by RandR for a given monitor should work with that monitor, but the preferred mode should work best (for

Re: support question regarding X11 1.20.1

2019-06-24 Thread Aaron Plattner
On 6/19/19 2:10 PM, Greenamyer, Shaun wrote: > Hey guys, > > I hope this message isn’t an annoyance but I’m not sure the best place > to turn right now. > > I am having an issue with my X windows not getting “erased” when the > window is closed. I’m probably not explaining well which is why my >

Re: Can X11 return an image with an NVIDIA GPU memory address?

2019-08-12 Thread Aaron Plattner
On 8/3/19 10:09 AM, Suhail Doshi wrote: Hi there, My goal is to try get the frame of a desktop to do low-latency remote desktop. I am interested in using X11 as the window manager. When I run nvidia-smi, I noticed that X11 is a process that interacts the GPU:

Re: Dummy video driver plea

2020-05-11 Thread Aaron Plattner
On 5/11/20 5:33 AM, Mgr. Janusz Chmiel wrote: Please, if I will only use dummi video driver and X11vnc. Does making setting monitor frequency to 80 Hertz some sense? Or it will not increase The app responsiveness performance at all? Because VNC protocol is being used? The only parts of the

Re: How to switch to a particular resolution? Xorg complains "No valid modes for "DFP-1:2560x1600"; removing." and ignores my setting.

2020-09-23 Thread Aaron Plattner
On 9/23/20 1:35 PM, Yuri wrote: On 2020-09-23 08:21, Pete Wright wrote: $ xrandr |grep 2560 Interestingly, this command shows nothing after Xorg was started, but shows 2560x1600 after the "NVidia settings" program changes the resolution to 2560x1600. I just need to find a way, if any, to

Re: How to switch to a particular resolution? Xorg complains "No valid modes for "DFP-1:2560x1600"; removing." and ignores my setting.

2020-09-24 Thread Aaron Plattner
On 9/23/20 11:31 PM, Yuri wrote: But how does it work when the NVidia utility switches it to this mode then? Does the output look blurry or fuzzy when that happens? I wonder if you're getting a 2560x1600 desktop scaled down to 1920x1080. You should be able to tell for sure by looking at the

Re: Feature request, but must be universallly accepted by ALL blanker authors

2020-10-02 Thread Aaron Plattner
On 10/2/20 1:17 PM, Dan Arena wrote: Gene, Following up more about xfce4, you should be able to go into their Settings Manager and you can turn off the Screensaver and uncheck Power Management under the Advanced tab. You will still want to add the lines I mentioned before into

Re: How to force Xorg/Firefox to use iGPU instead of discrete nvidia card on a Clevo laptop?

2021-03-24 Thread Aaron Plattner
Mesa lumps all of its drivers together under one vendor named "mesa" so you are supposed to be able to do something like this: DRI_PRIME=1 __GLX_VENDOR_LIBRARY_NAME=mesa glxinfo | grep vendor Unfortunately, Mesa doesn't like something about the GLX fbconfigs the NVIDIA driver provides, so it

Re: XrandR Verify Active Rate

2021-03-29 Thread Aaron Plattner
On 3/27/21 4:35 AM, re.mcclue wrote: A |XRRScreenResources| contains multiple |XRRModeInfo| and |RROutput| (which with |XRRGetOutputInfo()| effectively means |XRROutputInfo|). Therefore, a screen can have many outputs which in turn, can have many modes. I can restrict myself to only

Re: xorg.conf question

2021-03-04 Thread Aaron Plattner
These are NVIDIA-specific options so they're kinda offtopic here, but I'll try to address them below. You might want to consider posting on the NVIDIA forum [1] or by emailing linux-b...@nvidia.com

Re: XInitThreads multiple times

2021-08-11 Thread Aaron Plattner
On 8/5/21 8:36 AM, Keith Packard wrote: Dawid Kowalczyk writes: Hello, Is it possible to call |XInitThreads| multiple times, for example 20 times and not worry who calls it first? XInitThreads isn't re-entrant, so you need to ensure that it isn't getting invoked by multiple threads in

Re: [PATCH] randr: Add 'set' fields to SetCrtcConfigs request

2011-02-23 Thread Aaron Plattner
; - goto sendReply; } rep.status = RRSetConfigSuccess; scr_priv-lastSetTime = currentTime; -- 1.7.2.3 Other than the protocol doc comments and the existing bug noted above, Reviewed-by: Aaron Plattner aplatt...@nvidia.com However, I do need to object strongly to such a major

Re: [PATCH xf86-video-nv] Fix compiler warning.

2011-03-01 Thread Aaron Plattner
On Tue, Mar 01, 2011 at 05:09:10PM -0800, Cyril Brulebois wrote: Get rid of this with CFLAGS=-Wall -Werror: | CC g80_display.lo | cc1: warnings being treated as errors | g80_display.c: In function ‘G80CrtcSetPClk’: | g80_display.c:216: error: unused variable ‘i’ Signed-off-by: Cyril

Re: Resend: [PATCH xev] Show RandR events.

2011-03-03 Thread Aaron Plattner
On Wed, Feb 16, 2011 at 04:00:25AM -0800, Michael Thayer wrote: Signed-off-by: Michael Thayer michael.tha...@oracle.com --- [Resending this as there were no responses the first time, but I don't know whether it was due to lack of interest or whether it just went down in the flood.] I think

Re: Resend: [PATCH xev] Show RandR events.

2011-03-03 Thread Aaron Plattner
On Thu, Mar 03, 2011 at 01:35:15PM -0800, Michael Thayer wrote: On Thu, 2011-03-03 at 11:37 -0800, Aaron Plattner wrote: On Wed, Feb 16, 2011 at 04:00:25AM -0800, Michael Thayer wrote: +if (rr_major = 1 rr_minor = 2) This check seems wrong, since it will include these event

Re: Resend: [PATCH xev] Show RandR events.

2011-03-03 Thread Aaron Plattner
On Thu, Mar 03, 2011 at 02:08:49PM -0800, Michael Thayer wrote: On Thu, 2011-03-03 at 13:54 -0800, Aaron Plattner wrote: The updated change looks good to me. Reviewed-by: Aaron Plattner aplatt...@nvidia.com Tested-by: Aaron Plattner aplatt...@nvidia.com Do you have a freedesktop.org

Re: [PATCH] mi: Remove unused overlay support

2011-03-29 Thread Aaron Plattner
On Mon, Mar 28, 2011 at 11:36:04PM -0700, Mikhail Gusarov wrote: The only reference to it in server and free drivers is in XAA overlay code which would segfault as no miInitOverlay is called ever. Quoting Jamey Sharp: It wasn't obvious to me that the patch was correct, because you

Re: [PATCH] mi: Remove unused overlay support

2011-03-29 Thread Aaron Plattner
On Tue, Mar 29, 2011 at 09:47:19AM -0700, Mikhail Gusarov wrote: Twas brillig at 09:26:19 29.03.2011 UTC-07 when aplatt...@nvidia.com did gyre and gimble: AP NAK. This code is not NVIDIA-specific just because only the NVIDIA AP driver currently uses it. I did not claim it. I just said

Re: [PULL XTS] Misc fixes

2011-03-29 Thread Aaron Plattner
in the git repository at: git://people.freedesktop.org/~whot/xts.git master Peter Hutterer (21): Remove superfluous (void) casts. Remove EXPECT_NOTHING pointer to int cast. Purge wbzero - use memset instead. These three Reviewed-by: Aaron Plattner aplatt...@nvidia.com

Re: [PATCH 1/2] render: Move miCompositeRects() into fb as fbCompositeRects().

2011-03-31 Thread Aaron Plattner
On Wed, Mar 30, 2011 at 09:37:21AM -0700, Søren Sandmann wrote: From: Søren Sandmann Pedersen s...@redhat.com It is not used anywhere else. Signed-off-by: Soren Sandmann s...@redhat.com --- [...] +pixman_image_fill_rectangles ( + op, image, (pixman_color_t *)color, nRect, +

Re: [PATCH] [RFC] dix: don't use CopyWindow in driver level interface.

2011-03-31 Thread Aaron Plattner
On Wed, Mar 30, 2011 at 10:21:42PM -0700, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com This splits CopyWindow before the exa/fb layers, and uses a new interface called PixmapCopyRegion to do the actual copy. The main point of this is a step towards removing WindowPtr's from the

Re: [PATCH] [RFC] dix: don't use CopyWindow in driver level interface.

2011-03-31 Thread Aaron Plattner
On Thu, Mar 31, 2011 at 02:14:03PM -0700, Dave Airlie wrote: On Fri, Apr 1, 2011 at 5:05 AM, Aaron Plattner aplatt...@nvidia.com wrote: On Wed, Mar 30, 2011 at 10:21:42PM -0700, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com This splits CopyWindow before the exa/fb layers

Re: [PATCH] [RFC] dix: don't use CopyWindow in driver level interface.

2011-03-31 Thread Aaron Plattner
On Thu, Mar 31, 2011 at 05:01:20PM -0700, Keith Packard wrote: On Thu, 31 Mar 2011 15:49:36 -0700, Aaron Plattner aplatt...@nvidia.com wrote: Would this be multiple backing pixmaps like how James envisioned, or something else? The problem here is that while pWin may be in the overlay

Re: [PATCH] [RFC] dix/mi: remove ChangeWindowAttributes from rendering path.

2011-04-04 Thread Aaron Plattner
On Wed, Mar 30, 2011 at 11:07:46PM -0700, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com This moves ChangeWindowAttributes to mi, and calls a new hook that fb/exa can use to fixup the pixmap properly. I've looked at XAA and it seems this should work there as well, and I'm sure

Re: [PATCH] [RFC] dix/mi: remove ChangeWindowAttributes from rendering path.

2011-04-04 Thread Aaron Plattner
On Mon, Apr 04, 2011 at 01:24:10PM -0700, Dave Airlie wrote: Are you planning on leaving the ChangeWindowAttrbutes screen hook there, or is it slated for deletion?  I ask because we currently wrap CWA so we can watch for colormap changes on PseduoColor windows. Can you give me a why?

Re: [PATCH] [RFC] dix/mi: remove ChangeWindowAttributes from rendering path.

2011-04-04 Thread Aaron Plattner
On Mon, Apr 04, 2011 at 02:26:42PM -0700, Dave Airlie wrote: On Tue, Apr 5, 2011 at 6:42 AM, Aaron Plattner aplatt...@nvidia.com wrote: On Mon, Apr 04, 2011 at 01:24:10PM -0700, Dave Airlie wrote: Are you planning on leaving the ChangeWindowAttrbutes screen hook there, or is it slated

[PATCH] Decouple the test suite from make check

2011-04-05 Thread Aaron Plattner
host X server, which is bad because some of the tests can have adverse effects. Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- Makefile.am |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/Makefile.am b/Makefile.am index b8de497..87040d2 100644 --- a/Makefile.am

Splitting protocol screens from the driver interface

2011-04-06 Thread Aaron Plattner
Hi Dave, I like the idea of separating the driver from the protocol screens, and am interested in helping out. I think it might be easiest in the short term to let the overlay functionality I talked about break and then I can add it back later in whatever form makes the most sense with the new

Re: Splitting protocol screens from the driver interface

2011-04-11 Thread Aaron Plattner
On Wed, Apr 06, 2011 at 08:51:52PM -0700, Dave Airlie wrote: On Thu, Apr 7, 2011 at 10:59 AM, Aaron Plattner aplatt...@nvidia.com wrote: Hi Dave, I like the idea of separating the driver from the protocol screens, and am interested in helping out.  I think it might be easiest

[PATCH XTS 0/2] Fix 'missing sentinel in function call' warnings

2011-04-12 Thread Aaron Plattner
mc. Aaron Plattner (2): mc: Add the ability to pass literal NULL arguments to XCALL functions xts5: Fix 'missing sentinel in function call' warnings by using literal NULL xts5/Xlib14/XGetICValues.m|2 +- xts5/Xlib14/XGetIMValues.m|2 +- xts5/Xlib14/XSetICFocus.m

[PATCH XTS 1/2] mc: Add the ability to pass literal NULL arguments to XCALL functions

2011-04-12 Thread Aaron Plattner
Check for NULL in the argument list at the top of the .m file. If found, skip creating a declaration or initializer for it but do include it in the argument list when emitting the code for XCALL. Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- xts5/src/bin/mc/code.c |8 xts5

[PATCH XTS 2/2] xts5: Fix 'missing sentinel in function call' warnings by using literal NULL

2011-04-12 Thread Aaron Plattner
= NULL; function(..., endlist); While this is perfectly valid, GCC warns about it anyway. Switch these to using literal NULLs too. Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- xts5/Xlib14/XGetICValues.m|2 +- xts5/Xlib14/XGetIMValues.m|2 +- xts5/Xlib14

Re: [PATCH xf86-video-nv] remove duplicated includes

2011-04-25 Thread Aaron Plattner
On Mon, Apr 25, 2011 at 04:26:33AM -0700, Nicolas Kaiser wrote: Remove duplicated includes of guarded headers. Signed-off-by: Nicolas Kaiser ni...@nikai.net Merged: 2b17d1a..4b03459 master - master ___ xorg-devel@lists.x.org: X.Org development

[PATCH] randr: check rotated virtual size limits correctly

2011-05-11 Thread Aaron Plattner
Serial number of failed request: 14 Current serial number in output stream: 14 Fix this by moving the check above the code that swaps the dimensions based on the rotation. Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- This patch is against server-1.10-branch. randr/rrscreen.c

Re: Howto determine screen geometry?

2011-05-23 Thread Aaron Plattner
On Sun, May 22, 2011 at 09:20:31AM -0700, Stef Bon wrote: On 05/22/2011 04:23 PM, Mikhail Gusarov wrote: Twas brillig at 16:19:49 22.05.2011 UTC+02 when s...@bononline.nl did gyre and gimble: SB Isn't it possible to just get that information from the Xserver?? xrandr + awk

[PATCH XTS] xts5: BadDevice takes an int*, not an XID*

2011-05-24 Thread Aaron Plattner
startup.c: In function ‘init_xinput’: startup.c:505: error: passing argument 2 of ‘_xibaddevice’ from incompatible pointer type /usr/include/X11/extensions/XInput.h:162: note: expected ‘int *’ but argument is of type ‘XID *’ Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- xts5/src/lib

[PATCH XTS] tet: tet_getvar needs to take a const char* argument

2011-05-24 Thread Aaron Plattner
config.c: In function ‘initconfig’: config.c:394: error: passing argument 1 of ‘_initconfig’ from incompatible pointer type config.c:328: note: expected ‘char * (*)(const char *)’ but argument is of type ‘char * (*)(char *)’ Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- I'm thinking

[PATCH v2] randr: check rotated virtual size limits correctly

2011-05-24 Thread Aaron Plattner
Serial number of failed request: 14 Current serial number in output stream: 14 Fix this by moving the check above the code that swaps the dimensions based on the rotation. Signed-off-by: Aaron Plattner aplatt...@nvidia.com Reviewed-by: Jeremy Huddleston jerem...@apple.com Tested-by: Robert

Re: xts warning fixes round 1 (was: [ANNOUNCE] xts 0.99.0)

2011-05-26 Thread Aaron Plattner
On Wed, May 25, 2011 at 03:57:04AM -0700, Cyril Brulebois wrote: Hi, Aaron Plattner aplatt...@nvidia.com (24/05/2011): Patches to fix warnings in the build or the tests themselves would be very much appreciated. […] w00t, an XTS release! Here's a first round with 16 patches

Re: [PATCH 2/4] Remove the cacheing of the last scratch PixmapRec

2011-06-08 Thread Aaron Plattner
On Sun, Jun 05, 2011 at 10:36:07PM -0700, Chris Wilson wrote: In order for the driver to be notified of when the resource backing the scratch pixmap becomes no longer accessible, it needs to be called on every FreeScratchPixmapHeader(). As we instead maybe cached the PixmapRec (to avoid the

Re: [PATCH 1/3] composite: Don't backfill non-bg-None windows

2010-10-04 Thread Aaron Plattner
On Mon, Oct 04, 2010 at 12:33:40PM -0700, Adam Jackson wrote: If there's a defined background then backfilling is a waste of effort, since exposure processing will paint that in for us. But note that we have to backfill if any children are bg=None to preserve semantics with non-composited

Re: ABI breakage

2010-10-14 Thread Aaron Plattner
On Thu, Oct 14, 2010 at 12:00:40PM -0700, Thomas Hellstrom wrote: On 10/14/2010 06:11 PM, Thomas Hellstrom wrote: Hi! Isn't the commit 5c6a2f93ebc16a78093782b442306de23ae94e78 xfree86: Kill pixmapPrivate with a vengeance (v2) a major ABI breakage (It alters the layout of the

Re: [PATCH 2/3] xfree86: Remove unused xf86AssertBlockedSIGIO

2010-10-14 Thread Aaron Plattner
On Thu, Oct 14, 2010 at 09:49:05AM -0700, Adam Jackson wrote: Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/os-support/shared/sigio.c | 11 --- hw/xfree86/os-support/shared/sigiostubs.c |5 - hw/xfree86/os-support/xf86_OSproc.h |1 - 3 files

Re: [PATCH 2/3] xfree86: Remove unused xf86AssertBlockedSIGIO

2010-10-14 Thread Aaron Plattner
On Thu, Oct 14, 2010 at 12:19:49PM -0700, Adam Jackson wrote: On Thu, 2010-10-14 at 12:12 -0700, Aaron Plattner wrote: On Thu, Oct 14, 2010 at 09:49:05AM -0700, Adam Jackson wrote: Signed-off-by: Adam Jackson a...@redhat.com --- hw/xfree86/os-support/shared/sigio.c | 11

[PATCH] Revert Set DamageSetReportAfterOp to true for the damage extension (#30260)

2010-10-17 Thread Aaron Plattner
This change does not solve the problem it claims to solve because it makes two assumptions about the driver that are not always true: 1. that it sends rendering requests to the GPU immediately, which can kill performance on some GPUs, and 2. that the GPU processes all requests from multiple

[PATCH] xts5: Bail out with UNTESTED for BAD_LENGTH requests that would have length 0

2010-10-17 Thread Aaron Plattner
request handling in the X server. This makes the server interpret the next request as the size part of the Bell request, and hilarity ensues. Instead, detect this case and bail out with an UNTESTED result. Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- pBell-2 and a lot of tests like

Re: [PATCH] Revert Set DamageSetReportAfterOp to true for the damage extension (#30260)

2010-10-20 Thread Aaron Plattner
Thanks for this discussion. More precise semantics are always better. Comments below: On Wed, Oct 20, 2010 at 11:05:29AM -0700, Keith Packard wrote: On Sun, 17 Oct 2010 09:58:50 -0700, Aaron Plattner aplatt...@nvidia.com wrote: This change does not solve the problem it claims to solve

Re: [PATCH] Revert Set DamageSetReportAfterOp to true for the damage extension (#30260)

2010-10-20 Thread Aaron Plattner
On Wed, Oct 20, 2010 at 01:54:35PM -0700, Keith Packard wrote: On Wed, 20 Oct 2010 11:35:46 -0700, Aaron Plattner aplatt...@nvidia.com wrote: By sending X protocol to explicitly fence the GL rendering with the X rendering, which is the whole purpose of James's XDamageSubtractAndTrigger

Re: [PATCH] Revert Set DamageSetReportAfterOp to true for the damage extension (#30260)

2010-10-20 Thread Aaron Plattner
On Wed, Oct 20, 2010 at 02:57:27PM -0700, Keith Packard wrote: On Wed, 20 Oct 2010 14:29:33 -0700, Aaron Plattner aplatt...@nvidia.com wrote: Then you're implementing what sounds like a nice vendor-specific feature. There isn't any other mechanism provided today to perform

Re: [PATCH] Revert Set DamageSetReportAfterOp to true for the damage extension (#30260)

2010-10-20 Thread Aaron Plattner
On Wed, Oct 20, 2010 at 03:03:59PM -0700, Aaron Plattner wrote: On Wed, Oct 20, 2010 at 02:57:27PM -0700, Keith Packard wrote: On Wed, 20 Oct 2010 14:29:33 -0700, Aaron Plattner aplatt...@nvidia.com wrote: The driver can do that by enabling ReportAfter from a damage create hook

Re: [PATCH] Revert Set DamageSetReportAfterOp to true for the damage extension (#30260)

2010-10-20 Thread Aaron Plattner
On Wed, Oct 20, 2010 at 03:49:47PM -0700, Keith Packard wrote: On Wed, 20 Oct 2010 15:03:59 -0700, Aaron Plattner aplatt...@nvidia.com wrote: Fine, but will you be willing to move this call to the drivers that need it when we have a real sync extension? I don't see the point

[PATCH] os: Fix BigReq ignoring when another request is pending

2010-10-25 Thread Aaron Plattner
-Xproto122 389 367 219 0 0 0 1 0 0 0 +Xproto122 389 368 219 0 0 0 0 0 0 0 Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- Sorry I screwed this up the first time. Do you think I should revert the first

Re: [PATCH] xfree86: Fix rotation of 2-color non-interleaved cursor images

2010-11-29 Thread Aaron Plattner
Does anyone else want to review this, or are Robert's (@nvidia.com) Reviewed-by and Cyril's Tested-by sufficient? On Tue, Nov 16, 2010 at 12:17:33PM -0800, Aaron Plattner wrote: When RandR 1.2's transformation code is enabled, it rotates the cursor image so that it appears upright on a rotated

Re: rendercheck segfaults [Re: rendercheck: 10 commits]

2010-12-01 Thread Aaron Plattner
On Wed, Dec 01, 2010 at 01:47:47PM -0800, Chris Wilson wrote: On Wed, 1 Dec 2010 13:06:47 -0800, Aaron Plattner aplatt...@nvidia.com wrote: This series causes a segmentation fault in blend_test because the loop at the top of the while (k num_dst) loop doesn't ever increment y because

Re: rendercheck segfaults [FIXED]

2010-12-01 Thread Aaron Plattner
On Wed, Dec 01, 2010 at 02:27:16PM -0800, Chris Wilson wrote: On Wed, 1 Dec 2010 13:57:15 -0800, Aaron Plattner aplatt...@nvidia.com wrote: On Wed, Dec 01, 2010 at 01:47:47PM -0800, Chris Wilson wrote: On Wed, 1 Dec 2010 13:06:47 -0800, Aaron Plattner aplatt...@nvidia.com wrote

Re: [PATCH damageproto (rev3) 1/3] Document changes in damage proto version 1.2

2010-12-02 Thread Aaron Plattner
On Thu, Dec 02, 2010 at 05:02:49PM -0800, Keith Packard wrote: On Thu, 2 Dec 2010 16:58:58 -0800, James Jones jajo...@nvidia.com wrote: OK, I've pushed the requested updates to proto and lib with your reviewed-by line to my github repositories. Once I get a review for the Xext changes,

Re: [PATCH xextproto (rev4) 0/5] XSync Fence Objects

2010-12-03 Thread Aaron Plattner
On Thu, Dec 02, 2010 at 09:15:30AM -0800, James Jones wrote: Changes since the last version: -Rebased at fd8a26edefc53b370c554a60c75ff32fc60b99c8 -Added version bump so other packages can check for fence support Adds support for binary sync objects. Objects are set to triggered using X

Re: [PATCH libXext (rev3) 0/5] XSync Fence Objects, lib portion

2010-12-03 Thread Aaron Plattner
On Thu, Dec 02, 2010 at 09:26:26AM -0800, James Jones wrote: Changes since last version: -Rebased at d35e95fdf433a249d0293744d3e1ef6196422700 -Added xextproto version check. Adds client library support for binary sync objects added in the X Synchronization protocol version 3.1. libXext

Re: Sprite transforms in RandR

2010-12-05 Thread Aaron Plattner
master There are some unrelated whitespace changes, and a tab-space-tab pattern in the argument list for XRRSetCrtcConfigs. xrandr changes: git://people.freedesktop.org/~keithp/xrandr master I just skimmed this. It seemed okay. Acked-by: Aaron Plattner aplatt...@nvidia.com xserver

Re: Sprite transforms in RandR

2010-12-05 Thread Aaron Plattner
On Sat, Dec 04, 2010 at 08:34:40PM -0800, Keith Packard wrote: Would it be useful to be able to return more information from the server to the application? How would such information get encoded? Unless it's very generic or just a string to present to the user, I don't think trying to encode

Re: Sprite transforms in RandR

2010-12-05 Thread Aaron Plattner
On Sun, Dec 05, 2010 at 06:15:04PM -0800, Keith Packard wrote: On Sun, 5 Dec 2010 16:38:52 -0800, Aaron Plattner aplatt...@nvidia.com wrote: Thanks for reviewing over the weekend. There are a few whitespace bugs, such as this: ┌─── RRQueryScanoutPixmaps \twindow: WINDOW

Re: [PATCH] RandR version 1.4 additions

2010-12-06 Thread Aaron Plattner
On Sun, Dec 05, 2010 at 08:43:07PM -0800, Keith Packard wrote: On Sun, 5 Dec 2010 16:38:52 -0800, Aaron Plattner aplatt...@nvidia.com wrote: This adds the specification and protocol header definitions for the RandR 1.4 protocol changes, including * Per crtc pixmaps and pixmap origins

Re: Sprite transforms in RandR

2010-12-06 Thread Aaron Plattner
On Sun, Dec 05, 2010 at 08:47:15PM -0800, Keith Packard wrote: On Sun, 5 Dec 2010 16:38:52 -0800, Aaron Plattner aplatt...@nvidia.com wrote: The crtc shadow functions seem obsolete given the new scanout pixmap creation function, but they can be removed in a later ABI. So, I

Re: Sprite transforms in RandR

2010-12-06 Thread Aaron Plattner
On Sun, Dec 05, 2010 at 06:17:12PM -0800, Keith Packard wrote: On Sun, 5 Dec 2010 16:42:43 -0800, Aaron Plattner aplatt...@nvidia.com wrote: Unless it's very generic or just a string to present to the user, I don't think trying to encode this in the protocol is going to be reasonable. We

Re: Sprite transforms in RandR

2010-12-06 Thread Aaron Plattner
On Mon, Dec 06, 2010 at 08:44:41AM -0800, Keith Packard wrote: On Mon, 6 Dec 2010 15:20:01 +, Daniel Stone dan...@fooishbar.org wrote: Nope, we don't have any per-CRTC differences there, it's just that rotation is essentially another property of the mode that we need to know about in

Re: RandR 1.4 additional X server patches

2010-12-06 Thread Aaron Plattner
out earlier, plus I guess the two at git://people.freedesktop.org/~keithp/xserver master that I can't find emails for Reviewed-by: Aaron Plattner aplatt...@nvidia.com The change to decouple the screen and screen pixmap sizes scares me. We're going to have to go through our driver with a fine

Re: RandR 1.4 additional X server patches

2010-12-06 Thread Aaron Plattner
On Mon, Dec 06, 2010 at 02:44:08PM -0800, Keith Packard wrote: On Mon, 6 Dec 2010 14:36:43 -0800, Aaron Plattner aplatt...@nvidia.com wrote: The change to decouple the screen and screen pixmap sizes scares me. We're going to have to go through our driver with a fine-toothed comb to make

Re: [PATCH 3/3] xfree86: Fix the sdk headers to be multilib-safe

2011-01-07 Thread Aaron Plattner
On Fri, Jan 07, 2011 at 09:49:44AM -0800, Daniel Stone wrote: On Fri, Jan 07, 2011 at 09:41:19AM -0800, Alan Coopersmith wrote: On 01/ 7/11 03:10 AM, Daniel Stone wrote: On Fri, Jan 07, 2011 at 12:51:17AM +0100, Mark Kettenis wrote: diff --git a/include/colormapst.h b/include/colormapst.h

Re: [REVIEW] DMX2 DIX merge

2010-01-07 Thread Aaron Plattner
On Thu, Jan 07, 2010 at 07:04:45AM -0800, Adam Jackson wrote: On Wed, 2010-01-06 at 17:26 -0800, Keith Packard wrote: On Wed, 06 Jan 2010 17:39:15 -0500, Adam Jackson a...@nwnk.net wrote: But that's still kind of gross, since your per-screen pixmaps are still going to end up in VRAM, and

[PATCH] x86emu: Respect the LEA 67h address size prefix.

2010-01-11 Thread Aaron Plattner
From: Christian Zander chzan...@nvidia.com Signed-off-by: Christian Zander chzan...@nvidia.com Signed-off-by: Aaron Plattner aplatt...@nvidia.com --- This change fixes console restore on some newer NVIDIA boards. Tiago, you'll want this in libx86 once it's reviewed. hw/xfree86/x86emu/ops.c

[PATCH] DRI2: Allow multiple driver names.

2010-01-13 Thread Aaron Plattner
Each driver type (e.g. DRI2DriverDRI or DRI2DriverVDPAU) can have a name in the driverNames array in DRI2InfoRec. DRI2Connect returns the name for the driver specified by driverType. Also print names of supported drivers in DRI2ScreenInit. Signed-off-by: Aaron Plattner aplatt...@nvidia.com

Re: [PATCH] DRI2: Allow multiple driver names.

2010-01-19 Thread Aaron Plattner
On Wed, Jan 13, 2010 at 01:58:05PM -0800, Keith Packard wrote: On Wed, 13 Jan 2010 11:35:52 -0800, Aaron Plattner aplatt...@nvidia.com wrote: Each driver type (e.g. DRI2DriverDRI or DRI2DriverVDPAU) can have a name in the driverNames array in DRI2InfoRec. DRI2Connect returns the name

Re: [PATCH] xfree: dri2: add AuthMagic hook for driver side support

2010-01-23 Thread Aaron Plattner
On Fri, Jan 22, 2010 at 09:04:26AM -0800, Tiago Vignatti wrote: With this new hook drmAuthMagic becomes useless and should be deprecated. You might want to implement AuthMagic on driver side instead. Attention: ABI being break. I'll leave the review to the DRI2 experts, but I think you

[PATCH] DRI2: Bump the dri2 module version to 1.2.0 to reflect recent API changes.

2010-01-28 Thread Aaron Plattner
Signed-off-by: Aaron Plattner aplatt...@nvidia.com Acked-by: Jesse Barnes jbar...@virtuousgeek.org --- This was reviewed by Jesse. Keith, please apply. hw/xfree86/dri2/dri2.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86/dri2/dri2

Re: [PATCH 0/2] RENDER: Fix gradient and solid fill pictures with Xinerama, and misc cleanup

2010-01-29 Thread Aaron Plattner
Keith, is my review on these sufficient, or do you need sombody outside NVIDIA to review them too? On Thu, Jan 21, 2010 at 12:38:35PM -0800, Robert Morell wrote: This series of patches against the RENDER extension provides two things: 1. Misc cleanup of use of improper types in size calculation

Re: [xserver patch v4 1/5] Add a PreConfigureWindow hook.

2010-02-09 Thread Aaron Plattner
On Mon, Feb 08, 2010 at 10:25:12AM -0800, Francisco Jerez wrote: Executed from the ConfigureWindow request, right before sending ConfigureNotify to the clients. This commit breaks the ScreenRec ABI. Signed-off-by: Francisco Jerez curroje...@riseup.net --- dix/window.c

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

2010-02-23 Thread Aaron Plattner
On Tue, Feb 23, 2010 at 10:15:31PM -0800, Oliver McFadden wrote: This is very similar to the RunFromSmartParent option, except we do not send the signal to our parent process, but our own process instead; also the signal is SIGSTOP, not SIGUSR1. Upstart will detect that our process has

  1   2   3   4   5   6   >