From 40acc203c043fd419f3c045dc6f116c3a28411d8 Mon Sep 17 00:00:00 2001 From: Zbigniew Jędrzejewski-Szmek Date: Tue, 14 Apr 2015 20:47:20 -0500 Subject: systemctl: avoid bumping NOFILE rlimit unless needed We actually only use the journal when showing status. Move setrlimit call so it is only called for status. https://bugzilla.redhat.com/show_bug.cgi?id=1184712 --- src/systemctl/systemctl.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'src/systemctl') diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c index 75d709d317..4e702fb8bc 100644 --- a/src/systemctl/systemctl.c +++ b/src/systemctl/systemctl.c @@ -4466,6 +4466,12 @@ static int show(sd_bus *bus, char **args) { if (show_properties) pager_open_if_enabled(); + if (show_status) + /* Increase max number of open files to 16K if we can, we + * might needs this when browsing journal files, which might + * be split up into many files. */ + setrlimit_closest(RLIMIT_NOFILE, &RLIMIT_MAKE_CONST(16384)); + /* If no argument is specified inspect the manager itself */ if (show_properties && strv_length(args) <= 1) @@ -7164,11 +7170,6 @@ found: } } - /* Increase max number of open files to 16K if we can, we - * might needs this when browsing journal files, which might - * be split up into many files. */ - setrlimit_closest(RLIMIT_NOFILE, &RLIMIT_MAKE_CONST(16384)); - return verb->dispatch(bus, argv + optind); } -- cgit v1.2.3-54-g00ecf