]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph-fs: kill the public ceph_file_layout_is_valid() method
authorxie xingguo <xie.xingguo@zte.com.cn>
Thu, 16 Jun 2016 06:42:50 +0000 (14:42 +0800)
committerxie xingguo <xie.xingguo@zte.com.cn>
Sat, 18 Jun 2016 01:54:57 +0000 (09:54 +0800)
Which is deprecated file_layout_t::is_valid(), and removing
the unnecessary code is always the preferred option.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
src/common/ceph_fs.cc
src/include/ceph_fs.h

index 6b69e267b70f33c036905d07c1746a2f90cda39c..a4f71be943acf7aa6d0cb5021e10b2ddc299d36c 100644 (file)
  */
 #include "include/types.h"
 
-/*
- * return true if @layout appears to be valid
- */
-int ceph_file_layout_is_valid(const struct ceph_file_layout *layout)
-{
-       __u32 su = le32_to_cpu(layout->fl_stripe_unit);
-       __u32 sc = le32_to_cpu(layout->fl_stripe_count);
-       __u32 os = le32_to_cpu(layout->fl_object_size);
-
-       /* stripe unit, object size must be non-zero, 64k increment */
-       if (!su || (su & (CEPH_MIN_STRIPE_UNIT-1)))
-               return 0;
-       if (!os || (os & (CEPH_MIN_STRIPE_UNIT-1)))
-               return 0;
-       /* object size must be a multiple of stripe unit */
-       if (os < su || os % su)
-               return 0;
-       /* stripe count must be non-zero */
-       if (!sc)
-               return 0;
-       return 1;
-}
-
-
 int ceph_flags_to_mode(int flags)
 {
        /* because CEPH_FILE_MODE_PIN is zero, so mode = -1 is error */
index 2a3992522e4c423c412f95907f5b83fa2f96ef3e..45188ddb637857bbe4e10d35efaab128703b5cc1 100644 (file)
@@ -56,8 +56,6 @@ struct ceph_file_layout {
 
 #define CEPH_MIN_STRIPE_UNIT 65536
 
-int ceph_file_layout_is_valid(const struct ceph_file_layout *layout);
-
 struct ceph_dir_layout {
        __u8   dl_dir_hash;   /* see ceph_hash.h for ids */
        __u8   dl_unused1;