]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
sd-journal: drop redundant re-read of entry array object
authorYu Watanabe <watanabe.yu+github@gmail.com>
Mon, 25 Sep 2023 01:44:50 +0000 (10:44 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Sat, 30 Sep 2023 11:10:15 +0000 (20:10 +0900)
This effectively reverts d9b61db922404a216de018cc5ddff9b69bcaf1db.

In the do-while loop, we do not read any other entry array object, hence
the current object is always in the mmap cache and not necessary to re-read it.

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

index 5d3f85e4cc38f7d5eda26492bae6da33334e66f0..31d4db97c0c792cf0ecd6c88518de99439d325e9 100644 (file)
@@ -2840,10 +2840,6 @@ static int generic_array_get(
                          * disk properly, let's see if the next one might work for us instead. */
                         log_debug_errno(r, "Entry item %" PRIu64 " is bad, skipping over it.", i);
 
-                        r = journal_file_move_to_object(f, OBJECT_ENTRY_ARRAY, a, &o);
-                        if (r < 0)
-                                return r;
-
                 } while (bump_array_index(&i, direction, k) > 0);
 
                 r = bump_entry_array(f, o, a, first, direction, &a);