On Mon, Oct 07, 2019 at 04:17:35PM -0400, Dennis Zhou wrote:
> --- a/fs/btrfs/free-space-cache.h
> +++ b/fs/btrfs/free-space-cache.h
> @@ -7,6 +7,7 @@
> #define BTRFS_FREE_SPACE_CACHE_H
>
> #define BTRFS_FSC_TRIMMED (1UL << 0)
> +#define BTRFS_FSC_TRIMMING_BITMAP (1UL << 1)
>
> struct btrfs_free_space {
> struct rb_node offset_index;
> @@ -23,6 +24,12 @@ static inline bool btrfs_free_space_trimmed(struct btrfs_free_space *info)
> return (info->flags & BTRFS_FSC_TRIMMED);
> }
>
> +static inline
> +bool btrfs_free_space_trimming_bitmap(struct btrfs_free_space *info)
Please keep the specifiers and type on the same line as the function,
static inline bool btrfs_free_space_trimming_bitmap(struct btrfs_free_space *info)
Short 80 column overflow of ( or { is ok (though chekpatch would report
that). I've seen split type/name in several other patches and will not
point out every occurence, so please fix it up as you find them.
