no more indexes - clean up API

Aaron Plattner aplattner at nvidia.com
Mon May 14 15:29:41 PDT 2012


On 05/14/2012 02:25 PM, Alan Coopersmith wrote:
> On 05/14/12 09:40 AM, Dave Airlie wrote:
>> The screen/scrninfo APIs are very into passing index values, that
>> are then immediately used to lookup a global array, this is not
>> an API, its whatever the opposite is.
>>
>> This set of patches changes all the API passing of screen indices
>> (apart from the logging functions). It seriously breaks the driver
>> API/ABI, and I've got a demo of some compat for it I've sent out before.
>>
>> I'll try and co-ordinate with Keith, Alex/Chris/Ben so that main 3
>> drivers are compatible when we push the server with all these in it.
> 
> And presumably with Aaron to get the ABI version bumped appropriately.

Yeah.  I don't care who bumps it, but I attached a patch to do it which should get tacked onto your pull request somewhere.  I don't *think* the extension ABI needs to be bumped.  I'll let Peter comment on whether the input ABI needs to be bumped too.

(sorry, I tried to paste it after a scissors line but I can't get my mailer to not expand tabs)

>> I was holding back on this to try and get some bit by bit review, but
>> it might be easier for someone to just block off an hour and go mad,
>> so here goes.
> 
> I mostly just looked at what you had, didn't double check you caught all the
> places that needed updating (that's what git grep&  compilers are for).   Thanks
> for taking care to update the docs to match.
> 
> For 1-12,14,16-17,19:
> Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

I just skimmed the changes but they all seem sane to me, so
Acked-by: Aaron Plattner <aplattner at nvidia.com>

-- Aaron

> Comments sent separately on the rest.


More information about the xorg-devel mailing list