diff options
| author | Pranith Kumar K <pkarampu@redhat.com> | 2019-03-11 14:04:39 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2019-03-14 04:47:47 +0000 |
| commit | f5987d38f216a3142dfe45f03bf66ff4827d9b55 (patch) | |
| tree | c661924947831c9b9317758aaa4d0653ca3076e7 /xlators/storage/posix/src/posix-inode-fd-ops.c | |
| parent | 08a9f198d576bbae3596536bbd2c4d34dadd1a93 (diff) | |
storage/posix: Remove nr_files usage
nr_files is supposed to represent the number of files opened in posix.
Present logic doesn't seem to handle anon-fds because of which the
counts would always be wrong.
I don't remember anyone using this value in debugging any problem probably
because we always have 'ls -l /proc/<pid>/fd' which not only prints the
fds that are active but also prints their paths. It also handles directories
and anon-fds which actually opened the file. So removing this code
instead of fixing the buggy logic to have the nr_files.
fixes bz#1688106
Change-Id: Ibf8713fdfdc1ef094e08e6818152637206a54040
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
Diffstat (limited to 'xlators/storage/posix/src/posix-inode-fd-ops.c')
| -rw-r--r-- | xlators/storage/posix/src/posix-inode-fd-ops.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/xlators/storage/posix/src/posix-inode-fd-ops.c b/xlators/storage/posix/src/posix-inode-fd-ops.c index 0ee6803880d..93497da812b 100644 --- a/xlators/storage/posix/src/posix-inode-fd-ops.c +++ b/xlators/storage/posix/src/posix-inode-fd-ops.c @@ -1490,7 +1490,6 @@ posix_open(call_frame_t *frame, xlator_t *this, loc_t *loc, int32_t flags, "failed to set the fd context gfid-handle=%s path=%s fd=%p", real_path, loc->path, fd); - GF_ATOMIC_INC(priv->nr_files); op_ret = 0; out: @@ -2369,7 +2368,6 @@ posix_release(xlator_t *this, fd_t *fd) if (!priv) goto out; - GF_ATOMIC_DEC(priv->nr_files); out: return 0; } |