Re: [PATCH rt-tests v1 1/2] rt-utils: Call get_timestmap() in rt_init()
From: Kurt Kanzenbach <kurt@linutronix.de>
Date: 2021-06-29 06:43:52
Attachments
- signature.asc [application/pgp-signature] 861 bytes
From: Kurt Kanzenbach <kurt@linutronix.de>
Date: 2021-06-29 06:43:52
Hi Daniel, On Mon Jun 28 2021, Daniel Wagner wrote:
Move the get_timestamp() from rt_test_start() to rt_init(). The idea of rt_test_start() was to get the start timestamp right before the 'main loop' for the test starts. At least for cyclictest the rt_test_start() was placed wrongly so that the first test cycle could hit the pagefault when strftime() wrote into the tsbuf. We don't have an exact semantic description what start test timestamp means, so the simplest thing to avoid any further problems with it, is to take the timestamp right at the beginning when the program starts. Most test programs have a very short setup phase anyway. Reported-by: Mike Galbraith <redacted> Signed-off-by: Daniel Wagner <redacted>
This fixes the issue for me: Tested-by: Kurt Kanzenbach <kurt@linutronix.de> Thanks, Kurt