[PATCH 02/13] glamor: Add glamor_program based copy acceleration
Keith Packard
keithp at keithp.com
Thu May 22 18:40:25 PDT 2014
Michel Dänzer <michel at daenzer.net> writes:
> This results in a crash / memory corruption when confronted with
> GLAMOR_DRM_ONLY pixmaps. glamor_copy_bail calls down to fb, but the
> pixmap's devPrivate.ptr does not point to any usable storage.
>
> This happens for me when a piglit test creates a depth 16 GLXPixmap.
This should fix the problem by bailing when attempting to prepare a
pixmap which cannot be downloaded:
diff --git a/glamor/glamor_prepare.c b/glamor/glamor_prepare.c
index 60a4229..6fb8a91 100644
--- a/glamor/glamor_prepare.c
+++ b/glamor/glamor_prepare.c
@@ -39,6 +39,9 @@ glamor_prep_pixmap_box(PixmapPtr pixmap, glamor_access_t access, BoxPtr box)
int gl_access, gl_usage;
RegionRec region;
+ if (priv->type == GLAMOR_DRM_ONLY)
+ return FALSE;
+
if (!GLAMOR_PIXMAP_PRIV_HAS_FBO(priv))
return TRUE;
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140522/9d80ea6a/attachment.sig>
More information about the xorg-devel
mailing list