mallocstress: extend test time on arm
PATCH 1/1 mallocstress: extend test time on arm
Arm systems with lot of memory and THP enabled are hitting a timeout at ~5 minute mark as reported here:
mallocstress poor performance with THP on arm64 system
https://marc.info/?l=linux-mm&m=151864950330870&w=2
Extend the timeout to 10m (only for arm).
--- testcases/kernel/mem/mtest07/mallocstress.c | 8 ++++++++ 1 file changed, 8 insertions(+)
testcases/kernel/mem/mtest07/mallocstress.c +8 -0
@@ -243,6 +243,14 @@ static void cleanup(void) } static struct tst_test test = { +/* +* extend test time on arm due to: +* "mallocstress poor performance with THP on arm64 system" +* https://marc.info/?l=linux-mm&m=151864950330870&w=2 +*/ +#if defined(__arm__) || defined(__aarch64__) + .timeout = 600, +#endif .needs_checkpoints = 1, .setup = setup, .cleanup = cleanup,
Signature · 1 line
1.8.3.1
PATCH 1/1 mallocstress: extend test time on arm
Hi!
Quoted 6 lines
Arm systems with lot of memory and THP enabled are hitting a timeout at ~5 minute mark as reported here:
mallocstress poor performance with THP on arm64 system
https://marc.info/?l=linux-mm&m=151864950330870&w=2Extend the timeout to 10m (only for arm).
What about extending it unconditionally? It's not like it will hurt in generall case and there are possible other slow systems that would benefit from that as well.
Or do you have a strong reason to extend the timeout only on arm?
Signature · 2 lines
Cyril Hrubis [email protected]
PATCH 1/1 mallocstress: extend test time on arm
----- Original Message -----
Quoted 6 lines
Hi!
Quoted 5 lines
Arm systems with lot of memory and THP enabled are hitting a timeout at ~5 minute mark as reported here:
mallocstress poor performance with THP on arm64 system
https://marc.info/?l=linux-mm&m=151864950330870&w=2
Quoted 7 lines
Quoted 1 line
Extend the timeout to 10m (only for arm).
What about extending it unconditionally? It's not like it will hurt in generall case and there are possible other slow systems that would benefit from that as well.
Or do you have a strong reason to extend the timeout only on arm?
Just wanted to go with minimal impact on other arches (since they worked OK so far as I'm aware). I don't have strong opinion either way, I can make the timeout unconditional.
Regards,
Jan
PATCH 1/1 mallocstress: extend test time on arm
Hi!
Quoted 3 lines
Just wanted to go with minimal impact on other arches (since they worked OK so far as I'm aware). I don't have strong opinion either way, I can make the timeout unconditional.
Given that the test seems to take 4m on successful run 5m timeout seems to be a bit too strict to me. We are not primarily benchmarking the kernel, so these timeouts are better to be set to be double of the expecte runtime.
Also I like to avoid #ifdefs whenever possible.
Signature · 2 lines
Cyril Hrubis [email protected]
PATCH 1/1 mallocstress: extend test time on arm
----- Original Message -----
Quoted 11 lines
Hi!
Quoted 3 lines
Just wanted to go with minimal impact on other arches (since they worked OK so far as I'm aware). I don't have strong opinion either way, I can make the timeout unconditional.
Given that the test seems to take 4m on successful run 5m timeout seems to be a bit too strict to me. We are not primarily benchmarking the kernel, so these timeouts are better to be set to be double of the expecte runtime.
Also I like to avoid #ifdefs whenever possible.
Pushed with your ACK as unconditional test time change to 10m.
Regards,
Jan