Keith Packard <kei...@keithp.com> writes: > I don't think the per-port Xv data which glamor allocates is getting > freed at server reset time, so I put together a short patch sequence > which does it. Seems to work for me at least.
I looked into this a bit further yesterday and discovered that glamor is depending on StopVideo being called to discard the temporary storage used for Xv PutImage. However, StopVideo is only called for overlays, never for textured video in xf86xv.c (and kxv.c) -- all of the calls are predicated on pPriv->isOn == XV_ON while 'pPriv->isOn' is only set here: if ((ret == Success) && (portPriv->AdaptorRec->flags & VIDEO_OVERLAID_IMAGES)) { portPriv->isOn = XV_ON; I've posted a patch under separate cover that just frees the glamor xv data right after the image is presented; that's the simplest solution I could think of and works just fine. -- keith.pack...@intel.com
pgpM5uenuPSXD.pgp
Description: PGP signature
_______________________________________________ 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