From 569770afe52726e1df970c2219b8aa2c080e8a31 Mon Sep 17 00:00:00 2001 From: Xiubo Li Date: Mon, 20 Dec 2021 10:21:50 +0800 Subject: [PATCH] client: force a request to update the perms If the caller perms is different with the current dirtier's, we can still cache the attributes locally if possible, but will mark the ctime needs to be updated forcely and the request will implicitly flush the dirty caps for us. Signed-off-by: Xiubo Li --- src/client/Client.cc | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/client/Client.cc b/src/client/Client.cc index 89b512303bfe..463584d1c27b 100644 --- a/src/client/Client.cc +++ b/src/client/Client.cc @@ -7489,9 +7489,7 @@ int Client::_do_setattr(Inode *in, struct ceph_statx *stx, int mask, * case later, we can build a more complex pipelined cap writeback * infrastructure... */ - if (!mask) - mask |= CEPH_SETATTR_CTIME; - goto force_request; + mask |= CEPH_SETATTR_CTIME; } if (!mask) { @@ -7691,7 +7689,6 @@ int Client::_do_setattr(Inode *in, struct ceph_statx *stx, int mask, return 0; } -force_request: MetaRequest *req = new MetaRequest(CEPH_MDS_OP_SETATTR); filepath path; -- 2.47.3