Whamcloud - gitweb
LU-1199 build: Cleanup ldiskfs kernel config defines
[fs/lustre-release.git] / lustre / lvfs / fsfilt_ext3.c
index 7e88750..b7eab7a 100644 (file)
@@ -1,26 +1,41 @@
-/* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*-
- * vim:expandtab:shiftwidth=8:tabstop=8:
+/*
+ * GPL HEADER START
+ *
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
- *  lustre/lib/fsfilt_ext3.c
- *  Lustre filesystem abstraction routines
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 only,
+ * as published by the Free Software Foundation.
  *
- *  Copyright (C) 2002, 2003 Cluster File Systems, Inc.
- *   Author: Andreas Dilger <adilger@clusterfs.com>
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License version 2 for more details (a copy is included
+ * in the LICENSE file that accompanied this code).
  *
- *   This file is part of Lustre, http://www.lustre.org.
+ * You should have received a copy of the GNU General Public License
+ * version 2 along with this program; If not, see
+ * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
  *
- *   Lustre is free software; you can redistribute it and/or
- *   modify it under the terms of version 2 of the GNU General Public
- *   License as published by the Free Software Foundation.
+ * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
+ * CA 95054 USA or visit www.sun.com if you need additional information or
+ * have any questions.
  *
- *   Lustre is distributed in the hope that it will be useful,
- *   but WITHOUT ANY WARRANTY; without even the implied warranty of
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *   GNU General Public License for more details.
+ * GPL HEADER END
+ */
+/*
+ * Copyright (c) 2002, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Use is subject to license terms.
  *
- *   You should have received a copy of the GNU General Public License
- *   along with Lustre; if not, write to the Free Software
- *   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ * Copyright (c) 2011, 2012, Intel Corporation.
+ */
+/*
+ * This file is part of Lustre, http://www.lustre.org/
+ * Lustre is a trademark of Sun Microsystems, Inc.
+ *
+ * lustre/lvfs/fsfilt_ext3.c
+ *
+ * Author: Andreas Dilger <adilger@clusterfs.com>
  */
 
 #define DEBUG_SUBSYSTEM S_FILTER
 #include <linux/init.h>
 #include <linux/module.h>
 #include <linux/fs.h>
-#include <linux/jbd.h>
 #include <linux/slab.h>
 #include <linux/pagemap.h>
-#include <linux/quotaops.h>
-#include <linux/ext3_fs.h>
-#include <linux/ext3_jbd.h>
-#include <linux/ext3_extents.h>
+#include <ldiskfs/ldiskfs_config.h>
+#include <ext4/ext4.h>
+#include <ext4/ext4_jbd2.h>
 #include <linux/version.h>
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0))
-#include <linux/ext3_xattr.h>
-#else
-#include <ext3/xattr.h>
-#endif
+#include <linux/bitops.h>
+#include <linux/quota.h>
 
-#include <libcfs/kp30.h>
-#include <linux/lustre_fsfilt.h>
-#include <linux/obd.h>
-#include <linux/obd_class.h>
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0))
-#include <linux/module.h>
-#include <linux/iobuf.h>
-#endif
+#include <libcfs/libcfs.h>
+#include <lustre_fsfilt.h>
+#include <obd.h>
+#include <linux/lustre_compat25.h>
+#include <linux/lprocfs_status.h>
 
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,6,7))
-# define lock_24kernel() lock_kernel()
-# define unlock_24kernel() unlock_kernel()
-#else
-# define lock_24kernel() do {} while (0)
-# define unlock_24kernel() do {} while (0)
-#endif
+#include <ext4/ext4_extents.h>
+
+/* for kernels 2.6.18 and later */
+#define FSFILT_SINGLEDATA_TRANS_BLOCKS(sb) EXT3_SINGLEDATA_TRANS_BLOCKS(sb)
+
+#define fsfilt_ext3_ext_insert_extent(handle, inode, path, newext, flag) \
+               ext3_ext_insert_extent(handle, inode, path, newext, flag)
+
+#define ext3_mb_discard_inode_preallocations(inode) \
+                 ext3_discard_preallocations(inode)
 
-static kmem_cache_t *fcb_cache;
-static atomic_t fcb_cache_count = ATOMIC_INIT(0);
+#define fsfilt_log_start_commit(journal, tid) jbd2_log_start_commit(journal, tid)
+#define fsfilt_log_wait_commit(journal, tid) jbd2_log_wait_commit(journal, tid)
+
+static cfs_mem_cache_t *fcb_cache;
 
 struct fsfilt_cb_data {
-        struct journal_callback cb_jcb; /* jbd private data - MUST BE FIRST */
-        fsfilt_cb_t cb_func;            /* MDS/OBD completion function */
-        struct obd_device *cb_obd;      /* MDS/OBD completion device */
-        __u64 cb_last_num;              /* MDS/OST last committed operation */
-        void *cb_data;                  /* MDS/OST completion function data */
+       struct ext4_journal_cb_entry cb_jcb; /* private data - MUST BE FIRST */
+       fsfilt_cb_t cb_func;            /* MDS/OBD completion function */
+       struct obd_device *cb_obd;      /* MDS/OBD completion device */
+       __u64 cb_last_rcvd;             /* MDS/OST last committed operation */
+       void *cb_data;                  /* MDS/OST completion function data */
 };
 
-#ifndef EXT3_XATTR_INDEX_TRUSTED        /* temporary until we hit l28 kernel */
-#define EXT3_XATTR_INDEX_TRUSTED        4
+static char *fsfilt_ext3_get_label(struct super_block *sb)
+{
+        return EXT3_SB(sb)->s_es->s_volume_name;
+}
+
+/* kernel has ext4_blocks_for_truncate since linux-3.1.1 */
+#ifdef HAVE_BLOCKS_FOR_TRUNCATE
+# include <ext4/truncate.h>
+#else
+static inline unsigned long ext4_blocks_for_truncate(struct inode *inode)
+{
+       ext4_lblk_t needed;
+
+       needed = inode->i_blocks >> (inode->i_sb->s_blocksize_bits - 9);
+       if (needed < 2)
+               needed = 2;
+       if (needed > EXT4_MAX_TRANS_DATA)
+               needed = EXT4_MAX_TRANS_DATA;
+       return EXT4_DATA_TRANS_BLOCKS(inode->i_sb) + needed;
+}
 #endif
 
 /*
@@ -83,8 +113,8 @@ struct fsfilt_cb_data {
 static void *fsfilt_ext3_start(struct inode *inode, int op, void *desc_private,
                                int logs)
 {
-        /* For updates to the last recieved file */
-        int nblocks = EXT3_SINGLEDATA_TRANS_BLOCKS;
+        /* For updates to the last received file */
+        int nblocks = FSFILT_SINGLEDATA_TRANS_BLOCKS(inode->i_sb);
         journal_t *journal;
         void *handle;
 
@@ -94,54 +124,23 @@ static void *fsfilt_ext3_start(struct inode *inode, int op, void *desc_private,
                 goto journal_start;
         }
 
-       if (logs)
-               nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-                            EXT3_SINGLEDATA_TRANS_BLOCKS) * logs;
-                
         switch(op) {
-        case FSFILT_OP_RMDIR:
         case FSFILT_OP_UNLINK:
-                /* delete one file + create/update logs for each stripe */
-                nblocks += EXT3_DELETE_TRANS_BLOCKS;
-                /*nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-                            EXT3_SINGLEDATA_TRANS_BLOCKS) * logs;*/
-                break;
-        case FSFILT_OP_RENAME:
-                /* modify additional directory */
-                nblocks += EXT3_SINGLEDATA_TRANS_BLOCKS;
-                /* no break */
-        case FSFILT_OP_SYMLINK:
-                /* additional block + block bitmap + GDT for long symlink */
-                nblocks += 3;
-                /* no break */
-        case FSFILT_OP_CREATE:
-                /* create/update logs for each stripe */
-                /*nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-                            EXT3_SINGLEDATA_TRANS_BLOCKS) * logs;*/
-                /* no break */
-        case FSFILT_OP_MKDIR:
-        case FSFILT_OP_MKNOD:
-                /* modify one inode + block bitmap + GDT */
-                nblocks += 3;
-                /* no break */
-        case FSFILT_OP_LINK:
-                /* modify parent directory */
-                nblocks += EXT3_INDEX_EXTRA_TRANS_BLOCKS +
-                        EXT3_DATA_TRANS_BLOCKS;
-                break;
-        case FSFILT_OP_SETATTR:
-                /* Setattr on inode */
-                nblocks += 1;
-                break;
+               /* delete one file + create/update logs for each stripe */
+               nblocks += EXT3_DELETE_TRANS_BLOCKS(inode->i_sb);
+               nblocks += (EXT3_INDEX_EXTRA_TRANS_BLOCKS +
+                           FSFILT_SINGLEDATA_TRANS_BLOCKS(inode->i_sb)) * logs;
+               break;
         case FSFILT_OP_CANCEL_UNLINK:
-                /* blocks for log header bitmap update OR
-                 * blocks for catalog header bitmap update + unlink of logs */
-                nblocks = (LLOG_CHUNK_SIZE >> inode->i_blkbits) +
-                        EXT3_DELETE_TRANS_BLOCKS * logs;
-                break;
-        case FSFILT_OP_NOOP:
-                nblocks += EXT3_INDEX_EXTRA_TRANS_BLOCKS+EXT3_DATA_TRANS_BLOCKS;
-                break;
+               LASSERT(logs == 1);
+
+               /* blocks for log header bitmap update OR
+                * blocks for catalog header bitmap update + unlink of logs +
+                * blocks for delete the inode (include blocks truncating). */
+               nblocks = (LLOG_CHUNK_SIZE >> inode->i_blkbits) +
+                         EXT3_DELETE_TRANS_BLOCKS(inode->i_sb) +
+                         ext4_blocks_for_truncate(inode) + 3;
+               break;
         default: CERROR("unknown transaction start op %d\n", op);
                 LBUG();
         }
@@ -149,16 +148,14 @@ static void *fsfilt_ext3_start(struct inode *inode, int op, void *desc_private,
         LASSERT(current->journal_info == desc_private);
         journal = EXT3_SB(inode->i_sb)->s_journal;
         if (nblocks > journal->j_max_transaction_buffers) {
-                CERROR("too many credits %d for op %ux%u using %d instead\n",
+                CWARN("too many credits %d for op %ux%u using %d instead\n",
                        nblocks, op, logs, journal->j_max_transaction_buffers);
                 nblocks = journal->j_max_transaction_buffers;
         }
 
  journal_start:
         LASSERTF(nblocks > 0, "can't start %d credit transaction\n", nblocks);
-        lock_24kernel();
-        handle = journal_start(EXT3_JOURNAL(inode), nblocks);
-        unlock_24kernel();
+        handle = ext3_journal_start(inode, nblocks);
 
         if (!IS_ERR(handle))
                 LASSERT(current->journal_info == handle);
@@ -168,149 +165,7 @@ static void *fsfilt_ext3_start(struct inode *inode, int op, void *desc_private,
         return handle;
 }
 
-/*
- * Calculate the number of buffer credits needed to write multiple pages in
- * a single ext3 transaction.  No, this shouldn't be here, but as yet ext3
- * doesn't have a nice API for calculating this sort of thing in advance.
- *
- * See comment above ext3_writepage_trans_blocks for details.  We assume
- * no data journaling is being done, but it does allow for all of the pages
- * being non-contiguous.  If we are guaranteed contiguous pages we could
- * reduce the number of (d)indirect blocks a lot.
- *
- * With N blocks per page and P pages, for each inode we have at most:
- * N*P indirect
- * min(N*P, blocksize/4 + 1) dindirect blocks
- * niocount tindirect
- *
- * For the entire filesystem, we have at most:
- * min(sum(nindir + P), ngroups) bitmap blocks (from the above)
- * min(sum(nindir + P), gdblocks) group descriptor blocks (from the above)
- * objcount inode blocks
- * 1 superblock
- * 2 * EXT3_SINGLEDATA_TRANS_BLOCKS for the quota files
- *
- * 1 EXT3_DATA_TRANS_BLOCKS for the last_rcvd update.
- */
-static int fsfilt_ext3_credits_needed(int objcount, struct fsfilt_objinfo *fso,
-                                      int niocount, struct niobuf_local *nb)
-{
-        struct super_block *sb = fso->fso_dentry->d_inode->i_sb;
-        __u64 next_indir;
-        const int blockpp = 1 << (PAGE_CACHE_SHIFT - sb->s_blocksize_bits);
-        int nbitmaps = 0, ngdblocks;
-        int needed = objcount + 1; /* inodes + superblock */
-        int i, j;
-
-        for (i = 0, j = 0; i < objcount; i++, fso++) {
-                /* two or more dindirect blocks in case we cross boundary */
-                int ndind = (long)((nb[j + fso->fso_bufcnt - 1].offset -
-                                    nb[j].offset) >>
-                                   sb->s_blocksize_bits) /
-                        (EXT3_ADDR_PER_BLOCK(sb) * EXT3_ADDR_PER_BLOCK(sb));
-                nbitmaps += min(fso->fso_bufcnt, ndind > 0 ? ndind : 2);
-
-                /* leaf, indirect, tindirect blocks for first block */
-                nbitmaps += blockpp + 2;
-
-                j += fso->fso_bufcnt;
-        }
-
-        next_indir = nb[0].offset +
-                (EXT3_ADDR_PER_BLOCK(sb) << sb->s_blocksize_bits);
-        for (i = 1; i < niocount; i++) {
-                if (nb[i].offset >= next_indir) {
-                        nbitmaps++;     /* additional indirect */
-                        next_indir = nb[i].offset +
-                                (EXT3_ADDR_PER_BLOCK(sb)<<sb->s_blocksize_bits);
-                } else if (nb[i].offset != nb[i - 1].offset + sb->s_blocksize) {
-                        nbitmaps++;     /* additional indirect */
-                }
-                nbitmaps += blockpp;    /* each leaf in different group? */
-        }
-
-        ngdblocks = nbitmaps;
-        if (nbitmaps > EXT3_SB(sb)->s_groups_count)
-                nbitmaps = EXT3_SB(sb)->s_groups_count;
-        if (ngdblocks > EXT3_SB(sb)->s_gdb_count)
-                ngdblocks = EXT3_SB(sb)->s_gdb_count;
-
-        needed += nbitmaps + ngdblocks;
-
-        /* last_rcvd update */
-        needed += EXT3_DATA_TRANS_BLOCKS;
-
-#if defined(CONFIG_QUOTA) && !defined(__x86_64__) /* XXX */
-        /* We assume that there will be 1 bit set in s_dquot.flags for each
-         * quota file that is active.  This is at least true for now.
-         */
-        needed += hweight32(sb_any_quota_enabled(sb)) *
-                EXT3_SINGLEDATA_TRANS_BLOCKS;
-#endif
-
-        return needed;
-}
-
-/* We have to start a huge journal transaction here to hold all of the
- * metadata for the pages being written here.  This is necessitated by
- * the fact that we do lots of prepare_write operations before we do
- * any of the matching commit_write operations, so even if we split
- * up to use "smaller" transactions none of them could complete until
- * all of them were opened.  By having a single journal transaction,
- * we eliminate duplicate reservations for common blocks like the
- * superblock and group descriptors or bitmaps.
- *
- * We will start the transaction here, but each prepare_write will
- * add a refcount to the transaction, and each commit_write will
- * remove a refcount.  The transaction will be closed when all of
- * the pages have been written.
- */
-static void *fsfilt_ext3_brw_start(int objcount, struct fsfilt_objinfo *fso,
-                                   int niocount, struct niobuf_local *nb,
-                                   void *desc_private, int logs)
-{
-        journal_t *journal;
-        handle_t *handle;
-        int needed;
-        ENTRY;
-
-        LASSERT(current->journal_info == desc_private);
-        journal = EXT3_SB(fso->fso_dentry->d_inode->i_sb)->s_journal;
-        needed = fsfilt_ext3_credits_needed(objcount, fso, niocount, nb);
-
-        /* The number of blocks we could _possibly_ dirty can very large.
-         * We reduce our request if it is absurd (and we couldn't get that
-         * many credits for a single handle anyways).
-         *
-         * At some point we have to limit the size of I/Os sent at one time,
-         * increase the size of the journal, or we have to calculate the
-         * actual journal requirements more carefully by checking all of
-         * the blocks instead of being maximally pessimistic.  It remains to
-         * be seen if this is a real problem or not.
-         */
-        if (needed > journal->j_max_transaction_buffers) {
-                CERROR("want too many journal credits (%d) using %d instead\n",
-                       needed, journal->j_max_transaction_buffers);
-                needed = journal->j_max_transaction_buffers;
-        }
-
-        LASSERTF(needed > 0, "can't start %d credit transaction\n", needed);
-        lock_24kernel();
-        handle = journal_start(journal, needed);
-        unlock_24kernel();
-        if (IS_ERR(handle)) {
-                CERROR("can't get handle for %d credits: rc = %ld\n", needed,
-                       PTR_ERR(handle));
-        } else {
-                LASSERT(handle->h_buffer_credits >= needed);
-                LASSERT(current->journal_info == handle);
-        }
-
-        RETURN(handle);
-}
-
-static int fsfilt_ext3_commit(struct super_block *sb, struct inode *inode, 
-                              void *h, int force_sync)
+static int fsfilt_ext3_commit(struct inode *inode, void *h, int force_sync)
 {
         int rc;
         handle_t *handle = h;
@@ -319,453 +174,36 @@ static int fsfilt_ext3_commit(struct super_block *sb, struct inode *inode,
         if (force_sync)
                 handle->h_sync = 1; /* recovery likes this */
 
-        lock_24kernel();
-        rc = journal_stop(handle);
-        unlock_24kernel();
+        rc = ext3_journal_stop(handle);
 
         return rc;
 }
 
-static int fsfilt_ext3_commit_async(struct inode *inode, void *h,
-                                    void **wait_handle)
-{
-        unsigned long tid;
-        transaction_t *transaction;
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
-        unsigned long rtid;
-#endif
-        handle_t *handle = h;
-        journal_t *journal;
-        int rc;
-
-        LASSERT(current->journal_info == handle);
-
-        lock_24kernel();
-        transaction = handle->h_transaction;
-        journal = transaction->t_journal;
-        tid = transaction->t_tid;
-        /* we don't want to be blocked */
-        handle->h_sync = 0;
-        rc = journal_stop(handle);
-        if (rc) {
-                CERROR("error while stopping transaction: %d\n", rc);
-                unlock_kernel();
-                return rc;
-        }
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
-        rtid = log_start_commit(journal, transaction);
-        if (rtid != tid)
-                CERROR("strange race: %lu != %lu\n",
-                       (unsigned long) tid, (unsigned long) rtid);
-#else
-        log_start_commit(journal, tid);
-#endif
-        unlock_24kernel();
-
-        *wait_handle = (void *) tid;
-        CDEBUG(D_INODE, "commit async: %lu\n", (unsigned long) tid);
-        return 0;
-}
-
-static int fsfilt_ext3_commit_wait(struct inode *inode, void *h)
-{
-        journal_t *journal = EXT3_JOURNAL(inode);
-        tid_t tid = (tid_t)(long)h;
-
-        CDEBUG(D_INODE, "commit wait: %lu\n", (unsigned long) tid);
-        if (unlikely(is_journal_aborted(journal)))
-                return -EIO;
-
-        log_wait_commit(EXT3_JOURNAL(inode), tid);
-
-        if (unlikely(is_journal_aborted(journal)))
-                return -EIO;
-        return 0;
-}
-
-static int fsfilt_ext3_setattr(struct dentry *dentry, void *handle,
-                               struct iattr *iattr, int do_trunc)
-{
-        struct inode *inode = dentry->d_inode;
-        int rc;
-
-        lock_24kernel();
-
-        /* A _really_ horrible hack to avoid removing the data stored
-         * in the block pointers; this is really the "small" stripe MD data.
-         * We can avoid further hackery by virtue of the MDS file size being
-         * zero all the time (which doesn't invoke block truncate at unlink
-         * time), so we assert we never change the MDS file size from zero. */
-        if (iattr->ia_valid & ATTR_SIZE && !do_trunc) {
-                /* ATTR_SIZE would invoke truncate: clear it */
-                iattr->ia_valid &= ~ATTR_SIZE;
-                EXT3_I(inode)->i_disksize = inode->i_size = iattr->ia_size;
-
-                /* make sure _something_ gets set - so new inode
-                 * goes to disk (probably won't work over XFS */
-                if (!(iattr->ia_valid & (ATTR_MODE | ATTR_MTIME | ATTR_CTIME))){
-                        iattr->ia_valid |= ATTR_MODE;
-                        iattr->ia_mode = inode->i_mode;
-                }
-        }
-
-        /* Don't allow setattr to change file type */
-        iattr->ia_mode = (inode->i_mode & S_IFMT)|(iattr->ia_mode & ~S_IFMT);
-
-        /* We set these flags on the client, but have already checked perms
-         * so don't confuse inode_change_ok. */
-        iattr->ia_valid &= ~(ATTR_MTIME_SET | ATTR_ATIME_SET);
-
-        if (inode->i_op->setattr) {
-                rc = inode->i_op->setattr(dentry, iattr);
-        } else {
-                rc = inode_change_ok(inode, iattr);
-                if (!rc)
-                        rc = inode_setattr(inode, iattr);
-        }
-
-        unlock_24kernel();
-
-        return rc;
-}
-
-static int fsfilt_ext3_iocontrol(struct inode * inode, struct file *file,
-                                 unsigned int cmd, unsigned long arg)
-{
-        int rc = 0;
-        ENTRY;
-
-        if (inode->i_fop->ioctl)
-                rc = inode->i_fop->ioctl(inode, file, cmd, arg);
-        else
-                RETURN(-ENOTTY);
-
-        RETURN(rc);
-}
-
-static int fsfilt_ext3_set_xattr(struct inode * inode, void *handle, char *name,
-                                 void *buffer, int buffer_size)
-{
-        int rc = 0;
-
-        lock_24kernel();
-
-        rc = ext3_xattr_set_handle(handle, inode, EXT3_XATTR_INDEX_TRUSTED,
-                                   name, buffer, buffer_size, 0);
-        unlock_24kernel();
-        if (rc)
-                CERROR("set xattr %s from inode %lu: rc %d\n",
-                       name,  inode->i_ino, rc);
-        return rc;
-}
-
-static int fsfilt_ext3_get_xattr(struct inode *inode, char *name,
-                                 void *buffer, int buffer_size)
-{
-        int rc = 0;
-       
-        lock_24kernel();
-
-        rc = ext3_xattr_get(inode, EXT3_XATTR_INDEX_TRUSTED,
-                            name, buffer, buffer_size);
-        unlock_24kernel();
-
-        if (buffer == NULL)
-                return (rc == -ENODATA) ? 0 : rc;
-        if (rc < 0) {
-                CDEBUG(D_INFO, "error getting EA %s from inode %lu: rc %d\n",
-                       name,  inode->i_ino, rc);
-                memset(buffer, 0, buffer_size);
-                return (rc == -ENODATA) ? 0 : rc;
-        }
-
-        return rc;
-}
-
-static int fsfilt_ext3_set_md(struct inode *inode, void *handle,
-                              void *lmm, int lmm_size,
-                              enum ea_type type)
-{
-        int rc;
-        
-        switch(type) {
-        case EA_LOV:
-                rc = fsfilt_ext3_set_xattr(inode, handle,
-                                           XATTR_LUSTRE_MDS_LOV_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_MEA:
-                rc = fsfilt_ext3_set_xattr(inode, handle,
-                                           XATTR_LUSTRE_MDS_MEA_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_SID:
-                rc = fsfilt_ext3_set_xattr(inode, handle,
-                                           XATTR_LUSTRE_MDS_SID_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_PID:
-                 rc = fsfilt_ext3_set_xattr(inode, handle,
-                                            XATTR_LUSTRE_MDS_PID_EA,
-                                            lmm, lmm_size);
-                break;
-        case EA_KEY:
-                rc = fsfilt_ext3_set_xattr(inode, handle,
-                                           XATTR_LUSTRE_MDS_KEY_EA,
-                                           lmm, lmm_size);
-                break;
-        default:
-                return -EINVAL;
-        }
-
-        return rc;
-}
-
-static int fsfilt_ext3_get_md(struct inode *inode, void *lmm,
-                              int lmm_size, enum ea_type type)
-{
-        int rc;
-        
-        switch (type) {
-        case EA_LOV:
-                rc = fsfilt_ext3_get_xattr(inode,
-                                           XATTR_LUSTRE_MDS_LOV_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_MEA:
-                rc = fsfilt_ext3_get_xattr(inode,
-                                           XATTR_LUSTRE_MDS_MEA_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_SID:
-                rc = fsfilt_ext3_get_xattr(inode,
-                                           XATTR_LUSTRE_MDS_SID_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_PID:
-                rc = fsfilt_ext3_get_xattr(inode,
-                                           XATTR_LUSTRE_MDS_PID_EA,
-                                           lmm, lmm_size);
-                break;
-        case EA_KEY:
-                rc = fsfilt_ext3_get_xattr(inode, 
-                                           XATTR_LUSTRE_MDS_KEY_EA,
-                                           lmm, lmm_size);
-                break;
-        default:
-                return -EINVAL;
-        }
-        
-        return rc;
-}
-
-static int fsfilt_ext3_send_bio(int rw, struct inode *inode, void *bio)
-{
-       int rc = 0;
-#if (LINUX_VERSION_CODE > KERNEL_VERSION(2,5,0))
-        submit_bio(rw, (struct bio *)bio);
-#else
-       struct bio *b = (struct kiobuf *)bio;
-        int blocks_per_page;
-       
-        rc = brw_kiovec(rw, 1, &b, inode->i_dev,
-                        b->blocks, 1 << inode->i_blkbits);
-
-        blocks_per_page = PAGE_SIZE >> inode->i_blkbits;
-
-        if (rc != (1 << inode->i_blkbits) * b->nr_pages * blocks_per_page) {
-                CERROR("short write?  expected %d, wrote %d\n",
-                       (1 << inode->i_blkbits) * b->nr_pages *
-                       blocks_per_page, rc);
-        }
-#endif
-        return rc;
-}
-
-static struct page *fsfilt_ext3_getpage(struct inode *inode, long int index)
-{
-        int rc;
-        struct page *page;
-
-        page = grab_cache_page(inode->i_mapping, index);
-        if (page == NULL)
-                return ERR_PTR(-ENOMEM);
-
-        if (PageUptodate(page)) {
-                unlock_page(page);
-                return page;
-        }
-
-        rc = inode->i_mapping->a_ops->readpage(NULL, page);
-        if (rc < 0) {
-                page_cache_release(page);
-                return ERR_PTR(rc);
-        }
-
-        return page;
-}
-
-static ssize_t fsfilt_ext3_readpage(struct file *file, char *buf, size_t count,
-                                    loff_t *off)
-{
-        struct inode *inode = file->f_dentry->d_inode;
-        int rc = 0;
-
-        if (S_ISREG(inode->i_mode))
-                rc = file->f_op->read(file, buf, count, off);
-        else {
-                const int blkbits = inode->i_sb->s_blocksize_bits;
-                const int blksize = inode->i_sb->s_blocksize;
-
-                CDEBUG(D_EXT2, "reading "LPSZ" at dir %lu+%llu\n",
-                       count, inode->i_ino, *off);
-                while (count > 0) {
-                        struct buffer_head *bh;
-
-                        bh = NULL;
-                        if (*off < inode->i_size) {
-                                int err = 0;
-
-                                bh = ext3_bread(NULL, inode, *off >> blkbits,
-                                                0, &err);
-
-                                CDEBUG(D_EXT2, "read %u@%llu\n", blksize, *off);
-
-                                if (bh) {
-                                        memcpy(buf, bh->b_data, blksize);
-                                        brelse(bh);
-                                } else if (err) {
-                                        /* XXX in theory we should just fake
-                                         * this buffer and continue like ext3,
-                                         * especially if this is a partial read
-                                         */
-                                        CERROR("error read dir %lu+%llu: %d\n",
-                                               inode->i_ino, *off, err);
-                                        RETURN(err);
-                                }
-                        }
-                        if (!bh) {
-                                struct ext3_dir_entry_2 *fake = (void *)buf;
-
-                                CDEBUG(D_EXT2, "fake %u@%llu\n", blksize, *off);
-                                memset(fake, 0, sizeof(*fake));
-                                fake->rec_len = cpu_to_le32(blksize);
-                        }
-                        count -= blksize;
-                        buf += blksize;
-                        *off += blksize;
-                        rc += blksize;
-                }
-        }
-
-        return rc;
-}
-
-static void fsfilt_ext3_cb_func(struct journal_callback *jcb, int error)
-{
-        struct fsfilt_cb_data *fcb = (struct fsfilt_cb_data *)jcb;
-
-        fcb->cb_func(fcb->cb_obd, fcb->cb_last_num, fcb->cb_data, error);
-
-        OBD_SLAB_FREE(fcb, fcb_cache, sizeof *fcb);
-        atomic_dec(&fcb_cache_count);
-}
-
-static int fsfilt_ext3_add_journal_cb(struct obd_device *obd,
-                                      struct super_block *sb,
-                                      __u64 last_num, void *handle,
-                                      fsfilt_cb_t cb_func,
-                                      void *cb_data)
-{
-        struct fsfilt_cb_data *fcb;
-
-        OBD_SLAB_ALLOC(fcb, fcb_cache, GFP_NOFS, sizeof *fcb);
-        if (fcb == NULL)
-                RETURN(-ENOMEM);
-
-        atomic_inc(&fcb_cache_count);
-        fcb->cb_func = cb_func;
-        fcb->cb_obd = obd;
-        fcb->cb_last_num = last_num;
-        fcb->cb_data = cb_data;
-
-        CDEBUG(D_EXT2, "set callback for last_num: "LPD64"\n", last_num);
-        lock_24kernel();
-        journal_callback_set(handle, fsfilt_ext3_cb_func,
-                             (struct journal_callback *)fcb);
-        unlock_24kernel();
-        return 0;
-}
-
-/*
- * We need to hack the return value for the free inode counts because
- * the current EA code requires one filesystem block per inode with EAs,
- * so it is possible to run out of blocks before we run out of inodes.
- *
- * This can be removed when the ext3 EA code is fixed.
- */
-static int fsfilt_ext3_statfs(struct super_block *sb, struct obd_statfs *osfs)
-{
-        struct kstatfs sfs;
-        int rc;
-
-        memset(&sfs, 0, sizeof(sfs));
-
-        rc = sb->s_op->statfs(sb, &sfs);
-
-        if (!rc && sfs.f_bfree < sfs.f_ffree) {
-                sfs.f_files = (sfs.f_files - sfs.f_ffree) + sfs.f_bfree;
-                sfs.f_ffree = sfs.f_bfree;
-        }
-
-        statfs_pack(osfs, &sfs);
-        return rc;
-}
-
-static int fsfilt_ext3_sync(struct super_block *sb)
-{
-        return ext3_force_commit(sb);
-}
-
-#if defined(EXT3_MULTIBLOCK_ALLOCATOR) && (!defined(EXT3_EXT_CACHE_NO) || defined(EXT_CACHE_MARK))
-#warning "kernel code has old extents/mballoc patch, disabling"
-#undef EXT3_MULTIBLOCK_ALLOCATOR
-#endif
 #ifndef EXT3_EXTENTS_FL
-#define EXT3_EXTENTS_FL                        0x00080000 /* Inode uses extents */
+#define EXT3_EXTENTS_FL                 0x00080000 /* Inode uses extents */
 #endif
 
-#ifdef EXT3_MULTIBLOCK_ALLOCATOR
-#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0))
-#define ext3_up_truncate_sem(inode)  up_write(&EXT3_I(inode)->truncate_sem);
-#define ext3_down_truncate_sem(inode)  down_write(&EXT3_I(inode)->truncate_sem);
-#else
-#define ext3_up_truncate_sem(inode)  up(&EXT3_I(inode)->truncate_sem);
-#define ext3_down_truncate_sem(inode)  down(&EXT3_I(inode)->truncate_sem);
+#ifndef EXT_ASSERT
+#define EXT_ASSERT(cond)  BUG_ON(!(cond))
 #endif
 
-#include <linux/lustre_version.h>
-#if EXT3_EXT_MAGIC == 0xf301
-#define ee_start e_start
-#define ee_block e_block
-#define ee_len   e_num
-#endif
-#ifndef EXT3_BB_MAX_BLOCKS
-#define ext3_mb_new_blocks(handle, inode, goal, count, aflags, err) \
-        ext3_new_blocks(handle, inode, count, goal, err)
-#endif
+#define EXT_GENERATION(inode)           (EXT4_I(inode)->i_ext_generation)
+#define ext3_ext_base                   inode
+#define ext3_ext_base2inode(inode)      (inode)
+#define EXT_DEPTH(inode)                ext_depth(inode)
+#define fsfilt_ext3_ext_walk_space(inode, block, num, cb, cbdata) \
+                        ext3_ext_walk_space(inode, block, num, cb, cbdata);
 
 struct bpointers {
         unsigned long *blocks;
-        int *created;
         unsigned long start;
         int num;
         int init_num;
         int create;
 };
 
-static int ext3_ext_find_goal(struct inode *inode, struct ext3_ext_path *path,
-                              unsigned long block, int *aflags)
+static long ext3_ext_find_goal(struct inode *inode, struct ext3_ext_path *path,
+                               unsigned long block, int *aflags)
 {
         struct ext3_inode_info *ei = EXT3_I(inode);
         unsigned long bg_start;
@@ -777,18 +215,8 @@ static int ext3_ext_find_goal(struct inode *inode, struct ext3_ext_path *path,
                 depth = path->p_depth;
 
                 /* try to predict block placement */
-                if ((ex = path[depth].p_ext)) {
-#if 0
-                        /* This prefers to eat into a contiguous extent
-                         * rather than find an extent that the whole
-                         * request will fit into.  This can fragment data
-                         * block allocation and prevents our lovely 1M I/Os
-                         * from reaching the disk intact. */
-                        if (ex->ee_block + ex->ee_len == block)
-                                *aflags |= 1;
-#endif
-                        return ex->ee_start + (block - ex->ee_block);
-                }
+                if ((ex = path[depth].p_ext))
+                        return ext_pblock(ex) + (block - le32_to_cpu(ex->ee_block));
 
                 /* it looks index is empty
                  * try to find starting from index itself */
@@ -804,24 +232,79 @@ static int ext3_ext_find_goal(struct inode *inode, struct ext3_ext_path *path,
         return bg_start + colour + block;
 }
 
-static int ext3_ext_new_extent_cb(struct ext3_extents_tree *tree,
+#define ll_unmap_underlying_metadata(sb, blocknr) \
+        unmap_underlying_metadata((sb)->s_bdev, blocknr)
+
+#ifndef EXT3_MB_HINT_GROUP_ALLOC
+static unsigned long new_blocks(handle_t *handle, struct ext3_ext_base *base,
+                                struct ext3_ext_path *path, unsigned long block,
+                                unsigned long *count, int *err)
+{
+        unsigned long pblock, goal;
+        int aflags = 0;
+        struct inode *inode = ext3_ext_base2inode(base);
+
+        goal = ext3_ext_find_goal(inode, path, block, &aflags);
+        aflags |= 2; /* block have been already reserved */
+        pblock = ext3_mb_new_blocks(handle, inode, goal, count, aflags, err);
+        return pblock;
+
+}
+#else
+static unsigned long new_blocks(handle_t *handle, struct ext3_ext_base *base,
+                                struct ext3_ext_path *path, unsigned long block,
+                                unsigned long *count, int *err)
+{
+        struct inode *inode = ext3_ext_base2inode(base);
+        struct ext3_allocation_request ar;
+        unsigned long pblock;
+        int aflags;
+
+        /* find neighbour allocated blocks */
+        ar.lleft = block;
+        *err = ext3_ext_search_left(base, path, &ar.lleft, &ar.pleft);
+        if (*err)
+                return 0;
+        ar.lright = block;
+        *err = ext3_ext_search_right(base, path, &ar.lright, &ar.pright);
+        if (*err)
+                return 0;
+
+        /* allocate new block */
+        ar.goal = ext3_ext_find_goal(inode, path, block, &aflags);
+        ar.inode = inode;
+        ar.logical = block;
+        ar.len = *count;
+        ar.flags = EXT3_MB_HINT_DATA;
+        pblock = ext3_mb_new_blocks(handle, &ar, err);
+        *count = ar.len;
+        return pblock;
+}
+#endif
+
+static int ext3_ext_new_extent_cb(struct ext3_ext_base *base,
                                   struct ext3_ext_path *path,
-                                  struct ext3_ext_cache *cex)
+                                  struct ext3_ext_cache *cex,
+#ifdef HAVE_EXT_PREPARE_CB_EXTENT
+                                   struct ext3_extent *ex,
+#endif
+                                  void *cbdata)
 {
-        struct inode *inode = tree->inode;
-        struct bpointers *bp = tree->private;
+        struct bpointers *bp = cbdata;
+        struct inode *inode = ext3_ext_base2inode(base);
         struct ext3_extent nex;
-        int count, err, goal;
         unsigned long pblock;
-        loff_t new_i_size;
+        unsigned long tgen;
+        int err, i;
+        unsigned long count;
         handle_t *handle;
-        int i, aflags = 0;
 
-        i = EXT_DEPTH(tree);
-        EXT_ASSERT(i == path->p_depth);
-        EXT_ASSERT(path[i].p_hdr);
-
-               if (cex->ec_type == EXT3_EXT_CACHE_EXTENT) {
+#ifdef EXT3_EXT_CACHE_EXTENT
+        if (cex->ec_type == EXT3_EXT_CACHE_EXTENT)
+#else
+        if ((cex->ec_len != 0) && (cex->ec_start != 0))
+#endif
+                                                  {
                 err = EXT_CONTINUE;
                 goto map;
         }
@@ -834,8 +317,6 @@ static int ext3_ext_new_extent_cb(struct ext3_extents_tree *tree,
                         CERROR("nothing to do?! i = %d, e_num = %u\n",
                                         i, cex->ec_len);
                 for (; i < cex->ec_len && bp->num; i++) {
-                        *(bp->created) = 0;
-                        bp->created++;
                         *(bp->blocks) = 0;
                         bp->blocks++;
                         bp->num--;
@@ -845,70 +326,86 @@ static int ext3_ext_new_extent_cb(struct ext3_extents_tree *tree,
                 return EXT_CONTINUE;
         }
 
-        count = ext3_ext_calc_credits_for_insert(tree, path);
+       tgen = EXT_GENERATION(base);
+       count = ext3_ext_calc_credits_for_insert(base, path);
+
+       handle = ext3_journal_start(inode, count+EXT3_ALLOC_NEEDED+1);
+       if (IS_ERR(handle)) {
+               return PTR_ERR(handle);
+       }
 
-        lock_24kernel();
-        handle = journal_start(EXT3_JOURNAL(inode), count+EXT3_ALLOC_NEEDED+1);
-        unlock_24kernel();
-        if (IS_ERR(handle))
-                return PTR_ERR(handle);
+        if (tgen != EXT_GENERATION(base)) {
+                /* the tree has changed. so path can be invalid at moment */
+                ext3_journal_stop(handle);
+                return EXT_REPEAT;
+        }
+
+        /* In 2.6.32 kernel, ext4_ext_walk_space()'s callback func is not
+         * protected by i_data_sem as whole. so we patch it to store
+        * generation to path and now verify the tree hasn't changed */
+        down_write((&EXT4_I(inode)->i_data_sem));
+
+        /* validate extent, make sure the extent tree does not changed */
+       if (EXT_GENERATION(base) != path[0].p_generation) {
+                /* cex is invalid, try again */
+                up_write(&EXT4_I(inode)->i_data_sem);
+                ext3_journal_stop(handle);
+                return EXT_REPEAT;
+        }
 
         count = cex->ec_len;
-        goal = ext3_ext_find_goal(inode, path, cex->ec_block, &aflags);
-        aflags |= 2; /* block have been already reserved */
-        lock_24kernel();
-        pblock = ext3_mb_new_blocks(handle, inode, goal, &count, aflags, &err);
-        unlock_24kernel();
+        pblock = new_blocks(handle, base, path, cex->ec_block, &count, &err);
         if (!pblock)
                 goto out;
         EXT_ASSERT(count <= cex->ec_len);
-        //if (count < cex->ec_len)
-        //        CDEBUG(D_ERROR, "%d < %lu\n", count, cex->ec_len);
 
         /* insert new extent */
-        nex.ee_block = cex->ec_block;
-        nex.ee_start = pblock;
-        nex.ee_len = count;
-        err = ext3_ext_insert_extent(handle, tree, path, &nex);
-        if (err)
+        nex.ee_block = cpu_to_le32(cex->ec_block);
+        ext3_ext_store_pblock(&nex, pblock);
+        nex.ee_len = cpu_to_le16(count);
+        err = fsfilt_ext3_ext_insert_extent(handle, base, path, &nex, 0);
+        if (err) {
+                /* free data blocks we just allocated */
+                /* not a good idea to call discard here directly,
+                 * but otherwise we'd need to call it every free() */
+#ifdef EXT3_MB_HINT_GROUP_ALLOC
+                ext3_mb_discard_inode_preallocations(inode);
+#endif
+                ext3_free_blocks(handle, inode, ext_pblock(&nex),
+                                 cpu_to_le16(nex.ee_len), 0);
                 goto out;
+        }
 
         /*
          * Putting len of the actual extent we just inserted,
          * we are asking ext3_ext_walk_space() to continue
          * scaning after that block
          */
-        cex->ec_len = nex.ee_len;
-        cex->ec_start = nex.ee_start;
-        BUG_ON(nex.ee_len == 0);
-        BUG_ON(nex.ee_block != cex->ec_block);
-
-        /* correct on-disk inode size */
-        if (nex.ee_len > 0) {
-                new_i_size = (loff_t) nex.ee_block + nex.ee_len;
-                new_i_size = new_i_size << inode->i_blkbits;
-                if (new_i_size > EXT3_I(inode)->i_disksize) {
-                        EXT3_I(inode)->i_disksize = new_i_size;
-                        err = ext3_mark_inode_dirty(handle, inode);
-                }
-        }
+        cex->ec_len = le16_to_cpu(nex.ee_len);
+        cex->ec_start = ext_pblock(&nex);
+        BUG_ON(le16_to_cpu(nex.ee_len) == 0);
+        BUG_ON(le32_to_cpu(nex.ee_block) != cex->ec_block);
 
 out:
-        lock_24kernel();
-        journal_stop(handle);
-        unlock_24kernel();
+        up_write((&EXT4_I(inode)->i_data_sem));
+        ext3_journal_stop(handle);
 map:
         if (err >= 0) {
-                struct block_device *bdev = inode->i_sb->s_bdev;
-
                 /* map blocks */
                 if (bp->num == 0) {
                         CERROR("hmm. why do we find this extent?\n");
                         CERROR("initial space: %lu:%u\n",
                                 bp->start, bp->init_num);
-                        CERROR("current extent: %u/%u/%u %d\n",
+#ifdef EXT3_EXT_CACHE_EXTENT
+                        CERROR("current extent: %u/%u/%llu %d\n",
+                                cex->ec_block, cex->ec_len,
+                                (unsigned long long)cex->ec_start,
+                                cex->ec_type);
+#else
+                        CERROR("current extent: %u/%u/%llu\n",
                                 cex->ec_block, cex->ec_len,
-                                cex->ec_start, cex->ec_type);
+                                (unsigned long long)cex->ec_start);
+#endif
                 }
                 i = 0;
                 if (cex->ec_block < bp->start)
@@ -918,17 +415,17 @@ map:
                                         i, cex->ec_len);
                 for (; i < cex->ec_len && bp->num; i++) {
                         *(bp->blocks) = cex->ec_start + i;
-                        if (cex->ec_type == EXT3_EXT_CACHE_EXTENT) {
-                                *(bp->created) = 0;
-                        } else {
-                                *(bp->created) = 1;
+#ifdef EXT3_EXT_CACHE_EXTENT
+                       if (cex->ec_type != EXT3_EXT_CACHE_EXTENT)
+#else
+                       if ((cex->ec_len == 0) || (cex->ec_start == 0))
+#endif
+                                                                       {
                                 /* unmap any possible underlying metadata from
-                                 * the block device mapping.  bug 6998.
-                                 * This only compiles on 2.6, but there are
-                                 * no users of mballoc on 2.4. */
-                                unmap_underlying_metadata(bdev, *(bp->blocks));
+                                 * the block device mapping.  bug 6998. */
+                                ll_unmap_underlying_metadata(inode->i_sb,
+                                                             *(bp->blocks));
                         }
-                        bp->created++;
                         bp->blocks++;
                         bp->num--;
                         bp->start++;
@@ -938,35 +435,33 @@ map:
 }
 
 int fsfilt_map_nblocks(struct inode *inode, unsigned long block,
-                       unsigned long num, unsigned long *blocks,
-                       int *created, int create)
+                      unsigned long num, unsigned long *blocks,
+                      int create)
 {
-        struct ext3_extents_tree tree;
+        struct ext3_ext_base *base = inode;
         struct bpointers bp;
         int err;
 
         CDEBUG(D_OTHER, "blocks %lu-%lu requested for inode %u\n",
-                block, block + num, (unsigned) inode->i_ino);
+               block, block + num - 1, (unsigned) inode->i_ino);
 
-        ext3_init_tree_desc(&tree, inode);
-        tree.private = &bp;
         bp.blocks = blocks;
-        bp.created = created;
         bp.start = block;
         bp.init_num = bp.num = num;
         bp.create = create;
 
-        err = ext3_ext_walk_space(&tree, block, num, ext3_ext_new_extent_cb);
-        ext3_ext_invalidate_cache(&tree);
+       err = fsfilt_ext3_ext_walk_space(base, block, num,
+                                        ext3_ext_new_extent_cb, &bp);
+       ext3_ext_invalidate_cache(base);
 
         return err;
 }
 
 int fsfilt_ext3_map_ext_inode_pages(struct inode *inode, struct page **page,
-                                    int pages, unsigned long *blocks,
-                                    int *created, int create)
+                                   int pages, unsigned long *blocks,
+                                   int create)
 {
-        int blocks_per_page = PAGE_SIZE >> inode->i_blkbits;
+        int blocks_per_page = CFS_PAGE_SIZE >> inode->i_blkbits;
         int rc = 0, i = 0;
         struct page *fp = NULL;
         int clen = 0;
@@ -993,102 +488,85 @@ int fsfilt_ext3_map_ext_inode_pages(struct inode *inode, struct page **page,
 
                 /* process found extent */
                 rc = fsfilt_map_nblocks(inode, fp->index * blocks_per_page,
-                                        clen * blocks_per_page, blocks,
-                                        created, create);
+                                       clen * blocks_per_page, blocks,
+                                       create);
                 if (rc)
                         GOTO(cleanup, rc);
 
                 /* look for next extent */
                 fp = NULL;
                 blocks += blocks_per_page * clen;
-                created += blocks_per_page * clen;
         }
 
         if (fp)
                 rc = fsfilt_map_nblocks(inode, fp->index * blocks_per_page,
-                                        clen * blocks_per_page, blocks,
-                                        created, create);
+                                       clen * blocks_per_page, blocks,
+                                       create);
 cleanup:
         return rc;
 }
-#endif
 
-extern int ext3_map_inode_page(struct inode *inode, struct page *page,
-                               unsigned long *blocks, int *created, int create);
 int fsfilt_ext3_map_bm_inode_pages(struct inode *inode, struct page **page,
-                                   int pages, unsigned long *blocks,
-                                   int *created, int create)
+                                  int pages, unsigned long *blocks,
+                                  int create)
 {
-        int blocks_per_page = PAGE_SIZE >> inode->i_blkbits;
-        unsigned long *b;
-        int rc = 0, i, *cr;
+       int blocks_per_page = CFS_PAGE_SIZE >> inode->i_blkbits;
+       unsigned long *b;
+       int rc = 0, i;
 
-        for (i = 0, cr = created, b = blocks; i < pages; i++, page++) {
-                rc = ext3_map_inode_page(inode, *page, b, cr, create);
+       for (i = 0, b = blocks; i < pages; i++, page++) {
+               rc = ext3_map_inode_page(inode, *page, b, create);
                 if (rc) {
-                        CERROR("ino %lu, blk %lu cr %u create %d: rc %d\n",
-                               inode->i_ino, *b, *cr, create, rc);
+                       CERROR("ino %lu, blk %lu create %d: rc %d\n",
+                              inode->i_ino, *b, create, rc);
                         break;
                 }
 
                 b += blocks_per_page;
-                cr += blocks_per_page;
         }
         return rc;
 }
 
 int fsfilt_ext3_map_inode_pages(struct inode *inode, struct page **page,
-                                int pages, unsigned long *blocks,
-                                int *created, int create,
-                                struct semaphore *optional_sem)
+                               int pages, unsigned long *blocks,
+                               int create, struct mutex *optional_mutex)
 {
-        int rc = 0;
-        if (optional_sem != NULL)
-                down(optional_sem);
-#ifdef EXT3_MULTIBLOCK_ALLOCATOR
-        if (EXT3_I(inode)->i_flags & EXT3_EXTENTS_FL)
-                rc = fsfilt_ext3_map_ext_inode_pages(inode, page, pages,
-                                                     blocks, created, create);
-        else
-#endif
-                rc = fsfilt_ext3_map_bm_inode_pages(inode, page, pages, blocks,
-                                                    created, create);
-        if (optional_sem != NULL)
-                up(optional_sem);
+        int rc;
 
-        return rc;
-}
+        if (EXT3_I(inode)->i_flags & EXT3_EXTENTS_FL) {
+               rc = fsfilt_ext3_map_ext_inode_pages(inode, page, pages,
+                                                    blocks, create);
+                return rc;
+        }
+        if (optional_mutex != NULL)
+               mutex_lock(optional_mutex);
+       rc = fsfilt_ext3_map_bm_inode_pages(inode, page, pages, blocks, create);
+        if (optional_mutex != NULL)
+               mutex_unlock(optional_mutex);
 
-extern int ext3_prep_san_write(struct inode *inode, long *blocks,
-                               int nblocks, loff_t newsize);
-static int fsfilt_ext3_prep_san_write(struct inode *inode, long *blocks,
-                                      int nblocks, loff_t newsize)
-{
-        return ext3_prep_san_write(inode, blocks, nblocks, newsize);
+        return rc;
 }
 
-static int fsfilt_ext3_read_record(struct file * file, void *buf,
-                                   int size, loff_t *offs)
+int fsfilt_ext3_read(struct inode *inode, void *buf, int size, loff_t *offs)
 {
-        struct inode *inode = file->f_dentry->d_inode;
         unsigned long block;
         struct buffer_head *bh;
-        int err, blocksize, csize, boffs;
+        int err, blocksize, csize, boffs, osize = size;
 
         /* prevent reading after eof */
-        lock_kernel();
-        if (inode->i_size < *offs + size) {
-                size = inode->i_size - *offs;
-                unlock_kernel();
+       spin_lock(&inode->i_lock);
+       if (i_size_read(inode) < *offs + size) {
+               size = i_size_read(inode) - *offs;
+               spin_unlock(&inode->i_lock);
                 if (size < 0) {
-                        CERROR("size %llu is too short for read %u@%llu\n",
-                               inode->i_size, size, *offs);
-                        return -EIO;
+                        CDEBUG(D_EXT2, "size %llu is too short for read @%llu\n",
+                               i_size_read(inode), *offs);
+                        return -EBADR;
                 } else if (size == 0) {
                         return 0;
                 }
         } else {
-                unlock_kernel();
+               spin_unlock(&inode->i_lock);
         }
 
         blocksize = 1 << inode->i_blkbits;
@@ -1110,35 +588,28 @@ static int fsfilt_ext3_read_record(struct file * file, void *buf,
                 buf += csize;
                 size -= csize;
         }
-        return 0;
+        return osize;
 }
+EXPORT_SYMBOL(fsfilt_ext3_read);
 
-static int fsfilt_ext3_write_record(struct file *file, void *buf, int bufsize,
-                                    loff_t *offs, int force_sync)
+static int fsfilt_ext3_read_record(struct file * file, void *buf,
+                                   int size, loff_t *offs)
+{
+        int rc;
+        rc = fsfilt_ext3_read(file->f_dentry->d_inode, buf, size, offs);
+        if (rc > 0)
+                rc = 0;
+        return rc;
+}
+
+int fsfilt_ext3_write_handle(struct inode *inode, void *buf, int bufsize,
+                                loff_t *offs, handle_t *handle)
 {
         struct buffer_head *bh = NULL;
+        loff_t old_size = i_size_read(inode), offset = *offs;
+        loff_t new_size = i_size_read(inode);
         unsigned long block;
-        struct inode *inode = file->f_dentry->d_inode;
-        loff_t old_size = inode->i_size, offset = *offs;
-        loff_t new_size = inode->i_size;
-        journal_t *journal;
-        handle_t *handle;
-        int err, block_count = 0, blocksize, size, boffs;
-
-        /* Determine how many transaction credits are needed */
-        blocksize = 1 << inode->i_blkbits;
-        block_count = (*offs & (blocksize - 1)) + bufsize;
-        block_count = (block_count + blocksize - 1) >> inode->i_blkbits;
-
-        journal = EXT3_SB(inode->i_sb)->s_journal;
-        lock_24kernel();
-        handle = journal_start(journal,
-                               block_count * EXT3_DATA_TRANS_BLOCKS + 2);
-        unlock_24kernel();
-        if (IS_ERR(handle)) {
-                CERROR("can't start transaction\n");
-                return PTR_ERR(handle);
-        }
+        int err = 0, blocksize = 1 << inode->i_blkbits, size, boffs;
 
         while (bufsize > 0) {
                 if (bh != NULL)
@@ -1150,14 +621,14 @@ static int fsfilt_ext3_write_record(struct file *file, void *buf, int bufsize,
                 bh = ext3_bread(handle, inode, block, 1, &err);
                 if (!bh) {
                         CERROR("can't read/create block: %d\n", err);
-                        goto out;
+                        break;
                 }
 
                 err = ext3_journal_get_write_access(handle, bh);
                 if (err) {
                         CERROR("journal_get_write_access() returned error %d\n",
                                err);
-                        goto out;
+                        break;
                 }
                 LASSERT(bh->b_data + boffs + size <= bh->b_data + bh->b_size);
                 memcpy(bh->b_data + boffs, buf, size);
@@ -1165,7 +636,7 @@ static int fsfilt_ext3_write_record(struct file *file, void *buf, int bufsize,
                 if (err) {
                         CERROR("journal_dirty_metadata() returned error %d\n",
                                err);
-                        goto out;
+                        break;
                 }
                 if (offset + size > new_size)
                         new_size = offset + size;
@@ -1173,293 +644,96 @@ static int fsfilt_ext3_write_record(struct file *file, void *buf, int bufsize,
                 bufsize -= size;
                 buf += size;
         }
-
-        if (force_sync)
-                handle->h_sync = 1; /* recovery likes this */
-out:
         if (bh)
                 brelse(bh);
 
         /* correct in-core and on-disk sizes */
-        if (new_size > inode->i_size) {
-                lock_kernel();
-                if (new_size > inode->i_size)
-                        inode->i_size = new_size;
-                if (inode->i_size > EXT3_I(inode)->i_disksize)
-                        EXT3_I(inode)->i_disksize = inode->i_size;
-                if (inode->i_size > old_size)
-                        mark_inode_dirty(inode);
-                unlock_kernel();
+        if (new_size > i_size_read(inode)) {
+               spin_lock(&inode->i_lock);
+               if (new_size > i_size_read(inode))
+                       i_size_write(inode, new_size);
+               if (i_size_read(inode) > EXT3_I(inode)->i_disksize)
+                       EXT3_I(inode)->i_disksize = i_size_read(inode);
+               if (i_size_read(inode) > old_size) {
+                       spin_unlock(&inode->i_lock);
+                       mark_inode_dirty(inode);
+               } else {
+                       spin_unlock(&inode->i_lock);
+                }
         }
 
-        lock_24kernel();
-        journal_stop(handle);
-        unlock_24kernel();
-
         if (err == 0)
                 *offs = offset;
         return err;
 }
+EXPORT_SYMBOL(fsfilt_ext3_write_handle);
 
-static int fsfilt_ext3_setup(struct obd_device *obd, struct super_block *sb)
+static int fsfilt_ext3_write_record(struct file *file, void *buf, int bufsize,
+                                    loff_t *offs, int force_sync)
 {
-#ifdef EXT3_FEATURE_INCOMPAT_MDSNUM
-        struct mds_obd *mds = &obd->u.mds;
-#endif
-#if 0
-        EXT3_SB(sb)->dx_lock = fsfilt_ext3_dx_lock;
-        EXT3_SB(sb)->dx_unlock = fsfilt_ext3_dx_unlock;
-#endif
-#ifdef S_PDIROPS
-        CWARN("Enabling PDIROPS\n");
-        set_opt(EXT3_SB(sb)->s_mount_opt, PDIROPS);
-        sb->s_flags |= S_PDIROPS;
-#endif
-        /* setup mdsnum in underlying fs */
-#ifdef EXT3_FEATURE_INCOMPAT_MDSNUM
-        if (mds->mds_md_obd) {
-                struct ext3_sb_info *sbi = EXT3_SB(sb);
-                struct ext3_super_block *es = sbi->s_es;
-                handle_t *handle;
-                int err;
-                
-                if (!EXT3_HAS_INCOMPAT_FEATURE(sb, EXT3_FEATURE_INCOMPAT_MDSNUM)) {
-                        CWARN("%s: set mdsnum %d in ext3\n",
-                              obd->obd_name, mds->mds_num);
-                        lock_24kernel();
-                        handle = journal_start(sbi->s_journal, 1);
-                        unlock_24kernel();
-                        LASSERT(!IS_ERR(handle));
-                        err = ext3_journal_get_write_access(handle, sbi->s_sbh);
-                        LASSERT(err == 0);
-                        EXT3_SET_INCOMPAT_FEATURE(sb,
-                                                EXT3_FEATURE_INCOMPAT_MDSNUM);
-                        es->s_mdsnum = mds->mds_num;
-                        err = ext3_journal_dirty_metadata(handle, sbi->s_sbh);
-                        LASSERT(err == 0);
-                        lock_24kernel();
-                        journal_stop(handle);
-                        unlock_24kernel();
-                } else {
-                        CWARN("%s: mdsnum initialized to %u in ext3fs\n",
-                                obd->obd_name, es->s_mdsnum);
-                }
-                sbi->s_mdsnum = es->s_mdsnum;
-        }
-#endif
-        return 0;
-}
+        struct inode *inode = file->f_dentry->d_inode;
+        handle_t *handle;
+        int err, block_count = 0, blocksize;
 
-extern int ext3_add_dir_entry(struct dentry *dentry);
-extern int ext3_del_dir_entry(struct dentry *dentry);
+        /* Determine how many transaction credits are needed */
+        blocksize = 1 << inode->i_blkbits;
+        block_count = (*offs & (blocksize - 1)) + bufsize;
+        block_count = (block_count + blocksize - 1) >> inode->i_blkbits;
 
-static int fsfilt_ext3_add_dir_entry(struct obd_device *obd,
-                                     struct dentry *parent,
-                                     char *name, int namelen,
-                                     unsigned long ino,
-                                     unsigned long generation,
-                                     unsigned long mds, 
-                                     unsigned long fid)
-{
-#ifdef EXT3_FEATURE_INCOMPAT_MDSNUM
-        struct dentry *dentry;
-        int err;
-        LASSERT(ino != 0);
-        LASSERT(namelen != 0);
-        dentry = ll_lookup_one_len(name, parent, namelen);
-        if (IS_ERR(dentry)) {
-                CERROR("can't lookup %*s in %lu/%lu: %d\n", dentry->d_name.len,
-                       dentry->d_name.name, dentry->d_inode->i_ino,
-                       (unsigned long) dentry->d_inode->i_generation,
-                       (int) PTR_ERR(dentry));
-                RETURN(PTR_ERR(dentry));
-        }
-        if (dentry->d_inode != NULL || dentry->d_flags & DCACHE_CROSS_REF) {
-                CERROR("dentry %*s(0x%p) found\n", dentry->d_name.len,
-                       dentry->d_name.name, dentry);
-                l_dput(dentry);
-                RETURN(-EEXIST);
-        }
+       handle = ext3_journal_start(inode,
+                       block_count * EXT3_DATA_TRANS_BLOCKS(inode->i_sb) + 2);
+       if (IS_ERR(handle)) {
+               CERROR("can't start transaction for %d blocks (%d bytes)\n",
+                      block_count * EXT3_DATA_TRANS_BLOCKS(inode->i_sb) + 2,
+                      bufsize);
+               return PTR_ERR(handle);
+       }
 
-        /* mds_reint_rename() may use this method to add dir entry 
-         * that points onto local inode. and we don't want to find
-         * it cross-ref by subsequent lookups */
-        d_drop(dentry);
-
-        dentry->d_flags |= DCACHE_CROSS_REF;
-        dentry->d_inum = ino;
-        dentry->d_mdsnum = mds;
-        dentry->d_generation = generation;
-        dentry->d_fid = fid;
-        lock_24kernel();
-        err = ext3_add_dir_entry(dentry);
-        unlock_24kernel();
-        
-        l_dput(dentry);
-
-        RETURN(err);
-#else
-#error "rebuild kernel and lustre with ext3-mds-num patch!"
-        LASSERT(0);
-#endif
-}
+        err = fsfilt_ext3_write_handle(inode, buf, bufsize, offs, handle);
+
+        if (!err && force_sync)
+                handle->h_sync = 1; /* recovery likes this */
+
+        ext3_journal_stop(handle);
 
-static int fsfilt_ext3_del_dir_entry(struct obd_device *obd,
-                                 struct dentry *dentry)
-{
-#ifdef EXT3_FEATURE_INCOMPAT_MDSNUM
-        int err;
-        lock_24kernel();
-        err = ext3_del_dir_entry(dentry);
-        unlock_24kernel();
-        if (err == 0)
-                d_drop(dentry);
         return err;
-#else
-#error "rebuild kernel and lustre with ext3-mds-num patch!"
-        LASSERT(0);
-#endif
 }
 
-/* If fso is NULL, op is FSFILT operation, otherwise op is number of fso
-   objects. Logs is number of logfiles to update */
-static int fsfilt_ext3_get_op_len(int op, struct fsfilt_objinfo *fso, int logs)
+static int fsfilt_ext3_setup(struct super_block *sb)
 {
-        if ( !fso ) {
-                switch(op) {
-                case FSFILT_OP_CREATE:
-                                 /* directory leaf, index & indirect & EA*/
-                        return 4 + 3 * logs;
-                case FSFILT_OP_UNLINK:
-                        return 3 * logs;
-                }
-        } else {
-                int i;
-                int needed = 0;
-                struct super_block *sb = fso->fso_dentry->d_inode->i_sb;
-                int blockpp = 1 << (PAGE_CACHE_SHIFT - sb->s_blocksize_bits);
-                int addrpp = EXT3_ADDR_PER_BLOCK(sb) * blockpp;
-                for (i = 0; i < op; i++, fso++) {
-                        int nblocks = fso->fso_bufcnt * blockpp;
-                        int ndindirect = min(nblocks, addrpp + 1);
-                        int nindir = nblocks + ndindirect + 1;
-
-                        needed += nindir;
-                }
-                return needed + 3 * logs;
+        if (!EXT3_HAS_COMPAT_FEATURE(sb,
+                                EXT3_FEATURE_COMPAT_HAS_JOURNAL)) {
+                CERROR("ext3 mounted without journal\n");
+                return -EINVAL;
         }
 
+#ifdef S_PDIROPS
+        CWARN("Enabling PDIROPS\n");
+        set_opt(EXT3_SB(sb)->s_mount_opt, PDIROPS);
+        sb->s_flags |= S_PDIROPS;
+#endif
+        if (!EXT3_HAS_COMPAT_FEATURE(sb, EXT3_FEATURE_COMPAT_DIR_INDEX))
+                CWARN("filesystem doesn't have dir_index feature enabled\n");
         return 0;
 }
-
-
-#if 0
-#define EXTENTS_EA "write_extents"
-#define EXTENTS_EA_SIZE 64
-
-int ext3_ext_in_ea_alloc_space(struct inode *, int, const char *, unsigned long, unsigned long);
-int ext3_ext_in_ea_remove_space(struct inode *, int, const char *, unsigned long, unsigned long);
-int ext3_ext_in_ea_get_extents(struct inode *, int, const char *, char **, int *);
-int ext3_ext_in_ea_get_extents_num(struct inode *, int, const char *, int *);
-
-static int fsfilt_ext3_insert_extents_ea(struct inode *inode, 
-                                      unsigned long from, 
-                                      unsigned long num) 
-{
-        int rc = 0;
-
-        rc = ext3_ext_in_ea_alloc_space(inode, EXT3_XATTR_INDEX_TRUSTED,
-                                        EXTENTS_EA, from, num);  
-        return rc;
-}
-
-static int fsfilt_ext3_remove_extents_ea(struct inode *inode, 
-                                         unsigned long from, 
-                                         unsigned long num) 
-{
-        int rc = 0;
-
-        rc = ext3_ext_in_ea_remove_space(inode, EXT3_XATTR_INDEX_TRUSTED,
-                                         EXTENTS_EA, from, num);  
-        return rc;
-}
-
-extern int ext3_init_tree_in_ea(struct inode *inode, int name_index,
-                               const char *eaname, int size);
-                                
-static int fsfilt_ext3_init_extents_ea(struct inode *inode)
-{
-        int rc = 0;
-
-        rc = ext3_init_tree_in_ea(inode, EXT3_XATTR_INDEX_TRUSTED,
-                                  EXTENTS_EA, 64);  
-        return rc;
-}
-
-static int fsfilt_ext3_get_inode_write_extents(struct inode *inode, 
-                                         char **pbuf, int *size)
-{
-        int rc = 0;
-        
-        rc = ext3_ext_in_ea_get_extents(inode, EXT3_XATTR_INDEX_TRUSTED,
-                                        EXTENTS_EA,  pbuf, size);
-        return rc; 
-} 
-
-static int fsfilt_ext3_get_write_extents_num(struct inode *inode, int *size)
-{
-        int rc = 0;
-        
-        rc = ext3_ext_in_ea_get_extents_num(inode, EXT3_XATTR_INDEX_TRUSTED, 
-                                            EXTENTS_EA, size);
-        return rc; 
-} 
-#endif
-
 static struct fsfilt_operations fsfilt_ext3_ops = {
-        .fs_type                    = "ext3",
-        .fs_owner                   = THIS_MODULE,
-        .fs_start                   = fsfilt_ext3_start,
-        .fs_brw_start               = fsfilt_ext3_brw_start,
-        .fs_commit                  = fsfilt_ext3_commit,
-        .fs_commit_async            = fsfilt_ext3_commit_async,
-        .fs_commit_wait             = fsfilt_ext3_commit_wait,
-        .fs_setattr                 = fsfilt_ext3_setattr,
-        .fs_iocontrol               = fsfilt_ext3_iocontrol,
-        .fs_set_md                  = fsfilt_ext3_set_md,
-        .fs_get_md                  = fsfilt_ext3_get_md,
-        .fs_readpage                = fsfilt_ext3_readpage,
-        .fs_add_journal_cb          = fsfilt_ext3_add_journal_cb,
-        .fs_statfs                  = fsfilt_ext3_statfs,
-        .fs_sync                    = fsfilt_ext3_sync,
-        .fs_map_inode_pages         = fsfilt_ext3_map_inode_pages,
-        .fs_prep_san_write          = fsfilt_ext3_prep_san_write,
-        .fs_write_record            = fsfilt_ext3_write_record,
-        .fs_read_record             = fsfilt_ext3_read_record,
-        .fs_setup                   = fsfilt_ext3_setup,
-        .fs_getpage                 = fsfilt_ext3_getpage,
-        .fs_send_bio                = fsfilt_ext3_send_bio,
-        .fs_set_xattr               = fsfilt_ext3_set_xattr,
-        .fs_get_xattr               = fsfilt_ext3_get_xattr,
-        .fs_get_op_len              = fsfilt_ext3_get_op_len,
-        .fs_add_dir_entry           = fsfilt_ext3_add_dir_entry,
-        .fs_del_dir_entry           = fsfilt_ext3_del_dir_entry,
-#if 0
-        .fs_init_extents_ea         = fsfilt_ext3_init_extents_ea,
-        .fs_insert_extents_ea       = fsfilt_ext3_insert_extents_ea,
-        .fs_remove_extents_ea       = fsfilt_ext3_remove_extents_ea,
-        .fs_get_inode_write_extents = fsfilt_ext3_get_inode_write_extents,
-        .fs_get_write_extents_num   = fsfilt_ext3_get_write_extents_num,
-#endif
+        .fs_type                = "ext3",
+        .fs_owner               = THIS_MODULE,
+        .fs_getlabel            = fsfilt_ext3_get_label,
+        .fs_start               = fsfilt_ext3_start,
+        .fs_commit              = fsfilt_ext3_commit,
+        .fs_map_inode_pages     = fsfilt_ext3_map_inode_pages,
+        .fs_write_record        = fsfilt_ext3_write_record,
+        .fs_read_record         = fsfilt_ext3_read_record,
+        .fs_setup               = fsfilt_ext3_setup,
 };
 
 static int __init fsfilt_ext3_init(void)
 {
         int rc;
 
-        fcb_cache = kmem_cache_create("fsfilt_ext3_fcb",
-                                      sizeof(struct fsfilt_cb_data), 0,
-                                      0, NULL, NULL);
+        fcb_cache = cfs_mem_cache_create("fsfilt_ext3_fcb",
+                                         sizeof(struct fsfilt_cb_data), 0, 0);
         if (!fcb_cache) {
                 CERROR("error allocating fsfilt journal callback cache\n");
                 GOTO(out, rc = -ENOMEM);
@@ -1467,23 +741,26 @@ static int __init fsfilt_ext3_init(void)
 
         rc = fsfilt_register_ops(&fsfilt_ext3_ops);
 
-        if (rc)
-                kmem_cache_destroy(fcb_cache);
+        if (rc) {
+                int err = cfs_mem_cache_destroy(fcb_cache);
+                LASSERTF(err == 0, "error destroying new cache: rc %d\n", err);
+        }
 out:
         return rc;
 }
 
 static void __exit fsfilt_ext3_exit(void)
 {
+        int rc;
+
         fsfilt_unregister_ops(&fsfilt_ext3_ops);
-        LASSERTF(kmem_cache_destroy(fcb_cache) == 0,
-                 "can't free fsfilt callback cache: count %d\n",
-                 atomic_read(&fcb_cache_count));
+        rc = cfs_mem_cache_destroy(fcb_cache);
+        LASSERTF(rc == 0, "couldn't destroy fcb_cache slab\n");
 }
 
 module_init(fsfilt_ext3_init);
 module_exit(fsfilt_ext3_exit);
 
-MODULE_AUTHOR("Cluster File Systems, Inc. <info@clusterfs.com>");
+MODULE_AUTHOR("Sun Microsystems, Inc. <http://www.lustre.org/>");
 MODULE_DESCRIPTION("Lustre ext3 Filesystem Helper v0.1");
 MODULE_LICENSE("GPL");