On Thu, Apr 15, 2021 at 10:58:35PM +0900, Johannes Thumshirn wrote:
> --- a/fs/btrfs/ctree.h
> +++ b/fs/btrfs/ctree.h
> @@ -954,10 +954,14 @@ struct btrfs_fs_info {
> struct work_struct async_data_reclaim_work;
> struct work_struct preempt_reclaim_work;
>
> + /* Used to reclaim data space in the background */
> + struct work_struct reclaim_bgs_work;
> +
> spinlock_t unused_bgs_lock;
> struct list_head unused_bgs;
> struct mutex unused_bg_unpin_mutex;
> struct mutex reclaim_bgs_lock;
> + struct list_head reclaim_bgs;
>
> /* Cached block sizes */
> u32 nodesize;
> @@ -998,6 +1002,8 @@ struct btrfs_fs_info {
> spinlock_t treelog_bg_lock;
> u64 treelog_bg;
>
> + int bg_reclaim_threshold;
That's spreading related members over too many lines, please put them
together with comments what they mean if that's not obvious (like for
the work struct).
