From fb20713b380e1df8d7f9e9df96563be2f9144fd6 Mon Sep 17 00:00:00 2001 From: Mohit Agrawal Date: Thu, 12 Mar 2020 21:12:13 +0530 Subject: Posix: Use simple approach to close fd Problem: posix_release(dir) functions add the fd's into a ctx->janitor_fds and janitor thread closes the fd's.In brick_mux environment it is difficult to handle race condition in janitor threads because brick spawns a single janitor thread for all bricks. Solution: Use synctask to execute posix_release(dir) functions instead of using background a thread to close fds. Credits: Pranith Karampuri Change-Id: Iffb031f0695a7da83d5a2f6bac8863dad225317e Fixes: bz#1811631 Signed-off-by: Mohit Agrawal --- tests/features/ssl-authz.t | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'tests/features') diff --git a/tests/features/ssl-authz.t b/tests/features/ssl-authz.t index 132b598ff69..497083e5a3a 100755 --- a/tests/features/ssl-authz.t +++ b/tests/features/ssl-authz.t @@ -67,13 +67,14 @@ echo "Memory consumption for glusterfsd process" for i in $(seq 1 100); do gluster v heal $V0 info >/dev/null done - +#Wait to cleanup memory +sleep 10 end=`pmap -x $glusterfsd_pid | grep total | awk -F " " '{print $4}'` diff=$((end-start)) -# If memory consumption is more than 5M some leak in SSL code path +# If memory consumption is more than 15M some leak in SSL code path -TEST [ $diff -lt 5000 ] +TEST [ $diff -lt 15000 ] # Set ssl-allow to a wildcard that includes our identity. -- cgit