> -----Original Message----- > From: Chris Wilson [mailto:[email protected]] > Sent: Friday, August 25, 2017 1:47 PM > To: Lofstedt, Marta <[email protected]>; intel- > [email protected] > Subject: Re: [Intel-gfx] [PATCH i-g-t v2] tests/kms_frontbuffer_tracking: > increase FBC wait timeout to 5s > > Quoting Marta Lofstedt (2017-08-25 11:40:29) > > From: "Lofstedt, Marta" <[email protected]> > > > > The subtests: igt@kms_frontbuffer_tracking@fbc-*draw* > > has non-consistent results, pending between fail and pass. > > The fails are always due to "FBC disabled". > > With this increase in timeout the flip-flop behavior is no longer > > reproducible. > > > > This is a partial revert of: > > 64590c7b768dc8d8dd962f812d5ff5a39e7e8b54, > > where the timeout was decreased from 5s to 2s. > > After investigating the timeout needed, the conclusion is that the > > longer timeout is only needed when the test swaps between some > > specific draw domains, typically blt vs. mmap_cpu. > > The objective of the FBC part of the tests is not to benchmark draw > > domain changes, it is to check that FBC was (re-)enabled. > > > > V2: Added documentation > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101623 > > Signed-off-by: Marta Lofstedt <[email protected]> > > Acked-by: Paulo Zanoni <[email protected]> > > --- > > tests/kms_frontbuffer_tracking.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tests/kms_frontbuffer_tracking.c > > b/tests/kms_frontbuffer_tracking.c > > index e03524f1..2538450c 100644 > > --- a/tests/kms_frontbuffer_tracking.c > > +++ b/tests/kms_frontbuffer_tracking.c > > @@ -924,7 +924,7 @@ static bool fbc_stride_not_supported(void) > > > > static bool fbc_wait_until_enabled(void) { > > Try igt_drop_caches_set(device, DROP_RETIRE); instead of relaxing the > timeout. > -Chris
OK, I will test that and do a V3 if it works! /Marta _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
