gfapi: mark glfs_ipc() for internal use only

The 3.7 version of glfs_ipc() has never been used by external
applications. There is little use for internal xlator communication that
is triggered from outside of core GlusterFS executables. This function
has now been removed from libgfapi.so.

For Gluster 4.0 a new variation for glfs_ipc() has been added. The
function expects dict_t parameters, which are currently not available
for external applications. There is no sense in providing glfs_ipc() for
non-core GlusterFS executables. Therefore, glfs_ipc() has been marked as
private, and the declaration is now in the glfs-internal.h header.

The Python test case (tests/features/ipctest.py) is not correct and will
be re-written in C to prevent portability issues. This test is currently
disabled (commit d26f0bac149d495fa93710c3f7b6b63c36cb8387).

Cherry picked from commit 40d71d6a201d618e52555a062ac1a429d36ca9ca:
> Change-Id: Idbfe35570d34d45ce8b6b43084627a552ac21f59
> Fixes: #269
> Signed-off-by: Niels de Vos <ndevos@redhat.com>
> Reviewed-on: https://review.gluster.org/17854
> Smoke: Gluster Build System <jenkins@build.gluster.org>
> CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
> Reviewed-by: soumya k <skoduri@redhat.com>
> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>

Change-Id: Idbfe35570d34d45ce8b6b43084627a552ac21f59
Fixes: #269
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: https://review.gluster.org/17894
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Reviewed-by: jiffin tony Thottan <jthottan@redhat.com>

5 files changed, 10 insertions, 20 deletions

diff --git a/api/src/gfapi.aliases b/api/src/gfapi.aliases
index f3726a3348..b0facb717b 100644
--- a/api/src/gfapi.aliases
+++ b/api/src/gfapi.aliases
@@ -125,7 +125,6 @@ _pub_glfs_h_removexattrs _glfs_h_removexattrs$GFAPI_3.5.1
 _pub_glfs_get_volfile _glfs_get_volfile$GFAPI_3.6.0
 _pub_glfs_h_access _glfs_h_access$GFAPI_3.6.0
 
-_pub_glfs_ipc37 _glfs_ipc$GFAPI_3.7.0
 _pub_glfs_h_poll_upcall _glfs_h_poll_upcall$GFAPI_3.7.0
 _pub_glfs_h_acl_set _glfs_h_acl_set$GFAPI_3.7.0
 _pub_glfs_h_acl_get _glfs_h_acl_get$GFAPI_3.7.0
@@ -163,4 +162,4 @@ _pub_glfs_xreaddirplus_r_get_stat _glfs_xreaddirplus_r_get_stat$GFAPI_3.11.0
 _pub_glfs_xreaddirplus_r_get_object _glfs_xreaddirplus_r_get_object$GFAPI_3.11.0
 _pub_glfs_object_copy _glfs_object_copy$GFAPI_3.11.0
 
-_pub_glfs_ipc _glfs_ipc$GFAPI_4.0.0
+_priv_glfs_ipc _glfs_ipc$GFAPI_3.12.0
diff --git a/api/src/gfapi.map b/api/src/gfapi.map
index 5e0c877077..7f19e1ee4f 100644
--- a/api/src/gfapi.map
+++ b/api/src/gfapi.map
@@ -146,7 +146,6 @@ GFAPI_3.6.0 {
 
 GFAPI_3.7.0 {
 	global:
-		glfs_ipc;
 		glfs_h_poll_upcall;
 		glfs_h_acl_set;
 		glfs_h_acl_get;
@@ -206,7 +205,7 @@ GFAPI_3.11.0 {
                 glfs_object_copy;
 } GFAPI_3.10.0;
 
-GFAPI_4.0.0 {
+GFAPI_PRIVATE_3.12.0 {
 	global:
 		glfs_ipc;
 } GFAPI_3.11.0;
diff --git a/api/src/glfs-internal.h b/api/src/glfs-internal.h
index b5d3f78bf7..838b5c748b 100644
--- a/api/src/glfs-internal.h
+++ b/api/src/glfs-internal.h
@@ -490,4 +490,10 @@ glfd_entry_next (struct glfs_fd *glfd, int plus);
 void
 gf_dirent_to_dirent (gf_dirent_t *gf_dirent, struct dirent *dirent);
 
+/*
+ * Nobody needs this call at all yet except for the test script.
+ */
+int glfs_ipc (glfs_fd_t *fd, int cmd,  void *xd_in, void **xd_out) __THROW
+        GFAPI_PRIVATE(glfs_ipc, 3.12.0);
+
 #endif /* !_GLFS_INTERNAL_H */
diff --git a/api/src/glfs.c b/api/src/glfs.c
index 6d03aaa9b6..8369546667 100644
--- a/api/src/glfs.c
+++ b/api/src/glfs.c
@@ -1342,7 +1342,7 @@ invalid_fs:
 GFAPI_SYMVER_PUBLIC_DEFAULT(glfs_get_volfile, 3.6.0);
 
 int
-pub_glfs_ipc (struct glfs *fs, int opcode, void *xd_in, void **xd_out)
+priv_glfs_ipc (struct glfs *fs, int opcode, void *xd_in, void **xd_out)
 {
 	xlator_t        *subvol = NULL;
         int             ret = -1;
@@ -1368,15 +1368,7 @@ invalid_fs:
         return ret;
 }
 
-GFAPI_SYMVER_PUBLIC_DEFAULT(glfs_ipc, 4.0.0);
-
-int
-pub_glfs_ipc37 (struct glfs *fs, int opcode)
-{
-        return pub_glfs_ipc (fs, opcode, NULL, NULL);
-}
-
-GFAPI_SYMVER_PUBLIC(glfs_ipc37, glfs_ipc, 3.7.0);
+GFAPI_SYMVER_PRIVATE_DEFAULT(glfs_ipc, 3.12.0);
 
 
 void
diff --git a/api/src/glfs.h b/api/src/glfs.h
index 4e535bda16..5420a1dde6 100644
--- a/api/src/glfs.h
+++ b/api/src/glfs.h
@@ -855,12 +855,6 @@ glfs_xreaddirplus_r (struct glfs_fd *glfd, uint32_t flags,
                      struct dirent *ext, struct dirent **res);
         GFAPI_PUBLIC(glfs_xreaddirplus_r, 3.11.0);
 
-/*
- * Nobody needs this call at all yet except for the test script.
- */
-int glfs_ipc (glfs_fd_t *fd, int cmd,  void *xd_in, void **xd_out) __THROW
-        GFAPI_PUBLIC(glfs_ipc, 4.0.0);
-
 __END_DECLS
 
 #endif /* !_GLFS_H */