On Mon, 2019-12-16 at 00:12 -0500, xiubli@xxxxxxxxxx wrote: > From: Xiubo Li <xiubli@xxxxxxxxxx> > > For the caps having no any subset mask requested we shouldn't touch > them. > > Signed-off-by: Xiubo Li <xiubli@xxxxxxxxxx> > --- > fs/ceph/caps.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/fs/ceph/caps.c b/fs/ceph/caps.c > index 1d7f66902b0a..b9e5960df183 100644 > --- a/fs/ceph/caps.c > +++ b/fs/ceph/caps.c > @@ -908,7 +908,8 @@ int __ceph_caps_issued_mask(struct ceph_inode_info *ci, int mask, int touch) > ci_node); > if (!__cap_is_valid(cap)) > continue; > - __touch_cap(cap); > + if (cap->issued & mask) > + __touch_cap(cap); > } > } > return 1; Looks correct to me. Merged. Thanks, -- Jeff Layton <jlayton@xxxxxxxxxx>
