On Thu, Aug 30, 2018 at 10:42 AM, Josef Bacik <josef@xxxxxxxxxxxxxx> wrote: > From: Josef Bacik <jbacik@xxxxxx> > > We need to clear the max_extent_size when we clear bits from a bitmap > since it could have been from the range that contains the > max_extent_size. > Looks OK. Reviewed-by: Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx> thanks, liubo > Signed-off-by: Josef Bacik <jbacik@xxxxxx> > --- > fs/btrfs/free-space-cache.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c > index 53521027dd78..7faca05e61ea 100644 > --- a/fs/btrfs/free-space-cache.c > +++ b/fs/btrfs/free-space-cache.c > @@ -1683,6 +1683,8 @@ static inline void __bitmap_clear_bits(struct btrfs_free_space_ctl *ctl, > bitmap_clear(info->bitmap, start, count); > > info->bytes -= bytes; > + if (info->max_extent_size > ctl->unit) > + info->max_extent_size = 0; > } > > static void bitmap_clear_bits(struct btrfs_free_space_ctl *ctl, > -- > 2.14.3 >
