diff options
author | Jeff Darcy <jdarcy@redhat.com> | 2016-07-07 15:24:05 -0400 |
---|---|---|
committer | Atin Mukherjee <amukherj@redhat.com> | 2016-07-07 21:26:26 -0700 |
commit | ef08924ba7c568605d96af8b1f4ca50ede045204 (patch) | |
tree | 0c133ab4ded2f27cf06c578f1a0e0c2ddaf648fa /rpc | |
parent | 45f986a20f582c24c71172eb1ff097295c88aebc (diff) |
mgmt: remove unused and misleading pmap_signup
We use signin, but not signup. Having both just introduces confusion.
The proc number has been retained to avoid changes to the numbering of
other procs, and the mapping to a name has similarly been retained as a
placeholder, but the code and structure definitions have been removed.
Change-Id: I60f64f3b5d71ba6ed6862b36a38f90a9c8271c9f
Signed-off-by: Jeff Darcy <jdarcy@redhat.com>
Reviewed-on: http://review.gluster.org/14792
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'rpc')
-rw-r--r-- | rpc/rpc-lib/src/protocol-common.h | 5 | ||||
-rw-r--r-- | rpc/xdr/src/portmap-xdr.x | 11 |
2 files changed, 5 insertions, 11 deletions
diff --git a/rpc/rpc-lib/src/protocol-common.h b/rpc/rpc-lib/src/protocol-common.h index 915d358e707..b98ea3af62f 100644 --- a/rpc/rpc-lib/src/protocol-common.h +++ b/rpc/rpc-lib/src/protocol-common.h @@ -84,6 +84,11 @@ enum gf_pmap_procnum { GF_PMAP_NULL = 0, GF_PMAP_PORTBYBRICK, GF_PMAP_BRICKBYPORT, + /* + * SIGNUP is not used, and shouldn't be used. It was kept here only + * to avoid changing the numbers for things that come after it in this + * list. + */ GF_PMAP_SIGNUP, GF_PMAP_SIGNIN, GF_PMAP_SIGNOUT, diff --git a/rpc/xdr/src/portmap-xdr.x b/rpc/xdr/src/portmap-xdr.x index 7380de5e9ca..0e694f3359b 100644 --- a/rpc/xdr/src/portmap-xdr.x +++ b/rpc/xdr/src/portmap-xdr.x @@ -23,17 +23,6 @@ struct pmap_brick_by_port_rsp { }; -struct pmap_signup_req { - string brick<>; - int port; -}; - -struct pmap_signup_rsp { - int op_ret; - int op_errno; -}; - - struct pmap_signin_req { string brick<>; int port; |