On Fri, Apr 05, 2024 at 08:44:29AM -0700, Darrick J. Wong wrote: > On Fri, Apr 05, 2024 at 08:07:08AM +0200, Christoph Hellwig wrote: > > Move the handling of discarded entries into xfs_extent_busy_clear_one > > to reuse the length check and tidy up the logic in the caller. > > > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > > AFAICT, the return value of xfs_extent_busy_clear_one is whether > or not it actually changed the pagb_tree, right? And if that return > value is true, then we want to wake up anyone who might be waiting on > busy extents to clear the pagb_tree, which is what the @wakeup logic > does, right? Yes and yes.