DeadPage Reference Counter Test Case

src/llfs/ioring_page_file_device.cpp

@@ -222,6 +222,9 @@ void IoRingPageFileDevice::read_some(PageId page_id, i64 page_offset_in_file,
 void IoRingPageFileDevice::drop(PageId id, WriteHandler&& handler)
 {
   // TODO [tastolfi 2021-06-11] - trim at device level?
+  // Note that we are not clearing the data blocks here. This behavior is copied replicated for
+  // SimulatedPageDevice drop() too. Thus, when modifying this function do visit
+  // SimulatedPageDevice::Impl::drop() and make sure things are updated there.
   (void)id;
   handler(OkStatus());
 }

src/llfs/simulated_page_device_impl.cpp

@@ -309,9 +309,10 @@ void SimulatedPageDevice::Impl::drop(u32 device_create_step, PageId page_id,
           return;
         }
 
-        // No failure injected; erase the DataBlock at the given index and notify the op.
+        // No failure injected.
+        // Note that we are not not erasing the data block as we want to mimic the actual
+        // io_ring Page Device behavior.
         //
-        locked_blocks->erase(block_i);
         op->set_block_result(block_i - block_0, batt::OkStatus());
       });
     });