On Tue, 20 May 2025 12:56:43 -0700 nifan....@gmail.com wrote: > From: Fan Ni <fan...@samsung.com> > > Per cxl r3.2 Section 9.13.3.3, extent capacity tracking should include > extents in different states including added, pending, etc. > > Before the change, for the in-device extent number tracking purpose, we only > have "total_extent_count" defined, which only tracks the number of > extents accepted. However, we need to track number of extents in other > states also, for now it is extents pending-to-add. > > To fix that, we introduce a new counter for dynamic capacity > "nr_extents_accepted" which explicitly tracks number of the extents > accepted by the hosts, and fix "total_extent_count" to include > both accepted and pending extents counting.
Hi Fan, This is complex code but looks correct to me - I'd managed to forget all the weird ways the extent counts can go up and down - most of which Linux doesn't actually hit. For fixes like this can save a bit of time by +CC m...@redhat.com then it just needs a review tag from me (or you if it is me sending). I only have that other regloc fix and just sent that out for review with mst +CC so maybe just send a v2 of this (rebased on upstream) and add Reviewed-by: Jonathan Cameron <jonathan.came...@huawei.com>
