[PATCH] exa: add low VRAM option.
Michel Dänzer
michel at daenzer.net
Mon Oct 5 02:25:52 PDT 2009
On Mon, 2009-10-05 at 14:42 +1000, Dave Airlie wrote:
> On Mon, 2009-09-28 at 10:13 +0200, Michel Dänzer wrote:
> > On Mon, 2009-09-28 at 14:57 +1000, Dave Airlie wrote:
> > > From: Dave Airlie <airlied at linux.ie>
> > >
> > > This basically stops any accel to non screen pixmap and non
> > > window pixmaps when this option is enabled. It makes my
> > > RN50 at least useable again under KMS.
> > >
> > > I'm mainly producing this to open discussions on whether
> > > there is a better way. I'm defining low VRAM as < 32MB.
> >
> > There has to be a better way in the long run, but in the meantime this
> > could be handled in the driver PixmapIsOffscreen hook?
> >
>
> http://people.freedesktop.org/~airlied/scratch/radeon-return-FALSE-offscreen.patch
>
> I tried this, it fails, server crashes doing a composite operation in
> pixman.
>
> maybe you need to explain a bit better why you think this should work,
> as to me PixmapIsOffscreen seems like something you call possibly
> after migrating the pixmap.
Right, I was thinking of the code in exaFinishAccess_mixed which checks
IsOffscreen first, but of course that doesn't apply here.
Might it be feasible to return NULL from the CreatePixmap(2) hook for
non-screen pixmaps (or whichever ones you don't want in VRAM)? AFAICT
the mixed pixmaps code should basically handle that, though maybe we
should change exaCreateDriverPixmap_mixed to mark the pixmap as pinned
to prevent repeated calls of the CreatePixmap(2) hook.
--
Earthling Michel Dänzer | http://www.vmware.com
Libre software enthusiast | Debian, X and DRI developer
More information about the xorg-devel
mailing list