[Linux] JSTests/wasm/stress/oom.js should not run on Linux

https://bugs.webkit.org/show_bug.cgi?id=175100

Reviewed by Mark Lam.

The JSC test JSTests/wasm/stress/oom.js tries to use all the
available memory until an out of memory exception happens.

The Linux kernel is more tuned for server workloads than for GUI
responsiveness. When a process tries to use a lot of memory, Linux
will do its best to serve the request. This usually translates to
free physical RAM by writing to disk dirty pages and/or moving out
less recently used pages to swap (disk storage).
Meanwhile it does this, the system will become unresponsive and this
leads to freezes that can last even some minutes on the worst cases.

Therefore, let's skip this test on Linux as it will cause more harm
than good on the Linux bots or on the machines of Linux developers.

* wasm/stress/oom.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@220173 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/JSTests/ChangeLog b/JSTests/ChangeLog
index 97e8c51c15fbe264d933d6ad9ba938824395eace..b6596d8bddf0c228bbbe11d64c4f4e228f54ce0e 100644 (file)
--- a/JSTests/ChangeLog
+++ b/JSTests/ChangeLog
@@ -1,3 +1,26 @@
+2017-08-02  Carlos Alberto Lopez Perez  <clopez@igalia.com>
+
+        [Linux] JSTests/wasm/stress/oom.js should not run on Linux
+        https://bugs.webkit.org/show_bug.cgi?id=175100
+
+        Reviewed by Mark Lam.
+
+        The JSC test JSTests/wasm/stress/oom.js tries to use all the
+        available memory until an out of memory exception happens.
+
+        The Linux kernel is more tuned for server workloads than for GUI
+        responsiveness. When a process tries to use a lot of memory, Linux
+        will do its best to serve the request. This usually translates to
+        free physical RAM by writing to disk dirty pages and/or moving out
+        less recently used pages to swap (disk storage).
+        Meanwhile it does this, the system will become unresponsive and this
+        leads to freezes that can last even some minutes on the worst cases.
+
+        Therefore, let's skip this test on Linux as it will cause more harm
+        than good on the Linux bots or on the machines of Linux developers.
+
+        * wasm/stress/oom.js:
+
 2017-08-01  Oleksandr Skachkov  <gskachkov@gmail.com>
 
         [JSC] Remove unnecessary print from stress\promise-finally.js test

diff --git a/JSTests/wasm/stress/oom.js b/JSTests/wasm/stress/oom.js
index 076ea760339d74337d2fc2204e7c8948caadd52d..6c675aba262c96f312cceb0d52540f242ddd2ed9 100644 (file)
--- a/JSTests/wasm/stress/oom.js
+++ b/JSTests/wasm/stress/oom.js
@@ -1,5 +1,5 @@
 // We don't need N versions of this simultaneously filling up RAM.
-//@ runDefault
+//@ if $hostOS == "linux" then skip else runDefault end
 
 const verbose = false;