On Wed, Oct 23, 2019 at 06:53:01PM -0400, Dennis Zhou wrote: > The prior two patches added discarding via a background workqueue. This > just piggybacked off of the fstrim code to trim the whole block at once. > Well inevitably this is worse performance wise and will aggressively > overtrim. But it was nice to plumb the other infrastructure to keep the > patches easier to review. > > This adds the real goal of this series which is discarding slowly (ie a > slow long running fstrim). The discarding is split into two phases, > extents and then bitmaps. The reason for this is two fold. First, the > bitmap regions overlap the extent regions. Second, discarding the > extents first will let the newly trimmed bitmaps have the highest chance > of coalescing when being readded to the free space cache. > > Signed-off-by: Dennis Zhou <dennis@xxxxxxxxxx> Reviewed-by: Josef Bacik <josef@xxxxxxxxxxxxxx> Thanks, Josef
