On Fri, 6 May 2011, Dyweni - Qemu-Devel wrote: > Hi Sage/Lists! > > Yes! The entire Ceph cluster (1 Mon, 1 MSD, 3 OSD) are 32bit linux. > > The machine running Qemu is 64bit linux.
Excellent. This is now fixed by 48d94f6e34da8ace2b5cb128de1bcfb34b3c40b5 in the stable and master branches of ceph.git. Or you can apply the patch below. Thanks! sage >From 48d94f6e34da8ace2b5cb128de1bcfb34b3c40b5 Mon Sep 17 00:00:00 2001 From: Sage Weil <[email protected]> Date: Fri, 6 May 2011 13:42:23 -0700 Subject: [PATCH] osd: used fixed size types for fiemap/mapext/sparseread encoding The client expects <uint64_t,uint64_t>, so this breaks on any 32-bit osd. Signed-off-by: Sage Weil <[email protected]> --- src/os/FileStore.cc | 4 ++-- src/osd/ReplicatedPG.cc | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/os/FileStore.cc b/src/os/FileStore.cc index 81bc91d..ca11435 100644 --- a/src/os/FileStore.cc +++ b/src/os/FileStore.cc @@ -2198,7 +2198,7 @@ int FileStore::fiemap(coll_t cid, const sobject_t& oid, { if (!ioctl_fiemap) { - map<off_t, size_t> m; + map<uint64_t, uint64_t> m; m[offset] = len; ::encode(m, bl); return 0; @@ -2207,7 +2207,7 @@ int FileStore::fiemap(coll_t cid, const sobject_t& oid, char fn[PATH_MAX]; struct fiemap *fiemap = NULL; - map<off_t, size_t> extmap; + map<uint64_t, uint64_t> extmap; get_coname(cid, oid, fn, sizeof(fn)); diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc index ba76b05..84399ca 100644 --- a/src/osd/ReplicatedPG.cc +++ b/src/osd/ReplicatedPG.cc @@ -1142,10 +1142,10 @@ int ReplicatedPG::do_osd_ops(OpContext *ctx, vector<OSDOp>& ops, result = r; break; } - map<off_t, size_t> m; + map<uint64_t, uint64_t> m; bufferlist::iterator iter = bl.begin(); ::decode(m, iter); - map<off_t, size_t>::iterator miter; + map<uint64_t, uint64_t>::iterator miter; bufferlist data_bl; for (miter = m.begin(); miter != m.end(); ++miter) { bufferlist tmpbl; -- 1.7.0 -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
