Re: [PATCH] tests/gem_reloc_overflow: Add gen8+ specifc tests

Daniel Vetter
2013-11-07T13:15:11Z

On Thu, Nov 07, 2013 at 02:05:55PM +0100, Daniel Vetter wrote:
> Cool, thanks for testing. Since I'm paranoid about this I've added an
> assert into the relevant fixture block to make sure the batch would really
> work safe for the condition we're testing. That way we're maximally robust
> against kernel changes that move the tests around.
>
> Also fixed some compile warnings in a follow-up.
>
> Thanks for your patch, applied to i-g-t.

Meh, I've been a bit too quick, two important fixups where needed:
- Make the subtest enumeration work (it's supposed to work without an
intel gpu present, e.g. for central build/test servers).
- Always enumerate all subtests (again to facilitate testing across
platforms).

Maybe we need some docs about this stuff, currently most of it is encoded
as runtime asserts in the test suite library.