Bug 1256606

+++ This bug was initially created as a clone of Bug #1248669 +++

Description of problem:

As described in the bug1245636, we have encountered an issue where in the 'snapview-client' xlator lookup fop name 'svc_lookup' matches with one of the routines provided by libntirpc, used by NFS-Ganesha. Hence while using 'uss' feature over NFS-Ganesha, during dlopen() of the snapview-client xlator, as  'svc_lookup' is already loaded, 'xlator->fops->lookup' incorrectly refers to that symbol.

To avoid similar issues, it would be better and cleaner to mark all the xlator fops static. This bug is to track all the required changes.

--- Additional comment from Soumya Koduri on 2015-07-31 03:07:59 EDT ---



--- Additional comment from Anand Avati on 2015-07-31 05:02:54 EDT ---

REVIEW: http://review.gluster.org/11805 (snapshot: Make fops static for correct resolution of symbols) posted (#3) for review on master by soumya k (skoduri)

--- Additional comment from Anand Avati on 2015-08-04 08:54:22 EDT ---

REVIEW: http://review.gluster.org/11814 (build: export minimum symbols from xlators for correct resolution) posted (#21) for review on master by Kaleb KEITHLEY (kkeithle)

--- Additional comment from Anand Avati on 2015-08-24 13:23:13 EDT ---

COMMIT: http://review.gluster.org/11805 committed in master by Niels de Vos (ndevos) 
------
commit d5e03b7f02f68b3a9aaccf586e1f6ed901224ba7
Author: Soumya Koduri <skoduri>
Date:   Thu Jul 30 16:37:45 2015 +0530

    snapshot: Make fops static for correct resolution of symbols
    
    Few of the snapshot fops (like 'svc_lookup') may not get resolved
    while using dynamic loading as there could be other libraries(like libntirpc)
    with same routine names. Making them static to resolve the same.
    
    Change-Id: I6577bf3705864f5583425c94427b4e1025a59bcd
    BUG: 1248669
    Signed-off-by: Soumya Koduri <skoduri>
    Reviewed-on: http://review.gluster.org/11805
    Reviewed-by: Niels de Vos <ndevos>
    Tested-by: NetBSD Build System <jenkins.org>
    Reviewed-by: Rajesh Joseph <rjoseph>
    Tested-by: Gluster Build System <jenkins.com>