[Mesa-dev] [PATCH 20/26] mesa/tests: Add tests for the generated dispatch table

Ian Romanick idr at freedesktop.org
Thu May 31 19:59:59 PDT 2012


On 05/30/2012 03:19 PM, Eric Anholt wrote:
> On Tue, 29 May 2012 15:51:47 -0700, "Ian Romanick"<idr at freedesktop.org>  wrote:
>> +TEST(GetProcAddress, ABINameByOffset)
>> +{
>> +   /* 408 functions have had their locations in the dispatch table set since
>> +    * the danw of time.  Verify that all of these functions are at the correct
>               ^dawn

D'oh!

> (also copy and pasted)
>
> I don't like the "as big as at some git sha1" test given that you
> immediately go on to show that the test is invalid and just hack around
> it.  That sounds to me like "this test should be removed".  Maybe just
> check that it's bigger than the fixed ABI?

The intention is for the test to catch changes that accidentally cause 
the dispatch table to shrink.  The dispatch table pretty much only ever 
grows.  The most recent change to gl_API.xml that caused the dispatch 
table to shrink was a5f220607 in 2006.  I'd be surprised if there were 
more changes after this series that shrink the dispatch table before 
another 6 years. :)

Anything else that causes a shrink is a bug.  I want to have some kind 
of test for that, but it doesn't necessarily have to be this test.


More information about the mesa-dev mailing list