[sheepdog] [PATCH v6] try to set RLIMIT_NOFILE to recommended value

Vasiliy Tolstov v.tolstov at selfip.ru
Wed Apr 8 08:14:29 CEST 2015


Signed-off-by: Vasiliy Tolstov <v.tolstov at selfip.ru>
---
 sheep/sheep.c | 26 +++++++++++++++--------------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/sheep/sheep.c b/sheep/sheep.c
index 86444b0..38ffed0 100644
--- a/sheep/sheep.c
+++ b/sheep/sheep.c
@@ -526,22 +526,24 @@ static void check_host_env(void)
 	struct rlimit r;
 
 	if (getrlimit(RLIMIT_NOFILE, &r) < 0)
-		sd_err("failed to get nofile %m");
-	/*
-	 * 1024 is default for NOFILE on most distributions, which is very
-	 * dangerous to run Sheepdog cluster.
-	 */
-	else if (r.rlim_cur == 1024)
-		sd_warn("Allowed open files 1024 too small, suggested %u",
-			SD_RLIM_NOFILE);
-	else if (r.rlim_cur < SD_RLIM_NOFILE)
-		sd_info("Allowed open files %lu, suggested %u", r.rlim_cur,
-			SD_RLIM_NOFILE);
+                sd_err("failed to getrlimit");
+        else if (r.rlim_cur < SD_RLIM_NOFILE) {
+                r.rlim_cur = SD_RLIM_NOFILE;
+                r.rlim_max = SD_RLIM_NOFILE;
+                if (setrlimit(RLIMIT_NOFILE, &r) != 0) {
+                        sd_warn("failed to set nofile to suggested %lu",
+                                r.rlim_cur);
+                        sd_warn("increase nofile via sysctl fs.nr_open");
+                } else {
+                        sd_info("allowed open files set to suggested %lu",
+                                r.rlim_cur);
+                }
+        }
 
 	if (getrlimit(RLIMIT_CORE, &r) < 0)
 		sd_debug("failed to get core %m");
 	else if (r.rlim_cur < RLIM_INFINITY)
-		sd_debug("Allowed core file size %lu, suggested unlimited",
+		sd_debug("allowed core file size %lu, suggested unlimited",
 			 r.rlim_cur);
 
 	/*



More information about the sheepdog mailing list