[RFC] radeon bo/cs API cleanup

Michel Dänzer michel at daenzer.net
Thu Dec 17 04:13:46 PST 2009


On Thu, 2009-12-17 at 14:34 +1000, Dave Airlie wrote: 
> Hi,
> 
> so Michel pointed out that the libdrm_radeon API was pretty unmaintainble.
> 
> Reasons:
> 1. inlines in headers mean API/ABI is fixed completely.
> 2. All of structure internals for bo/cs are exposed.
> 3. RADEON_BO_TRACK changed the library API when on/off.
> 
> So I
> a) removed bo tracking
> b) split the cs/bo structs into public/private parts and minimised the
> members, the cs_int/bo_int need to reflect the public bits.
> c) left some members intact for fast paths for functions that need it
> (write_dword)
> 
> 3 patches attached, libdrm/ddx/mesa (no r300g bits yet) to clean the API of
> all of this.
> 
> I'd like to then declare this as the stable API going forward.

Thanks Dave, these look good.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the xorg-driver-ati mailing list