[PATCH] XAA: Verify "pixmap" ops are really called on pixmaps before reading privates.

[Tiago Vignatti]

Jamey,

On Sun, Jul 04, 2010 at 05:44:40PM +0200, ext Jamey Sharp wrote:
> The pixmap ops certainly are used for pixmaps (if they aren't
> offscreen), but they're also used for windows when this screen doesn't
> have the VT semaphore. I assume that could lead to an privates assertion
> failure in xserver 1.9.
> 
> Signed-off-by: Jamey Sharp <jamey at minilop.net>
> ---
> I'd appreciate if somebody would test and confirm that xserver 1.9 does
> indeed assert-fail in this case, and report whether this patch fixes it.

I was trying to find the reason of a pixmap leak that I'm still seeing with
XAA when the server regenerates and thought that this patch could magically
solves it :) So, I tried the following:

- started a server
- chvt 1
- start xterm 
- chvt 7   # default display number


The results: mga stills hanging somewhere and VT switch is broken for me with
and without you patch (it's worse. Very like it's related with VGA
arbitration. Sigh). nv driver works nicely with and without your patch.

So at least in practice, your patch doesn't seems to bring anything better for
me. It doesn't regress either.


Thanks,
             Tiago