On Tue, 20 May 2025 12:56:43 -0700 [email protected] wrote: > From: Fan Ni <[email protected]> > > 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 [email protected] 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 <[email protected]>
