[PATCH 1/4] fb: fix screen pixmap leak on server reset

Keith Packard keithp at keithp.com
Thu Jun 26 15:09:55 PDT 2014


Dave Airlie <airlied at gmail.com> writes:

> is pScreen->devPrivate guaranteed to be a pixmap, or is that just a
> convention?

It's always a pixmap if you use fb:

#define fbGetScreenPixmap(s)	((PixmapPtr) (s)->devPrivate)

> Do we have anything to say when it is?

A driver *not* using fb could put something else there. I think we
should probably just declare a PixmapPtr in the screen and be done with
it though.

> I think I'm also confusing this with the screwed up miScreenInit
> private ptr hiding stuff.

Agreed. We should just create a new structure to pass the screen init
stuff through instead of re-purposing devPrivate.

-- 
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/20140626/ab3119c3/attachment.sig>


More information about the xorg-devel mailing list