From 169a1006582d1dbef3e16bd94c14abd9437d6b2b Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Fri, 15 Sep 2023 17:54:02 +0200 Subject: tests/shell: suggest 4Mb /proc/sys/net/core/{wmem_max,rmem_max} for rootless 2Mb was not enough to pass "tests/shell/testcases/sets/0030add_many_elements_interval_0" in an unprivileged/rootless namespace. Instead, bump the suggestion to 4Mb, which lets the test pass. Note that the 4Mb are only the recommended value when running the test as rootless, and is used to autodetect NFT_TEST_HAS_SOCKET_LIMITS=y. You can set whatever values are suitable for your environment, and explicitly indicate whether the limits are appropriate or not via NFT_TEST_HAS_SOCKET_LIMITS=n|y. Signed-off-by: Thomas Haller Signed-off-by: Florian Westphal --- tests/shell/run-tests.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'tests') diff --git a/tests/shell/run-tests.sh b/tests/shell/run-tests.sh index 1527b2a6..d11b4a63 100755 --- a/tests/shell/run-tests.sh +++ b/tests/shell/run-tests.sh @@ -180,7 +180,7 @@ usage() { echo " with rootless the test would fail. Tests will check for [ "\$NFT_TEST_HAS_SOCKET_LIMITS" = y ]" echo " and skip. You may set NFT_TEST_HAS_SOCKET_LIMITS=n if you ensure those limits are" echo " suitable to run the test rootless. Otherwise will be autodetected." - echo " Set /proc/sys/net/core/{wmem_max,rmem_max} to at least 2MB to get them to pass automatically." + echo " Set /proc/sys/net/core/{wmem_max,rmem_max} to at least 4MB to get them to pass automatically." echo " NFT_TEST_UNSHARE_CMD=cmd : when set, this is the command line for an unshare" echo " command, which is used to sandbox each test invocation. By" echo " setting it to empty, no unsharing is done." @@ -391,8 +391,8 @@ export NFT_TEST_HAS_REALROOT if [ "$NFT_TEST_HAS_SOCKET_LIMITS" = "" ] ; then if [ "$NFT_TEST_HAS_REALROOT" = y ] ; then NFT_TEST_HAS_SOCKET_LIMITS=n - elif [ "$(cat /proc/sys/net/core/wmem_max 2>/dev/null)" -ge $((2000*1024)) ] 2>/dev/null && \ - [ "$(cat /proc/sys/net/core/rmem_max 2>/dev/null)" -ge $((2000*1024)) ] 2>/dev/null ; then + elif [ "$(cat /proc/sys/net/core/wmem_max 2>/dev/null)" -ge $((4000*1024)) ] 2>/dev/null && \ + [ "$(cat /proc/sys/net/core/rmem_max 2>/dev/null)" -ge $((4000*1024)) ] 2>/dev/null ; then NFT_TEST_HAS_SOCKET_LIMITS=n else NFT_TEST_HAS_SOCKET_LIMITS=y -- cgit v1.2.3