Project

General

Profile

Tasks #46768

Tasks #23844: client: break client_lock

client: clean up the unnecessary client_lock for _conf->client_trace

Added by Xiubo Li 6 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Performance/Resource Usage
Target version:
% Done:

0%

Tags:
Reviewed:
Affected Versions:
Component(FS):
Client
Labels (FS):
Pull request ID:

Description

There is no need to make the "cct->_conf->client_trace" under the
client_lock, it is for the "ceph-syn" and it almost won't change,
even it will but it still won't cause any race.

For the "inode->ino" and "inode->snapid", since they won't change
once initialized when creating the inode, and the inode here will
be protected by the reference count, so no need the client_lock too.

History

#1 Updated by Xiubo Li 6 months ago

  • Pull request ID set to 36364

#2 Updated by Xiubo Li 6 months ago

  • Status changed from New to Fix Under Review

#3 Updated by Patrick Donnelly 6 months ago

  • Category set to Performance/Resource Usage
  • Status changed from Fix Under Review to Resolved
  • Target version set to v16.0.0

Also available in: Atom PDF