Removing XAA render accel in radeon

[Richard Wilbur]

On Mon, 2009-12-14 at 16:22 -0500, Alex Deucher wrote:
> On Mon, Dec 14, 2009 at 4:17 PM, Roland Scheidegger
> <sroland at tungstengraphics.com> wrote:
> > On 14.12.2009 20:33, Alex Deucher wrote:
> >> On Mon, Dec 14, 2009 at 2:21 PM, Roland Scheidegger
> >> <sroland at tungstengraphics.com> wrote:
> >>> On 14.12.2009 15:22, Alex Deucher wrote:
> >>>> XAA render support seems to be broken (fdo bug 22055, possibly
> >>>> others).  It never really did much to begin with, and it only
> >>>> supported r1xx and r2xx.  Any objections to me dumping XAA render
> >>>> accel in the radeon driver?
> >>> Hmm, personally I was never able to get good performance out of EXA with
> >>> a 32MB r100 card (read: completely unusable). Maybe that has changed
> >>> with more recent xorg server versions, and I guess it should have
> >>> changed with KMS (I certainly used to have problems even with a 64MB
> >>> card without KMS, if the resolution was more than XGA), but unless
> >>> you're willing to ditch non-KMS too that might be a reason to keep it
> >>> for now, though I guess that would mean fixing...
> >>
> >> I think the XAA render problem is due to bitrot in the XAA code in the
> >> xserver as the XAA render code in the driver hasn't been touched in
> >> years and only recently have we started getting reports of it being
> >> broken.  Does the XAA render code actually make much of a difference
> >> performance-wise?  I never really noticed much difference with it.
> >> It's especially useless since offscreen pixmap support for XAA broke a
> >> year or two ago.  We could just default render accel to false with
> >> XAA, but leave the code there, but I don't see anyone fixing XAA, so
> >> there's not much point in keeping it.
> >
> > Oh atually I misread that I thought you were going to nuke xaa
> > completely from the driver. I guess it's ok to to remove xaa render
> > code. Is radeon actually the only driver left with xaa render accel?
> 
> Could be.  I'm not sure what other drivers ever supported it.

Alex,

1.  What kind of bitrot?
2.  Are the XAA render code bugs tagged similarly?
3.  What kind of functionality did XAA render code provide?
4.  What level of effort do you think it would require to repair?

Richard
-- 
PGP public key at
http://keyserver.ubuntu.com:11371/pks/lookup?op=get&search=0xA7CEB3C711AFF0B0
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-driver-ati/attachments/20100106/077fdbc5/attachment.pgp