]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
osdc: use __u64, not size_t
authorSage Weil <sage@newdream.net>
Thu, 19 Nov 2009 18:11:19 +0000 (10:11 -0800)
committerSage Weil <sage@newdream.net>
Thu, 19 Nov 2009 18:11:19 +0000 (10:11 -0800)
size_t is too small on 32 bit platforms.

src/osdc/Filer.cc
src/osdc/Filer.h
src/osdc/ObjectCacher.cc
src/osdc/ObjectCacher.h
src/osdc/Objecter.cc
src/osdc/Objecter.h

index 0e6879636a573ade11606790f9d2b726d04a4a6c..cba7c126db1d089193e4ac6e4491251748e0dcf2 100644 (file)
@@ -211,7 +211,7 @@ void Filer::_probed(Probe *probe, const object_t& oid, __u64 size, utime_t mtime
 
 
 void Filer::file_to_extents(inodeno_t ino, ceph_file_layout *layout,
-                            __u64 offset, size_t len,
+                            __u64 offset, __u64 len,
                             vector<ObjectExtent>& extents)
 {
   dout(10) << "file_to_extents " << offset << "~" << len 
index 08b64dc3e031d9eda800e2de5da8a358b6179aa8..7ccfa4f9dd170b762991d0cf16bd647f7b5359e2 100644 (file)
@@ -97,7 +97,7 @@ class Filer {
    * ranges in objects on (primary) osds)
    */
   void file_to_extents(inodeno_t ino, ceph_file_layout *layout,
-                      __u64 offset, size_t len,
+                      __u64 offset, __u64 len,
                       vector<ObjectExtent>& extents);
 
 
@@ -109,7 +109,7 @@ class Filer {
           ceph_file_layout *layout,
           snapid_t snap,
            __u64 offset, 
-           size_t len, 
+           __u64 len, 
            bufferlist *bl,   // ptr to data
           int flags,
            Context *onfinish) {
@@ -125,7 +125,7 @@ class Filer {
            ceph_file_layout *layout,
            const SnapContext& snapc,
            __u64 offset, 
-            size_t len, 
+            __u64 len, 
             bufferlist& bl,
            utime_t mtime,
             int flags, 
@@ -141,7 +141,7 @@ class Filer {
               ceph_file_layout *layout,
               const SnapContext& snapc,
               __u64 offset,
-              size_t len,
+              __u64 len,
               __u32 truncate_seq,
               utime_t mtime,
               int flags,
@@ -178,7 +178,7 @@ class Filer {
           ceph_file_layout *layout,
           const SnapContext& snapc,
           __u64 offset,
-           size_t len,
+           __u64 len,
           utime_t mtime,
           int flags,
            Context *onack,
@@ -208,7 +208,7 @@ class Filer {
             ceph_file_layout *layout,
             const SnapContext& snapc,
             __u64 offset,
-            size_t len,
+            __u64 len,
             utime_t mtime,
             int flags,
             Context *onack,
index 6418a9b810a41f0376394acd2ecc10b22474f18e..3916410f87be8b50aaac7b7a7780e7b5bef68ccf 100644 (file)
@@ -444,7 +444,7 @@ void ObjectCacher::bh_read(BufferHead *bh)
                 onfinish);
 }
 
-void ObjectCacher::bh_read_finish(sobject_t oid, loff_t start, size_t length, bufferlist &bl)
+void ObjectCacher::bh_read_finish(sobject_t oid, loff_t start, __u64 length, bufferlist &bl)
 {
   //lock.Lock();
   dout(7) << "bh_read_finish " 
@@ -608,7 +608,7 @@ void ObjectCacher::lock_ack(list<sobject_t>& oids, tid_t tid)
   }
 }
 
-void ObjectCacher::bh_write_ack(sobject_t oid, loff_t start, size_t length, tid_t tid)
+void ObjectCacher::bh_write_ack(sobject_t oid, loff_t start, __u64 length, tid_t tid)
 {
   //lock.Lock();
   
@@ -677,7 +677,7 @@ void ObjectCacher::bh_write_ack(sobject_t oid, loff_t start, size_t length, tid_
   //lock.Unlock();
 }
 
-void ObjectCacher::bh_write_commit(sobject_t oid, loff_t start, size_t length, tid_t tid)
+void ObjectCacher::bh_write_commit(sobject_t oid, loff_t start, __u64 length, tid_t tid)
 {
   //lock.Lock();
   
@@ -809,7 +809,7 @@ int ObjectCacher::readx(OSDRead *rd, inodeno_t ino, Context *onfinish)
 {
   bool success = true;
   list<BufferHead*> hit_ls;
-  map<size_t, bufferlist> stripe_map;  // final buffer offset -> substring
+  map<__u64, bufferlist> stripe_map;  // final buffer offset -> substring
 
   for (vector<ObjectExtent>::iterator ex_it = rd->extents.begin();
        ex_it != rd->extents.end();
@@ -868,9 +868,9 @@ int ObjectCacher::readx(OSDRead *rd, inodeno_t ino, Context *onfinish)
       loff_t opos = ex_it->offset;
       map<loff_t, BufferHead*>::iterator bh_it = hits.begin();
       assert(bh_it->second->start() <= opos);
-      size_t bhoff = opos - bh_it->second->start();
+      __u64 bhoff = opos - bh_it->second->start();
       map<__u32,__u32>::iterator f_it = ex_it->buffer_extents.begin();
-      size_t foff = 0;
+      __u64 foff = 0;
       while (1) {
         BufferHead *bh = bh_it->second;
         assert(opos == (loff_t)(bh->start() + bhoff));
@@ -880,7 +880,7 @@ int ObjectCacher::readx(OSDRead *rd, inodeno_t ino, Context *onfinish)
                  << " frag " << f_it->first << "~" << f_it->second << " +" << foff
                  << dendl;
 
-        size_t len = MIN(f_it->second - foff,
+        __u64 len = MIN(f_it->second - foff,
                          bh->length() - bhoff);
        bufferlist bit;  // put substr here first, since substr_of clobbers, and
                         // we may get multiple bh's at this stripe_map position
@@ -921,10 +921,10 @@ int ObjectCacher::readx(OSDRead *rd, inodeno_t ino, Context *onfinish)
   dout(10) << "readx has all buffers" << dendl;
   
   // ok, assemble into result buffer.
-  size_t pos = 0;
+  __u64 pos = 0;
   if (rd->bl) {
     rd->bl->clear();
-    for (map<size_t,bufferlist>::iterator i = stripe_map.begin();
+    for (map<__u64,bufferlist>::iterator i = stripe_map.begin();
         i != stripe_map.end();
         i++) {
       assert(pos == i->first);
@@ -972,7 +972,7 @@ int ObjectCacher::writex(OSDWrite *wr, inodeno_t ino)
          f_it != ex_it->buffer_extents.end();
          f_it++) {
       dout(10) << "writex writing " << f_it->first << "~" << f_it->second << " into " << *bh << " at " << opos << dendl;
-      size_t bhoff = bh->start() - opos;
+      __u64 bhoff = bh->start() - opos;
       assert(f_it->second <= bh->length() - bhoff);
 
       // get the frag we're mapping in
@@ -1006,7 +1006,7 @@ int ObjectCacher::writex(OSDWrite *wr, inodeno_t ino)
  
 
 // blocking wait for write.
-bool ObjectCacher::wait_for_write(size_t len, Mutex& lock)
+bool ObjectCacher::wait_for_write(__u64 len, Mutex& lock)
 {
   int blocked = 0;
 
index 9ab97e03d4654542cad881e5b7710119fa89dc4b..0447c1dc05a4176a65b324b824d27ee1afe75ceb 100644 (file)
@@ -415,19 +415,19 @@ class ObjectCacher {
   void wrunlock(Object *o);
 
  public:
-  void bh_read_finish(sobject_t oid, loff_t offset, size_t length, bufferlist &bl);
-  void bh_write_ack(sobject_t oid, loff_t offset, size_t length, tid_t t);
-  void bh_write_commit(sobject_t oid, loff_t offset, size_t length, tid_t t);
+  void bh_read_finish(sobject_t oid, loff_t offset, __u64 length, bufferlist &bl);
+  void bh_write_ack(sobject_t oid, loff_t offset, __u64 length, tid_t t);
+  void bh_write_commit(sobject_t oid, loff_t offset, __u64 length, tid_t t);
   void lock_ack(list<sobject_t>& oids, tid_t tid);
 
   class C_ReadFinish : public Context {
     ObjectCacher *oc;
     sobject_t oid;
     loff_t start;
-    size_t length;
+    __u64 length;
   public:
     bufferlist bl;
-    C_ReadFinish(ObjectCacher *c, sobject_t o, loff_t s, size_t l) : oc(c), oid(o), start(s), length(l) {}
+    C_ReadFinish(ObjectCacher *c, sobject_t o, loff_t s, __u64 l) : oc(c), oid(o), start(s), length(l) {}
     void finish(int r) {
       oc->bh_read_finish(oid, start, length, bl);
     }
@@ -437,10 +437,10 @@ class ObjectCacher {
     ObjectCacher *oc;
     sobject_t oid;
     loff_t start;
-    size_t length;
+    __u64 length;
   public:
     tid_t tid;
-    C_WriteAck(ObjectCacher *c, sobject_t o, loff_t s, size_t l) : oc(c), oid(o), start(s), length(l) {}
+    C_WriteAck(ObjectCacher *c, sobject_t o, loff_t s, __u64 l) : oc(c), oid(o), start(s), length(l) {}
     void finish(int r) {
       oc->bh_write_ack(oid, start, length, tid);
     }
@@ -449,10 +449,10 @@ class ObjectCacher {
     ObjectCacher *oc;
     sobject_t oid;
     loff_t start;
-    size_t length;
+    __u64 length;
   public:
     tid_t tid;
-    C_WriteCommit(ObjectCacher *c, sobject_t o, loff_t s, size_t l) : oc(c), oid(o), start(s), length(l) {}
+    C_WriteCommit(ObjectCacher *c, sobject_t o, loff_t s, __u64 l) : oc(c), oid(o), start(s), length(l) {}
     void finish(int r) {
       oc->bh_write_commit(oid, start, length, tid);
     }
@@ -525,7 +525,7 @@ class ObjectCacher {
   bool is_cached(inodeno_t ino, vector<ObjectExtent>& extents, snapid_t snapid);
 
   // write blocking
-  bool wait_for_write(size_t len, Mutex& lock);
+  bool wait_for_write(__u64 len, Mutex& lock);
   
   // blocking.  atomic+sync.
   int atomic_sync_readx(OSDRead *rd, inodeno_t ino, Mutex& lock);
@@ -555,14 +555,14 @@ class ObjectCacher {
 
   /*** async+caching (non-blocking) file interface ***/
   int file_is_cached(inodeno_t ino, ceph_file_layout *layout, snapid_t snapid,
-                    loff_t offset, size_t len) {
+                    loff_t offset, __u64 len) {
     vector<ObjectExtent> extents;
     filer.file_to_extents(ino, layout, offset, len, extents);
     return is_cached(ino, extents, snapid);
   }
 
   int file_read(inodeno_t ino, ceph_file_layout *layout, snapid_t snapid,
-                loff_t offset, size_t len, 
+                loff_t offset, __u64 len, 
                 bufferlist *bl,
                int flags,
                 Context *onfinish) {
@@ -572,7 +572,7 @@ class ObjectCacher {
   }
 
   int file_write(inodeno_t ino, ceph_file_layout *layout, const SnapContext& snapc,
-                 loff_t offset, size_t len, 
+                 loff_t offset, __u64 len, 
                  bufferlist& bl, utime_t mtime, int flags) {
     OSDWrite *wr = prepare_write(snapc, bl, mtime, flags);
     filer.file_to_extents(ino, layout, offset, len, wr->extents);
@@ -585,7 +585,7 @@ class ObjectCacher {
 
   int file_atomic_sync_read(inodeno_t ino, ceph_file_layout *layout, 
                            snapid_t snapid,
-                            loff_t offset, size_t len, 
+                            loff_t offset, __u64 len, 
                             bufferlist *bl, int flags,
                             Mutex &lock) {
     OSDRead *rd = prepare_read(snapid, bl, flags);
@@ -595,7 +595,7 @@ class ObjectCacher {
 
   int file_atomic_sync_write(inodeno_t ino, ceph_file_layout *layout, 
                             const SnapContext& snapc,
-                             loff_t offset, size_t len, 
+                             loff_t offset, __u64 len, 
                              bufferlist& bl, utime_t mtime, int flags,
                              Mutex &lock) {
     OSDWrite *wr = prepare_write(snapc, bl, mtime, flags);
index eb1f97d011163109948b16f02cfef1872f8b1860..91981cd5993c58d7502e825624be40cdf068c43a 100644 (file)
@@ -824,7 +824,7 @@ void Objecter::_sg_read_finish(vector<ObjectExtent>& extents, vector<bufferlist>
                               bufferlist *bl, Context *onfinish)
 {
   // all done
-  size_t bytes_read = 0;
+  __u64 bytes_read = 0;
   
   dout(15) << "_sg_read_finish" << dendl;
 
index 87b046ab0aa6b4980f7ee53f505ffda82cf23f98..8042a39eeaace20e6aaec805328cb1e7c12e70f9 100644 (file)
@@ -480,7 +480,7 @@ private:
   }
 
   tid_t read(const object_t& oid, ceph_object_layout ol, 
-            __u64 off, size_t len, snapid_t snap, bufferlist *pbl, int flags,
+            __u64 off, __u64 len, snapid_t snap, bufferlist *pbl, int flags,
             Context *onfinish) {
     vector<OSDOp> ops(1);
     ops[0].op.op = CEPH_OSD_OP_READ;
@@ -536,7 +536,7 @@ private:
     return op_submit(o);
   }
   tid_t write(const object_t& oid, ceph_object_layout ol,
-             __u64 off, size_t len, const SnapContext& snapc, const bufferlist &bl,
+             __u64 off, __u64 len, const SnapContext& snapc, const bufferlist &bl,
              utime_t mtime, int flags,
               Context *onack, Context *oncommit) {
     vector<OSDOp> ops(1);
@@ -563,7 +563,7 @@ private:
     return op_submit(o);
   }
   tid_t zero(const object_t& oid, ceph_object_layout ol, 
-            __u64 off, size_t len, const SnapContext& snapc, utime_t mtime, int flags,
+            __u64 off, __u64 len, const SnapContext& snapc, utime_t mtime, int flags,
              Context *onack, Context *oncommit) {
     vector<OSDOp> ops(1);
     ops[0].op.op = CEPH_OSD_OP_ZERO;