myoungwon oh
myoungwon oh
@athanatos The purpose of this PR is to enable CBJournal trim with renaming existing segment_cleaner. After this work, I probably can add RBM related works into EPM for ool writes....
@athanatos I added two commits to address your comments and left comments as above.
Can you take a look at updated commits?
@cyx1231st @athanatos This is a draft version document for next RBM works. Please take a look https://docs.google.com/document/d/1O9h1dMrnTAqMV7_tImoxTovgC8v1nyetXLbKsKYrxfc/edit
> Still works to do, I can help with the below two items first if they also make sense to you: > re-introduce journal_head with update_journal_head() > simplify out journal_tail_committed...
@cyx1231st Can you take a look to check if I addressed your comments?
@cyx1231st Rebase is done.
@cyx1231st OK. I'll keep my eyes on that commit. Apart from the paddr_t work, I wonder if your work (fix AsyncCleaner::SegmentMap inconsistency) is done to know when this PR will...
@cyx1231st I'll rebase Trimmer parts after https://github.com/ceph/ceph/pull/47489 is merged. Except for that, could you take a look at other updated commits?
I'm trying to understand why AsyncCleaner needs to be moved to EPM (because commits here seems like cleanup). In my understanding, EPM is in charge of data placement, and AsyncCleaner...