[PATCH] EXA/mixed: Don't consider devKind for detecting dimension change.
Keith Packard
keithp at keithp.com
Wed Dec 9 15:14:38 PST 2009
On Thu, 10 Dec 2009 00:04:53 +0100, Michel Dänzer <michel at daenzer.net> wrote:
> From: Michel Dänzer <daenzer at vmware.com>
>
> Fixes http://bugs.freedesktop.org/show_bug.cgi?id=25536 .
>
> There should be no reason for the pitch to actually change when none of the
> other dimensions do, and this could lead to incorrectly freeing the system
> memory copy when allocating a GPU copy, resulting in loss of valid pixmap
> contents.
One can imagine the driver wanting to change padding requirements to hit
some different tiling mode; is this not a reasonable thing to account
for?
And, of course, why would someone send a devKind that wasn't the same
without intending to change it? Sure seems like you're masking a bug in
other code.
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20091209/66857ec4/attachment.pgp
More information about the xorg-devel
mailing list