On Thu, Nov 19, 2015 at 08:00:09PM -0800, Ian Romanick wrote: > From: Ian Romanick <ian.d.roman...@intel.com> > > If this test has problems with odd sample counts, the test should detect > that and SKIP.
I'm not sure I agree. I think it's pretty reasonable for all.py to only pass valid inputs to a test binary. > > Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> > --- > tests/all.py | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/tests/all.py b/tests/all.py > index 1038e07..abe7da2 100644 > --- a/tests/all.py > +++ b/tests/all.py > @@ -2758,12 +2758,10 @@ with profile.group_manager( > 'alpha-blending slow_cc') > g(['ext_framebuffer_multisample-fast-clear'], 'fast-clear') > > - for num_samples in MSAA_SAMPLE_COUNTS: > - if num_samples % 2 != 0: > - continue > + for sample_count in (str(x) for x in MSAA_SAMPLE_COUNTS): > g(['ext_framebuffer_multisample-alpha-blending-after-rendering', > - str(num_samples)], > - 'alpha-blending-after-rendering {}'.format(num_samples)) > + sample_count], > + 'alpha-blending-after-rendering {}'.format(sample_count)) > > for num_samples in ('all_samples', ) + MSAA_SAMPLE_COUNTS: > g(['ext_framebuffer_multisample-formats', str(num_samples)], > -- > 2.1.0 > > _______________________________________________ > Piglit mailing list > Piglit@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/piglit
signature.asc
Description: PGP signature
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/piglit