Re: [PATCH v2 1/4] selftests/livepatch: simplify test-klp-callbacks busy target tests

From: Petr Mladek
Date: Fri Jun 05 2020 - 08:58:36 EST


On Wed 2020-06-03 14:20:55, Yannick Cote wrote:
> From: Joe Lawrence <joe.lawrence@xxxxxxxxxx>
>
> The test-klp-callbacks script includes a few tests which rely on kernel
> task timings that may not always execute as expected under system load.
> These may generate out of sequence kernel log messages that result in
> test failure.
>
> Instead of using sleep timing windows to orchestrate these tests, add a
> block_transition module parameter to communicate the test purpose and
> utilize flush_queue() to serialize the test module's task output.
>
> Suggested-by: Petr Mladek <pmladek@xxxxxxxx>

I am not sure if I deserve this. I suggested only few implementation details.

> Signed-off-by: Joe Lawrence <joe.lawrence@xxxxxxxxxx>

Looks good now:

Reviewed-by: Petr Mladek <pmladek@xxxxxxxx>

Best Regards,
Petr