>From Daniel Gryniewicz <[email protected]>:
Daniel Gryniewicz has uploaded this change for review. (
https://review.gerrithub.io/390670
Change subject: pNFS - Fix memory leaks for fsal_pnfs_ds
......................................................................
pNFS - Fix memory leaks for fsal_pnfs_ds
Instances of fsal_pnfs_ds can be of 3 varieties: 1) Associated with an
export (MDS/DS co-located). This will be cleaned up when the export is
cleaned up. 2) Stand-alone DS (No MDS). This was leaking. 3)
Placeholder when no DS is configured. This was leaking.
Fix case 2) by tracking all DSs and cleaning them up on shutdown, like
is done for exports.
Fix case 3) by detecting it and using a static instance, rather than a
dynamically allocated instance. This needs to be done because the
placeholder instance is never committed, so it's otherwise completely
untracked.
Change-Id: Id66c8c3c9056a69c5306fca6895c66782985b1a5
Signed-off-by: Daniel Gryniewicz <[email protected]>
---
M src/MainNFSD/nfs_admin_thread.c
M src/include/fsal_api.h
M src/include/pnfs_utils.h
M src/support/ds.c
4 files changed, 51 insertions(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha
refs/changes/70/390670/1
--
To view, visit https://review.gerrithub.io/390670
To unsubscribe, visit https://review.gerrithub.io/settings
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id66c8c3c9056a69c5306fca6895c66782985b1a5
Gerrit-Change-Number: 390670
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz <[email protected]>
------------------------------------------------------------------------------
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
