diff mbox

generic/072: limit max cpu number to 8 to avoid long run time and high system pressure

Message ID 1455182926-7782-1-git-send-email-eguan@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Eryu Guan Feb. 11, 2016, 9:28 a.m. UTC
Right now generic/072 scales the loop count based on the cpu count. But
on hosts with many cpus(100+), generic/072 runs for hours and generates
very high system load.

Given that the original bug can be reproduced easily on unpatched
kernel, the great number of loops and long run time are not needed. So
limiting the cpu number to 8 (which gives around 20 seconds run time on
my test vm with 8 vcpus) seems reasonable.

Signed-off-by: Eryu Guan <eguan@redhat.com>
---
 tests/generic/072 | 4 ++++
 1 file changed, 4 insertions(+)
diff mbox

Patch

diff --git a/tests/generic/072 b/tests/generic/072
index 6e59d84..ece71fb 100755
--- a/tests/generic/072
+++ b/tests/generic/072
@@ -51,6 +51,10 @@  _require_xfs_io_command "fcollapse"
 
 testfile=$TEST_DIR/testfile.$seq
 NCPUS=`$here/src/feature -o`
+# cap NCPUS to 8 at maximum to avoid long run time on hosts with many cpus
+if [ $NCPUS -gt 8 ]; then
+	NCPUS=8
+fi
 OUTER_LOOPS=$((10 * $NCPUS * $LOAD_FACTOR))
 INNER_LOOPS=$((50 * $NCPUS * $LOAD_FACTOR))
 # fcollapse/truncate continuously and simultaneously a same file