Hi Stephen,
On Mon, 8 Jun 2020 at 10:43, Stephen Warren swarren@wwwdotorg.org wrote:
On 6/7/20 7:45 AM, Simon Glass wrote:
On Thu, 4 Jun 2020 at 09:24, Heiko Schocher hs@denx.de wrote:
make the sleep time and the margin configurable.
Signed-off-by: Heiko Schocher hs@denx.de
travis build: https://travis-ci.org/github/hsdenx/u-boot-test/builds/694545225
This patch is needed as I start test/py now within tbot [1]. On some configurations U-Boot is compiled on a build machine for example in munich, while the board under test is in my lab in hungary.
So the 0.25 seconds default margin is often to low because of latencies on the net.
See as an example configuration (within tbot):
https://github.com/EmbLux-Kft/tbot-tbot2go/blob/devel/boards/aristainetos.py...
[1] http://tbot.tools/modules/tc.html#u-boot-test-py
test/py/tests/test_sleep.py | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-)
Reviewed-by: Simon Glass sjg@chromium.org
Related, at some point we should change sandbox to fake the time movement since this test currently waits for three seconds even on sandbox.
We definitely shouldn't do that; that's the exact kind of failure this test is intended to detect.
No, we're not looking for bugs in sandbox's time handling. We are just testing the plumbing associated with delaying (timer driver, etc.).
Regards, Simon