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 <sage@xxxxxxxxxxxx>
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 <sage@xxxxxxxxxxxx>
---
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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
