Commit 94045e11 authored by Sage Weil's avatar Sage Weil

ceph: decode updated mdsmap format

The mds map now uses the global_id as the 'key' (instead of the addr,
which was a poor choice).

This is protocol change.
Signed-off-by: default avatarSage Weil <sage@newdream.net>
parent 0743304d
...@@ -39,7 +39,7 @@ ...@@ -39,7 +39,7 @@
#define CEPH_MDS_PROTOCOL 9 /* cluster internal */ #define CEPH_MDS_PROTOCOL 9 /* cluster internal */
#define CEPH_MON_PROTOCOL 5 /* cluster internal */ #define CEPH_MON_PROTOCOL 5 /* cluster internal */
#define CEPH_OSDC_PROTOCOL 22 /* server/client */ #define CEPH_OSDC_PROTOCOL 22 /* server/client */
#define CEPH_MDSC_PROTOCOL 29 /* server/client */ #define CEPH_MDSC_PROTOCOL 30 /* server/client */
#define CEPH_MONC_PROTOCOL 15 /* server/client */ #define CEPH_MONC_PROTOCOL 15 /* server/client */
......
...@@ -76,6 +76,7 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end) ...@@ -76,6 +76,7 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end)
/* pick out active nodes from mds_info (state > 0) */ /* pick out active nodes from mds_info (state > 0) */
n = ceph_decode_32(p); n = ceph_decode_32(p);
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
u64 global_id;
u32 namelen; u32 namelen;
s32 mds, inc, state; s32 mds, inc, state;
u64 state_seq; u64 state_seq;
...@@ -84,10 +85,10 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end) ...@@ -84,10 +85,10 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end)
u32 num_export_targets; u32 num_export_targets;
void *pexport_targets = NULL; void *pexport_targets = NULL;
ceph_decode_need(p, end, sizeof(addr) + 1 + sizeof(u32), bad); ceph_decode_need(p, end, sizeof(u64)*2 + 1 + sizeof(u32), bad);
ceph_decode_copy(p, &addr, sizeof(addr)); global_id = ceph_decode_64(p);
ceph_decode_addr(&addr);
infoversion = ceph_decode_8(p); infoversion = ceph_decode_8(p);
*p += sizeof(u64);
namelen = ceph_decode_32(p); /* skip mds name */ namelen = ceph_decode_32(p); /* skip mds name */
*p += namelen; *p += namelen;
...@@ -99,7 +100,8 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end) ...@@ -99,7 +100,8 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end)
inc = ceph_decode_32(p); inc = ceph_decode_32(p);
state = ceph_decode_32(p); state = ceph_decode_32(p);
state_seq = ceph_decode_64(p); state_seq = ceph_decode_64(p);
*p += sizeof(addr); ceph_decode_copy(p, &addr, sizeof(addr));
ceph_decode_addr(&addr);
*p += sizeof(struct ceph_timespec); *p += sizeof(struct ceph_timespec);
*p += sizeof(u32); *p += sizeof(u32);
ceph_decode_32_safe(p, end, namelen, bad); ceph_decode_32_safe(p, end, namelen, bad);
...@@ -112,10 +114,11 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end) ...@@ -112,10 +114,11 @@ struct ceph_mdsmap *ceph_mdsmap_decode(void **p, void *end)
num_export_targets = 0; num_export_targets = 0;
} }
dout("mdsmap_decode %d/%d mds%d.%d %s %s\n", dout("mdsmap_decode %d/%d %lld mds%d.%d %s %s\n",
i+1, n, mds, inc, pr_addr(&addr.in_addr), i+1, n, global_id, mds, inc, pr_addr(&addr.in_addr),
ceph_mds_state_name(state)); ceph_mds_state_name(state));
if (mds >= 0 && mds < m->m_max_mds && state > 0) { if (mds >= 0 && mds < m->m_max_mds && state > 0) {
m->m_info[mds].global_id = global_id;
m->m_info[mds].state = state; m->m_info[mds].state = state;
m->m_info[mds].addr = addr; m->m_info[mds].addr = addr;
m->m_info[mds].num_export_targets = num_export_targets; m->m_info[mds].num_export_targets = num_export_targets;
......
...@@ -9,6 +9,7 @@ ...@@ -9,6 +9,7 @@
* we limit fields to those the client actually xcares about * we limit fields to those the client actually xcares about
*/ */
struct ceph_mds_info { struct ceph_mds_info {
u64 global_id;
struct ceph_entity_addr addr; struct ceph_entity_addr addr;
s32 state; s32 state;
int num_export_targets; int num_export_targets;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment