mbox series

[v2,0/1,SRU,E/F] kselftests/timers: disable timeout

Message ID 20200226093754.20463-1-paolo.pisati@canonical.com
Headers show
Series kselftests/timers: disable timeout | expand

Message

Paolo Pisati Feb. 26, 2020, 9:37 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1864626

Several timers kselftests are failing since the introduction of a timeout
threshold in commit 852c8cbf34d3 "selftests/kselftest/runner.sh: Add 45 second
timeout per test": we hit the same problem with ftrace kselftests, and the fix
there was to disable this timeout completely.

Sam proposes a similar fix for kselftests/timers timeout failures:

https://lists.linaro.org/pipermail/linux-kselftest-mirror/2020-February/011044.html

Po-Hsu Lin (1):
  selftests/timers: Turn off timeout setting

 tools/testing/selftests/timers/Makefile | 1 +
 tools/testing/selftests/timers/settings | 1 +
 2 files changed, 2 insertions(+)
 create mode 100644 tools/testing/selftests/timers/settings

Comments

Andrea Righi Feb. 26, 2020, 12:15 p.m. UTC | #1
On Wed, Feb 26, 2020 at 10:37:53AM +0100, Paolo Pisati wrote:
> BugLink: https://bugs.launchpad.net/bugs/1864626
> 
> Several timers kselftests are failing since the introduction of a timeout
> threshold in commit 852c8cbf34d3 "selftests/kselftest/runner.sh: Add 45 second
> timeout per test": we hit the same problem with ftrace kselftests, and the fix
> there was to disable this timeout completely.
> 
> Sam proposes a similar fix for kselftests/timers timeout failures:
> 
> https://lists.linaro.org/pipermail/linux-kselftest-mirror/2020-February/011044.html
> 
> Po-Hsu Lin (1):
>   selftests/timers: Turn off timeout setting
> 
>  tools/testing/selftests/timers/Makefile | 1 +
>  tools/testing/selftests/timers/settings | 1 +
>  2 files changed, 2 insertions(+)
>  create mode 100644 tools/testing/selftests/timers/settings

Looks good to me.

Acked-by: Andrea Righi <andrea.righi@canonical.com>
Seth Forshee Feb. 26, 2020, 1:04 p.m. UTC | #2
On Wed, Feb 26, 2020 at 10:37:53AM +0100, Paolo Pisati wrote:
> BugLink: https://bugs.launchpad.net/bugs/1864626
> 
> Several timers kselftests are failing since the introduction of a timeout
> threshold in commit 852c8cbf34d3 "selftests/kselftest/runner.sh: Add 45 second
> timeout per test": we hit the same problem with ftrace kselftests, and the fix
> there was to disable this timeout completely.
> 
> Sam proposes a similar fix for kselftests/timers timeout failures:
> 
> https://lists.linaro.org/pipermail/linux-kselftest-mirror/2020-February/011044.html

Acked-by: Seth Forshee <seth.forshee@canonical.com>

Applied to focal/master-next and unstable/master, thanks!
Khalid Elmously March 11, 2020, 6:15 a.m. UTC | #3
On 2020-02-26 10:37:53 , Paolo Pisati wrote:
> BugLink: https://bugs.launchpad.net/bugs/1864626
> 
> Several timers kselftests are failing since the introduction of a timeout
> threshold in commit 852c8cbf34d3 "selftests/kselftest/runner.sh: Add 45 second
> timeout per test": we hit the same problem with ftrace kselftests, and the fix
> there was to disable this timeout completely.
> 
> Sam proposes a similar fix for kselftests/timers timeout failures:
> 
> https://lists.linaro.org/pipermail/linux-kselftest-mirror/2020-February/011044.html
> 
> Po-Hsu Lin (1):
>   selftests/timers: Turn off timeout setting
> 
>  tools/testing/selftests/timers/Makefile | 1 +
>  tools/testing/selftests/timers/settings | 1 +
>  2 files changed, 2 insertions(+)
>  create mode 100644 tools/testing/selftests/timers/settings
> 
> -- 
> 2.25.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team