diff mbox series

[PULL,10/16] os-posix: Expand setrlimit() syscall compatibility

Message ID 20240630165327.38153-11-mjt@tls.msk.ru (mailing list archive)
State New, archived
Headers show
Series [PULL,01/16] hmp-commands-info.hx: Add missing info command for stats subcommand | expand

Commit Message

Michael Tokarev June 30, 2024, 4:53 p.m. UTC 
From: Trent Huber <trentmhuber@gmail.com>

Darwin uses a subtly different version of the setrlimit() syscall as
described in the COMPATIBILITY section of the macOS man page. The value
of the rlim_cur member has been adjusted accordingly for Darwin-based
systems.

Signed-off-by: Trent Huber <trentmhuber@gmail.com>
Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
---
 os-posix.c | 4 ++++
 1 file changed, 4 insertions(+)
diff mbox series

Patch

diff --git a/os-posix.c b/os-posix.c
index a4284e2c07..43f9a43f3f 100644
--- a/os-posix.c
+++ b/os-posix.c
@@ -270,7 +270,11 @@  void os_setup_limits(void)
         return;
     }
 
+#ifdef CONFIG_DARWIN
+    nofile.rlim_cur = OPEN_MAX < nofile.rlim_max ? OPEN_MAX : nofile.rlim_max;
+#else
     nofile.rlim_cur = nofile.rlim_max;
+#endif
 
     if (setrlimit(RLIMIT_NOFILE, &nofile) < 0) {
         warn_report("unable to set NOFILE limit: %s", strerror(errno));