On Fri, Apr 01, 2016 at 04:14:24PM -0400, Jeff Mahoney wrote:
> The BTRFS_QGROUP_LIMIT_* flags are required to tell the kernel which
> fields are valid when using the BTRFS_IOC_QGROUP_LIMIT ioctl.

Looks good to me.

Reviewed-by: Liu Bo <[email protected]>

Thanks,

-liubo
> 
> Signed-off-by: Jeff Mahoney <[email protected]>
> ---
>  fs/btrfs/ctree.h           |  8 --------
>  include/uapi/linux/btrfs.h | 22 +++++++++++++++++++++-
>  2 files changed, 21 insertions(+), 9 deletions(-)
> 
> diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h
> index 3beaa24..c228b39 100644
> --- a/fs/btrfs/ctree.h
> +++ b/fs/btrfs/ctree.h
> @@ -1154,14 +1154,6 @@ struct btrfs_qgroup_info_item {
>       __le64 excl_cmpr;
>  } __attribute__ ((__packed__));
>  
> -/* flags definition for qgroup limits */
> -#define BTRFS_QGROUP_LIMIT_MAX_RFER  (1ULL << 0)
> -#define BTRFS_QGROUP_LIMIT_MAX_EXCL  (1ULL << 1)
> -#define BTRFS_QGROUP_LIMIT_RSV_RFER  (1ULL << 2)
> -#define BTRFS_QGROUP_LIMIT_RSV_EXCL  (1ULL << 3)
> -#define BTRFS_QGROUP_LIMIT_RFER_CMPR (1ULL << 4)
> -#define BTRFS_QGROUP_LIMIT_EXCL_CMPR (1ULL << 5)
> -
>  struct btrfs_qgroup_limit_item {
>       /*
>        * only updated when any of the other values change
> diff --git a/include/uapi/linux/btrfs.h b/include/uapi/linux/btrfs.h
> index 11eee34..9651af3 100644
> --- a/include/uapi/linux/btrfs.h
> +++ b/include/uapi/linux/btrfs.h
> @@ -41,7 +41,19 @@ struct btrfs_ioctl_vol_args {
>  #define BTRFS_UUID_SIZE 16
>  #define BTRFS_UUID_UNPARSED_SIZE     37
>  
> -#define BTRFS_QGROUP_INHERIT_SET_LIMITS      (1ULL << 0)
> +/*
> + * flags definition for qgroup limits
> + *
> + * Used by:
> + * struct btrfs_qgroup_limit.flags
> + * struct btrfs_qgroup_limit_item.flags
> + */
> +#define BTRFS_QGROUP_LIMIT_MAX_RFER  (1ULL << 0)
> +#define BTRFS_QGROUP_LIMIT_MAX_EXCL  (1ULL << 1)
> +#define BTRFS_QGROUP_LIMIT_RSV_RFER  (1ULL << 2)
> +#define BTRFS_QGROUP_LIMIT_RSV_EXCL  (1ULL << 3)
> +#define BTRFS_QGROUP_LIMIT_RFER_CMPR (1ULL << 4)
> +#define BTRFS_QGROUP_LIMIT_EXCL_CMPR (1ULL << 5)
>  
>  struct btrfs_qgroup_limit {
>       __u64   flags;
> @@ -51,6 +63,14 @@ struct btrfs_qgroup_limit {
>       __u64   rsv_excl;
>  };
>  
> +/*
> + * flags definition for qgroup inheritance
> + *
> + * Used by:
> + * struct btrfs_qgroup_inherit.flags
> + */
> +#define BTRFS_QGROUP_INHERIT_SET_LIMITS      (1ULL << 0)
> +
>  struct btrfs_qgroup_inherit {
>       __u64   flags;
>       __u64   num_qgroups;
> -- 
> 2.7.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to