]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
osd_types: Restore new_object marking for delete missing entries 62705/head
authorNitzan Mordechai <nmordech@redhat.com>
Wed, 26 Mar 2025 08:20:15 +0000 (08:20 +0000)
committerNitzan Mordechai <nmordech@redhat.com>
Mon, 7 Apr 2025 13:00:49 +0000 (13:00 +0000)
Recent changes (PR #29893) removed the “new_object” parameter from missing.add() and the
pg_missing_item constructor. As a result, when processing delete log entries,
if an object is found on disk, its on‑disk version is stored as “have” instead
of the default eversion_t() (0'0). The invariant in read_log_and_missing() then
fails because delete entries are expected to have “have” set to eversion_t().

This patch reintroduces the following check:
if (have == eversion_t())
    clean_regions.mark_object_new();

By doing so, we ensure that when the on‑disk “have” is default, the missing record
is marked as new—restoring the previous behavior and satisfying the invariant for
delete operations.

Fixes: https://tracker.ceph.com/issues/45702
Signed-off-by: Nitzan Mordechai <nmordech@redhat.com>
src/osd/osd_types.h

index ebc1d359ee2ebaa8c8fef54ac830d090443d59de..d33007822f0f7e42cbd0f63ef5524319f1eca754 100644 (file)
@@ -4787,6 +4787,8 @@ struct pg_missing_item {
     set_delete(is_delete);
     if (old_style)
       clean_regions.mark_fully_dirty();
+    if (have == eversion_t())
+      clean_regions.mark_object_new();
   }
 
   void encode(ceph::buffer::list& bl, uint64_t features) const {