mm: fault feedback #1
[safe/jmp/linux-2.6] / fs / gfs2 / ops_file.c
index 0f356fc..1a5e8e8 100644 (file)
@@ -1,13 +1,12 @@
 /*
  * Copyright (C) Sistina Software, Inc.  1997-2003 All rights reserved.
- * Copyright (C) 2004-2005 Red Hat, Inc.  All rights reserved.
+ * Copyright (C) 2004-2006 Red Hat, Inc.  All rights reserved.
  *
  * This copyrighted material is made available to anyone wishing to use,
  * modify, copy, or redistribute it subject to the terms and conditions
- * of the GNU General Public License v.2.
+ * of the GNU General Public License version 2.
  */
 
-#include <linux/sched.h>
 #include <linux/slab.h>
 #include <linux/spinlock.h>
 #include <linux/completion.h>
 #include <linux/uio.h>
 #include <linux/blkdev.h>
 #include <linux/mm.h>
-#include <linux/smp_lock.h>
-#include <linux/gfs2_ioctl.h>
-#include <asm/semaphore.h>
+#include <linux/fs.h>
+#include <linux/gfs2_ondisk.h>
+#include <linux/ext2_fs.h>
+#include <linux/crc32.h>
+#include <linux/lm_interface.h>
+#include <linux/writeback.h>
 #include <asm/uaccess.h>
 
 #include "gfs2.h"
+#include "incore.h"
 #include "bmap.h"
 #include "dir.h"
 #include "glock.h"
 #include "glops.h"
 #include "inode.h"
-#include "jdata.h"
 #include "lm.h"
 #include "log.h"
 #include "meta_io.h"
 #include "quota.h"
 #include "rgrp.h"
 #include "trans.h"
+#include "util.h"
+#include "eaops.h"
 
-/* "bad" is for NFS support */
-struct filldir_bad_entry {
-       char *fbe_name;
-       unsigned int fbe_length;
-       uint64_t fbe_offset;
-       struct gfs2_inum fbe_inum;
-       unsigned int fbe_type;
+/*
+ * Most fields left uninitialised to catch anybody who tries to
+ * use them. f_flags set to prevent file_accessed() from touching
+ * any other part of this. Its use is purely as a flag so that we
+ * know (in readpage()) whether or not do to locking.
+ */
+struct file gfs2_internal_file_sentinel = {
+       .f_flags = O_NOATIME|O_RDONLY,
 };
 
-struct filldir_bad {
-       struct gfs2_sbd *fdb_sbd;
-
-       struct filldir_bad_entry *fdb_entry;
-       unsigned int fdb_entry_num;
-       unsigned int fdb_entry_off;
+static int gfs2_read_actor(read_descriptor_t *desc, struct page *page,
+                          unsigned long offset, unsigned long size)
+{
+       char *kaddr;
+       unsigned long count = desc->count;
 
-       char *fdb_name;
-       unsigned int fdb_name_size;
-       unsigned int fdb_name_off;
-};
+       if (size > count)
+               size = count;
 
-/* For regular, non-NFS */
-struct filldir_reg {
-       struct gfs2_sbd *fdr_sbd;
-       int fdr_prefetch;
+       kaddr = kmap(page);
+       memcpy(desc->arg.data, kaddr + offset, size);
+       kunmap(page);
 
-       filldir_t fdr_filldir;
-       void *fdr_opaque;
-};
+       desc->count = count - size;
+       desc->written += size;
+       desc->arg.buf += size;
+       return size;
+}
 
-typedef ssize_t(*do_rw_t) (struct file *file,
-                  char __user *buf,
-                  size_t size, loff_t *offset,
-                  unsigned int num_gh, struct gfs2_holder *ghs);
+int gfs2_internal_read(struct gfs2_inode *ip, struct file_ra_state *ra_state,
+                      char *buf, loff_t *pos, unsigned size)
+{
+       struct inode *inode = &ip->i_inode;
+       read_descriptor_t desc;
+       desc.written = 0;
+       desc.arg.data = buf;
+       desc.count = size;
+       desc.error = 0;
+       do_generic_mapping_read(inode->i_mapping, ra_state,
+                               &gfs2_internal_file_sentinel, pos, &desc,
+                               gfs2_read_actor);
+       return desc.written ? desc.written : desc.error;
+}
 
 /**
  * gfs2_llseek - seek to a location in a file
@@ -86,12 +99,10 @@ typedef ssize_t(*do_rw_t) (struct file *file,
 
 static loff_t gfs2_llseek(struct file *file, loff_t offset, int origin)
 {
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
+       struct gfs2_inode *ip = GFS2_I(file->f_mapping->host);
        struct gfs2_holder i_gh;
        loff_t error;
 
-       atomic_inc(&ip->i_sbd->sd_ops_file);
-
        if (origin == 2) {
                error = gfs2_glock_nq_init(ip->i_gl, LM_ST_SHARED, LM_FLAG_ANY,
                                           &i_gh);
@@ -105,845 +116,8 @@ static loff_t gfs2_llseek(struct file *file, loff_t offset, int origin)
        return error;
 }
 
-static inline unsigned int vma2state(struct vm_area_struct *vma)
-{
-       if ((vma->vm_flags & (VM_MAYWRITE | VM_MAYSHARE)) ==
-           (VM_MAYWRITE | VM_MAYSHARE))
-               return LM_ST_EXCLUSIVE;
-       return LM_ST_SHARED;
-}
-
-static ssize_t walk_vm_hard(struct file *file, const char __user *buf, size_t size,
-                   loff_t *offset, do_rw_t operation)
-{
-       struct gfs2_holder *ghs;
-       unsigned int num_gh = 0;
-       ssize_t count;
-       struct super_block *sb = file->f_dentry->d_inode->i_sb;
-       struct mm_struct *mm = current->mm;
-       struct vm_area_struct *vma;
-       unsigned long start = (unsigned long)buf;
-       unsigned long end = start + size;
-       int dumping = (current->flags & PF_DUMPCORE);
-       unsigned int x = 0;
-
-       for (vma = find_vma(mm, start); vma; vma = vma->vm_next) {
-               if (end <= vma->vm_start)
-                       break;
-               if (vma->vm_file &&
-                   vma->vm_file->f_dentry->d_inode->i_sb == sb) {
-                       num_gh++;
-               }
-       }
-
-       ghs = kcalloc((num_gh + 1), sizeof(struct gfs2_holder), GFP_KERNEL);
-       if (!ghs) {
-               if (!dumping)
-                       up_read(&mm->mmap_sem);
-               return -ENOMEM;
-       }
-
-       for (vma = find_vma(mm, start); vma; vma = vma->vm_next) {
-               if (end <= vma->vm_start)
-                       break;
-               if (vma->vm_file) {
-                       struct inode *inode = vma->vm_file->f_dentry->d_inode;
-                       if (inode->i_sb == sb)
-                               gfs2_holder_init(get_v2ip(inode)->i_gl,
-                                                vma2state(vma), 0, &ghs[x++]);
-               }
-       }
-
-       if (!dumping)
-               up_read(&mm->mmap_sem);
-
-       gfs2_assert(get_v2sdp(sb), x == num_gh);
-
-       count = operation(file, buf, size, offset, num_gh, ghs);
-
-       while (num_gh--)
-               gfs2_holder_uninit(&ghs[num_gh]);
-       kfree(ghs);
-
-       return count;
-}
-
-/**
- * walk_vm - Walk the vmas associated with a buffer for read or write.
- *    If any of them are gfs2, pass the gfs2 inode down to the read/write
- *    worker function so that locks can be acquired in the correct order.
- * @file: The file to read/write from/to
- * @buf: The buffer to copy to/from
- * @size: The amount of data requested
- * @offset: The current file offset
- * @operation: The read or write worker function
- *
- * Outputs: Offset - updated according to number of bytes written
- *
- * Returns: The number of bytes written, errno on failure
- */
-
-static ssize_t walk_vm(struct file *file, const char __user *buf, size_t size,
-              loff_t *offset, do_rw_t operation)
-{
-       struct gfs2_holder gh;
-
-       if (current->mm) {
-               struct super_block *sb = file->f_dentry->d_inode->i_sb;
-               struct mm_struct *mm = current->mm;
-               struct vm_area_struct *vma;
-               unsigned long start = (unsigned long)buf;
-               unsigned long end = start + size;
-               int dumping = (current->flags & PF_DUMPCORE);
-
-               if (!dumping)
-                       down_read(&mm->mmap_sem);
-
-               for (vma = find_vma(mm, start); vma; vma = vma->vm_next) {
-                       if (end <= vma->vm_start)
-                               break;
-                       if (vma->vm_file &&
-                           vma->vm_file->f_dentry->d_inode->i_sb == sb)
-                               goto do_locks;
-               }
-
-               if (!dumping)
-                       up_read(&mm->mmap_sem);
-       }
-
-       return operation(file, buf, size, offset, 0, &gh);
-
-do_locks:
-       return walk_vm_hard(file, buf, size, offset, operation);
-}
-
-static ssize_t do_jdata_read(struct file *file, char __user *buf, size_t size,
-                            loff_t *offset)
-{
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
-       ssize_t count = 0;
-
-       if (*offset < 0)
-               return -EINVAL;
-       if (!access_ok(VERIFY_WRITE, buf, size))
-               return -EFAULT;
-
-       if (!(file->f_flags & O_LARGEFILE)) {
-               if (*offset >= MAX_NON_LFS)
-                       return -EFBIG;
-               if (*offset + size > MAX_NON_LFS)
-                       size = MAX_NON_LFS - *offset;
-       }
-
-       count = gfs2_jdata_read(ip, buf, *offset, size, gfs2_copy2user);
-
-       if (count > 0)
-               *offset += count;
-
-       return count;
-}
-
-/**
- * do_read_direct - Read bytes from a file
- * @file: The file to read from
- * @buf: The buffer to copy into
- * @size: The amount of data requested
- * @offset: The current file offset
- * @num_gh: The number of other locks we need to do the read
- * @ghs: the locks we need plus one for our lock
- *
- * Outputs: Offset - updated according to number of bytes read
- *
- * Returns: The number of bytes read, errno on failure
- */
-
-static ssize_t do_read_direct(struct file *file, char __user *buf, size_t size,
-                             loff_t *offset, unsigned int num_gh,
-                             struct gfs2_holder *ghs)
-{
-       struct inode *inode = file->f_mapping->host;
-       struct gfs2_inode *ip = get_v2ip(inode);
-       unsigned int state = LM_ST_DEFERRED;
-       int flags = 0;
-       unsigned int x;
-       ssize_t count = 0;
-       int error;
-
-       for (x = 0; x < num_gh; x++)
-               if (ghs[x].gh_gl == ip->i_gl) {
-                       state = LM_ST_SHARED;
-                       flags |= GL_LOCAL_EXCL;
-                       break;
-               }
-
-       gfs2_holder_init(ip->i_gl, state, flags, &ghs[num_gh]);
-
-       error = gfs2_glock_nq_m(num_gh + 1, ghs);
-       if (error)
-               goto out;
-
-       error = -EINVAL;
-       if (gfs2_is_jdata(ip))
-               goto out_gunlock;
-
-       if (gfs2_is_stuffed(ip)) {
-               size_t mask = bdev_hardsect_size(inode->i_sb->s_bdev) - 1;
-
-               if (((*offset) & mask) || (((unsigned long)buf) & mask))
-                       goto out_gunlock;
-
-               count = do_jdata_read(file, buf, size & ~mask, offset);
-       } else
-               count = generic_file_read(file, buf, size, offset);
-
-       error = 0;
-
- out_gunlock:
-       gfs2_glock_dq_m(num_gh + 1, ghs);
-
- out:
-       gfs2_holder_uninit(&ghs[num_gh]);
-
-       return (count) ? count : error;
-}
-
-/**
- * do_read_buf - Read bytes from a file
- * @file: The file to read from
- * @buf: The buffer to copy into
- * @size: The amount of data requested
- * @offset: The current file offset
- * @num_gh: The number of other locks we need to do the read
- * @ghs: the locks we need plus one for our lock
- *
- * Outputs: Offset - updated according to number of bytes read
- *
- * Returns: The number of bytes read, errno on failure
- */
-
-static ssize_t do_read_buf(struct file *file, char __user *buf, size_t size,
-                          loff_t *offset, unsigned int num_gh,
-                          struct gfs2_holder *ghs)
-{
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
-       ssize_t count = 0;
-       int error;
-
-       gfs2_holder_init(ip->i_gl, LM_ST_SHARED, GL_ATIME, &ghs[num_gh]);
-
-       error = gfs2_glock_nq_m_atime(num_gh + 1, ghs);
-       if (error)
-               goto out;
-
-       if (gfs2_is_jdata(ip))
-               count = do_jdata_read(file, buf, size, offset);
-       else
-               count = generic_file_read(file, buf, size, offset);
-
-       gfs2_glock_dq_m(num_gh + 1, ghs);
-
- out:
-       gfs2_holder_uninit(&ghs[num_gh]);
-
-       return (count) ? count : error;
-}
-
-/**
- * gfs2_read - Read bytes from a file
- * @file: The file to read from
- * @buf: The buffer to copy into
- * @size: The amount of data requested
- * @offset: The current file offset
- *
- * Outputs: Offset - updated according to number of bytes read
- *
- * Returns: The number of bytes read, errno on failure
- */
-
-static ssize_t gfs2_read(struct file *file, char __user *buf, size_t size,
-                        loff_t *offset)
-{
-       atomic_inc(&get_v2sdp(file->f_mapping->host->i_sb)->sd_ops_file);
-
-       if (file->f_flags & O_DIRECT)
-               return walk_vm(file, buf, size, offset, do_read_direct);
-       else
-               return walk_vm(file, buf, size, offset, do_read_buf);
-}
-
-/**
- * grope_mapping - feel up a mapping that needs to be written
- * @buf: the start of the memory to be written
- * @size: the size of the memory to be written
- *
- * We do this after acquiring the locks on the mapping,
- * but before starting the write transaction.  We need to make
- * sure that we don't cause recursive transactions if blocks
- * need to be allocated to the file backing the mapping.
- *
- * Returns: errno
- */
-
-static int grope_mapping(const char __user *buf, size_t size)
-{
-       const char __user *stop = buf + size;
-       char c;
-
-       while (buf < stop) {
-               if (copy_from_user(&c, buf, 1))
-                       return -EFAULT;
-               buf += PAGE_CACHE_SIZE;
-               buf = (const char __user *)PAGE_ALIGN((unsigned long)buf);
-       }
-
-       return 0;
-}
-
-/**
- * do_write_direct_alloc - Write bytes to a file
- * @file: The file to write to
- * @buf: The buffer to copy from
- * @size: The amount of data requested
- * @offset: The current file offset
- *
- * Outputs: Offset - updated according to number of bytes written
- *
- * Returns: The number of bytes written, errno on failure
- */
-
-static ssize_t do_write_direct_alloc(struct file *file, const char __user *buf, size_t size,
-                                    loff_t *offset)
-{
-       struct inode *inode = file->f_mapping->host;
-       struct gfs2_inode *ip = get_v2ip(inode);
-       struct gfs2_sbd *sdp = ip->i_sbd;
-       struct gfs2_alloc *al = NULL;
-       struct iovec local_iov = { .iov_base = buf, .iov_len = size };
-       struct buffer_head *dibh;
-       unsigned int data_blocks, ind_blocks;
-       ssize_t count;
-       int error;
-
-       gfs2_write_calc_reserv(ip, size, &data_blocks, &ind_blocks);
-
-       al = gfs2_alloc_get(ip);
-
-       error = gfs2_quota_lock(ip, NO_QUOTA_CHANGE, NO_QUOTA_CHANGE);
-       if (error)
-               goto fail;
-
-       error = gfs2_quota_check(ip, ip->i_di.di_uid, ip->i_di.di_gid);
-       if (error)
-               goto fail_gunlock_q;
-
-       al->al_requested = data_blocks + ind_blocks;
-
-       error = gfs2_inplace_reserve(ip);
-       if (error)
-               goto fail_gunlock_q;
-
-       error = gfs2_trans_begin(sdp,
-                                al->al_rgd->rd_ri.ri_length + ind_blocks +
-                                RES_DINODE + RES_STATFS + RES_QUOTA, 0);
-       if (error)
-               goto fail_ipres;
-
-       if ((ip->i_di.di_mode & (S_ISUID | S_ISGID)) && !capable(CAP_FSETID)) {
-               error = gfs2_meta_inode_buffer(ip, &dibh);
-               if (error)
-                       goto fail_end_trans;
-
-               ip->i_di.di_mode &= (ip->i_di.di_mode & S_IXGRP) ?
-                       (~(S_ISUID | S_ISGID)) : (~S_ISUID);
-
-               gfs2_trans_add_bh(ip->i_gl, dibh, 1);
-               gfs2_dinode_out(&ip->i_di, dibh->b_data);
-               brelse(dibh);
-       }
-
-       if (gfs2_is_stuffed(ip)) {
-               error = gfs2_unstuff_dinode(ip, gfs2_unstuffer_sync, NULL);
-               if (error)
-                       goto fail_end_trans;
-       }
-
-       count = generic_file_write_nolock(file, &local_iov, 1, offset);
-       if (count < 0) {
-               error = count;
-               goto fail_end_trans;
-       }
-
-       error = gfs2_meta_inode_buffer(ip, &dibh);
-       if (error)
-               goto fail_end_trans;
-
-       if (ip->i_di.di_size < inode->i_size)
-               ip->i_di.di_size = inode->i_size;
-       ip->i_di.di_mtime = ip->i_di.di_ctime = get_seconds();
-
-       gfs2_trans_add_bh(ip->i_gl, dibh, 1);
-       gfs2_dinode_out(&ip->i_di, dibh->b_data);
-       brelse(dibh);
-
-       gfs2_trans_end(sdp);
-
-       if (file->f_flags & O_SYNC)
-               gfs2_log_flush_glock(ip->i_gl);
-
-       gfs2_inplace_release(ip);
-       gfs2_quota_unlock(ip);
-       gfs2_alloc_put(ip);
-
-       if (file->f_mapping->nrpages) {
-               error = filemap_fdatawrite(file->f_mapping);
-               if (!error)
-                       error = filemap_fdatawait(file->f_mapping);
-       }
-       if (error)
-               return error;
-
-       return count;
-
- fail_end_trans:
-       gfs2_trans_end(sdp);
-
- fail_ipres:
-       gfs2_inplace_release(ip);
-
- fail_gunlock_q:
-       gfs2_quota_unlock(ip);
-
- fail:
-       gfs2_alloc_put(ip);
-
-       return error;
-}
-
 /**
- * do_write_direct - Write bytes to a file
- * @file: The file to write to
- * @buf: The buffer to copy from
- * @size: The amount of data requested
- * @offset: The current file offset
- * @num_gh: The number of other locks we need to do the read
- * @gh: the locks we need plus one for our lock
- *
- * Outputs: Offset - updated according to number of bytes written
- *
- * Returns: The number of bytes written, errno on failure
- */
-
-static ssize_t do_write_direct(struct file *file, const char __user *buf, size_t size,
-                              loff_t *offset, unsigned int num_gh,
-                              struct gfs2_holder *ghs)
-{
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
-       struct gfs2_sbd *sdp = ip->i_sbd;
-       struct gfs2_file *fp = get_v2fp(file);
-       unsigned int state = LM_ST_DEFERRED;
-       int alloc_required;
-       unsigned int x;
-       size_t s;
-       ssize_t count = 0;
-       int error;
-
-       if (test_bit(GFF_DID_DIRECT_ALLOC, &fp->f_flags))
-               state = LM_ST_EXCLUSIVE;
-       else
-               for (x = 0; x < num_gh; x++)
-                       if (ghs[x].gh_gl == ip->i_gl) {
-                               state = LM_ST_EXCLUSIVE;
-                               break;
-                       }
-
- restart:
-       gfs2_holder_init(ip->i_gl, state, 0, &ghs[num_gh]);
-
-       error = gfs2_glock_nq_m(num_gh + 1, ghs);
-       if (error)
-               goto out;
-
-       error = -EINVAL;
-       if (gfs2_is_jdata(ip))
-               goto out_gunlock;
-
-       if (num_gh) {
-               error = grope_mapping(buf, size);
-               if (error)
-                       goto out_gunlock;
-       }
-
-       if (file->f_flags & O_APPEND)
-               *offset = ip->i_di.di_size;
-
-       if (!(file->f_flags & O_LARGEFILE)) {
-               error = -EFBIG;
-               if (*offset >= MAX_NON_LFS)
-                       goto out_gunlock;
-               if (*offset + size > MAX_NON_LFS)
-                       size = MAX_NON_LFS - *offset;
-       }
-
-       if (gfs2_is_stuffed(ip) ||
-           *offset + size > ip->i_di.di_size ||
-           ((ip->i_di.di_mode & (S_ISUID | S_ISGID)) && !capable(CAP_FSETID)))
-               alloc_required = 1;
-       else {
-               error = gfs2_write_alloc_required(ip, *offset, size,
-                                                &alloc_required);
-               if (error)
-                       goto out_gunlock;
-       }
-
-       if (alloc_required && state != LM_ST_EXCLUSIVE) {
-               gfs2_glock_dq_m(num_gh + 1, ghs);
-               gfs2_holder_uninit(&ghs[num_gh]);
-               state = LM_ST_EXCLUSIVE;
-               goto restart;
-       }
-
-       if (alloc_required) {
-               set_bit(GFF_DID_DIRECT_ALLOC, &fp->f_flags);
-
-               /* split large writes into smaller atomic transactions */
-               while (size) {
-                       s = gfs2_tune_get(sdp, gt_max_atomic_write);
-                       if (s > size)
-                               s = size;
-
-                       error = do_write_direct_alloc(file, buf, s, offset);
-                       if (error < 0)
-                               goto out_gunlock;
-
-                       buf += error;
-                       size -= error;
-                       count += error;
-               }
-       } else {
-               struct iovec local_iov = { .iov_base = buf, .iov_len = size };
-               struct gfs2_holder t_gh;
-
-               clear_bit(GFF_DID_DIRECT_ALLOC, &fp->f_flags);
-
-               error = gfs2_glock_nq_init(sdp->sd_trans_gl, LM_ST_SHARED,
-                                          GL_NEVER_RECURSE, &t_gh);
-               if (error)
-                       goto out_gunlock;
-
-               count = generic_file_write_nolock(file, &local_iov, 1, offset);
-
-               gfs2_glock_dq_uninit(&t_gh);
-       }
-
-       error = 0;
-
- out_gunlock:
-       gfs2_glock_dq_m(num_gh + 1, ghs);
-
- out:
-       gfs2_holder_uninit(&ghs[num_gh]);
-
-       return (count) ? count : error;
-}
-
-/**
- * do_do_write_buf - Write bytes to a file
- * @file: The file to write to
- * @buf: The buffer to copy from
- * @size: The amount of data requested
- * @offset: The current file offset
- *
- * Outputs: Offset - updated according to number of bytes written
- *
- * Returns: The number of bytes written, errno on failure
- */
-
-static ssize_t do_do_write_buf(struct file *file, const char __user *buf, size_t size,
-                              loff_t *offset)
-{
-       struct inode *inode = file->f_mapping->host;
-       struct gfs2_inode *ip = get_v2ip(inode);
-       struct gfs2_sbd *sdp = ip->i_sbd;
-       struct gfs2_alloc *al = NULL;
-       struct buffer_head *dibh;
-       unsigned int data_blocks, ind_blocks;
-       int alloc_required, journaled;
-       ssize_t count;
-       int error;
-
-       journaled = gfs2_is_jdata(ip);
-
-       gfs2_write_calc_reserv(ip, size, &data_blocks, &ind_blocks);
-
-       error = gfs2_write_alloc_required(ip, *offset, size, &alloc_required);
-       if (error)
-               return error;
-
-       if (alloc_required) {
-               al = gfs2_alloc_get(ip);
-
-               error = gfs2_quota_lock(ip, NO_QUOTA_CHANGE, NO_QUOTA_CHANGE);
-               if (error)
-                       goto fail;
-
-               error = gfs2_quota_check(ip, ip->i_di.di_uid, ip->i_di.di_gid);
-               if (error)
-                       goto fail_gunlock_q;
-
-               al->al_requested = data_blocks + ind_blocks;
-
-               error = gfs2_inplace_reserve(ip);
-               if (error)
-                       goto fail_gunlock_q;
-
-               error = gfs2_trans_begin(sdp,
-                                        al->al_rgd->rd_ri.ri_length +
-                                        ind_blocks +
-                                        ((journaled) ? data_blocks : 0) +
-                                        RES_DINODE + RES_STATFS + RES_QUOTA,
-                                        0);
-               if (error)
-                       goto fail_ipres;
-       } else {
-               error = gfs2_trans_begin(sdp,
-                                       ((journaled) ? data_blocks : 0) +
-                                       RES_DINODE,
-                                       0);
-               if (error)
-                       goto fail_ipres;
-       }
-
-       if ((ip->i_di.di_mode & (S_ISUID | S_ISGID)) && !capable(CAP_FSETID)) {
-               error = gfs2_meta_inode_buffer(ip, &dibh);
-               if (error)
-                       goto fail_end_trans;
-
-               ip->i_di.di_mode &= (ip->i_di.di_mode & S_IXGRP) ?
-                                         (~(S_ISUID | S_ISGID)) : (~S_ISUID);
-
-               gfs2_trans_add_bh(ip->i_gl, dibh, 1);
-               gfs2_dinode_out(&ip->i_di, dibh->b_data);
-               brelse(dibh);
-       }
-
-       if (journaled) {
-               count = gfs2_jdata_write(ip, buf, *offset, size,
-                                        gfs2_copy_from_user);
-               if (count < 0) {
-                       error = count;
-                       goto fail_end_trans;
-               }
-
-               *offset += count;
-       } else {
-               struct iovec local_iov = { .iov_base = buf, .iov_len = size };
-
-               count = generic_file_write_nolock(file, &local_iov, 1, offset);
-               if (count < 0) {
-                       error = count;
-                       goto fail_end_trans;
-               }
-
-               error = gfs2_meta_inode_buffer(ip, &dibh);
-               if (error)
-                       goto fail_end_trans;
-
-               if (ip->i_di.di_size < inode->i_size)
-                       ip->i_di.di_size = inode->i_size;
-               ip->i_di.di_mtime = ip->i_di.di_ctime = get_seconds();
-
-               gfs2_trans_add_bh(ip->i_gl, dibh, 1);
-               gfs2_dinode_out(&ip->i_di, dibh->b_data);
-               brelse(dibh);
-       }
-
-       gfs2_trans_end(sdp);
-
-       if (file->f_flags & O_SYNC || IS_SYNC(inode)) {
-               gfs2_log_flush_glock(ip->i_gl);
-               error = filemap_fdatawrite(file->f_mapping);
-               if (error == 0)
-                       error = filemap_fdatawait(file->f_mapping);
-               if (error)
-                       goto fail_ipres;
-       }
-
-       if (alloc_required) {
-               gfs2_assert_warn(sdp, count != size ||
-                                al->al_alloced);
-               gfs2_inplace_release(ip);
-               gfs2_quota_unlock(ip);
-               gfs2_alloc_put(ip);
-       }
-
-       return count;
-
- fail_end_trans:
-       gfs2_trans_end(sdp);
-
- fail_ipres:
-       if (alloc_required)
-               gfs2_inplace_release(ip);
-
- fail_gunlock_q:
-       if (alloc_required)
-               gfs2_quota_unlock(ip);
-
- fail:
-       if (alloc_required)
-               gfs2_alloc_put(ip);
-
-       return error;
-}
-
-/**
- * do_write_buf - Write bytes to a file
- * @file: The file to write to
- * @buf: The buffer to copy from
- * @size: The amount of data requested
- * @offset: The current file offset
- * @num_gh: The number of other locks we need to do the read
- * @gh: the locks we need plus one for our lock
- *
- * Outputs: Offset - updated according to number of bytes written
- *
- * Returns: The number of bytes written, errno on failure
- */
-
-static ssize_t do_write_buf(struct file *file, const char __user *buf, size_t size,
-                           loff_t *offset, unsigned int num_gh,
-                           struct gfs2_holder *ghs)
-{
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
-       struct gfs2_sbd *sdp = ip->i_sbd;
-       size_t s;
-       ssize_t count = 0;
-       int error;
-
-       gfs2_holder_init(ip->i_gl, LM_ST_EXCLUSIVE, 0, &ghs[num_gh]);
-
-       error = gfs2_glock_nq_m(num_gh + 1, ghs);
-       if (error)
-               goto out;
-
-       if (num_gh) {
-               error = grope_mapping(buf, size);
-               if (error)
-                       goto out_gunlock;
-       }
-
-       if (file->f_flags & O_APPEND)
-               *offset = ip->i_di.di_size;
-
-       if (!(file->f_flags & O_LARGEFILE)) {
-               error = -EFBIG;
-               if (*offset >= MAX_NON_LFS)
-                       goto out_gunlock;
-               if (*offset + size > MAX_NON_LFS)
-                       size = MAX_NON_LFS - *offset;
-       }
-
-       /* split large writes into smaller atomic transactions */
-       while (size) {
-               s = gfs2_tune_get(sdp, gt_max_atomic_write);
-               if (s > size)
-                       s = size;
-
-               error = do_do_write_buf(file, buf, s, offset);
-               if (error < 0)
-                       goto out_gunlock;
-
-               buf += error;
-               size -= error;
-               count += error;
-       }
-
-       error = 0;
-
- out_gunlock:
-       gfs2_glock_dq_m(num_gh + 1, ghs);
-
- out:
-       gfs2_holder_uninit(&ghs[num_gh]);
-
-       return (count) ? count : error;
-}
-
-/**
- * gfs2_write - Write bytes to a file
- * @file: The file to write to
- * @buf: The buffer to copy from
- * @size: The amount of data requested
- * @offset: The current file offset
- *
- * Outputs: Offset - updated according to number of bytes written
- *
- * Returns: The number of bytes written, errno on failure
- */
-
-static ssize_t gfs2_write(struct file *file, const char __user *buf,
-                         size_t size, loff_t *offset)
-{
-       struct inode *inode = file->f_mapping->host;
-       ssize_t count;
-
-       atomic_inc(&get_v2sdp(inode->i_sb)->sd_ops_file);
-
-       if (*offset < 0)
-               return -EINVAL;
-       if (!access_ok(VERIFY_READ, buf, size))
-               return -EFAULT;
-
-       mutex_lock(&inode->i_mutex);
-       if (file->f_flags & O_DIRECT)
-               count = walk_vm(file, buf, size, offset,
-                               do_write_direct);
-       else
-               count = walk_vm(file, buf, size, offset, do_write_buf);
-       mutex_unlock(&inode->i_mutex);
-
-       return count;
-}
-
-/**
- * filldir_reg_func - Report a directory entry to the caller of gfs2_dir_read()
- * @opaque: opaque data used by the function
- * @name: the name of the directory entry
- * @length: the length of the name
- * @offset: the entry's offset in the directory
- * @inum: the inode number the entry points to
- * @type: the type of inode the entry points to
- *
- * Returns: 0 on success, 1 if buffer full
- */
-
-static int filldir_reg_func(void *opaque, const char *name, unsigned int length,
-                           uint64_t offset, struct gfs2_inum *inum,
-                           unsigned int type)
-{
-       struct filldir_reg *fdr = (struct filldir_reg *)opaque;
-       struct gfs2_sbd *sdp = fdr->fdr_sbd;
-       int error;
-
-       error = fdr->fdr_filldir(fdr->fdr_opaque, name, length, offset,
-                                inum->no_formal_ino, type);
-       if (error)
-               return 1;
-
-       if (fdr->fdr_prefetch && !(length == 1 && *name == '.')) {
-               gfs2_glock_prefetch_num(sdp,
-                                      inum->no_addr, &gfs2_inode_glops,
-                                      LM_ST_SHARED, LM_FLAG_TRY | LM_FLAG_ANY);
-               gfs2_glock_prefetch_num(sdp,
-                                      inum->no_addr, &gfs2_iopen_glops,
-                                      LM_ST_SHARED, LM_FLAG_TRY);
-       }
-
-       return 0;
-}
-
-/**
- * readdir_reg - Read directory entries from a directory
+ * gfs2_readdir - Read directory entries from a directory
  * @file: The directory to read from
  * @dirent: Buffer for dirents
  * @filldir: Function used to do the copying
@@ -951,19 +125,14 @@ static int filldir_reg_func(void *opaque, const char *name, unsigned int length,
  * Returns: errno
  */
 
-static int readdir_reg(struct file *file, void *dirent, filldir_t filldir)
+static int gfs2_readdir(struct file *file, void *dirent, filldir_t filldir)
 {
-       struct gfs2_inode *dip = get_v2ip(file->f_mapping->host);
-       struct filldir_reg fdr;
+       struct inode *dir = file->f_mapping->host;
+       struct gfs2_inode *dip = GFS2_I(dir);
        struct gfs2_holder d_gh;
-       uint64_t offset = file->f_pos;
+       u64 offset = file->f_pos;
        int error;
 
-       fdr.fdr_sbd = dip->i_sbd;
-       fdr.fdr_prefetch = 1;
-       fdr.fdr_filldir = filldir;
-       fdr.fdr_opaque = dirent;
-
        gfs2_holder_init(dip->i_gl, LM_ST_SHARED, GL_ATIME, &d_gh);
        error = gfs2_glock_nq_atime(&d_gh);
        if (error) {
@@ -971,7 +140,7 @@ static int readdir_reg(struct file *file, void *dirent, filldir_t filldir)
                return error;
        }
 
-       error = gfs2_dir_read(dip, &offset, &fdr, filldir_reg_func);
+       error = gfs2_dir_read(dir, &offset, dirent, filldir);
 
        gfs2_glock_dq_uninit(&d_gh);
 
@@ -981,248 +150,190 @@ static int readdir_reg(struct file *file, void *dirent, filldir_t filldir)
 }
 
 /**
- * filldir_bad_func - Report a directory entry to the caller of gfs2_dir_read()
- * @opaque: opaque data used by the function
- * @name: the name of the directory entry
- * @length: the length of the name
- * @offset: the entry's offset in the directory
- * @inum: the inode number the entry points to
- * @type: the type of inode the entry points to
+ * fsflags_cvt
+ * @table: A table of 32 u32 flags
+ * @val: a 32 bit value to convert
  *
- * For supporting NFS.
+ * This function can be used to convert between fsflags values and
+ * GFS2's own flags values.
  *
- * Returns: 0 on success, 1 if buffer full
+ * Returns: the converted flags
  */
-
-static int filldir_bad_func(void *opaque, const char *name, unsigned int length,
-                           uint64_t offset, struct gfs2_inum *inum,
-                           unsigned int type)
+static u32 fsflags_cvt(const u32 *table, u32 val)
 {
-       struct filldir_bad *fdb = (struct filldir_bad *)opaque;
-       struct gfs2_sbd *sdp = fdb->fdb_sbd;
-       struct filldir_bad_entry *fbe;
-
-       if (fdb->fdb_entry_off == fdb->fdb_entry_num ||
-           fdb->fdb_name_off + length > fdb->fdb_name_size)
-               return 1;
-
-       fbe = &fdb->fdb_entry[fdb->fdb_entry_off];
-       fbe->fbe_name = fdb->fdb_name + fdb->fdb_name_off;
-       memcpy(fbe->fbe_name, name, length);
-       fbe->fbe_length = length;
-       fbe->fbe_offset = offset;
-       fbe->fbe_inum = *inum;
-       fbe->fbe_type = type;
-
-       fdb->fdb_entry_off++;
-       fdb->fdb_name_off += length;
-
-       if (!(length == 1 && *name == '.')) {
-               gfs2_glock_prefetch_num(sdp,
-                                      inum->no_addr, &gfs2_inode_glops,
-                                      LM_ST_SHARED, LM_FLAG_TRY | LM_FLAG_ANY);
-               gfs2_glock_prefetch_num(sdp,
-                                      inum->no_addr, &gfs2_iopen_glops,
-                                      LM_ST_SHARED, LM_FLAG_TRY);
-       }
-
-       return 0;
+       u32 res = 0;
+       while(val) {
+               if (val & 1)
+                       res |= *table;
+               table++;
+               val >>= 1;
+       }
+       return res;
 }
 
-/**
- * readdir_bad - Read directory entries from a directory
- * @file: The directory to read from
- * @dirent: Buffer for dirents
- * @filldir: Function used to do the copying
- *
- * For supporting NFS.
- *
- * Returns: errno
- */
+static const u32 fsflags_to_gfs2[32] = {
+       [3] = GFS2_DIF_SYNC,
+       [4] = GFS2_DIF_IMMUTABLE,
+       [5] = GFS2_DIF_APPENDONLY,
+       [7] = GFS2_DIF_NOATIME,
+       [12] = GFS2_DIF_EXHASH,
+       [14] = GFS2_DIF_JDATA,
+       [20] = GFS2_DIF_DIRECTIO,
+};
 
-static int readdir_bad(struct file *file, void *dirent, filldir_t filldir)
+static const u32 gfs2_to_fsflags[32] = {
+       [gfs2fl_Sync] = FS_SYNC_FL,
+       [gfs2fl_Immutable] = FS_IMMUTABLE_FL,
+       [gfs2fl_AppendOnly] = FS_APPEND_FL,
+       [gfs2fl_NoAtime] = FS_NOATIME_FL,
+       [gfs2fl_ExHash] = FS_INDEX_FL,
+       [gfs2fl_Jdata] = FS_JOURNAL_DATA_FL,
+       [gfs2fl_Directio] = FS_DIRECTIO_FL,
+       [gfs2fl_InheritDirectio] = FS_DIRECTIO_FL,
+       [gfs2fl_InheritJdata] = FS_JOURNAL_DATA_FL,
+};
+
+static int gfs2_get_flags(struct file *filp, u32 __user *ptr)
 {
-       struct gfs2_inode *dip = get_v2ip(file->f_mapping->host);
-       struct gfs2_sbd *sdp = dip->i_sbd;
-       struct filldir_reg fdr;
-       unsigned int entries, size;
-       struct filldir_bad *fdb;
-       struct gfs2_holder d_gh;
-       uint64_t offset = file->f_pos;
-       unsigned int x;
-       struct filldir_bad_entry *fbe;
+       struct inode *inode = filp->f_path.dentry->d_inode;
+       struct gfs2_inode *ip = GFS2_I(inode);
+       struct gfs2_holder gh;
        int error;
+       u32 fsflags;
 
-       entries = gfs2_tune_get(sdp, gt_entries_per_readdir);
-       size = sizeof(struct filldir_bad) +
-           entries * (sizeof(struct filldir_bad_entry) + GFS2_FAST_NAME_SIZE);
-
-       fdb = kzalloc(size, GFP_KERNEL);
-       if (!fdb)
-               return -ENOMEM;
-
-       fdb->fdb_sbd = sdp;
-       fdb->fdb_entry = (struct filldir_bad_entry *)(fdb + 1);
-       fdb->fdb_entry_num = entries;
-       fdb->fdb_name = ((char *)fdb) + sizeof(struct filldir_bad) +
-               entries * sizeof(struct filldir_bad_entry);
-       fdb->fdb_name_size = entries * GFS2_FAST_NAME_SIZE;
-
-       gfs2_holder_init(dip->i_gl, LM_ST_SHARED, GL_ATIME, &d_gh);
-       error = gfs2_glock_nq_atime(&d_gh);
-       if (error) {
-               gfs2_holder_uninit(&d_gh);
-               goto out;
-       }
-
-       error = gfs2_dir_read(dip, &offset, fdb, filldir_bad_func);
-
-       gfs2_glock_dq_uninit(&d_gh);
-
-       fdr.fdr_sbd = sdp;
-       fdr.fdr_prefetch = 0;
-       fdr.fdr_filldir = filldir;
-       fdr.fdr_opaque = dirent;
-
-       for (x = 0; x < fdb->fdb_entry_off; x++) {
-               fbe = &fdb->fdb_entry[x];
-
-               error = filldir_reg_func(&fdr,
-                                        fbe->fbe_name, fbe->fbe_length,
-                                        fbe->fbe_offset,
-                                        &fbe->fbe_inum, fbe->fbe_type);
-               if (error) {
-                       file->f_pos = fbe->fbe_offset;
-                       error = 0;
-                       goto out;
-               }
-       }
-
-       file->f_pos = offset;
+       gfs2_holder_init(ip->i_gl, LM_ST_SHARED, GL_ATIME, &gh);
+       error = gfs2_glock_nq_atime(&gh);
+       if (error)
+               return error;
 
- out:
-       kfree(fdb);
+       fsflags = fsflags_cvt(gfs2_to_fsflags, ip->i_di.di_flags);
+       if (put_user(fsflags, ptr))
+               error = -EFAULT;
 
+       gfs2_glock_dq_m(1, &gh);
+       gfs2_holder_uninit(&gh);
        return error;
 }
 
+void gfs2_set_inode_flags(struct inode *inode)
+{
+       struct gfs2_inode *ip = GFS2_I(inode);
+       struct gfs2_dinode_host *di = &ip->i_di;
+       unsigned int flags = inode->i_flags;
+
+       flags &= ~(S_SYNC|S_APPEND|S_IMMUTABLE|S_NOATIME|S_DIRSYNC);
+       if (di->di_flags & GFS2_DIF_IMMUTABLE)
+               flags |= S_IMMUTABLE;
+       if (di->di_flags & GFS2_DIF_APPENDONLY)
+               flags |= S_APPEND;
+       if (di->di_flags & GFS2_DIF_NOATIME)
+               flags |= S_NOATIME;
+       if (di->di_flags & GFS2_DIF_SYNC)
+               flags |= S_SYNC;
+       inode->i_flags = flags;
+}
+
+/* Flags that can be set by user space */
+#define GFS2_FLAGS_USER_SET (GFS2_DIF_JDATA|                   \
+                            GFS2_DIF_DIRECTIO|                 \
+                            GFS2_DIF_IMMUTABLE|                \
+                            GFS2_DIF_APPENDONLY|               \
+                            GFS2_DIF_NOATIME|                  \
+                            GFS2_DIF_SYNC|                     \
+                            GFS2_DIF_SYSTEM|                   \
+                            GFS2_DIF_INHERIT_DIRECTIO|         \
+                            GFS2_DIF_INHERIT_JDATA)
+
 /**
- * gfs2_readdir - Read directory entries from a directory
- * @file: The directory to read from
- * @dirent: Buffer for dirents
- * @filldir: Function used to do the copying
+ * gfs2_set_flags - set flags on an inode
+ * @inode: The inode
+ * @flags: The flags to set
+ * @mask: Indicates which flags are valid
  *
- * Returns: errno
  */
-
-static int gfs2_readdir(struct file *file, void *dirent, filldir_t filldir)
+static int do_gfs2_set_flags(struct file *filp, u32 reqflags, u32 mask)
 {
+       struct inode *inode = filp->f_path.dentry->d_inode;
+       struct gfs2_inode *ip = GFS2_I(inode);
+       struct gfs2_sbd *sdp = GFS2_SB(inode);
+       struct buffer_head *bh;
+       struct gfs2_holder gh;
        int error;
+       u32 new_flags, flags;
 
-       atomic_inc(&get_v2sdp(file->f_mapping->host->i_sb)->sd_ops_file);
-
-       if (strcmp(current->comm, "nfsd") != 0)
-               error = readdir_reg(file, dirent, filldir);
-       else
-               error = readdir_bad(file, dirent, filldir);
-
-       return error;
-}
+       error = gfs2_glock_nq_init(ip->i_gl, LM_ST_EXCLUSIVE, 0, &gh);
+       if (error)
+               return error;
 
-static int gfs2_ioctl_flags(struct gfs2_inode *ip, unsigned int cmd, unsigned long arg)
-{
-       unsigned int lmode = (cmd == GFS2_IOCTL_SETFLAGS) ? LM_ST_EXCLUSIVE : LM_ST_SHARED;
-       struct buffer_head *dibh;
-       struct gfs2_holder i_gh;
-       int error;
-       __u32 flags = 0, change;
+       flags = ip->i_di.di_flags;
+       new_flags = (flags & ~mask) | (reqflags & mask);
+       if ((new_flags ^ flags) == 0)
+               goto out;
 
-       if (cmd == GFS2_IOCTL_SETFLAGS) {
-               error = get_user(flags, (__u32 __user *)arg);
-               if (error)
-                       return -EFAULT;
+       if (S_ISDIR(inode->i_mode)) {
+               if ((new_flags ^ flags) & GFS2_DIF_JDATA)
+                       new_flags ^= (GFS2_DIF_JDATA|GFS2_DIF_INHERIT_JDATA);
+               if ((new_flags ^ flags) & GFS2_DIF_DIRECTIO)
+                       new_flags ^= (GFS2_DIF_DIRECTIO|GFS2_DIF_INHERIT_DIRECTIO);
        }
 
-       error = gfs2_glock_nq_init(ip->i_gl, lmode, 0, &i_gh);
-       if (error)
-               return error;
-
-       if (cmd == GFS2_IOCTL_SETFLAGS) {
-               change = flags ^ ip->i_di.di_flags;
-               error = -EPERM;
-               if (change & (GFS2_DIF_IMMUTABLE|GFS2_DIF_APPENDONLY)) {
-                       if (!capable(CAP_LINUX_IMMUTABLE))
-                               goto out;
-               }
-               error = -EINVAL;
-               if (flags & (GFS2_DIF_JDATA|GFS2_DIF_DIRECTIO)) {
-                       if (!S_ISREG(ip->i_di.di_mode))
-                               goto out;
-                       /* FIXME: Would be nice not to require the following test */
-                       if ((flags & GFS2_DIF_JDATA) && ip->i_di.di_size)
-                               goto out;
-               }
-               if (flags & (GFS2_DIF_INHERIT_JDATA|GFS2_DIF_INHERIT_DIRECTIO)) {
-                       if (!S_ISDIR(ip->i_di.di_mode))
-                               goto out;
-               }
+       error = -EINVAL;
+       if ((new_flags ^ flags) & ~GFS2_FLAGS_USER_SET)
+               goto out;
 
-               error = gfs2_trans_begin(ip->i_sbd, RES_DINODE, 0);
+       error = -EPERM;
+       if (IS_IMMUTABLE(inode) && (new_flags & GFS2_DIF_IMMUTABLE))
+               goto out;
+       if (IS_APPEND(inode) && (new_flags & GFS2_DIF_APPENDONLY))
+               goto out;
+       if (((new_flags ^ flags) & GFS2_DIF_IMMUTABLE) &&
+           !capable(CAP_LINUX_IMMUTABLE))
+               goto out;
+       if (!IS_IMMUTABLE(inode)) {
+               error = permission(inode, MAY_WRITE, NULL);
                if (error)
                        goto out;
+       }
 
-               error = gfs2_meta_inode_buffer(ip, &dibh);
-               if (error)
-                       goto out_trans_end;
-
-               ip->i_di.di_flags = flags;
-
-               gfs2_trans_add_bh(ip->i_gl, dibh, 1);
-               gfs2_dinode_out(&ip->i_di, dibh->b_data);
-
-               brelse(dibh);
-
+       error = gfs2_trans_begin(sdp, RES_DINODE, 0);
+       if (error)
+               goto out;
+       error = gfs2_meta_inode_buffer(ip, &bh);
+       if (error)
+               goto out_trans_end;
+       gfs2_trans_add_bh(ip->i_gl, bh, 1);
+       ip->i_di.di_flags = new_flags;
+       gfs2_dinode_out(ip, bh->b_data);
+       brelse(bh);
+       gfs2_set_inode_flags(inode);
 out_trans_end:
-               gfs2_trans_end(ip->i_sbd);
-       } else {
-               flags = ip->i_di.di_flags;
-       }
+       gfs2_trans_end(sdp);
 out:
-       gfs2_glock_dq_uninit(&i_gh);
-       if (cmd == GFS2_IOCTL_GETFLAGS) {
-               if (put_user(flags, (__u32 __user *)arg))
-                       return -EFAULT;
-       }
+       gfs2_glock_dq_uninit(&gh);
        return error;
 }
 
-/**
- * gfs2_ioctl - do an ioctl on a file
- * @inode: the inode
- * @file: the file pointer
- * @cmd: the ioctl command
- * @arg: the argument
- *
- * Returns: errno
- */
-
-static int gfs2_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
-                     unsigned long arg)
+static int gfs2_set_flags(struct file *filp, u32 __user *ptr)
 {
-       struct gfs2_inode *ip = get_v2ip(inode);
-
-       atomic_inc(&ip->i_sbd->sd_ops_file);
-
-       switch (cmd) {
-       case GFS2_IOCTL_SETFLAGS:
-       case GFS2_IOCTL_GETFLAGS:
-               return gfs2_ioctl_flags(ip, cmd, arg);
+       u32 fsflags, gfsflags;
+       if (get_user(fsflags, ptr))
+               return -EFAULT;
+       gfsflags = fsflags_cvt(fsflags_to_gfs2, fsflags);
+       return do_gfs2_set_flags(filp, gfsflags, ~0);
+}
 
-       default:
-               return -ENOTTY;
+static long gfs2_ioctl(struct file *filp, unsigned int cmd, unsigned long arg)
+{
+       switch(cmd) {
+       case FS_IOC_GETFLAGS:
+               return gfs2_get_flags(filp, (u32 __user *)arg);
+       case FS_IOC_SETFLAGS:
+               return gfs2_set_flags(filp, (u32 __user *)arg);
        }
+       return -ENOTTY;
 }
 
+
 /**
  * gfs2_mmap -
  * @file: The file to map
@@ -1233,12 +344,10 @@ static int gfs2_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
 
 static int gfs2_mmap(struct file *file, struct vm_area_struct *vma)
 {
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
+       struct gfs2_inode *ip = GFS2_I(file->f_mapping->host);
        struct gfs2_holder i_gh;
        int error;
 
-       atomic_inc(&ip->i_sbd->sd_ops_file);
-
        gfs2_holder_init(ip->i_gl, LM_ST_SHARED, GL_ATIME, &i_gh);
        error = gfs2_glock_nq_atime(&i_gh);
        if (error) {
@@ -1246,21 +355,14 @@ static int gfs2_mmap(struct file *file, struct vm_area_struct *vma)
                return error;
        }
 
-       if (gfs2_is_jdata(ip)) {
-               if (vma->vm_flags & VM_MAYSHARE)
-                       error = -EOPNOTSUPP;
-               else
-                       vma->vm_ops = &gfs2_vm_ops_private;
-       } else {
-               /* This is VM_MAYWRITE instead of VM_WRITE because a call
-                  to mprotect() can turn on VM_WRITE later. */
-
-               if ((vma->vm_flags & (VM_MAYSHARE | VM_MAYWRITE)) ==
-                   (VM_MAYSHARE | VM_MAYWRITE))
-                       vma->vm_ops = &gfs2_vm_ops_sharewrite;
-               else
-                       vma->vm_ops = &gfs2_vm_ops_private;
-       }
+       /* This is VM_MAYWRITE instead of VM_WRITE because a call
+          to mprotect() can turn on VM_WRITE later. */
+
+       if ((vma->vm_flags & (VM_MAYSHARE | VM_MAYWRITE)) ==
+           (VM_MAYSHARE | VM_MAYWRITE))
+               vma->vm_ops = &gfs2_vm_ops_sharewrite;
+       else
+               vma->vm_ops = &gfs2_vm_ops_private;
 
        gfs2_glock_dq_uninit(&i_gh);
 
@@ -1277,26 +379,21 @@ static int gfs2_mmap(struct file *file, struct vm_area_struct *vma)
 
 static int gfs2_open(struct inode *inode, struct file *file)
 {
-       struct gfs2_inode *ip = get_v2ip(inode);
+       struct gfs2_inode *ip = GFS2_I(inode);
        struct gfs2_holder i_gh;
        struct gfs2_file *fp;
        int error;
 
-       atomic_inc(&ip->i_sbd->sd_ops_file);
-
        fp = kzalloc(sizeof(struct gfs2_file), GFP_KERNEL);
        if (!fp)
                return -ENOMEM;
 
-       init_MUTEX(&fp->f_fl_mutex);
-
-       fp->f_inode = ip;
-       fp->f_vfile = file;
+       mutex_init(&fp->f_fl_mutex);
 
-       gfs2_assert_warn(ip->i_sbd, !get_v2fp(file));
-       set_v2fp(file, fp);
+       gfs2_assert_warn(GFS2_SB(inode), !file->private_data);
+       file->private_data = fp;
 
-       if (S_ISREG(ip->i_di.di_mode)) {
+       if (S_ISREG(ip->i_inode.i_mode)) {
                error = gfs2_glock_nq_init(ip->i_gl, LM_ST_SHARED, LM_FLAG_ANY,
                                           &i_gh);
                if (error)
@@ -1313,25 +410,16 @@ static int gfs2_open(struct inode *inode, struct file *file)
                if (ip->i_di.di_flags & GFS2_DIF_DIRECTIO)
                        file->f_flags |= O_DIRECT;
 
-               /* Don't let the user open O_DIRECT on a jdata file */
-
-               if ((file->f_flags & O_DIRECT) && gfs2_is_jdata(ip)) {
-                       error = -EINVAL;
-                       goto fail_gunlock;
-               }
-
                gfs2_glock_dq_uninit(&i_gh);
        }
 
        return 0;
 
- fail_gunlock:
+fail_gunlock:
        gfs2_glock_dq_uninit(&i_gh);
-
- fail:
-       set_v2fp(file, NULL);
+fail:
+       file->private_data = NULL;
        kfree(fp);
-
        return error;
 }
 
@@ -1345,13 +433,11 @@ static int gfs2_open(struct inode *inode, struct file *file)
 
 static int gfs2_close(struct inode *inode, struct file *file)
 {
-       struct gfs2_sbd *sdp = get_v2sdp(inode->i_sb);
+       struct gfs2_sbd *sdp = inode->i_sb->s_fs_info;
        struct gfs2_file *fp;
 
-       atomic_inc(&sdp->sd_ops_file);
-
-       fp = get_v2fp(file);
-       set_v2fp(file, NULL);
+       fp = file->private_data;
+       file->private_data = NULL;
 
        if (gfs2_assert_warn(sdp, fp))
                return -EIO;
@@ -1366,17 +452,63 @@ static int gfs2_close(struct inode *inode, struct file *file)
  * @file: the file that points to the dentry (we ignore this)
  * @dentry: the dentry that points to the inode to sync
  *
+ * The VFS will flush "normal" data for us. We only need to worry
+ * about metadata here. For journaled data, we just do a log flush
+ * as we can't avoid it. Otherwise we can just bale out if datasync
+ * is set. For stuffed inodes we must flush the log in order to
+ * ensure that all data is on disk.
+ *
+ * The call to write_inode_now() is there to write back metadata and
+ * the inode itself. It does also try and write the data, but thats
+ * (hopefully) a no-op due to the VFS having already called filemap_fdatawrite()
+ * for us.
+ *
  * Returns: errno
  */
 
 static int gfs2_fsync(struct file *file, struct dentry *dentry, int datasync)
 {
-       struct gfs2_inode *ip = get_v2ip(dentry->d_inode);
+       struct inode *inode = dentry->d_inode;
+       int sync_state = inode->i_state & (I_DIRTY_SYNC|I_DIRTY_DATASYNC);
+       int ret = 0;
 
-       atomic_inc(&ip->i_sbd->sd_ops_file);
-       gfs2_log_flush_glock(ip->i_gl);
+       if (gfs2_is_jdata(GFS2_I(inode))) {
+               gfs2_log_flush(GFS2_SB(inode), GFS2_I(inode)->i_gl);
+               return 0;
+       }
 
-       return 0;
+       if (sync_state != 0) {
+               if (!datasync)
+                       ret = write_inode_now(inode, 0);
+
+               if (gfs2_is_stuffed(GFS2_I(inode)))
+                       gfs2_log_flush(GFS2_SB(inode), GFS2_I(inode)->i_gl);
+       }
+
+       return ret;
+}
+
+/**
+ * gfs2_setlease - acquire/release a file lease
+ * @file: the file pointer
+ * @arg: lease type
+ * @fl: file lock
+ *
+ * Returns: errno
+ */
+
+static int gfs2_setlease(struct file *file, long arg, struct file_lock **fl)
+{
+       struct gfs2_sbd *sdp = GFS2_SB(file->f_mapping->host);
+
+       /*
+        * We don't currently have a way to enforce a lease across the whole
+        * cluster; until we do, disable leases (by just returning -EINVAL),
+        * unless the administrator has requested purely local locking.
+        */
+       if (!sdp->sd_args.ar_localflocks)
+               return -EINVAL;
+       return setlease(file, arg, fl);
 }
 
 /**
@@ -1390,38 +522,31 @@ static int gfs2_fsync(struct file *file, struct dentry *dentry, int datasync)
 
 static int gfs2_lock(struct file *file, int cmd, struct file_lock *fl)
 {
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
-       struct gfs2_sbd *sdp = ip->i_sbd;
+       struct gfs2_inode *ip = GFS2_I(file->f_mapping->host);
+       struct gfs2_sbd *sdp = GFS2_SB(file->f_mapping->host);
        struct lm_lockname name =
-               { .ln_number = ip->i_num.no_addr,
+               { .ln_number = ip->i_no_addr,
                  .ln_type = LM_TYPE_PLOCK };
 
-       atomic_inc(&sdp->sd_ops_file);
-
        if (!(fl->fl_flags & FL_POSIX))
                return -ENOLCK;
-       if ((ip->i_di.di_mode & (S_ISGID | S_IXGRP)) == S_ISGID)
+       if ((ip->i_inode.i_mode & (S_ISGID | S_IXGRP)) == S_ISGID)
                return -ENOLCK;
 
        if (sdp->sd_args.ar_localflocks) {
                if (IS_GETLK(cmd)) {
-                       struct file_lock *tmp;
-                       lock_kernel();
-                       tmp = posix_test_lock(file, fl);
-                       fl->fl_type = F_UNLCK;
-                       if (tmp)
-                               memcpy(fl, tmp, sizeof(struct file_lock));
-                       unlock_kernel();
+                       posix_test_lock(file, fl);
                        return 0;
                } else {
-                       int error;
-                       lock_kernel();
-                       error = posix_lock_file_wait(file, fl);
-                       unlock_kernel();
-                       return error;
+                       return posix_lock_file_wait(file, fl);
                }
        }
 
+       if (cmd == F_CANCELLK) {
+               /* Hack: */
+               cmd = F_SETLK;
+               fl->fl_type = F_UNLCK;
+       }
        if (IS_GETLK(cmd))
                return gfs2_lm_plock_get(sdp, &name, file, fl);
        else if (fl->fl_type == F_UNLCK)
@@ -1430,61 +555,20 @@ static int gfs2_lock(struct file *file, int cmd, struct file_lock *fl)
                return gfs2_lm_plock(sdp, &name, file, cmd, fl);
 }
 
-/**
- * gfs2_sendfile - Send bytes to a file or socket
- * @in_file: The file to read from
- * @out_file: The file to write to
- * @count: The amount of data
- * @offset: The beginning file offset
- *
- * Outputs: offset - updated according to number of bytes read
- *
- * Returns: The number of bytes sent, errno on failure
- */
-
-static ssize_t gfs2_sendfile(struct file *in_file, loff_t *offset, size_t count,
-                            read_actor_t actor, void *target)
-{
-       struct gfs2_inode *ip = get_v2ip(in_file->f_mapping->host);
-       struct gfs2_holder gh;
-       ssize_t retval;
-
-       atomic_inc(&ip->i_sbd->sd_ops_file);
-
-       gfs2_holder_init(ip->i_gl, LM_ST_SHARED, GL_ATIME, &gh);
-
-       retval = gfs2_glock_nq_atime(&gh);
-       if (retval)
-               goto out;
-
-       if (gfs2_is_jdata(ip))
-               retval = -EOPNOTSUPP;
-       else
-               retval = generic_file_sendfile(in_file, offset, count, actor,
-                                              target);
-
-       gfs2_glock_dq(&gh);
-
- out:
-       gfs2_holder_uninit(&gh);
-
-       return retval;
-}
-
 static int do_flock(struct file *file, int cmd, struct file_lock *fl)
 {
-       struct gfs2_file *fp = get_v2fp(file);
+       struct gfs2_file *fp = file->private_data;
        struct gfs2_holder *fl_gh = &fp->f_fl_gh;
-       struct gfs2_inode *ip = fp->f_inode;
+       struct gfs2_inode *ip = GFS2_I(file->f_path.dentry->d_inode);
        struct gfs2_glock *gl;
        unsigned int state;
        int flags;
        int error = 0;
 
        state = (fl->fl_type == F_WRLCK) ? LM_ST_EXCLUSIVE : LM_ST_SHARED;
-       flags = ((IS_SETLKW(cmd)) ? 0 : LM_FLAG_TRY) | GL_EXACT | GL_NOCACHE;
+       flags = (IS_SETLKW(cmd) ? 0 : LM_FLAG_TRY) | GL_EXACT | GL_NOCACHE;
 
-       down(&fp->f_fl_mutex);
+       mutex_lock(&fp->f_fl_mutex);
 
        gl = fl_gh->gh_gl;
        if (gl) {
@@ -1492,11 +576,11 @@ static int do_flock(struct file *file, int cmd, struct file_lock *fl)
                        goto out;
                gfs2_glock_hold(gl);
                flock_lock_file_wait(file,
-                                    &(struct file_lock){.fl_type = F_UNLCK});          
+                                    &(struct file_lock){.fl_type = F_UNLCK});
                gfs2_glock_dq_uninit(fl_gh);
        } else {
-               error = gfs2_glock_get(ip->i_sbd,
-                                     ip->i_num.no_addr, &gfs2_flock_glops,
+               error = gfs2_glock_get(GFS2_SB(&ip->i_inode),
+                                     ip->i_no_addr, &gfs2_flock_glops,
                                      CREATE, &gl);
                if (error)
                        goto out;
@@ -1512,25 +596,24 @@ static int do_flock(struct file *file, int cmd, struct file_lock *fl)
                        error = -EAGAIN;
        } else {
                error = flock_lock_file_wait(file, fl);
-               gfs2_assert_warn(ip->i_sbd, !error);
+               gfs2_assert_warn(GFS2_SB(&ip->i_inode), !error);
        }
 
- out:
-       up(&fp->f_fl_mutex);
-
+out:
+       mutex_unlock(&fp->f_fl_mutex);
        return error;
 }
 
 static void do_unflock(struct file *file, struct file_lock *fl)
 {
-       struct gfs2_file *fp = get_v2fp(file);
+       struct gfs2_file *fp = file->private_data;
        struct gfs2_holder *fl_gh = &fp->f_fl_gh;
 
-       down(&fp->f_fl_mutex);
+       mutex_lock(&fp->f_fl_mutex);
        flock_lock_file_wait(file, fl);
        if (fl_gh->gh_gl)
                gfs2_glock_dq_uninit(fl_gh);
-       up(&fp->f_fl_mutex);
+       mutex_unlock(&fp->f_fl_mutex);
 }
 
 /**
@@ -1544,14 +627,12 @@ static void do_unflock(struct file *file, struct file_lock *fl)
 
 static int gfs2_flock(struct file *file, int cmd, struct file_lock *fl)
 {
-       struct gfs2_inode *ip = get_v2ip(file->f_mapping->host);
-       struct gfs2_sbd *sdp = ip->i_sbd;
-
-       atomic_inc(&ip->i_sbd->sd_ops_file);
+       struct gfs2_inode *ip = GFS2_I(file->f_mapping->host);
+       struct gfs2_sbd *sdp = GFS2_SB(file->f_mapping->host);
 
        if (!(fl->fl_flags & FL_FLOCK))
                return -ENOLCK;
-       if ((ip->i_di.di_mode & (S_ISGID | S_IXGRP)) == S_ISGID)
+       if ((ip->i_inode.i_mode & (S_ISGID | S_IXGRP)) == S_ISGID)
                return -ENOLCK;
 
        if (sdp->sd_args.ar_localflocks)
@@ -1560,31 +641,36 @@ static int gfs2_flock(struct file *file, int cmd, struct file_lock *fl)
        if (fl->fl_type == F_UNLCK) {
                do_unflock(file, fl);
                return 0;
-       } else
+       } else {
                return do_flock(file, cmd, fl);
+       }
 }
 
-struct file_operations gfs2_file_fops = {
-       .llseek = gfs2_llseek,
-       .read = gfs2_read,
-       .write = gfs2_write,
-       .ioctl = gfs2_ioctl,
-       .mmap = gfs2_mmap,
-       .open = gfs2_open,
-       .release = gfs2_close,
-       .fsync = gfs2_fsync,
-       .lock = gfs2_lock,
-       .sendfile = gfs2_sendfile,
-       .flock = gfs2_flock,
+const struct file_operations gfs2_file_fops = {
+       .llseek         = gfs2_llseek,
+       .read           = do_sync_read,
+       .aio_read       = generic_file_aio_read,
+       .write          = do_sync_write,
+       .aio_write      = generic_file_aio_write,
+       .unlocked_ioctl = gfs2_ioctl,
+       .mmap           = gfs2_mmap,
+       .open           = gfs2_open,
+       .release        = gfs2_close,
+       .fsync          = gfs2_fsync,
+       .lock           = gfs2_lock,
+       .flock          = gfs2_flock,
+       .splice_read    = generic_file_splice_read,
+       .splice_write   = generic_file_splice_write,
+       .setlease       = gfs2_setlease,
 };
 
-struct file_operations gfs2_dir_fops = {
-       .readdir = gfs2_readdir,
-       .ioctl = gfs2_ioctl,
-       .open = gfs2_open,
-       .release = gfs2_close,
-       .fsync = gfs2_fsync,
-       .lock = gfs2_lock,
-       .flock = gfs2_flock,
+const struct file_operations gfs2_dir_fops = {
+       .readdir        = gfs2_readdir,
+       .unlocked_ioctl = gfs2_ioctl,
+       .open           = gfs2_open,
+       .release        = gfs2_close,
+       .fsync          = gfs2_fsync,
+       .lock           = gfs2_lock,
+       .flock          = gfs2_flock,
 };