[Intel-gfx] [PATCH] Make XAA support optional

[Chris Wilson]

On Wed, 18 Jan 2012 13:04:43 +0100, Daniel Vetter <daniel at ffwll.ch> wrote:
> On Wed, Jan 18, 2012 at 10:37:24AM +0000, Chris Wilson wrote:
> > On Wed, 18 Jan 2012 14:45:51 +1100, Daniel Stone <daniel at fooishbar.org> wrote:
> > > Does what it says on the box.
> > 
> > Danvet can you please test this on your elderly i810 box and make sure
> > we don't accidentally open the can of worms. Any sign of regressions?
> 
> Why don't we just rip it out entirely? If a bored soul ever ports this
> driver to kms we likely just need to reimplement proper accel with sna, so
> I don't think there's much value.

If we did rip it out, what happens? The X server uses the vesa driver,
which is equivalent to i810 for modesetting, no 3d support, no overlays,
and no accelerated blits (which is dying anyway). Am I right in thinking
that after removal of XAA, the legacy i810 offers no more functionality
or performance than is available through VESA?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre