]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
journal-file: drop unused journal_file_reload() 38199/head
authorYu Watanabe <watanabe.yu+github@gmail.com>
Tue, 15 Jul 2025 19:33:54 +0000 (04:33 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Fri, 18 Jul 2025 06:27:37 +0000 (15:27 +0900)
This partially reverts df5b3426f60bb626f46f93fbdacecae274c1645b.

src/libsystemd/sd-journal/journal-file.c
src/libsystemd/sd-journal/journal-file.h

index 209c58b95a3d42dc3cf5bd6cfbeaa834f025b6d8..f47e94fac8ba3e582beafe6ed33b7fae95d1c239 100644 (file)
@@ -4082,54 +4082,6 @@ static void journal_default_metrics(JournalMetrics *m, int fd, bool compact) {
                   m->n_max_files);
 }
 
-static uint64_t get_compress_threshold_bytes(uint64_t compress_threshold_bytes) {
-        return compress_threshold_bytes == UINT64_MAX ?
-                DEFAULT_COMPRESS_THRESHOLD :
-                MAX(MIN_COMPRESS_THRESHOLD, compress_threshold_bytes);
-}
-
-static int set_metrics(JournalFile *f, JournalMetrics *metrics, JournalFile *template) {
-        assert(f);
-        int r;
-
-        if (!journal_file_writable(f))
-                return 0;
-
-        if (metrics) {
-                journal_default_metrics(metrics, f->fd, JOURNAL_HEADER_COMPACT(f->header));
-                f->metrics = *metrics;
-        } else if (template)
-                f->metrics = template->metrics;
-
-        r = journal_file_refresh_header(f);
-        if (r < 0)
-                return log_error_errno(r, "Failed to refresh journal file header. Error to be handled by caller.");
-
-        return 0;
-}
-
-int journal_file_reload(
-                JournalFile *f,
-                JournalFileFlags file_flags,
-                uint64_t compress_threshold_bytes,
-                JournalMetrics *metrics) {
-
-        assert(f);
-        assert((file_flags & ~_JOURNAL_FILE_FLAGS_ALL) == 0);
-        assert(metrics);
-
-        int r;
-
-        f->compress_threshold_bytes = get_compress_threshold_bytes(compress_threshold_bytes);
-
-        r = set_metrics(f, metrics, /* template */ NULL);
-        if (r < 0)
-                /* Journal file header failed to be rotated. The changes may not have taken effect in this case. */
-                return r;
-
-        return 0;
-}
-
 int journal_file_open(
                 int fd,
                 const char *fname,
@@ -4169,7 +4121,9 @@ int journal_file_open(
                 .fd = fd,
                 .mode = mode,
                 .open_flags = open_flags,
-                .compress_threshold_bytes = get_compress_threshold_bytes(compress_threshold_bytes),
+                .compress_threshold_bytes = compress_threshold_bytes == UINT64_MAX ?
+                                            DEFAULT_COMPRESS_THRESHOLD :
+                                            MAX(MIN_COMPRESS_THRESHOLD, compress_threshold_bytes),
                 .strict_order = FLAGS_SET(file_flags, JOURNAL_STRICT_ORDER),
                 .newest_boot_id_prioq_idx = PRIOQ_IDX_NULL,
                 .last_direction = _DIRECTION_INVALID,
@@ -4278,9 +4232,17 @@ int journal_file_open(
         }
 #endif
 
-        r = set_metrics(f, metrics, template);
-        if (r < 0)
-                goto fail;
+        if (journal_file_writable(f)) {
+                if (metrics) {
+                        journal_default_metrics(metrics, f->fd, JOURNAL_HEADER_COMPACT(f->header));
+                        f->metrics = *metrics;
+                } else if (template)
+                        f->metrics = template->metrics;
+
+                r = journal_file_refresh_header(f);
+                if (r < 0)
+                        goto fail;
+        }
 
 #if HAVE_GCRYPT
         r = journal_file_hmac_setup(f);
index d1be26ca4c9b3cabf9cc02944e2246b20d576df3..f36db997449f4ba9c82f85205b8d3c2a08e8a9f3 100644 (file)
@@ -149,12 +149,6 @@ int journal_file_open(
                 JournalFile *template,
                 JournalFile **ret);
 
-int journal_file_reload(
-        JournalFile *f,
-        JournalFileFlags file_flags,
-        uint64_t compress_threshold_bytes,
-        JournalMetrics *metrics);
-
 int journal_file_set_offline_thread_join(JournalFile *f);
 JournalFile* journal_file_close(JournalFile *j);
 int journal_file_fstat(JournalFile *f);