libceph: define ceph_decode_pgid() only once

There are two basically identical definitions of __decode_pgid()
in libceph, one in "net/ceph/osdmap.c" and the other in
"net/ceph/osd_client.c".  Get rid of both, and instead define
a single inline version in "include/linux/ceph/osdmap.h".

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>

1 files changed, 2 insertions, 20 deletions

diff --git a/net/ceph/osdmap.c b/net/ceph/osdmap.c
index 09898711f2fd..603ddd92db19 100644
--- a/net/ceph/osdmap.c
+++ b/net/ceph/osdmap.c
@@ -654,24 +654,6 @@ static int osdmap_set_max_osd(struct ceph_osdmap *map, int max)
         return 0;
 }
 
-static int __decode_pgid(void **p, void *end, struct ceph_pg *pg)
-{
-        u8 v;
-
-        ceph_decode_need(p, end, 1+8+4+4, bad);
-        v = ceph_decode_8(p);
-        if (v != 1)
-                goto bad;
-        pg->pool = ceph_decode_64(p);
-        pg->seed = ceph_decode_32(p);
-        *p += 4; /* skip preferred */
-        return 0;
-
-bad:
-        dout("error decoding pgid\n");
-        return -EINVAL;
-}
-
 /*
  * decode a full map.
  */
@@ -765,7 +747,7 @@ struct ceph_osdmap *osdmap_decode(void **p, void *end)
                 struct ceph_pg pgid;
                 struct ceph_pg_mapping *pg;
 
-                err = __decode_pgid(p, end, &pgid);
+                err = ceph_decode_pgid(p, end, &pgid);
                 if (err)
                         goto bad;
                 ceph_decode_need(p, end, sizeof(u32), bad);
@@ -983,7 +965,7 @@ struct ceph_osdmap *osdmap_apply_incremental(void **p, void *end,
                 struct ceph_pg pgid;
                 u32 pglen;
 
-                err = __decode_pgid(p, end, &pgid);
+                err = ceph_decode_pgid(p, end, &pgid);
                 if (err)
                         goto bad;
                 ceph_decode_need(p, end, sizeof(u32), bad);