Ah, okay, that makes sense, I'll drop this patch
On Wed, Sep 02, 2015 at 09:27:16PM +0100, Jose Fonseca wrote: > On 02/09/15 20:51, Dylan Baker wrote: > > This was never added to all.py > > Yes, that was because the test uses random numbers, hence might yield > variable results (fail once, pass another). > > Rather than all.py, it probably makes sense to move this sort of tests > to a new stress.py test-list. > > The other alternative is use reproduceable pseudo-random numbers, or a > pre-generated set of data from a plaintext file. > > Jose > > > > > > cc: José Fonseca <[email protected]> > > Signed-off-by: Dylan Baker <[email protected]> > > --- > > > > Does this actually need run_concurrent=False? > > > > tests/all.py | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/tests/all.py b/tests/all.py > > index 9c55dd7..ca72b65 100644 > > --- a/tests/all.py > > +++ b/tests/all.py > > @@ -2249,6 +2249,7 @@ with profile.group_manager( > > PiglitGLTest, > > grouptools.join('spec', 'ARB_robustness')) as g: > > g(['arb_robustness_client-mem-bounds'], run_concurrent=False) > > + g(['arb_robustness_draw-vbo-bounds'], run_concurrent=False) > > > > # Group ARB_shader_texture_lod > > with profile.group_manager( > > >
signature.asc
Description: Digital signature
_______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
