Current tinderbox regressions (video drivers)

[Eric Anholt]

On Wed, 2009-02-25 at 17:05 +0100, Matthias Hopf wrote:
> On Feb 24, 09 19:46:24 +0100, Michel Dänzer wrote:
> > On Tue, 2009-02-24 at 13:18 -0500, Chris Ball wrote:
> > >    > EXA: Handle separate alpha maps properly in Composite fallback
> > >    http://cgit.freedesktop.org/xorg/xserver/commit/?id=170cf1270dff38d3cce7f5ba5b940d1c0d70eff5
> > > Driver maintainers:  from this commit onwards, EXA requires you to pass
> > > in -DEXA_DRIVER_KNOWN_MAJOR=3, else it'll fail to build.  If you use
> > > Prepare/FinishAccess, UploadToScratch or ExaOffscreenSwap*, you'll need
> > > to make other changes too.
> 
> There used to be a time where API changes were supposed to be upwards
> and downwards compatible. I grieve for that time.
> 
> Seriously, this change would have been easy to do in a less invasive
> manner by not nuking the hook, but just not using it any more. And by
> adding an additional hook for the new buffer types.
> 
> Enough ranting, radeonhd should be fixed now. Haven't validated, though.
> And the r6xx-r7xx-support branch probably needs similar treatment.

One of the design goals of EXA initialization was to prevent this sort
of breakage.  This commit should be reverted.

-- 
Eric Anholt
eric at anholt.net                         eric.anholt at intel.com


-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20090225/71dac53d/attachment.pgp