Whamcloud - gitweb
b=19742 fixed the fiemap functionality
authorRahul Deshmukh <Rahul.Deshmukh@Sun.COM>
Tue, 2 Feb 2010 08:44:41 +0000 (14:14 +0530)
committerRobert Read <rread@sun.com>
Wed, 3 Feb 2010 19:54:31 +0000 (11:54 -0800)
Fixed the fiemap functionality and enable sanity test_130*.

i=adilger
i=johann

ldiskfs/kernel_patches/patches/ext3-fiemap-2.6-rhel5.patch
ldiskfs/kernel_patches/patches/ext4-fiemap-2.6-rhel5.patch
ldiskfs/kernel_patches/series/ldiskfs-2.6-rhel5-ext4.series
ldiskfs/kernel_patches/series/ldiskfs-2.6-rhel5.series
lustre/llite/file.c
lustre/tests/sanity.sh

index 49e8f8f..f76beb6 100644 (file)
@@ -1,81 +1,13 @@
-A large part of this code is from the generic VFS code in fs/ioctl.c in the
-upstream kernel.
-
-Index: linux-2.6.18-128.1.6/fs/ext3/ioctl.c
-===================================================================
---- linux-2.6.18-128.1.6.orig/fs/ext3/ioctl.c
-+++ linux-2.6.18-128.1.6/fs/ext3/ioctl.c
-@@ -15,7 +15,161 @@
- #include <linux/time.h>
+diff -rupN linux-2.6.18-164.6.1_1/fs/ext3/ioctl.c linux-2.6.18-164.6.1_2/fs/ext3/ioctl.c
+--- linux-2.6.18-164.6.1_1/fs/ext3/ioctl.c     2009-12-22 12:37:55.000000000 +0530
++++ linux-2.6.18-164.6.1_2/fs/ext3/ioctl.c     2009-12-22 12:40:44.000000000 +0530
+@@ -16,6 +16,70 @@
  #include <asm/uaccess.h>
  #include <linux/namei.h>
  #include <asm/uaccess.h>
  #include <linux/namei.h>
-+#include "fiemap.h"
  
 +/* So that the fiemap access checks can't overflow on 32 bit machines. */
 +#define FIEMAP_MAX_EXTENTS     (UINT_MAX / sizeof(struct fiemap_extent))
 +
  
 +/* So that the fiemap access checks can't overflow on 32 bit machines. */
 +#define FIEMAP_MAX_EXTENTS     (UINT_MAX / sizeof(struct fiemap_extent))
 +
-+/**
-+ * fiemap_fill_next_extent - Fiemap helper function
-+ * @fieinfo:   Fiemap context passed into ->fiemap
-+ * @logical:   Extent logical start offset, in bytes
-+ * @phys:      Extent physical start offset, in bytes
-+ * @len:       Extent length, in bytes
-+ * @flags:     FIEMAP_EXTENT flags that describe this extent
-+ * @lun:       LUN on which this extent resides
-+ *
-+ * Called from file system ->fiemap callback. Will populate extent
-+ * info as passed in via arguments and copy to user memory. On
-+ * success, extent count on fieinfo is incremented.
-+ *
-+ * Returns 0 on success, -errno on error, 1 if this was the last
-+ * extent that will fit in user array.
-+ */
-+#define SET_UNKNOWN_FLAGS      (FIEMAP_EXTENT_DELALLOC)
-+#define SET_NO_DIRECT_FLAGS    (FIEMAP_EXTENT_DATA_ENCRYPTED  \
-+                              |FIEMAP_EXTENT_NET)
-+#define SET_NO_UNMOUNTED_IO_FLAGS       (FIEMAP_EXTENT_DATA_ENCRYPTED)
-+#define SET_NOT_ALIGNED_FLAGS  (FIEMAP_EXTENT_DATA_TAIL|FIEMAP_EXTENT_DATA_INLINE)
-+int fiemap_fill_next_extent(struct fiemap_extent_info *fieinfo, u64 logical,
-+                          u64 phys, u64 len, u32 flags, dev_t dev)
-+{
-+      struct fiemap_extent extent = { 0 };
-+      struct fiemap_extent *dest = fieinfo->fi_extents_start;
-+
-+      /* only count the extents */
-+      if (fieinfo->fi_extents_max == 0) {
-+              fieinfo->fi_extents_mapped++;
-+              return (flags & FIEMAP_EXTENT_LAST) ? 1 : 0;
-+      }
-+
-+      if (fieinfo->fi_extents_mapped >= fieinfo->fi_extents_max)
-+              return 1;
-+
-+      if (flags & SET_UNKNOWN_FLAGS)
-+              flags |= FIEMAP_EXTENT_UNKNOWN;
-+      if (flags & SET_NO_DIRECT_FLAGS)
-+              flags |= FIEMAP_EXTENT_NO_DIRECT;
-+      if (flags & SET_NOT_ALIGNED_FLAGS)
-+              flags |= FIEMAP_EXTENT_NOT_ALIGNED;
-+      if (flags & SET_NO_UNMOUNTED_IO_FLAGS)
-+              flags |= FIEMAP_EXTENT_ENCODED;
-+
-+      extent.fe_logical = logical;
-+      extent.fe_physical = phys;
-+      extent.fe_length = len;
-+      extent.fe_flags = flags;
-+      extent.fe_device = new_encode_dev(dev);
-+
-+      dest += fieinfo->fi_extents_mapped;
-+      if (copy_to_user(dest, &extent, sizeof(extent)))
-+              return -EFAULT;
-+
-+      fieinfo->fi_extents_mapped++;
-+      if (fieinfo->fi_extents_mapped == fieinfo->fi_extents_max)
-+              return 1;
-+
-+      return (flags & FIEMAP_EXTENT_LAST) ? 1 : 0;
-+}
-+
 +static int fiemap_check_ranges(struct super_block *sb,
 +                             u64 start, u64 len, u64 *new_len)
 +{
 +static int fiemap_check_ranges(struct super_block *sb,
 +                             u64 start, u64 len, u64 *new_len)
 +{
@@ -97,33 +29,6 @@ Index: linux-2.6.18-128.1.6/fs/ext3/ioctl.c
 +      return 0;
 +}
 +
 +      return 0;
 +}
 +
-+/*
-+ * fiemap_check_flags - check validity of requested flags for fiemap
-+ * @fieinfo:   Fiemap context passed into ->fiemap
-+ * @fs_flags:  Set of fiemap flags that the file system understands
-+ *
-+ * Called from file system ->fiemap callback. This will compute the
-+ * intersection of valid fiemap flags and those that the fs supports. That
-+ * value is then compared against the user supplied flags. In case of bad user
-+ * flags, the invalid values will be written into the fieinfo structure, and
-+ * -EBADR is returned, which tells ioctl_fiemap() to return those values to
-+ * userspace. For this reason, a return code of -EBADR should be preserved.
-+ *
-+ * Returns 0 on success, -EBADR on bad flags.
-+*/
-+int fiemap_check_flags(struct fiemap_extent_info *fieinfo, u32 fs_flags)
-+{
-+      u32 incompat_flags;
-+
-+      incompat_flags = fieinfo->fi_flags & ~(FIEMAP_FLAGS_COMPAT & fs_flags);
-+      if (incompat_flags) {
-+              fieinfo->fi_flags = incompat_flags;
-+              return -EBADR;
-+      }
-+
-+      return 0;
-+}
-+
 +int ioctl_fiemap(struct inode *inode, struct file *filp, unsigned long arg)
 +{
 +      struct fiemap fiemap;
 +int ioctl_fiemap(struct inode *inode, struct file *filp, unsigned long arg)
 +{
 +      struct fiemap fiemap;
@@ -167,7 +72,7 @@ Index: linux-2.6.18-128.1.6/fs/ext3/ioctl.c
  
  int ext3_ioctl (struct inode * inode, struct file * filp, unsigned int cmd,
                unsigned long arg)
  
  int ext3_ioctl (struct inode * inode, struct file * filp, unsigned int cmd,
                unsigned long arg)
-@@ -276,7 +430,9 @@ flags_err:
+@@ -276,7 +340,9 @@ flags_err:
  
                return err;
        }
  
                return err;
        }
@@ -178,319 +83,14 @@ Index: linux-2.6.18-128.1.6/fs/ext3/ioctl.c
  
        default:
                return -ENOTTY;
  
        default:
                return -ENOTTY;
-Index: linux-2.6.18-128.1.6/include/linux/ext3_fs.h
-===================================================================
---- linux-2.6.18-128.1.6.orig/include/linux/ext3_fs.h
-+++ linux-2.6.18-128.1.6/include/linux/ext3_fs.h
-@@ -258,15 +258,19 @@ struct ext3_new_group_data {
- #define       EXT3_IOC_SETFLAGS               _IOW('f', 2, long)
- #define       EXT3_IOC_GETVERSION             _IOR('f', 3, long)
+diff -rupN linux-2.6.18-164.6.1_1/include/linux/ext3_fs.h linux-2.6.18-164.6.1_2/include/linux/ext3_fs.h
+--- linux-2.6.18-164.6.1_1/include/linux/ext3_fs.h     2009-12-22 12:37:55.000000000 +0530
++++ linux-2.6.18-164.6.1_2/include/linux/ext3_fs.h     2009-12-22 12:43:02.000000000 +0530
+@@ -260,6 +260,7 @@ struct ext3_new_group_data {
  #define       EXT3_IOC_SETVERSION             _IOW('f', 4, long)
  #define       EXT3_IOC_SETVERSION             _IOW('f', 4, long)
-+#define EXT3_IOC_GETRSVSZ             _IOR('f', 5, long)
-+#define EXT3_IOC_SETRSVSZ             _IOW('f', 6, long)
  #define EXT3_IOC_GROUP_EXTEND         _IOW('f', 7, unsigned long)
  #define EXT3_IOC_GROUP_ADD            _IOW('f', 8,struct ext3_new_group_input)
  #define EXT3_IOC_GROUP_EXTEND         _IOW('f', 7, unsigned long)
  #define EXT3_IOC_GROUP_ADD            _IOW('f', 8,struct ext3_new_group_input)
-+#define EXT3_IOC_FIEMAP                       _IOWR('f', 11, struct fiemap)
++#define EXT3_IOC_FIEMAP               _IOWR('f', 11, struct fiemap)
  #define       EXT3_IOC_GETVERSION_OLD         _IOR('v', 1, long)
  #define       EXT3_IOC_SETVERSION_OLD         _IOW('v', 2, long)
  #ifdef CONFIG_JBD_DEBUG
  #define       EXT3_IOC_GETVERSION_OLD         _IOR('v', 1, long)
  #define       EXT3_IOC_SETVERSION_OLD         _IOW('v', 2, long)
  #ifdef CONFIG_JBD_DEBUG
- #define EXT3_IOC_WAIT_FOR_READONLY    _IOR('f', 99, long)
- #endif
--#define EXT3_IOC_GETRSVSZ             _IOR('f', 5, long)
--#define EXT3_IOC_SETRSVSZ             _IOW('f', 6, long)
-+
-+/* FIEMAP flags supported by ext3 */
-+#define EXT3_FIEMAP_FLAGS_COMPAT (FIEMAP_FLAG_SYNC)
- /*
-  *  Mount options
-@@ -1041,6 +1045,9 @@ extern int ext3_block_truncate_page(hand
- /* ioctl.c */
- extern int ext3_ioctl (struct inode *, struct file *, unsigned int,
-                      unsigned long);
-+struct fiemap_extent_info;
-+extern int ext3_fiemap(struct inode *, struct fiemap_extent_info *, __u64,
-+                     __u64);
- /* namei.c */
- extern int ext3_orphan_add(handle_t *, struct inode *);
-@@ -1118,7 +1125,6 @@ ext3_get_blocks_wrap(handle_t *handle, s
-       return ret;
- }
--
- #endif        /* __KERNEL__ */
- /* EXT3_IOC_CREATE_INUM at bottom of file (visible to kernel and user). */
-Index: linux-2.6.18-128.1.6/include/linux/ext3_extents.h
-===================================================================
---- linux-2.6.18-128.1.6.orig/include/linux/ext3_extents.h
-+++ linux-2.6.18-128.1.6/include/linux/ext3_extents.h
-@@ -142,8 +142,10 @@ struct ext3_ext_path {
-  * callback must return valid extent (passed or newly created)
-  */
- typedef int (*ext_prepare_callback)(struct inode *, struct ext3_ext_path *,
--                                      struct ext3_ext_cache *,
--                                      void *);
-+                                  struct ext3_ext_cache *,
-+                                  struct ext3_extent *, void *);
-+
-+#define HAVE_EXT_PREPARE_CB_EXTENT
- #define EXT_CONTINUE  0
- #define EXT_BREAK     1
-@@ -153,6 +155,26 @@ typedef int (*ext_prepare_callback)(stru
- #define EXT_MAX_BLOCK 0xffffffff
- #define EXT_UNSET_BLOCK 1
-+/*
-+ * EXT_INIT_MAX_LEN is the maximum number of blocks we can have in an
-+ * initialized extent. This is 2^15 and not (2^16 - 1), since we use the
-+ * MSB of ee_len field in the extent datastructure to signify if this
-+ * particular extent is an initialized extent or an uninitialized (i.e.
-+ * preallocated).
-+ * EXT_UNINIT_MAX_LEN is the maximum number of blocks we can have in an
-+ * uninitialized extent.
-+ * If ee_len is <= 0x8000, it is an initialized extent. Otherwise, it is an
-+ * uninitialized one. In other words, if MSB of ee_len is set, it is an
-+ * uninitialized extent with only one special scenario when ee_len = 0x8000.
-+ * In this case we can not have an uninitialized extent of zero length and
-+ * thus we make it as a special case of initialized extent with 0x8000 length.
-+ * This way we get better extent-to-group alignment for initialized extents.
-+ * Hence, the maximum number of blocks we can have in an *initialized*
-+ * extent is 2^15 (32768) and in an *uninitialized* extent is 2^15-1 (32767).
-+ */
-+#define EXT_INIT_MAX_LEN      (1UL << 15)
-+#define EXT_UNINIT_MAX_LEN    (EXT_INIT_MAX_LEN - 1)
-+
- #define EXT_FLAGS_CLR_UNKNOWN 0x7  /* Flags cleared on modification */
- #define EXT_HDR_GEN_BITS      24
- #define EXT_HDR_GEN_MASK      ((1 << EXT_HDR_GEN_BITS) - 1)
-@@ -220,6 +242,12 @@ ext3_ext_invalidate_cache(struct inode *
-       EXT3_I(inode)->i_cached_extent.ec_type = EXT3_EXT_CACHE_NO;
- }
-+static inline int ext3_ext_is_uninitialized(struct ext3_extent *ext)
-+{
-+      /* Extent with ee_len of 0x8000 is treated as an initialized extent */
-+      return (le16_to_cpu(ext->ee_len) > EXT_INIT_MAX_LEN);
-+}
-+
- extern int ext3_ext_search_left(struct inode *, struct ext3_ext_path *, unsigned long *, unsigned long *);
- extern int ext3_ext_search_right(struct inode *, struct ext3_ext_path *, unsigned long *, unsigned long *);
- extern int ext3_extent_tree_init(handle_t *, struct inode *);
-Index: linux-2.6.18-128.1.6/fs/ext3/extents.c
-===================================================================
---- linux-2.6.18-128.1.6.orig/fs/ext3/extents.c
-+++ linux-2.6.18-128.1.6/fs/ext3/extents.c
-@@ -41,6 +41,7 @@
- #include <linux/string.h>
- #include <linux/slab.h>
- #include <linux/ext3_extents.h>
-+#include "fiemap.h"
- #include <asm/uaccess.h>
-@@ -1481,7 +1482,7 @@ int ext3_ext_walk_space(struct inode *in
-               }
-               BUG_ON(cbex.ec_len == 0);
--              err = func(inode, path, &cbex, cbdata);
-+              err = func(inode, path, &cbex, ex, cbdata);
-               ext3_ext_drop_refs(path);
-               if (err < 0)
-@@ -2296,6 +2297,103 @@ int ext3_ext_writepage_trans_blocks(stru
-       return needed;
- }
-+/*
-+ * Callback function called for each extent to gather FIEMAP information.
-+ */
-+int ext3_ext_fiemap_cb(struct inode *inode, struct ext3_ext_path *path,
-+                     struct ext3_ext_cache *newex, struct ext3_extent *ex,
-+                     void *data)
-+{
-+      struct fiemap_extent_info *fieinfo = data;
-+      unsigned char blksize_bits = inode->i_sb->s_blocksize_bits;
-+      __u64   logical;
-+      __u64   physical;
-+      __u64   length;
-+      __u32   flags = 0;
-+      int     error;
-+
-+      logical =  (__u64)newex->ec_block << blksize_bits;
-+
-+      if (newex->ec_type == EXT3_EXT_CACHE_GAP) {
-+              pgoff_t offset;
-+              struct page *page;
-+              struct buffer_head *bh = NULL;
-+
-+              offset = logical >> PAGE_SHIFT;
-+              page = find_get_page(inode->i_mapping, offset);
-+              if (!page || !page_has_buffers(page))
-+                      return EXT_CONTINUE;
-+
-+              bh = page_buffers(page);
-+
-+              if (!bh)
-+                      return EXT_CONTINUE;
-+
-+              if (buffer_delay(bh)) {
-+                      flags |= FIEMAP_EXTENT_DELALLOC;
-+                      page_cache_release(page);
-+              } else {
-+                      page_cache_release(page);
-+                      return EXT_CONTINUE;
-+              }
-+      }
-+
-+      physical = (__u64)newex->ec_start << blksize_bits;
-+      length =   (__u64)newex->ec_len << blksize_bits;
-+
-+      if (ex && ext3_ext_is_uninitialized(ex))
-+              flags |= FIEMAP_EXTENT_UNWRITTEN;
-+
-+      /*
-+       * If this extent reaches EXT_MAX_BLOCK, it must be last.
-+       *
-+       * Or if ext3_ext_next_allocated_block is EXT_MAX_BLOCK,
-+       * this indicates no more allocated blocks.
-+       *
-+       * XXX this might miss a single-block extent at EXT_MAX_BLOCK
-+       */
-+      if (ext3_ext_next_allocated_block(path) == EXT_MAX_BLOCK ||
-+          newex->ec_block + newex->ec_len - 1 == EXT_MAX_BLOCK)
-+              flags |= FIEMAP_EXTENT_LAST;
-+
-+      error = fiemap_fill_next_extent(fieinfo, logical, physical,
-+                                      length, flags, inode->i_sb->s_dev);
-+      if (error < 0)
-+              return error;
-+      if (error == 1)
-+              return EXT_BREAK;
-+
-+      return EXT_CONTINUE;
-+}
-+
-+int ext3_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
-+              __u64 start, __u64 len)
-+{
-+      ext3_fsblk_t start_blk;
-+      ext3_fsblk_t len_blks;
-+      int error = 0;
-+
-+      if (!(EXT3_I(inode)->i_flags & EXT3_EXTENTS_FL))
-+              return -EOPNOTSUPP;
-+
-+      if (fiemap_check_flags(fieinfo, EXT3_FIEMAP_FLAGS_COMPAT))
-+              return -EBADR;
-+
-+      start_blk = start >> inode->i_sb->s_blocksize_bits;
-+      len_blks = (len + inode->i_sb->s_blocksize - 1) >> inode->i_sb->s_blocksize_bits;
-+
-+      /*
-+        * Walk the extent tree gathering extent information.
-+        * ext3_ext_fiemap_cb will push extents back to user.
-+        */
-+      mutex_lock(&EXT3_I(inode)->truncate_mutex);
-+      error = ext3_ext_walk_space(inode, start_blk, len_blks,
-+                                ext3_ext_fiemap_cb, fieinfo);
-+      mutex_unlock(&EXT3_I(inode)->truncate_mutex);
-+
-+      return error;
-+}
-+
- EXPORT_SYMBOL(ext3_mark_inode_dirty);
- EXPORT_SYMBOL(ext3_ext_invalidate_cache);
- EXPORT_SYMBOL(ext3_ext_insert_extent);
-Index: linux-2.6.18-128.1.6/fs/ext3/fiemap.h
-===================================================================
---- /dev/null
-+++ linux-2.6.18-128.1.6/fs/ext3/fiemap.h
-@@ -0,0 +1,85 @@
-+/*
-+ * FIEMAP ioctl infrastructure.
-+ *
-+ * Copyright 2008 Sun Microsystems, Inc.
-+ *
-+ * Author: Kalpak Shah <kalpak.shah@sun.com>
-+ *     Andreas Dilger <adilger@sun.com>
-+ */
-+
-+#ifndef _LINUX_EXT3_FIEMAP_H
-+#define _LINUX_EXT3_FIEMAP_H
-+
-+struct fiemap_extent {
-+      __u64 fe_logical;  /* logical offset in bytes for the start of
-+                          * the extent from the beginning of the file */
-+      __u64 fe_physical; /* physical offset in bytes for the start
-+                          * of the extent from the beginning of the disk */
-+      __u64 fe_length;   /* length in bytes for this extent */
-+      __u64 fe_reserved64[2];
-+      __u32 fe_flags;    /* FIEMAP_EXTENT_* flags for this extent */
-+      __u32 fe_device;   /* device number for this extent */
-+      __u32 fe_reserved[2];
-+};
-+
-+struct fiemap {
-+      __u64 fm_start;  /* logical offset (inclusive) at
-+                               * which to start mapping (in) */
-+      __u64 fm_length;        /* logical length of mapping which
-+                               * userspace wants (in) */
-+      __u32 fm_flags;  /* FIEMAP_FLAG_* flags for request (in/out) */
-+      __u32 fm_mapped_extents;/* number of extents that were mapped (out) */
-+      __u32 fm_extent_count;  /* size of fm_extents array (in) */
-+      __u32 fm_reserved;
-+      struct fiemap_extent fm_extents[0]; /* array of mapped extents (out) */
-+};
-+
-+/*
-+ * FIEMAP helper definition.
-+ */
-+struct fiemap_extent_info {
-+      unsigned int    fi_flags;               /* Flags as passed from user */
-+      unsigned int    fi_extents_mapped;      /* Number of mapped extents */
-+      unsigned int    fi_extents_max;         /* Size of fiemap_extent array*/
-+      struct fiemap_extent *fi_extents_start; /* Start of fiemap_extent array */
-+};
-+
-+int fiemap_check_flags(struct fiemap_extent_info *fieinfo, u32 fs_flags);
-+int fiemap_fill_next_extent(struct fiemap_extent_info *info, u64 logical,
-+                          u64 phys, u64 len, u32 flags, u32 lun);
-+
-+#define       FIEMAP_MAX_OFFSET       (~0ULL)
-+
-+#define       FIEMAP_FLAG_SYNC        0x00000001 /* sync file data before map */
-+#define       FIEMAP_FLAG_XATTR       0x00000002 /* map extended attribute tree */
-+
-+/* ldiskfs only supports FLAG_SYNC flag currently */
-+#define FIEMAP_FLAGS_COMPAT (FIEMAP_FLAG_SYNC | FIEMAP_FLAG_XATTR)
-+
-+#define FIEMAP_EXTENT_LAST            0x00000001 /* Last extent in file. */
-+#define FIEMAP_EXTENT_UNKNOWN         0x00000002 /* Data location unknown. */
-+#define FIEMAP_EXTENT_DELALLOC                0x00000004 /* Location still pending.
-+                                                  * Sets EXTENT_UNKNOWN. */
-+#define FIEMAP_EXTENT_ENCODED         0x00000008 /* Data can not be read
-+                                                  * while fs is unmounted */
-+#define FIEMAP_EXTENT_DATA_ENCRYPTED  0x00000080 /* Data is encrypted by fs.
-+                                                  * Sets EXTENT_NO_DIRECT. */
-+#define FIEMAP_EXTENT_NOT_ALIGNED     0x00000100 /* Extent offsets may not be
-+                                                  * block aligned. */
-+#define FIEMAP_EXTENT_DATA_INLINE     0x00000200 /* Data mixed with metadata.
-+                                                  * Sets EXTENT_NOT_ALIGNED.*/
-+#define FIEMAP_EXTENT_DATA_TAIL               0x00000400 /* Multiple files in block.
-+                                                  * Sets EXTENT_NOT_ALIGNED.*/
-+#define FIEMAP_EXTENT_UNWRITTEN               0x00000800 /* Space allocated, but
-+                                                  * no data (i.e. zero). */
-+#define FIEMAP_EXTENT_MERGED          0x00001000 /* File does not natively
-+                                                  * support extents. Result
-+                                                  * merged for efficiency. */
-+
-+/* Lustre specific flags - use a high bit, don't conflict with upstream flag */
-+#define FIEMAP_EXTENT_NO_DIRECT               0x40000000 /* Data mapping undefined */
-+#define FIEMAP_EXTENT_NET             0x80000000 /* Data stored remotely.
-+                                                  * Sets NO_DIRECT flag */
-+
-+#endif /* _LINUX_EXT3_FIEMAP_H */
-+
index b9468f8..ade2692 100644 (file)
@@ -1,81 +1,24 @@
-A large part of this code is from the generic VFS code in fs/ioctl.c in the
-upstream kernel.
-
-Index: linux-2.6.18-128.1.6/fs/ext4/ioctl.c
-===================================================================
---- linux-2.6.18-128.1.6.orig/fs/ext4/ioctl.c
-+++ linux-2.6.18-128.1.6/fs/ext4/ioctl.c
-@@ -17,6 +17,162 @@
+diff -rupN linux-2.6.18-164.6.1_1/fs/ext4/ext4.h linux-2.6.18-164.6.1_2/fs/ext4/ext4.h
+--- linux-2.6.18-164.6.1_1/fs/ext4/ext4.h      2009-12-22 13:07:27.000000000 +0530
++++ linux-2.6.18-164.6.1_2/fs/ext4/ext4.h      2009-12-22 13:10:18.000000000 +0530
+@@ -305,6 +305,7 @@ struct ext4_new_group_data {
+ #define EXT4_IOC_GROUP_EXTEND         _IOW('f', 7, unsigned long)
+ #define EXT4_IOC_GROUP_ADD            _IOW('f', 8, struct ext4_new_group_input)
+ #define EXT4_IOC_MIGRATE              _IO('f', 9)
++#define EXT4_IOC_FIEMAP               _IOWR('f', 11, struct fiemap)
+  /* note ioctl 11 reserved for filesystem-independent FIEMAP ioctl */
+ /*
+diff -rupN linux-2.6.18-164.6.1_1/fs/ext4/ioctl.c linux-2.6.18-164.6.1_2/fs/ext4/ioctl.c
+--- linux-2.6.18-164.6.1_1/fs/ext4/ioctl.c     2009-12-22 13:06:51.000000000 +0530
++++ linux-2.6.18-164.6.1_2/fs/ext4/ioctl.c     2009-12-22 13:09:45.000000000 +0530
+@@ -17,6 +17,71 @@
  #include "ext4_jbd2.h"
  #include "ext4.h"
  
  #include "ext4_jbd2.h"
  #include "ext4.h"
  
-+#include "fiemap.h"
-+
 +/* So that the fiemap access checks can't overflow on 32 bit machines. */
 +#define FIEMAP_MAX_EXTENTS     (UINT_MAX / sizeof(struct fiemap_extent))
 +
 +/* So that the fiemap access checks can't overflow on 32 bit machines. */
 +#define FIEMAP_MAX_EXTENTS     (UINT_MAX / sizeof(struct fiemap_extent))
 +
-+/**
-+ * fiemap_fill_next_extent - Fiemap helper function
-+ * @fieinfo:   Fiemap context passed into ->fiemap
-+ * @logical:   Extent logical start offset, in bytes
-+ * @phys:      Extent physical start offset, in bytes
-+ * @len:       Extent length, in bytes
-+ * @flags:     FIEMAP_EXTENT flags that describe this extent
-+ * @lun:       LUN on which this extent resides
-+ *
-+ * Called from file system ->fiemap callback. Will populate extent
-+ * info as passed in via arguments and copy to user memory. On
-+ * success, extent count on fieinfo is incremented.
-+ *
-+ * Returns 0 on success, -errno on error, 1 if this was the last
-+ * extent that will fit in user array.
-+ */
-+#define SET_UNKNOWN_FLAGS      (FIEMAP_EXTENT_DELALLOC)
-+#define SET_NO_DIRECT_FLAGS    (FIEMAP_EXTENT_DATA_ENCRYPTED  \
-+                              |FIEMAP_EXTENT_NET)
-+#define SET_NO_UNMOUNTED_IO_FLAGS       (FIEMAP_EXTENT_DATA_ENCRYPTED)
-+#define SET_NOT_ALIGNED_FLAGS  (FIEMAP_EXTENT_DATA_TAIL|FIEMAP_EXTENT_DATA_INLINE)
-+int fiemap_fill_next_extent(struct fiemap_extent_info *fieinfo, u64 logical,
-+                          u64 phys, u64 len, u32 flags, dev_t dev)
-+{
-+      struct fiemap_extent extent = { 0 };
-+      struct fiemap_extent *dest = fieinfo->fi_extents_start;
-+
-+      /* only count the extents */
-+      if (fieinfo->fi_extents_max == 0) {
-+              fieinfo->fi_extents_mapped++;
-+              return (flags & FIEMAP_EXTENT_LAST) ? 1 : 0;
-+      }
-+
-+      if (fieinfo->fi_extents_mapped >= fieinfo->fi_extents_max)
-+              return 1;
-+
-+      if (flags & SET_UNKNOWN_FLAGS)
-+              flags |= FIEMAP_EXTENT_UNKNOWN;
-+      if (flags & SET_NO_DIRECT_FLAGS)
-+              flags |= FIEMAP_EXTENT_NO_DIRECT;
-+      if (flags & SET_NOT_ALIGNED_FLAGS)
-+              flags |= FIEMAP_EXTENT_NOT_ALIGNED;
-+      if (flags & SET_NO_UNMOUNTED_IO_FLAGS)
-+              flags |= FIEMAP_EXTENT_ENCODED;
-+
-+      extent.fe_logical = logical;
-+      extent.fe_physical = phys;
-+      extent.fe_length = len;
-+      extent.fe_flags = flags;
-+      extent.fe_device = new_encode_dev(dev);
-+
-+      dest += fieinfo->fi_extents_mapped;
-+      if (copy_to_user(dest, &extent, sizeof(extent)))
-+              return -EFAULT;
-+
-+      fieinfo->fi_extents_mapped++;
-+      if (fieinfo->fi_extents_mapped == fieinfo->fi_extents_max)
-+              return 1;
-+
-+      return (flags & FIEMAP_EXTENT_LAST) ? 1 : 0;
-+}
-+
 +static int fiemap_check_ranges(struct super_block *sb,
 +                             u64 start, u64 len, u64 *new_len)
 +{
 +static int fiemap_check_ranges(struct super_block *sb,
 +                             u64 start, u64 len, u64 *new_len)
 +{
@@ -97,33 +40,6 @@ Index: linux-2.6.18-128.1.6/fs/ext4/ioctl.c
 +      return 0;
 +}
 +
 +      return 0;
 +}
 +
-+/*
-+ * fiemap_check_flags - check validity of requested flags for fiemap
-+ * @fieinfo:   Fiemap context passed into ->fiemap
-+ * @fs_flags:  Set of fiemap flags that the file system understands
-+ *
-+ * Called from file system ->fiemap callback. This will compute the
-+ * intersection of valid fiemap flags and those that the fs supports. That
-+ * value is then compared against the user supplied flags. In case of bad user
-+ * flags, the invalid values will be written into the fieinfo structure, and
-+ * -EBADR is returned, which tells ioctl_fiemap() to return those values to
-+ * userspace. For this reason, a return code of -EBADR should be preserved.
-+ *
-+ * Returns 0 on success, -EBADR on bad flags.
-+ */
-+int fiemap_check_flags(struct fiemap_extent_info *fieinfo, u32 fs_flags)
-+{
-+      u32 incompat_flags;
-+
-+      incompat_flags = fieinfo->fi_flags & ~(FIEMAP_FLAGS_COMPAT & fs_flags);
-+      if (incompat_flags) {
-+              fieinfo->fi_flags = incompat_flags;
-+              return -EBADR;
-+      }
-+
-+      return 0;
-+}
-+
 +int ioctl_fiemap(struct inode *inode, struct file *filp, unsigned long arg)
 +{
 +      struct fiemap fiemap;
 +int ioctl_fiemap(struct inode *inode, struct file *filp, unsigned long arg)
 +{
 +      struct fiemap fiemap;
@@ -168,135 +84,13 @@ Index: linux-2.6.18-128.1.6/fs/ext4/ioctl.c
  long ext4_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
  {
        struct inode *inode = filp->f_dentry->d_inode;
  long ext4_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
  {
        struct inode *inode = filp->f_dentry->d_inode;
-@@ -257,6 +413,10 @@ flags_err:
-       case EXT4_IOC_MIGRATE:
-               return ext4_ext_migrate(inode, filp, cmd, arg);
+@@ -249,6 +314,9 @@ flags_out:
+               mutex_unlock(&(inode->i_mutex));
+               return err;
+       }
 +      case EXT4_IOC_FIEMAP: {
 +              return ioctl_fiemap(inode, filp, arg);
 +      }
 +      case EXT4_IOC_FIEMAP: {
 +              return ioctl_fiemap(inode, filp, arg);
 +      }
-+
        default:
                return -ENOTTY;
        default:
                return -ENOTTY;
-       }
-Index: linux-2.6.18-128.1.6/fs/ext4/ext4.h
-===================================================================
---- linux-2.6.18-128.1.6.orig/fs/ext4/ext4.h
-+++ linux-2.6.18-128.1.6/fs/ext4/ext4.h
-@@ -300,6 +300,7 @@ struct ext4_new_group_data {
- #define EXT4_IOC_GROUP_EXTEND         _IOW('f', 7, unsigned long)
- #define EXT4_IOC_GROUP_ADD            _IOW('f', 8, struct ext4_new_group_input)
- #define EXT4_IOC_MIGRATE              _IO('f', 9)
-+#define EXT4_IOC_FIEMAP                       _IOWR('f', 11, struct fiemap)
-  /* note ioctl 11 reserved for filesystem-independent FIEMAP ioctl */
- /*
-@@ -317,6 +318,8 @@ struct ext4_new_group_data {
- #define EXT4_IOC32_GETVERSION_OLD     FS_IOC32_GETVERSION
- #define EXT4_IOC32_SETVERSION_OLD     FS_IOC32_SETVERSION
-+/* FIEMAP flags supported by ext4 */
-+#define EXT4_FIEMAP_FLAGS_COMPAT (FIEMAP_FLAG_SYNC)
- /*
-  *  Mount options
-@@ -1117,6 +1120,9 @@ extern int ext4_page_mkwrite(struct vm_a
- /* ioctl.c */
- extern long ext4_ioctl(struct file *, unsigned int, unsigned long);
- extern long ext4_compat_ioctl(struct file *, unsigned int, unsigned long);
-+struct fiemap_extent_info;
-+extern int ext4_fiemap(struct inode *, struct fiemap_extent_info *, __u64,
-+                     __u64);
- /* migrate.c */
- extern int ext4_ext_migrate(struct inode *, struct file *, unsigned int,
-Index: linux-2.6.18-128.1.6/fs/ext4/fiemap.h
-===================================================================
---- /dev/null
-+++ linux-2.6.18-128.1.6/fs/ext4/fiemap.h
-@@ -0,0 +1,85 @@
-+/*
-+ * FIEMAP ioctl infrastructure.
-+ *
-+ * Copyright 2008 Sun Microsystems, Inc
-+ *
-+ * Author: Kalpak Shah <kalpak.shah@sun.com>
-+ *     Andreas Dilger <adilger@sun.com>
-+ */
-+
-+#ifndef _LINUX_EXT4_FIEMAP_H
-+#define _LINUX_EXT4_FIEMAP_H
-+
-+struct fiemap_extent {
-+      __u64 fe_logical;  /* logical offset in bytes for the start of
-+                          * the extent from the beginning of the file */
-+      __u64 fe_physical; /* physical offset in bytes for the start
-+                          * of the extent from the beginning of the disk */
-+      __u64 fe_length;   /* length in bytes for this extent */
-+      __u64 fe_reserved64[2];
-+      __u32 fe_flags;    /* FIEMAP_EXTENT_* flags for this extent */
-+      __u32 fe_device;   /* device number for this extent */
-+      __u32 fe_reserved[2];
-+};
-+
-+struct fiemap {
-+      __u64 fm_start;  /* logical offset (inclusive) at
-+                               * which to start mapping (in) */
-+      __u64 fm_length;        /* logical length of mapping which
-+                               * userspace wants (in) */
-+      __u32 fm_flags;  /* FIEMAP_FLAG_* flags for request (in/out) */
-+      __u32 fm_mapped_extents;/* number of extents that were mapped (out) */
-+      __u32 fm_extent_count;  /* size of fm_extents array (in) */
-+      __u32 fm_reserved;
-+      struct fiemap_extent fm_extents[0]; /* array of mapped extents (out) */
-+};
-+
-+/*
-+ * FIEMAP helper definition.
-+ */
-+struct fiemap_extent_info {
-+      unsigned int    fi_flags;               /* Flags as passed from user */
-+      unsigned int    fi_extents_mapped;      /* Number of mapped extents */
-+      unsigned int    fi_extents_max;         /* Size of fiemap_extent array*/
-+      struct fiemap_extent *fi_extents_start; /* Start of fiemap_extent array */
-+};
-+
-+int fiemap_check_flags(struct fiemap_extent_info *fieinfo, u32 fs_flags);
-+int fiemap_fill_next_extent(struct fiemap_extent_info *info, u64 logical,
-+                          u64 phys, u64 len, u32 flags, u32 lun);
-+
-+#define       FIEMAP_MAX_OFFSET       (~0ULL)
-+
-+#define       FIEMAP_FLAG_SYNC        0x00000001 /* sync file data before map */
-+#define       FIEMAP_FLAG_XATTR       0x00000002 /* map extended attribute tree */
-+
-+/* ldiskfs only supports FLAG_SYNC flag currently */
-+#define FIEMAP_FLAGS_COMPAT (FIEMAP_FLAG_SYNC | FIEMAP_FLAG_XATTR)
-+
-+#define FIEMAP_EXTENT_LAST            0x00000001 /* Last extent in file. */
-+#define FIEMAP_EXTENT_UNKNOWN         0x00000002 /* Data location unknown. */
-+#define FIEMAP_EXTENT_DELALLOC                0x00000004 /* Location still pending.
-+                                                  * Sets EXTENT_UNKNOWN. */
-+#define FIEMAP_EXTENT_ENCODED         0x00000008 /* Data can not be read
-+                                                  * while fs is unmounted */
-+#define FIEMAP_EXTENT_DATA_ENCRYPTED  0x00000080 /* Data is encrypted by fs.
-+                                                  * Sets EXTENT_NO_DIRECT. */
-+#define FIEMAP_EXTENT_NOT_ALIGNED     0x00000100 /* Extent offsets may not be
-+                                                  * block aligned. */
-+#define FIEMAP_EXTENT_DATA_INLINE     0x00000200 /* Data mixed with metadata.
-+                                                  * Sets EXTENT_NOT_ALIGNED.*/
-+#define FIEMAP_EXTENT_DATA_TAIL               0x00000400 /* Multiple files in block.
-+                                                  * Sets EXTENT_NOT_ALIGNED.*/
-+#define FIEMAP_EXTENT_UNWRITTEN               0x00000800 /* Space allocated, but
-+                                                  * no data (i.e. zero). */
-+#define FIEMAP_EXTENT_MERGED          0x00001000 /* File does not natively
-+                                                  * support extents. Result
-+                                                  * merged for efficiency. */
-+
-+/* Lustre specific flags - use a high bit, don't conflict with upstream flag */
-+#define FIEMAP_EXTENT_NO_DIRECT               0x40000000 /* Data mapping undefined */
-+#define FIEMAP_EXTENT_NET             0x80000000 /* Data stored remotely.
-+                                                  * Sets NO_DIRECT flag */
-+
-+#endif /* _LINUX_EXT4_FIEMAP_H */
-+
index 1559a88..92f47a5 100644 (file)
@@ -28,3 +28,4 @@ ext4-hash-indexed-dir-dotdot-update-rhel5.patch
 ext4-ext_generation-sles11.patch
 ext4-kill-dx_root.patch
 ext4-extents-mount-option-rhel5.patch
 ext4-ext_generation-sles11.patch
 ext4-kill-dx_root.patch
 ext4-extents-mount-option-rhel5.patch
+ext4-fiemap-2.6-rhel5.patch
index ae426f0..3a8b689 100644 (file)
@@ -34,3 +34,4 @@ ext3-dynlocks-2.6-rhel5.patch
 ext3-hash-indexed-dir-dotdot-update.patch
 ext3-corrupted-orphans-2.6.patch
 ext3-kill-dx_root.patch
 ext3-hash-indexed-dir-dotdot-update.patch
 ext3-corrupted-orphans-2.6.patch
 ext3-kill-dx_root.patch
+ext3-fiemap-2.6-rhel5.patch
index d25de6c..8c8e1fa 100644 (file)
@@ -1557,6 +1557,19 @@ int ll_do_fiemap(struct inode *inode, struct ll_user_fiemap *fiemap,
         int rc;
         ENTRY;
 
         int rc;
         ENTRY;
 
+        /* Checks for fiemap flags */
+        if (fiemap->fm_flags & ~LUSTRE_FIEMAP_FLAGS_COMPAT) {
+                fiemap->fm_flags &= ~LUSTRE_FIEMAP_FLAGS_COMPAT;
+                return -EBADR;
+        }
+
+        /* Check for FIEMAP_FLAG_SYNC */
+        if (fiemap->fm_flags & FIEMAP_FLAG_SYNC) {
+                rc = filemap_fdatawrite(inode->i_mapping);
+                if (rc)
+                        return rc;
+        }
+
         /* If the stripe_count > 1 and the application does not understand
          * DEVICE_ORDER flag, then it cannot interpret the extents correctly.
          */
         /* If the stripe_count > 1 and the application does not understand
          * DEVICE_ORDER flag, then it cannot interpret the extents correctly.
          */
@@ -1622,6 +1635,58 @@ gf_free:
         RETURN(rc);
 }
 
         RETURN(rc);
 }
 
+static int ll_ioctl_fiemap(struct inode *inode, unsigned long arg)
+{
+        struct ll_user_fiemap *fiemap_s;
+        size_t num_bytes, ret_bytes;
+        unsigned int extent_count;
+        int rc = 0;
+
+        /* Get the extent count so we can calculate the size of
+         * required fiemap buffer */
+        if (get_user(extent_count,
+            &((struct ll_user_fiemap __user *)arg)->fm_extent_count))
+                RETURN(-EFAULT);
+        num_bytes = sizeof(*fiemap_s) + (extent_count *
+                                         sizeof(struct ll_fiemap_extent));
+
+        OBD_VMALLOC(fiemap_s, num_bytes);
+        if (fiemap_s == NULL)
+                RETURN(-ENOMEM);
+
+        /* get the fiemap value */
+        if (copy_from_user(fiemap_s,(struct ll_user_fiemap __user *)arg,
+                           sizeof(*fiemap_s)))
+                GOTO(error, rc = -EFAULT);
+
+        /* If fm_extent_count is non-zero, read the first extent since
+         * it is used to calculate end_offset and device from previous
+         * fiemap call. */
+        if (extent_count) {
+                if (copy_from_user(&fiemap_s->fm_extents[0],
+                    (char __user *)arg + sizeof(*fiemap_s),
+                    sizeof(struct ll_fiemap_extent)))
+                        GOTO(error, rc = -EFAULT);
+        }
+
+        rc = ll_do_fiemap(inode, fiemap_s, num_bytes);
+        if (rc)
+                GOTO(error, rc);
+
+        ret_bytes = sizeof(struct ll_user_fiemap);
+
+        if (extent_count != 0)
+                ret_bytes += (fiemap_s->fm_mapped_extents *
+                                 sizeof(struct ll_fiemap_extent));
+
+        if (copy_to_user((void *)arg, fiemap_s, ret_bytes))
+                rc = -EFAULT;
+
+error:
+        OBD_VFREE(fiemap_s, num_bytes);
+        RETURN(rc);
+}
+
 int ll_file_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
                   unsigned long arg)
 {
 int ll_file_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
                   unsigned long arg)
 {
@@ -1671,73 +1736,8 @@ int ll_file_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
                 RETURN(ll_lov_getstripe(inode, arg));
         case LL_IOC_RECREATE_OBJ:
                 RETURN(ll_lov_recreate_obj(inode, file, arg));
                 RETURN(ll_lov_getstripe(inode, arg));
         case LL_IOC_RECREATE_OBJ:
                 RETURN(ll_lov_recreate_obj(inode, file, arg));
-        case FSFILT_IOC_FIEMAP: {
-                struct ll_user_fiemap *fiemap_s;
-                size_t num_bytes, ret_bytes;
-                unsigned int extent_count;
-                int rc = 0;
-
-                /* Get the extent count so we can calculate the size of
-                 * required fiemap buffer */
-                if (get_user(extent_count,
-                    &((struct ll_user_fiemap __user *)arg)->fm_extent_count))
-                        RETURN(-EFAULT);
-                num_bytes = sizeof(*fiemap_s) + (extent_count *
-                                                 sizeof(struct ll_fiemap_extent));
-                OBD_VMALLOC(fiemap_s, num_bytes);
-                if (fiemap_s == NULL)
-                        RETURN(-ENOMEM);
-
-                if (cfs_copy_from_user(fiemap_s,
-                                       (struct ll_user_fiemap __user *)arg,
-                                       sizeof(*fiemap_s)))
-                        GOTO(error, rc = -EFAULT);
-
-                if (fiemap_s->fm_flags & ~LUSTRE_FIEMAP_FLAGS_COMPAT) {
-                        fiemap_s->fm_flags = fiemap_s->fm_flags &
-                                                    ~LUSTRE_FIEMAP_FLAGS_COMPAT;
-                        if (cfs_copy_to_user((char *)arg, fiemap_s,
-                                             sizeof(*fiemap_s)))
-                                GOTO(error, rc = -EFAULT);
-
-                        GOTO(error, rc = -EBADR);
-                }
-
-                /* If fm_extent_count is non-zero, read the first extent since
-                 * it is used to calculate end_offset and device from previous
-                 * fiemap call. */
-                if (extent_count) {
-                        if (cfs_copy_from_user(&fiemap_s->fm_extents[0],
-                            (char __user *)arg + sizeof(*fiemap_s),
-                            sizeof(struct ll_fiemap_extent)))
-                                GOTO(error, rc = -EFAULT);
-                }
-
-                if (fiemap_s->fm_flags & FIEMAP_FLAG_SYNC) {
-                        int rc;
-
-                        rc = filemap_fdatawrite(inode->i_mapping);
-                        if (rc)
-                                GOTO(error, rc);
-                }
-
-                rc = ll_do_fiemap(inode, fiemap_s, num_bytes);
-                if (rc)
-                        GOTO(error, rc);
-
-                ret_bytes = sizeof(struct ll_user_fiemap);
-
-                if (extent_count != 0)
-                        ret_bytes += (fiemap_s->fm_mapped_extents *
-                                         sizeof(struct ll_fiemap_extent));
-
-                if (cfs_copy_to_user((void *)arg, fiemap_s, ret_bytes))
-                        rc = -EFAULT;
-
-error:
-                OBD_VFREE(fiemap_s, num_bytes);
-                RETURN(rc);
-        }
+        case FSFILT_IOC_FIEMAP:
+                RETURN(ll_ioctl_fiemap(inode, arg));
         case FSFILT_IOC_GETFLAGS:
         case FSFILT_IOC_SETFLAGS:
                 RETURN(ll_iocontrol(inode, file, cmd, arg));
         case FSFILT_IOC_GETFLAGS:
         case FSFILT_IOC_SETFLAGS:
                 RETURN(ll_iocontrol(inode, file, cmd, arg));
@@ -2224,16 +2224,32 @@ int ll_fiemap(struct inode *inode, struct fiemap_extent_info *fieinfo,
                 __u64 start, __u64 len)
 {
         int rc;
                 __u64 start, __u64 len)
 {
         int rc;
-        struct ll_user_fiemap *fiemap = (struct ll_user_fiemap*)(
-                fieinfo->fi_extents_start - sizeof(ll_user_fiemap));
+        size_t num_bytes;
+        struct ll_user_fiemap *fiemap;
+        unsigned int extent_count = fieinfo->fi_extents_max;
+
+        num_bytes = sizeof(*fiemap) + (extent_count *
+                                       sizeof(struct ll_fiemap_extent));
+        OBD_VMALLOC(fiemap, num_bytes);
+
+        if (fiemap == NULL)
+                RETURN(-ENOMEM);
+
+        fiemap->fm_flags = fieinfo->fi_flags;
+        fiemap->fm_extent_count = fieinfo->fi_extents_max;
+        fiemap->fm_start = start;
+        fiemap->fm_length = len;
+        memcpy(&fiemap->fm_extents[0], fieinfo->fi_extents_start,
+               sizeof(struct ll_fiemap_extent));
 
 
-        rc = ll_do_fiemap(inode, fiemap, sizeof(*fiemap) +
-                          fiemap->fm_extent_count *
-                          sizeof(struct ll_fiemap_extent));
+        rc = ll_do_fiemap(inode, fiemap, num_bytes);
 
         fieinfo->fi_flags = fiemap->fm_flags;
         fieinfo->fi_extents_mapped = fiemap->fm_mapped_extents;
 
         fieinfo->fi_flags = fiemap->fm_flags;
         fieinfo->fi_extents_mapped = fiemap->fm_mapped_extents;
+        memcpy(fieinfo->fi_extents_start, &fiemap->fm_extents[0],
+               fiemap->fm_mapped_extents * sizeof(struct ll_fiemap_extent));
 
 
+        OBD_VFREE(fiemap, num_bytes);
         return rc;
 }
 #endif
         return rc;
 }
 #endif
index 7541e1e..d93a55b 100644 (file)
@@ -10,7 +10,7 @@ set -e
 
 ONLY=${ONLY:-"$*"}
 # bug number for skipped test: 13297 2108 9789 3637 9789 3561 12622 12653 12653 5188 16260 19742 
 
 ONLY=${ONLY:-"$*"}
 # bug number for skipped test: 13297 2108 9789 3637 9789 3561 12622 12653 12653 5188 16260 19742 
-ALWAYS_EXCEPT="                27u   42a  42b  42c  42d  45   51d   65a   65e   68b  119d  130 $SANITY_EXCEPT"
+ALWAYS_EXCEPT="                27u   42a  42b  42c  42d  45   51d   65a   65e   68b  119d $SANITY_EXCEPT"
 # bug number for skipped test: 2108 9789 3637 9789 3561 5188/5749 1443
 #ALWAYS_EXCEPT=${ALWAYS_EXCEPT:-"27m 42a 42b 42c 42d 45 68 76"}
 # UPDATE THE COMMENT ABOVE WITH BUG NUMBERS WHEN CHANGING ALWAYS_EXCEPT!
 # bug number for skipped test: 2108 9789 3637 9789 3561 5188/5749 1443
 #ALWAYS_EXCEPT=${ALWAYS_EXCEPT:-"27m 42a 42b 42c 42d 45 68 76"}
 # UPDATE THE COMMENT ABOVE WITH BUG NUMBERS WHEN CHANGING ALWAYS_EXCEPT!