* [PATCH] btrfs: add missing check for nocow and compression inode flags
@ 2020-07-10 10:05 David Sterba
2020-07-10 10:10 ` Nikolay Borisov
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: David Sterba @ 2020-07-10 10:05 UTC (permalink / raw)
To: linux-btrfs; +Cc: David Sterba, stable
User Forza reported on IRC that some invalid combinations of file
attributes are accepted by chattr.
The NODATACOW and compression file flags/attributes are mutually
exclusive, but they could be set by 'chattr +c +C' on an empty file. The
nodatacow will be in effect because it's checked first in
btrfs_run_delalloc_range.
Extend the flag validation to catch the following cases:
- input flags are conflicting
- old and new flags are conflicting
- initialize the local variable with inode flags after inode ls locked
CC: stable@vger.kernel.org # 4.4+
Signed-off-by: David Sterba <dsterba@suse.com>
---
fs/btrfs/ioctl.c | 30 ++++++++++++++++++++++--------
1 file changed, 22 insertions(+), 8 deletions(-)
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c
index 3a566cf71fc6..0c13bb38425b 100644
--- a/fs/btrfs/ioctl.c
+++ b/fs/btrfs/ioctl.c
@@ -164,8 +164,11 @@ static int btrfs_ioctl_getflags(struct file *file, void __user *arg)
return 0;
}
-/* Check if @flags are a supported and valid set of FS_*_FL flags */
-static int check_fsflags(unsigned int flags)
+/*
+ * Check if @flags are a supported and valid set of FS_*_FL flags and that
+ * the old and new flags are not conflicting
+ */
+static int check_fsflags(unsigned int old_flags, unsigned int flags)
{
if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
FS_NOATIME_FL | FS_NODUMP_FL | \
@@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
FS_NOCOW_FL))
return -EOPNOTSUPP;
+ /* COMPR and NOCOMP on new/old are valid */
if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
return -EINVAL;
+ if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
+ return -EINVAL;
+
+ /* NOCOW and compression options are mutually exclusive */
+ if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
+ return -EINVAL;
+ if ((flags & FS_NOCOW_FL) && (old_flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
+ return -EINVAL;
+
return 0;
}
@@ -190,7 +203,7 @@ static int btrfs_ioctl_setflags(struct file *file, void __user *arg)
unsigned int fsflags, old_fsflags;
int ret;
const char *comp = NULL;
- u32 binode_flags = binode->flags;
+ u32 binode_flags;
if (!inode_owner_or_capable(inode))
return -EPERM;
@@ -201,22 +214,23 @@ static int btrfs_ioctl_setflags(struct file *file, void __user *arg)
if (copy_from_user(&fsflags, arg, sizeof(fsflags)))
return -EFAULT;
- ret = check_fsflags(fsflags);
- if (ret)
- return ret;
-
ret = mnt_want_write_file(file);
if (ret)
return ret;
inode_lock(inode);
-
fsflags = btrfs_mask_fsflags_for_type(inode, fsflags);
old_fsflags = btrfs_inode_flags_to_fsflags(binode->flags);
+
ret = vfs_ioc_setflags_prepare(inode, old_fsflags, fsflags);
if (ret)
goto out_unlock;
+ ret = check_fsflags(old_fsflags, fsflags);
+ if (ret)
+ goto out_unlock;
+
+ binode_flags = binode->flags;
if (fsflags & FS_SYNC_FL)
binode_flags |= BTRFS_INODE_SYNC;
else
--
2.25.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] btrfs: add missing check for nocow and compression inode flags
2020-07-10 10:05 [PATCH] btrfs: add missing check for nocow and compression inode flags David Sterba
@ 2020-07-10 10:10 ` Nikolay Borisov
2020-07-10 13:34 ` David Sterba
2020-07-10 10:17 ` Johannes Thumshirn
` (2 subsequent siblings)
3 siblings, 1 reply; 9+ messages in thread
From: Nikolay Borisov @ 2020-07-10 10:10 UTC (permalink / raw)
To: David Sterba, linux-btrfs; +Cc: stable
On 10.07.20 г. 13:05 ч., David Sterba wrote:
> User Forza reported on IRC that some invalid combinations of file
> attributes are accepted by chattr.
>
> The NODATACOW and compression file flags/attributes are mutually
> exclusive, but they could be set by 'chattr +c +C' on an empty file. The
> nodatacow will be in effect because it's checked first in
> btrfs_run_delalloc_range.
>
> Extend the flag validation to catch the following cases:
>
> - input flags are conflicting
> - old and new flags are conflicting
> - initialize the local variable with inode flags after inode ls locked
>
> CC: stable@vger.kernel.org # 4.4+
> Signed-off-by: David Sterba <dsterba@suse.com>
> ---
> fs/btrfs/ioctl.c | 30 ++++++++++++++++++++++--------
> 1 file changed, 22 insertions(+), 8 deletions(-)
>
> diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c
> index 3a566cf71fc6..0c13bb38425b 100644
> --- a/fs/btrfs/ioctl.c
> +++ b/fs/btrfs/ioctl.c
> @@ -164,8 +164,11 @@ static int btrfs_ioctl_getflags(struct file *file, void __user *arg)
> return 0;
> }
>
> -/* Check if @flags are a supported and valid set of FS_*_FL flags */
> -static int check_fsflags(unsigned int flags)
> +/*
> + * Check if @flags are a supported and valid set of FS_*_FL flags and that
> + * the old and new flags are not conflicting
> + */
> +static int check_fsflags(unsigned int old_flags, unsigned int flags)
> {
> if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
> FS_NOATIME_FL | FS_NODUMP_FL | \
> @@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
> FS_NOCOW_FL))
> return -EOPNOTSUPP;
>
> + /* COMPR and NOCOMP on new/old are valid */
> if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
> return -EINVAL;
>
> + if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
> + return -EINVAL;
> +
> + /* NOCOW and compression options are mutually exclusive */
> + if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
Why is NOCOW and setting NOCOMP (which would really be a NOOP) an
invalid combination?
> + return -EINVAL;
> + if ((flags & FS_NOCOW_FL) && (old_flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
> + return -EINVAL;
Same thing here, just inverted?
> +
> return 0;
> }
>
> @@ -190,7 +203,7 @@ static int btrfs_ioctl_setflags(struct file *file, void __user *arg)
> unsigned int fsflags, old_fsflags;
> int ret;
> const char *comp = NULL;
> - u32 binode_flags = binode->flags;
> + u32 binode_flags;
>
> if (!inode_owner_or_capable(inode))
> return -EPERM;
> @@ -201,22 +214,23 @@ static int btrfs_ioctl_setflags(struct file *file, void __user *arg)
> if (copy_from_user(&fsflags, arg, sizeof(fsflags)))
> return -EFAULT;
>
> - ret = check_fsflags(fsflags);
> - if (ret)
> - return ret;
> -
> ret = mnt_want_write_file(file);
> if (ret)
> return ret;
>
> inode_lock(inode);
> -
> fsflags = btrfs_mask_fsflags_for_type(inode, fsflags);
> old_fsflags = btrfs_inode_flags_to_fsflags(binode->flags);
> +
> ret = vfs_ioc_setflags_prepare(inode, old_fsflags, fsflags);
> if (ret)
> goto out_unlock;
>
> + ret = check_fsflags(old_fsflags, fsflags);
> + if (ret)
> + goto out_unlock;
> +
> + binode_flags = binode->flags;
> if (fsflags & FS_SYNC_FL)
> binode_flags |= BTRFS_INODE_SYNC;
> else
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] btrfs: add missing check for nocow and compression inode flags
2020-07-10 10:05 [PATCH] btrfs: add missing check for nocow and compression inode flags David Sterba
2020-07-10 10:10 ` Nikolay Borisov
@ 2020-07-10 10:17 ` Johannes Thumshirn
2020-07-10 13:28 ` David Sterba
2020-07-13 10:33 ` [PATCH v2] " David Sterba
2020-07-16 0:27 ` [PATCH] " Sasha Levin
3 siblings, 1 reply; 9+ messages in thread
From: Johannes Thumshirn @ 2020-07-10 10:17 UTC (permalink / raw)
To: David Sterba, linux-btrfs; +Cc: stable
On 10/07/2020 12:06, David Sterba wrote:
> +static int check_fsflags(unsigned int old_flags, unsigned int flags)
> {
> if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
> FS_NOATIME_FL | FS_NODUMP_FL | \
> @@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
> FS_NOCOW_FL))
> return -EOPNOTSUPP;
>
> + /* COMPR and NOCOMP on new/old are valid */
> if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
> return -EINVAL;
>
> + if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
> + return -EINVAL;
> +
> + /* NOCOW and compression options are mutually exclusive */
> + if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
> + return -EINVAL;
> + if ((flags & FS_NOCOW_FL) && (old_flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
> + return -EINVAL;
> +
If we'd pass in fs_info to check_fsflags() we could also validate against mount options
which are incompatible with inode flags. Like -o nodatacow and FS_COMPR_FL or
-o auth_key and FS_NOCOW_FL.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] btrfs: add missing check for nocow and compression inode flags
2020-07-10 10:17 ` Johannes Thumshirn
@ 2020-07-10 13:28 ` David Sterba
0 siblings, 0 replies; 9+ messages in thread
From: David Sterba @ 2020-07-10 13:28 UTC (permalink / raw)
To: Johannes Thumshirn; +Cc: David Sterba, linux-btrfs, stable
On Fri, Jul 10, 2020 at 10:17:55AM +0000, Johannes Thumshirn wrote:
> On 10/07/2020 12:06, David Sterba wrote:
> > +static int check_fsflags(unsigned int old_flags, unsigned int flags)
> > {
> > if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
> > FS_NOATIME_FL | FS_NODUMP_FL | \
> > @@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
> > FS_NOCOW_FL))
> > return -EOPNOTSUPP;
> >
> > + /* COMPR and NOCOMP on new/old are valid */
> > if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
> > return -EINVAL;
> >
> > + if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
> > + return -EINVAL;
> > +
> > + /* NOCOW and compression options are mutually exclusive */
> > + if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
> > + return -EINVAL;
> > + if ((flags & FS_NOCOW_FL) && (old_flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
> > + return -EINVAL;
> > +
>
>
> If we'd pass in fs_info to check_fsflags() we could also validate against mount options
> which are incompatible with inode flags. Like -o nodatacow and FS_COMPR_FL or
> -o auth_key and FS_NOCOW_FL.
Same question was asked on IRC too, mount options are independent and
take lower precedence than the inode attributes. A scenario where user
wants to set a nodatacow attribute when the filesystem is mounted with
compress= is valid and can be quite common.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] btrfs: add missing check for nocow and compression inode flags
2020-07-10 10:10 ` Nikolay Borisov
@ 2020-07-10 13:34 ` David Sterba
2020-07-10 14:55 ` Nikolay Borisov
0 siblings, 1 reply; 9+ messages in thread
From: David Sterba @ 2020-07-10 13:34 UTC (permalink / raw)
To: Nikolay Borisov; +Cc: David Sterba, linux-btrfs, stable
On Fri, Jul 10, 2020 at 01:10:25PM +0300, Nikolay Borisov wrote:
> > +static int check_fsflags(unsigned int old_flags, unsigned int flags)
> > {
> > if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
> > FS_NOATIME_FL | FS_NODUMP_FL | \
> > @@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
> > FS_NOCOW_FL))
> > return -EOPNOTSUPP;
> >
> > + /* COMPR and NOCOMP on new/old are valid */
> > if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
> > return -EINVAL;
> >
> > + if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
> > + return -EINVAL;
> > +
> > + /* NOCOW and compression options are mutually exclusive */
> > + if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
>
> Why is NOCOW and setting NOCOMP (which would really be a NOOP) an
> invalid combination?
The options are not conflicting directly, like for the compression and
nodatacow, but it still is related to compression so it does not feel
right to allow that even if it's a noop.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] btrfs: add missing check for nocow and compression inode flags
2020-07-10 13:34 ` David Sterba
@ 2020-07-10 14:55 ` Nikolay Borisov
0 siblings, 0 replies; 9+ messages in thread
From: Nikolay Borisov @ 2020-07-10 14:55 UTC (permalink / raw)
To: dsterba, David Sterba, linux-btrfs, stable
On 10.07.20 г. 16:34 ч., David Sterba wrote:
> On Fri, Jul 10, 2020 at 01:10:25PM +0300, Nikolay Borisov wrote:
>>> +static int check_fsflags(unsigned int old_flags, unsigned int flags)
>>> {
>>> if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
>>> FS_NOATIME_FL | FS_NODUMP_FL | \
>>> @@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
>>> FS_NOCOW_FL))
>>> return -EOPNOTSUPP;
>>>
>>> + /* COMPR and NOCOMP on new/old are valid */
>>> if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
>>> return -EINVAL;
>>>
>>> + if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
>>> + return -EINVAL;
>>> +
>>> + /* NOCOW and compression options are mutually exclusive */
>>> + if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
>>
>> Why is NOCOW and setting NOCOMP (which would really be a NOOP) an
>> invalid combination?
>
> The options are not conflicting directly, like for the compression and
> nodatacow, but it still is related to compression so it does not feel
> right to allow that even if it's a noop.
>
Please put this reasoning in the changelog.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2] btrfs: add missing check for nocow and compression inode flags
2020-07-10 10:05 [PATCH] btrfs: add missing check for nocow and compression inode flags David Sterba
2020-07-10 10:10 ` Nikolay Borisov
2020-07-10 10:17 ` Johannes Thumshirn
@ 2020-07-13 10:33 ` David Sterba
2020-07-17 17:08 ` Sasha Levin
2020-07-16 0:27 ` [PATCH] " Sasha Levin
3 siblings, 1 reply; 9+ messages in thread
From: David Sterba @ 2020-07-13 10:33 UTC (permalink / raw)
To: linux-btrfs; +Cc: David Sterba, stable
User Forza reported on IRC that some invalid combinations of file
attributes are accepted by chattr.
The NODATACOW and compression file flags/attributes are mutually
exclusive, but they could be set by 'chattr +c +C' on an empty file. The
nodatacow will be in effect because it's checked first in
btrfs_run_delalloc_range.
Extend the flag validation to catch the following cases:
- input flags are conflicting
- old and new flags are conflicting
- initialize the local variable with inode flags after inode ls locked
Inode attributes take precedence over mount options and are an
independent setting.
Nocompress would be a no-op with nodatacow, but we don't want to mix
any compression-related options with nodatacow.
CC: stable@vger.kernel.org # 4.4+
Signed-off-by: David Sterba <dsterba@suse.com>
---
v2:
- update chanelog: mount options vs attributes, nocompress vs nodatacow
fs/btrfs/ioctl.c | 30 ++++++++++++++++++++++--------
1 file changed, 22 insertions(+), 8 deletions(-)
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c
index 3a566cf71fc6..0c13bb38425b 100644
--- a/fs/btrfs/ioctl.c
+++ b/fs/btrfs/ioctl.c
@@ -164,8 +164,11 @@ static int btrfs_ioctl_getflags(struct file *file, void __user *arg)
return 0;
}
-/* Check if @flags are a supported and valid set of FS_*_FL flags */
-static int check_fsflags(unsigned int flags)
+/*
+ * Check if @flags are a supported and valid set of FS_*_FL flags and that
+ * the old and new flags are not conflicting
+ */
+static int check_fsflags(unsigned int old_flags, unsigned int flags)
{
if (flags & ~(FS_IMMUTABLE_FL | FS_APPEND_FL | \
FS_NOATIME_FL | FS_NODUMP_FL | \
@@ -174,9 +177,19 @@ static int check_fsflags(unsigned int flags)
FS_NOCOW_FL))
return -EOPNOTSUPP;
+ /* COMPR and NOCOMP on new/old are valid */
if ((flags & FS_NOCOMP_FL) && (flags & FS_COMPR_FL))
return -EINVAL;
+ if ((flags & FS_COMPR_FL) && (flags & FS_NOCOW_FL))
+ return -EINVAL;
+
+ /* NOCOW and compression options are mutually exclusive */
+ if ((old_flags & FS_NOCOW_FL) && (flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
+ return -EINVAL;
+ if ((flags & FS_NOCOW_FL) && (old_flags & (FS_COMPR_FL | FS_NOCOMP_FL)))
+ return -EINVAL;
+
return 0;
}
@@ -190,7 +203,7 @@ static int btrfs_ioctl_setflags(struct file *file, void __user *arg)
unsigned int fsflags, old_fsflags;
int ret;
const char *comp = NULL;
- u32 binode_flags = binode->flags;
+ u32 binode_flags;
if (!inode_owner_or_capable(inode))
return -EPERM;
@@ -201,22 +214,23 @@ static int btrfs_ioctl_setflags(struct file *file, void __user *arg)
if (copy_from_user(&fsflags, arg, sizeof(fsflags)))
return -EFAULT;
- ret = check_fsflags(fsflags);
- if (ret)
- return ret;
-
ret = mnt_want_write_file(file);
if (ret)
return ret;
inode_lock(inode);
-
fsflags = btrfs_mask_fsflags_for_type(inode, fsflags);
old_fsflags = btrfs_inode_flags_to_fsflags(binode->flags);
+
ret = vfs_ioc_setflags_prepare(inode, old_fsflags, fsflags);
if (ret)
goto out_unlock;
+ ret = check_fsflags(old_fsflags, fsflags);
+ if (ret)
+ goto out_unlock;
+
+ binode_flags = binode->flags;
if (fsflags & FS_SYNC_FL)
binode_flags |= BTRFS_INODE_SYNC;
else
--
2.25.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] btrfs: add missing check for nocow and compression inode flags
2020-07-10 10:05 [PATCH] btrfs: add missing check for nocow and compression inode flags David Sterba
` (2 preceding siblings ...)
2020-07-13 10:33 ` [PATCH v2] " David Sterba
@ 2020-07-16 0:27 ` Sasha Levin
3 siblings, 0 replies; 9+ messages in thread
From: Sasha Levin @ 2020-07-16 0:27 UTC (permalink / raw)
To: Sasha Levin, David Sterba, linux-btrfs; +Cc: David Sterba, stable, stable
Hi
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: 4.4+
The bot has tested the following trees: v5.7.8, v5.4.51, v4.19.132, v4.14.188, v4.9.230, v4.4.230.
v5.7.8: Build OK!
v5.4.51: Build OK!
v4.19.132: Failed to apply! Possible dependencies:
04e6863b19c72 ("btrfs: split btrfs_setxattr calls regarding transaction")
262c96a3c3670 ("btrfs: refactor btrfs_set_prop and add btrfs_set_prop_trans")
7715da84f74d5 ("btrfs: merge _btrfs_set_prop helpers")
8b4d1efc9e6c3 ("btrfs: prop: open code btrfs_set_prop in inherit_prop")
cac237ae095f6 ("btrfs: rename btrfs_setxattr to btrfs_setxattr_trans")
d2b8fcfe43155 ("btrfs: modify local copy of btrfs_inode flags")
f22125e5d8ae1 ("btrfs: refactor btrfs_set_props to validate externally")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
v4.14.188: Failed to apply! Possible dependencies:
04e6863b19c72 ("btrfs: split btrfs_setxattr calls regarding transaction")
1905a0f7c7de3 ("btrfs: rename btrfs_mask_flags to reflect which flags it touches")
262c96a3c3670 ("btrfs: refactor btrfs_set_prop and add btrfs_set_prop_trans")
38e82de8ccd18 ("btrfs: user proper type for btrfs_mask_flags flags")
5ba76abfb2336 ("btrfs: rename check_flags to reflect which flags it touches")
5c57b8b6a4966 ("btrfs: unify naming of flags variables for SETFLAGS and XFLAGS")
7715da84f74d5 ("btrfs: merge _btrfs_set_prop helpers")
7852781d94b30 ("btrfs: drop underscores from exported xattr functions")
7b6a221e5b21f ("btrfs: rename btrfs_update_iflags to reflect which flags it touches")
8b4d1efc9e6c3 ("btrfs: prop: open code btrfs_set_prop in inherit_prop")
93370509c24cc ("btrfs: SETFLAGS ioctl: use helper for compression type conversion")
a157d4fd81dc7 ("btrfs: rename btrfs_flags_to_ioctl to reflect which flags it touches")
ab0d09361662b ("btrfs: drop extern from function declarations")
cac237ae095f6 ("btrfs: rename btrfs_setxattr to btrfs_setxattr_trans")
d2b8fcfe43155 ("btrfs: modify local copy of btrfs_inode flags")
f22125e5d8ae1 ("btrfs: refactor btrfs_set_props to validate externally")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
v4.9.230: Failed to apply! Possible dependencies:
0b246afa62b0c ("btrfs: root->fs_info cleanup, add fs_info convenience variables")
1905a0f7c7de3 ("btrfs: rename btrfs_mask_flags to reflect which flags it touches")
38e82de8ccd18 ("btrfs: user proper type for btrfs_mask_flags flags")
5ba76abfb2336 ("btrfs: rename check_flags to reflect which flags it touches")
5c57b8b6a4966 ("btrfs: unify naming of flags variables for SETFLAGS and XFLAGS")
62d1f9fe97dd2 ("btrfs: remove trivial helper btrfs_find_tree_block")
a157d4fd81dc7 ("btrfs: rename btrfs_flags_to_ioctl to reflect which flags it touches")
cf8cddd38bab3 ("btrfs: don't abuse REQ_OP_* flags for btrfs_map_block")
da17066c40472 ("btrfs: pull node/sector/stripe sizes out of root and into fs_info")
de143792253e2 ("btrfs: struct btrfsic_state->root should be an fs_info")
fb456252d3d9c ("btrfs: root->fs_info cleanup, use fs_info->dev_root everywhere")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
v4.4.230: Failed to apply! Possible dependencies:
0132761017e01 ("btrfs: fix string and comment grammatical issues and typos")
09cbfeaf1a5a6 ("mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros")
0b246afa62b0c ("btrfs: root->fs_info cleanup, add fs_info convenience variables")
0e749e54244ee ("dax: increase granularity of dax_clear_blocks() operations")
1905a0f7c7de3 ("btrfs: rename btrfs_mask_flags to reflect which flags it touches")
38e82de8ccd18 ("btrfs: user proper type for btrfs_mask_flags flags")
4420cfd3f51cf ("staging: lustre: format properly all comment blocks for LNet core")
52db400fcd502 ("pmem, dax: clean up clear_pmem()")
5ba76abfb2336 ("btrfs: rename check_flags to reflect which flags it touches")
5c57b8b6a4966 ("btrfs: unify naming of flags variables for SETFLAGS and XFLAGS")
5fd88337d209d ("staging: lustre: fix all conditional comparison to zero in LNet layer")
a157d4fd81dc7 ("btrfs: rename btrfs_flags_to_ioctl to reflect which flags it touches")
b2e0d1625e193 ("dax: fix lifetime of in-kernel dax mappings with dax_map_atomic()")
bb7ab3b92e46d ("btrfs: Fix misspellings in comments.")
cf8cddd38bab3 ("btrfs: don't abuse REQ_OP_* flags for btrfs_map_block")
d1a5f2b4d8a12 ("block: use DAX for partition table reads")
de143792253e2 ("btrfs: struct btrfsic_state->root should be an fs_info")
e10624f8c0971 ("pmem: fail io-requests to known bad blocks")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
NOTE: The patch will not be queued to stable trees until it is upstream.
How should we proceed with this patch?
--
Thanks
Sasha
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] btrfs: add missing check for nocow and compression inode flags
2020-07-13 10:33 ` [PATCH v2] " David Sterba
@ 2020-07-17 17:08 ` Sasha Levin
0 siblings, 0 replies; 9+ messages in thread
From: Sasha Levin @ 2020-07-17 17:08 UTC (permalink / raw)
To: Sasha Levin, David Sterba, linux-btrfs; +Cc: David Sterba, stable, stable
Hi
[This is an automated email]
This commit has been processed because it contains a -stable tag.
The stable tag indicates that it's relevant for the following trees: 4.4+
The bot has tested the following trees: v5.7.8, v5.4.51, v4.19.132, v4.14.188, v4.9.230, v4.4.230.
v5.7.8: Build OK!
v5.4.51: Build OK!
v4.19.132: Failed to apply! Possible dependencies:
04e6863b19c72 ("btrfs: split btrfs_setxattr calls regarding transaction")
262c96a3c3670 ("btrfs: refactor btrfs_set_prop and add btrfs_set_prop_trans")
7715da84f74d5 ("btrfs: merge _btrfs_set_prop helpers")
8b4d1efc9e6c3 ("btrfs: prop: open code btrfs_set_prop in inherit_prop")
cac237ae095f6 ("btrfs: rename btrfs_setxattr to btrfs_setxattr_trans")
d2b8fcfe43155 ("btrfs: modify local copy of btrfs_inode flags")
f22125e5d8ae1 ("btrfs: refactor btrfs_set_props to validate externally")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
v4.14.188: Failed to apply! Possible dependencies:
04e6863b19c72 ("btrfs: split btrfs_setxattr calls regarding transaction")
1905a0f7c7de3 ("btrfs: rename btrfs_mask_flags to reflect which flags it touches")
262c96a3c3670 ("btrfs: refactor btrfs_set_prop and add btrfs_set_prop_trans")
38e82de8ccd18 ("btrfs: user proper type for btrfs_mask_flags flags")
5ba76abfb2336 ("btrfs: rename check_flags to reflect which flags it touches")
5c57b8b6a4966 ("btrfs: unify naming of flags variables for SETFLAGS and XFLAGS")
7715da84f74d5 ("btrfs: merge _btrfs_set_prop helpers")
7852781d94b30 ("btrfs: drop underscores from exported xattr functions")
7b6a221e5b21f ("btrfs: rename btrfs_update_iflags to reflect which flags it touches")
8b4d1efc9e6c3 ("btrfs: prop: open code btrfs_set_prop in inherit_prop")
93370509c24cc ("btrfs: SETFLAGS ioctl: use helper for compression type conversion")
a157d4fd81dc7 ("btrfs: rename btrfs_flags_to_ioctl to reflect which flags it touches")
ab0d09361662b ("btrfs: drop extern from function declarations")
cac237ae095f6 ("btrfs: rename btrfs_setxattr to btrfs_setxattr_trans")
d2b8fcfe43155 ("btrfs: modify local copy of btrfs_inode flags")
f22125e5d8ae1 ("btrfs: refactor btrfs_set_props to validate externally")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
v4.9.230: Failed to apply! Possible dependencies:
0b246afa62b0c ("btrfs: root->fs_info cleanup, add fs_info convenience variables")
1905a0f7c7de3 ("btrfs: rename btrfs_mask_flags to reflect which flags it touches")
38e82de8ccd18 ("btrfs: user proper type for btrfs_mask_flags flags")
5ba76abfb2336 ("btrfs: rename check_flags to reflect which flags it touches")
5c57b8b6a4966 ("btrfs: unify naming of flags variables for SETFLAGS and XFLAGS")
62d1f9fe97dd2 ("btrfs: remove trivial helper btrfs_find_tree_block")
a157d4fd81dc7 ("btrfs: rename btrfs_flags_to_ioctl to reflect which flags it touches")
cf8cddd38bab3 ("btrfs: don't abuse REQ_OP_* flags for btrfs_map_block")
da17066c40472 ("btrfs: pull node/sector/stripe sizes out of root and into fs_info")
de143792253e2 ("btrfs: struct btrfsic_state->root should be an fs_info")
fb456252d3d9c ("btrfs: root->fs_info cleanup, use fs_info->dev_root everywhere")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
v4.4.230: Failed to apply! Possible dependencies:
0132761017e01 ("btrfs: fix string and comment grammatical issues and typos")
09cbfeaf1a5a6 ("mm, fs: get rid of PAGE_CACHE_* and page_cache_{get,release} macros")
0b246afa62b0c ("btrfs: root->fs_info cleanup, add fs_info convenience variables")
0e749e54244ee ("dax: increase granularity of dax_clear_blocks() operations")
1905a0f7c7de3 ("btrfs: rename btrfs_mask_flags to reflect which flags it touches")
38e82de8ccd18 ("btrfs: user proper type for btrfs_mask_flags flags")
4420cfd3f51cf ("staging: lustre: format properly all comment blocks for LNet core")
52db400fcd502 ("pmem, dax: clean up clear_pmem()")
5ba76abfb2336 ("btrfs: rename check_flags to reflect which flags it touches")
5c57b8b6a4966 ("btrfs: unify naming of flags variables for SETFLAGS and XFLAGS")
5fd88337d209d ("staging: lustre: fix all conditional comparison to zero in LNet layer")
a157d4fd81dc7 ("btrfs: rename btrfs_flags_to_ioctl to reflect which flags it touches")
b2e0d1625e193 ("dax: fix lifetime of in-kernel dax mappings with dax_map_atomic()")
bb7ab3b92e46d ("btrfs: Fix misspellings in comments.")
cf8cddd38bab3 ("btrfs: don't abuse REQ_OP_* flags for btrfs_map_block")
d1a5f2b4d8a12 ("block: use DAX for partition table reads")
de143792253e2 ("btrfs: struct btrfsic_state->root should be an fs_info")
e10624f8c0971 ("pmem: fail io-requests to known bad blocks")
ff9fef559babe ("btrfs: start transaction in btrfs_ioctl_setflags()")
NOTE: The patch will not be queued to stable trees until it is upstream.
How should we proceed with this patch?
--
Thanks
Sasha
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-07-17 17:08 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-10 10:05 [PATCH] btrfs: add missing check for nocow and compression inode flags David Sterba
2020-07-10 10:10 ` Nikolay Borisov
2020-07-10 13:34 ` David Sterba
2020-07-10 14:55 ` Nikolay Borisov
2020-07-10 10:17 ` Johannes Thumshirn
2020-07-10 13:28 ` David Sterba
2020-07-13 10:33 ` [PATCH v2] " David Sterba
2020-07-17 17:08 ` Sasha Levin
2020-07-16 0:27 ` [PATCH] " Sasha Levin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.