oom: Clean up includes

Split out of #37344.

diff --git a/src/oom/oomctl.c b/src/oom/oomctl.c
index 09df6080a89353d8da972069f3cdbb94517c71b4..d9098de0ebf1edffba4120f517bde9f0f44e3f86 100644 (file)
--- a/src/oom/oomctl.c
+++ b/src/oom/oomctl.c
@@ -1,7 +1,8 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
 #include <getopt.h>
-#include <unistd.h>
+
+#include "sd-bus.h"
 
 #include "alloc-util.h"
 #include "build.h"
@@ -12,7 +13,6 @@
 #include "main-func.h"
 #include "pager.h"
 #include "pretty-print.h"
-#include "terminal-util.h"
 #include "verbs.h"
 
 static PagerFlags arg_pager_flags = 0;

diff --git a/src/oom/oomd-conf.c b/src/oom/oomd-conf.c
index 1fcac362510a2267140c0e3c25a20831ac5bb6ec..299d2afe96c749a0af90be4c9467f2a3b36233d8 100644 (file)
--- a/src/oom/oomd-conf.c
+++ b/src/oom/oomd-conf.c
@@ -4,6 +4,9 @@
 #include "log.h"
 #include "oomd-conf.h"
 #include "oomd-manager.h"
+#include "parse-util.h"
+#include "string-util.h"
+#include "time-util.h"
 
 static int config_parse_duration(
                 const char *unit,

diff --git a/src/oom/oomd-conf.h b/src/oom/oomd-conf.h
index 0283b9e61dd992369f9c94bd76d9ebf985c9feb2..429b976b91be274045386d86289ef6a0c4284a11 100644 (file)
--- a/src/oom/oomd-conf.h
+++ b/src/oom/oomd-conf.h
@@ -1,8 +1,7 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "conf-parser.h"
-#include "oomd-manager.h"
+typedef struct Manager Manager;
 
 void manager_set_defaults(Manager *m);
 

diff --git a/src/oom/oomd-manager-bus.c b/src/oom/oomd-manager-bus.c
index f2bf6b41cb5d02c1184ed5beeef22b8fc55e3451..08f261fcc1a4fc3b5b63d4db97b7be6d9a477f3c 100644 (file)
--- a/src/oom/oomd-manager-bus.c
+++ b/src/oom/oomd-manager-bus.c
@@ -1,15 +1,13 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <linux/capability.h>
+#include "sd-bus.h"
 
 #include "alloc-util.h"
-#include "bus-common-errors.h"
-#include "bus-polkit.h"
+#include "bus-object.h"
 #include "fd-util.h"
 #include "memfd-util.h"
 #include "oomd-manager.h"
 #include "oomd-manager-bus.h"
-#include "user-util.h"
 
 static int bus_method_dump_by_fd(sd_bus_message *message, void *userdata, sd_bus_error *error) {
         _cleanup_free_ char *dump = NULL;

diff --git a/src/oom/oomd-manager-bus.h b/src/oom/oomd-manager-bus.h
index 7935b357a4dd20d324f2b0b74dff1f2fdcdd43a8..70c473f15be559c73d89a02f37eb134aad3e4a5d 100644 (file)
--- a/src/oom/oomd-manager-bus.h
+++ b/src/oom/oomd-manager-bus.h
@@ -1,8 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "bus-object.h"
-
-typedef struct Manager Manager;
+#include "forward.h"
 
 extern const BusObjectImplementation manager_object;

diff --git a/src/oom/oomd-manager.c b/src/oom/oomd-manager.c
index b494ea0c284c31a6b65bffeda63d0c75554a59e0..2cdd404bf00e1a6346767b0cfd8d6d26a5ab0cfa 100644 (file)
--- a/src/oom/oomd-manager.c
+++ b/src/oom/oomd-manager.c
@@ -1,25 +1,30 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include "sd-bus.h"
 #include "sd-daemon.h"
+#include "sd-event.h"
 #include "sd-json.h"
 
 #include "alloc-util.h"
 #include "bus-log-control-api.h"
-#include "bus-polkit.h"
+#include "bus-object.h"
 #include "bus-util.h"
 #include "cgroup-util.h"
+#include "constants.h"
 #include "daemon-util.h"
 #include "fd-util.h"
-#include "fileio.h"
 #include "format-util.h"
 #include "json-util.h"
-#include "memory-util.h"
 #include "memstream-util.h"
 #include "oomd-conf.h"
 #include "oomd-manager.h"
 #include "oomd-manager-bus.h"
+#include "parse-util.h"
 #include "path-util.h"
 #include "percent-util.h"
+#include "set.h"
+#include "string-util.h"
+#include "time-util.h"
 #include "varlink-io.systemd.oom.h"
 #include "varlink-io.systemd.service.h"
 #include "varlink-util.h"

diff --git a/src/oom/oomd-manager.h b/src/oom/oomd-manager.h
index a40946ce4e1426285f1fc892331282735a566da5..311250c6f35f31037b0bff47762a973093c74dc7 100644 (file)
--- a/src/oom/oomd-manager.h
+++ b/src/oom/oomd-manager.h
@@ -1,11 +1,8 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include "sd-bus.h"
-#include "sd-event.h"
-#include "sd-varlink.h"
-
-#include "conf-parser.h"
+#include "conf-parser-forward.h"
+#include "forward.h"
 #include "oomd-util.h"
 
 /* Polling interval for monitoring stats */
@@ -27,9 +24,7 @@
 #define RECLAIM_DURATION_USEC (30 * USEC_PER_SEC)
 #define POST_ACTION_DELAY_USEC (15 * USEC_PER_SEC)
 
-typedef struct Manager Manager;
-
-struct Manager {
+typedef struct Manager {
         sd_bus *bus;
         sd_event *event;
 
@@ -59,7 +54,7 @@ struct Manager {
         /* This varlink server object is used to manage systemd-oomd's varlink server which is used by user
          * managers to report changes in ManagedOOM settings (oomd server - systemd client). */
         sd_varlink_server *varlink_server;
-};
+} Manager;
 
 Manager* manager_free(Manager *m);
 DEFINE_TRIVIAL_CLEANUP_FUNC(Manager*, manager_free);

diff --git a/src/oom/oomd-util.c b/src/oom/oomd-util.c
index b6b6c2a2f1ab604059febb3af28033ae58db8343..de0f2e685409db142cbfa212db55f871e6122926 100644 (file)
--- a/src/oom/oomd-util.c
+++ b/src/oom/oomd-util.c
@@ -1,8 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
-#include <sys/xattr.h>
-#include <unistd.h>
-
+#include "alloc-util.h"
 #include "errno-util.h"
 #include "fd-util.h"
 #include "fileio.h"
@@ -12,12 +10,14 @@
 #include "oomd-util.h"
 #include "parse-util.h"
 #include "path-util.h"
+#include "pidref.h"
 #include "procfs-util.h"
+#include "set.h"
 #include "signal-util.h"
 #include "sort-util.h"
-#include "stat-util.h"
 #include "stdio-util.h"
-#include "user-util.h"
+#include "string-util.h"
+#include "time-util.h"
 
 DEFINE_HASH_OPS_WITH_VALUE_DESTRUCTOR(
                 oomd_cgroup_ctx_hash_ops,

diff --git a/src/oom/oomd-util.h b/src/oom/oomd-util.h
index 14fe5c5ebab31b5b5d53f1219b973226807bba45..cfb310ff32ce9f39af2160553c899b230083185c 100644 (file)
--- a/src/oom/oomd-util.h
+++ b/src/oom/oomd-util.h
@@ -1,10 +1,8 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 #pragma once
 
-#include <stdbool.h>
-
 #include "cgroup-util.h"
-#include "hashmap.h"
+#include "forward.h"
 #include "psi-util.h"
 
 #define DUMP_ON_KILL_COUNT 10u

diff --git a/src/oom/oomd.c b/src/oom/oomd.c
index d0876d789f0f751a840b8434f48bcebca38abe67..b1d3efb8f5733016edd9ff6bf65e9d6b0c730e45 100644 (file)
--- a/src/oom/oomd.c
+++ b/src/oom/oomd.c
@@ -2,12 +2,13 @@
 
 #include <getopt.h>
 
+#include "sd-event.h"
+
 #include "alloc-util.h"
 #include "build.h"
 #include "bus-log-control-api.h"
 #include "bus-object.h"
 #include "cgroup-util.h"
-#include "conf-parser.h"
 #include "daemon-util.h"
 #include "fileio.h"
 #include "log.h"
@@ -18,7 +19,6 @@
 #include "parse-util.h"
 #include "pretty-print.h"
 #include "psi-util.h"
-#include "signal-util.h"
 
 static bool arg_dry_run = false;
 

diff --git a/src/oom/test-oomd-util.c b/src/oom/test-oomd-util.c
index 2eca39c29585ce9d732fc10c5595d076a3a3efbe..3c5159129f73dd91a9c42364c535084e3ca71a97 100644 (file)
--- a/src/oom/test-oomd-util.c
+++ b/src/oom/test-oomd-util.c
@@ -1,5 +1,6 @@
 /* SPDX-License-Identifier: LGPL-2.1-or-later */
 
+#include <stdlib.h>
 #include <unistd.h>
 
 #include "alloc-util.h"
@@ -7,13 +8,12 @@
 #include "cgroup-util.h"
 #include "fd-util.h"
 #include "fileio.h"
-#include "fs-util.h"
 #include "oomd-util.h"
 #include "parse-util.h"
 #include "path-util.h"
-#include "string-util.h"
-#include "strv.h"
+#include "set.h"
 #include "tests.h"
+#include "time-util.h"
 #include "tmpfile-util.h"
 
 static int fork_and_sleep(unsigned sleep_min) {