Re: [PATCH v5] Ceph-fuse: Fallocate and punch hole support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sage,
  In general, these look ok to us.
One thing to discuss is that for libcephfs, it works in its own way, in spite of the FALLOC_FL_PUNCH_HOLE defined or not. Shall we limit it?

Cheers,
Li Wang

On 08/16/2013 02:06 PM, Sage Weil wrote:
Hi Li-

I pushed a couple small fixes on top of the wip-fallocate branch; let me
konw if they look ok to you.

	https://github.com/ceph/ceph/commits/wip-fallocate

Thanks!
sage


On Thu, 15 Aug 2013, Li Wang wrote:

This patch implements fallocate and punch hole support for Ceph fuse client.

Signed-off-by: Yunchuan Wen <yunchuanwen@xxxxxxxxxxxxxxx>
Signed-off-by: Li Wang <liwang@xxxxxxxxxxxxxxx>
---
Enable libcephfs to not delete the first object by passing in a flag.
---
  src/client/Client.cc           |   93 ++++++++++++++++++++++++++++++++++++++++
  src/client/Client.h            |    3 ++
  src/client/fuse_ll.cc          |   26 +++++++++++
  src/include/cephfs/libcephfs.h |   18 ++++++++
  src/libcephfs.cc               |    8 ++++
  src/osdc/Filer.h               |   23 +++++++++-
  6 files changed, 169 insertions(+), 2 deletions(-)

diff --git a/src/client/Client.cc b/src/client/Client.cc
index 7e26a43..fee0453 100644
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -22,6 +22,7 @@
  #include <sys/stat.h>
  #include <sys/param.h>
  #include <fcntl.h>
+#include <linux/falloc.h>

  #include <sys/statvfs.h>

@@ -7685,6 +7686,98 @@ int Client::ll_fsync(Fh *fh, bool syncdataonly)
    return _fsync(fh, syncdataonly);
  }

+int Client::_fallocate(Fh *fh, int mode, int64_t offset, int64_t length)
+{
+  if (offset < 0 || length <= 0)
+    return -EINVAL;
+
+  if (mode & ~(FALLOC_FL_KEEP_SIZE | FALLOC_FL_PUNCH_HOLE))
+    return -EOPNOTSUPP;
+
+  if ((mode & FALLOC_FL_PUNCH_HOLE) && !(mode & FALLOC_FL_KEEP_SIZE))
+    return -EOPNOTSUPP;
+
+  if (osdmap->test_flag(CEPH_OSDMAP_FULL) && !(mode & FALLOC_FL_PUNCH_HOLE))
+    return -ENOSPC;
+
+  Inode *in = fh->inode;
+
+  if (in->snapid != CEPH_NOSNAP)
+    return -EROFS;
+
+  if ((fh->mode & CEPH_FILE_MODE_WR) == 0)
+    return -EBADF;
+
+  int have;
+  int r = get_caps(in, CEPH_CAP_FILE_WR, CEPH_CAP_FILE_BUFFER, &have, -1);
+  if (r < 0)
+    return r;
+
+  if (mode & FALLOC_FL_PUNCH_HOLE) {
+    Mutex flock("Client::_punch_hole flock");
+    Cond cond;
+    bool done = false;
+    Context *onfinish = new C_SafeCond(&flock, &cond, &done);
+    Context *onsafe = new C_Client_SyncCommit(this, in);
+
+    unsafe_sync_write++;
+    get_cap_ref(in, CEPH_CAP_FILE_BUFFER);
+
+    _invalidate_inode_cache(in, offset, length, true);
+    r = filer->zero(in->ino, &in->layout,
+                    in->snaprealm->get_snap_context(),
+                    offset, length,
+                    ceph_clock_now(cct),
+                    0, true, onfinish, onsafe);
+    if (r < 0)
+      goto done;
+
+    client_lock.Unlock();
+    flock.Lock();
+    while (!done)
+      cond.Wait(flock);
+    flock.Unlock();
+    client_lock.Lock();
+  } else if (!(mode & FALLOC_FL_KEEP_SIZE)) {
+    uint64_t size = offset + length;
+    if (size > in->size) {
+      in->size = size;
+      mark_caps_dirty(in, CEPH_CAP_FILE_WR);
+
+      if ((in->size << 1) >= in->max_size &&
+          (in->reported_size << 1) < in->max_size)
+        check_caps(in, false);
+    }
+  }
+
+  in->mtime = ceph_clock_now(cct);
+  mark_caps_dirty(in, CEPH_CAP_FILE_WR);
+
+done:
+  put_cap_ref(in, CEPH_CAP_FILE_WR);
+  return r;
+}
+
+int Client::ll_fallocate(Fh *fh, int mode, loff_t offset, loff_t length)
+{
+  Mutex::Locker lock(client_lock);
+  ldout(cct, 3) << "ll_fallocate " << fh << " " << fh->inode->ino << " " << dendl;
+  tout(cct) << "ll_fallocate " << mode << " " << offset << " " << length << std::endl;
+  tout(cct) << (unsigned long)fh << std::endl;
+
+  return _fallocate(fh, mode, offset, length);
+}
+
+int Client::fallocate(int fd, int mode, loff_t offset, loff_t length)
+{
+  Mutex::Locker lock(client_lock);
+  tout(cct) << "fallocate " << " " << fd << mode << " " << offset << " " << length << std::endl;
+
+  Fh *fh = get_filehandle(fd);
+  if (!fh)
+    return -EBADF;
+  return _fallocate(fh, mode, offset, length);
+}

  int Client::ll_release(Fh *fh)
  {
diff --git a/src/client/Client.h b/src/client/Client.h
index 1117ff3..5adc4bf 100644
--- a/src/client/Client.h
+++ b/src/client/Client.h
@@ -560,6 +560,7 @@ private:
    int _flush(Fh *fh);
    int _fsync(Fh *fh, bool syncdataonly);
    int _sync_fs();
+  int _fallocate(Fh *fh, int mode, int64_t offset, int64_t length);

    int get_or_create(Inode *dir, const char* name,
  		    Dentry **pdn, bool expect_null=false);
@@ -658,6 +659,7 @@ public:
    int ftruncate(int fd, loff_t size);
    int fsync(int fd, bool syncdataonly);
    int fstat(int fd, struct stat *stbuf);
+  int fallocate(int fd, int mode, loff_t offset, loff_t length);

    // full path xattr ops
    int getxattr(const char *path, const char *name, void *value, size_t size);
@@ -727,6 +729,7 @@ public:
    int ll_write(Fh *fh, loff_t off, loff_t len, const char *data);
    int ll_flush(Fh *fh);
    int ll_fsync(Fh *fh, bool syncdataonly);
+  int ll_fallocate(Fh *fh, int mode, loff_t offset, loff_t length);
    int ll_release(Fh *fh);
    int ll_statfs(vinodeno_t vino, struct statvfs *stbuf);

diff --git a/src/client/fuse_ll.cc b/src/client/fuse_ll.cc
index 0812c9a..e62307d 100644
--- a/src/client/fuse_ll.cc
+++ b/src/client/fuse_ll.cc
@@ -400,6 +400,20 @@ static void fuse_ll_ioctl(fuse_req_t req, fuse_ino_t ino, int cmd, void *arg, st
  }
  #endif

+#if FUSE_VERSION >= FUSE_MAKE_VERSION(2, 9)
+
+static void fuse_ll_fallocate(fuse_req_t req, fuse_ino_t ino, int mode,
+                              off_t offset, off_t length,
+                              struct fuse_file_info *fi)
+{
+  CephFuse::Handle *cfuse = (CephFuse::Handle *)fuse_req_userdata(req);
+  Fh *fh = (Fh*)fi->fh;
+  int r = cfuse->client->ll_fallocate(fh, mode, offset, length);
+  fuse_reply_err(req, -r);
+}
+
+#endif
+
  static void fuse_ll_release(fuse_req_t req, fuse_ino_t ino, struct fuse_file_info *fi)
  {
    CephFuse::Handle *cfuse = (CephFuse::Handle *)fuse_req_userdata(req);
@@ -602,8 +616,20 @@ const static struct fuse_lowlevel_ops fuse_ll_oper = {
   getlk: 0,
   setlk: 0,
   bmap: 0,
+#if FUSE_VERSION >= FUSE_MAKE_VERSION(2, 8)
  #ifdef FUSE_IOCTL_COMPAT
   ioctl: fuse_ll_ioctl,
+#else
+ ioctl: 0,
+#endif
+ poll: 0,
+#if FUSE_VERSION >= FUSE_MAKE_VERSION(2, 9)
+ write_buf: 0,
+ retrieve_reply: 0,
+ forget_multi: 0,
+ flock: 0,
+ fallocate: fuse_ll_fallocate
+#endif
  #endif
  };

diff --git a/src/include/cephfs/libcephfs.h b/src/include/cephfs/libcephfs.h
index 93e86e7..9b74f63 100644
--- a/src/include/cephfs/libcephfs.h
+++ b/src/include/cephfs/libcephfs.h
@@ -709,6 +709,24 @@ int ceph_ftruncate(struct ceph_mount_info *cmount, int fd, loff_t size);
  int ceph_fsync(struct ceph_mount_info *cmount, int fd, int syncdataonly);

  /**
+ * Preallocate or release disk space for the file for the byte range.
+ *
+ * @param cmount the ceph mount handle to use for performing the fallocate.
+ * @param fd the file descriptor of the file to fallocate.
+ * @param mode the flags determines the operation to be performed on the given range.
+ *        default operation (0) allocate and initialize to zero the file in the byte range,
+ *        and the file size will be changed if offset + length is greater than
+ *        the file size. if the FALLOC_FL_KEEP_SIZE flag is specified in the mode,
+ *        the file size will not be changed. if the FALLOC_FL_PUNCH_HOLE flag is
+ *        specified in the mode, the operation is deallocate space and zero the byte range.
+ * @param offset the byte range starting.
+ * @param length the length of the range.
+ * @return 0 on success or a negative error code on failure.
+ */
+int ceph_fallocate(struct ceph_mount_info *cmount, int fd, int mode,
+	                      loff_t offset, loff_t length);
+
+/**
   * Get the open file's statistics.
   *
   * @param cmount the ceph mount handle to use for performing the fstat.
diff --git a/src/libcephfs.cc b/src/libcephfs.cc
index 16b130a..306c4ba 100644
--- a/src/libcephfs.cc
+++ b/src/libcephfs.cc
@@ -700,6 +700,14 @@ extern "C" int ceph_fsync(struct ceph_mount_info *cmount, int fd, int syncdataon
    return cmount->get_client()->fsync(fd, syncdataonly);
  }

+extern "C" int ceph_fallocate(struct ceph_mount_info *cmount, int fd, int mode,
+	                      loff_t offset, loff_t length)
+{
+  if (!cmount->is_mounted())
+    return -ENOTCONN;
+  return cmount->get_client()->fallocate(fd, mode, offset, length);
+}
+
  extern "C" int ceph_fstat(struct ceph_mount_info *cmount, int fd, struct stat *stbuf)
  {
    if (!cmount->is_mounted())
diff --git a/src/osdc/Filer.h b/src/osdc/Filer.h
index 86ff601..c069259 100644
--- a/src/osdc/Filer.h
+++ b/src/osdc/Filer.h
@@ -208,12 +208,14 @@ class Filer {
             uint64_t len,
  	   utime_t mtime,
  	   int flags,
+	   bool keep_first,
             Context *onack,
             Context *oncommit) {
      vector<ObjectExtent> extents;
      Striper::file_to_extents(cct, ino, layout, offset, len, 0, extents);
      if (extents.size() == 1) {
-      if (extents[0].offset == 0 && extents[0].length == layout->fl_object_size)
+      if (extents[0].offset == 0 && extents[0].length == layout->fl_object_size &&
+      	  (!keep_first || extents[0].objectno != 0))
  	objecter->remove(extents[0].oid, extents[0].oloc,
  			 snapc, mtime, flags, onack, oncommit);
        else
@@ -223,7 +225,8 @@ class Filer {
        C_GatherBuilder gack(cct, onack);
        C_GatherBuilder gcom(cct, oncommit);
        for (vector<ObjectExtent>::iterator p = extents.begin(); p != extents.end(); ++p) {
-	if (p->offset == 0 && p->length == layout->fl_object_size)
+	if (p->offset == 0 && p->length == layout->fl_object_size &&
+	    (!keep_first || p->objectno != 0))
  	  objecter->remove(p->oid, p->oloc,
  			   snapc, mtime, flags,
  			   onack ? gack.new_sub():0,
@@ -240,6 +243,22 @@ class Filer {
      return 0;
    }

+  int zero(inodeno_t ino,
+	   ceph_file_layout *layout,
+	   const SnapContext& snapc,
+	   uint64_t offset,
+           uint64_t len,
+	   utime_t mtime,
+	   int flags,
+           Context *onack,
+           Context *oncommit) {
+
+    return zero(ino, layout,
+                snapc, offset,
+                len, mtime,
+                flags, false,
+                onack, oncommit);
+  }
    // purge range of ino.### objects
    int purge_range(inodeno_t ino,
  		  ceph_file_layout *layout,
--
1.7.9.5

--
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



--
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




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux