[NFS-Ganesha-Devel] Change in ffilz/nfs-ganesha[next]: namespace cleanup: some poorly named and/or non-static symbols

From <kaleb(a)redhat.com&gt;: kaleb(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/427287 Change subject: namespace cleanup: some poorly named and/or non-static symbols ...................................................................... namespace cleanup: some poorly named and/or non-static symbols In Gluster are seeing a collision between the variable named 'options' in the gluster xlators and the variable named 'options' in ganesha.nfsd. ISTR a recent change made in Fedora to RT linker that affect this but am unable to find a reference to the discussion at this time. It is unclear to me whether this is a side effect of the introduction of the -z defs linker flags in Fedora 28. This probably affects RHEL 8 too. In a nutshell, references within a gluster xlator .so used to always be resolved to the "closest" definition and/or resolved at static link time; either way to the one in the xlator .so. Now though we see them resolved at runtime to the first definition of the symbol; in this case the one in ganesha.nfsd. In addition to 'options' there are a number of other variables in ganesha.nfsd that a) should be static, or b) should be renamed so as not to pollute the namespace. I picked a few that stood out. There may be others. Note too that gluster is working to clean up its own scribbling in the namespace. And, e.g., 'options' are exported from xlator .so files so that each xlators' options can be enumerated for display. Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com&gt; Change-Id: I0a5893ac4d54fce84e71f38997e1370f39d42ad3 --- M src/FSAL/FSAL_PROXY/handle.c M src/MainNFSD/9p_dispatcher.c M src/MainNFSD/9p_rdma_callbacks.c M src/MainNFSD/nfs_admin_thread.c M src/MainNFSD/nfs_init.c M src/MainNFSD/nfs_lib.c M src/MainNFSD/nfs_main.c M src/MainNFSD/nfs_rpc_callback.c M src/MainNFSD/nfs_rpc_dispatcher_thread.c M src/MainNFSD/nfs_worker_thread.c M src/Protocols/NFS/nfs4_Compound.c M src/SAL/nfs4_clientid.c M src/SAL/nfs4_state_id.c M src/include/nfs_core.h M src/include/nfs_lib.h M src/log/log_functions.c M src/support/client_mgr.c M src/support/export_mgr.c M src/support/server_stats.c 19 files changed, 114 insertions(+), 109 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/87/427287/1 -- To view, visit https://review.gerrithub.io/427287 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I0a5893ac4d54fce84e71f38997e1370f39d42ad3 Gerrit-Change-Number: 427287 Gerrit-PatchSet: 1 Gerrit-Owner: Anonymous Coward <kaleb(a)redhat.com&gt;