diff options
author | Mohit Agrawal <moagrawa@redhat.com> | 2020-07-27 18:08:00 +0530 |
---|---|---|
committer | MOHIT AGRAWAL <moagrawa@redhat.com> | 2020-08-20 12:11:43 +0000 |
commit | 41b9616435cbdf671805856e487e373060c9455b (patch) | |
tree | 6459211f8203ac2ae111bd992dd67aae3b1e34cf /rpc | |
parent | 742358ec1ab488a092d1fd9b2b47c717b627181e (diff) |
posix: Implement a janitor thread to close fd
Problem: In the commit fb20713b380e1df8d7f9e9df96563be2f9144fd6 we use
syntask to close fd but we have found the patch is reducing the
performance
Solution: Use janitor thread to close fd's and save the pfd ctx into
ctx janitor list and also save the posix_xlator into pfd object to
avoid the race condition during cleanup in brick_mux environment
Change-Id: Ifb3d18a854b267333a3a9e39845bfefb83fbc092
Fixes: #1396
Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>
Diffstat (limited to 'rpc')
-rw-r--r-- | rpc/rpc-lib/src/rpcsvc.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/rpc/rpc-lib/src/rpcsvc.c b/rpc/rpc-lib/src/rpcsvc.c index 4297af4609c..9e8c9af8a12 100644 --- a/rpc/rpc-lib/src/rpcsvc.c +++ b/rpc/rpc-lib/src/rpcsvc.c @@ -343,12 +343,6 @@ rpcsvc_program_actor(rpcsvc_request_t *req) req->ownthread = program->ownthread; req->synctask = program->synctask; - if (((req->procnum == GFS3_OP_RELEASE) || - (req->procnum == GFS3_OP_RELEASEDIR)) && - (program->prognum == GLUSTER_FOP_PROGRAM)) { - req->ownthread = _gf_false; - req->synctask = _gf_true; - } err = SUCCESS; gf_log(GF_RPCSVC, GF_LOG_TRACE, "Actor found: %s - %s for %s", |