>From Daniel Gryniewicz <d...@redhat.com>:

Daniel Gryniewicz has uploaded this change for review. ( 
https://review.gerrithub.io/369612


Change subject: Track global FDs for LRU
......................................................................

Track global FDs for LRU

The state FD changes broke accounting of the global FDs.  Fix this by:
- Ensuring all calls to obj_ops.close() go through fsal_close(), which
  can do the accounting.
- Converting obj_ops.close() to always return ERR_FSAL_NOT_OPEN if the
  global FD was not open

Note that state FDs are not accounted at the moment.  Horever, they
should always be associated with a client OP, so they couldn't be
background reaped anyway.

Change-Id: Id4f3340cca4af3a8f39777e6abf6a873c229e972
Signed-off-by: Daniel Gryniewicz <d...@redhat.com>
---
M src/FSAL/FSAL_CEPH/handle.c
M src/FSAL/FSAL_GLUSTER/handle.c
M src/FSAL/FSAL_GPFS/file.c
M src/FSAL/FSAL_MEM/mem_handle.c
M src/FSAL/FSAL_RGW/handle.c
M src/FSAL/FSAL_VFS/file.c
M src/FSAL/FSAL_ZFS/file.c
M src/FSAL/commonlib.c
M src/FSAL/fsal_helper.c
M src/Protocols/NFS/nfs4_op_open.c
M src/SAL/nfs4_state.c
M src/include/fsal.h
M src/include/fsal_api.h
13 files changed, 41 insertions(+), 10 deletions(-)



  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/12/369612/1
-- 
To view, visit https://review.gerrithub.io/369612
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4f3340cca4af3a8f39777e6abf6a873c229e972
Gerrit-Change-Number: 369612
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <d...@redhat.com>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to