linux-dentry-revalidation-always-revalidate-20010420
[openafs.git] / src / afs / LINUX / osi_vnodeops.c
index de2c51f..5958e65 100644 (file)
 #include "../afs/afs_stats.h"
 #include "../h/mm.h"
 #include "../h/pagemap.h"
+#if defined(AFS_LINUX24_ENV)
+#include "../h/smp_lock.h"
+#endif
+
+#ifdef pgoff2loff
+#define pageoff(pp) pgoff2loff((pp)->index)
+#else
+#define pageoff(pp) pp->offset
+#endif
 
 extern struct vcache *afs_globalVp;
 
 extern struct dentry_operations *afs_dops;
+#if defined(AFS_LINUX24_ENV)
+extern struct inode_operations afs_file_iops;
+extern struct address_space_operations afs_file_aops;
+struct address_space_operations afs_symlink_aops;
+#endif
 extern struct inode_operations afs_dir_iops;
 extern struct inode_operations afs_symlink_iops;
 
@@ -138,6 +152,7 @@ static ssize_t afs_linux_write(struct file *fp, const char *buf, size_t count,
 static int afs_linux_readdir(struct file *fp,
                             void *dirbuf, filldir_t filldir)
 {
+    extern struct DirEntry * afs_dir_GetBlob();
     struct vcache *avc = (struct vcache*)FILE_INODE(fp);
     struct vrequest treq;
     register struct dcache *tdc;
@@ -215,7 +230,41 @@ tagain:
        len = strlen(de->name);
 
        /* filldir returns -EINVAL when the buffer is full. */
-       code = (*filldir)(dirbuf, de->name, len, offset, ino);
+#ifdef AFS_LINUX24_ENV
+        {
+             unsigned int type=DT_UNKNOWN;
+             struct VenusFid afid;
+             struct vcache *tvc;
+             int vtype;
+             afid.Cell=avc->fid.Cell;
+             afid.Fid.Volume=avc->fid.Fid.Volume;
+             afid.Fid.Vnode=ntohl(de->fid.vnode);
+             afid.Fid.Unique=ntohl(de->fid.vunique);
+             if ((avc->states & CForeign) == 0 &&
+                 (ntohl(de->fid.vnode) & 1)) {
+                  type=DT_DIR;
+             } else if ((tvc=afs_FindVCache(&afid,0,0,0,0))) {
+                  if (tvc->mvstat) {
+                       type=DT_DIR;
+                  } else if (((tvc->states) & (CStatd|CTruth))) {
+                       /* CTruth will be set if the object has 
+                        *ever* been statd */
+                       vtype=vType(tvc);
+                       if (vtype == VDIR)
+                            type=DT_DIR;
+                       else if (vtype == VREG)
+                            type=DT_REG;
+                       /* Don't do this until we're sure it can't be a mtpt */
+                       /* else if (vtype == VLNK)
+                          type=DT_LNK; */
+                       /* what other types does AFS support? */
+                  }
+             }
+             code = (*filldir)(dirbuf, de->name, len, offset, ino, type);
+        }
+#else
+       code = (*filldir)(dirbuf, de->name, len, offset, ino); 
+#endif
        DRelease(de, 0);
        if (code)
            break;
@@ -301,10 +350,16 @@ static int afs_linux_mmap(struct file *fp, struct vm_area_struct *vmap)
     int code;
 
     AFS_GLOCK();
+#if defined(AFS_LINUX24_ENV)
+    afs_Trace3(afs_iclSetp, CM_TRACE_GMAP, ICL_TYPE_POINTER, vcp,
+              ICL_TYPE_POINTER, vmap->vm_start,
+              ICL_TYPE_INT32, vmap->vm_end - vmap->vm_start);
+#else
     afs_Trace4(afs_iclSetp, CM_TRACE_GMAP, ICL_TYPE_POINTER, vcp,
               ICL_TYPE_POINTER, vmap->vm_start,
               ICL_TYPE_INT32, vmap->vm_end - vmap->vm_start,
               ICL_TYPE_INT32, vmap->vm_offset);
+#endif
 
     /* get a validated vcache entry */
     code = afs_InitReq(&treq, credp);
@@ -322,36 +377,38 @@ static int afs_linux_mmap(struct file *fp, struct vm_area_struct *vmap)
        AFS_GLOCK();
     }
 
-    ObtainWriteLock(&vcp->lock,531);
-    /* Set out vma ops so we catch the close. The following test should be
-     * the same as used in generic_file_mmap.
-     */
-    if ((vmap->vm_flags & VM_SHARED) && (vmap->vm_flags & VM_MAYWRITE)) {
-       if (!afs_shared_mmap_ops_inited) {
-           afs_shared_mmap_ops_inited = 1;
-           afs_shared_mmap_ops = *vmap->vm_ops;
-           afs_shared_mmap_ops.close = afs_linux_vma_close;
+    if (code == 0) {
+       ObtainWriteLock(&vcp->lock,531);
+       /* Set out vma ops so we catch the close. The following test should be
+        * the same as used in generic_file_mmap.
+        */
+       if ((vmap->vm_flags & VM_SHARED) && (vmap->vm_flags & VM_MAYWRITE)) {
+           if (!afs_shared_mmap_ops_inited) {
+               afs_shared_mmap_ops_inited = 1;
+               afs_shared_mmap_ops = *vmap->vm_ops;
+               afs_shared_mmap_ops.close = afs_linux_vma_close;
+           }
+           vmap->vm_ops = &afs_shared_mmap_ops;
        }
-       vmap->vm_ops = &afs_shared_mmap_ops;
-    }
-    else {
-       if (!afs_private_mmap_ops_inited) {
-           afs_private_mmap_ops_inited = 1;
-           afs_private_mmap_ops = *vmap->vm_ops;
-           afs_private_mmap_ops.close = afs_linux_vma_close;
+       else {
+           if (!afs_private_mmap_ops_inited) {
+               afs_private_mmap_ops_inited = 1;
+               afs_private_mmap_ops = *vmap->vm_ops;
+               afs_private_mmap_ops.close = afs_linux_vma_close;
+           }
+           vmap->vm_ops = &afs_private_mmap_ops;
        }
-       vmap->vm_ops = &afs_private_mmap_ops;
-    }
     
     
-    /* Add an open reference on the first mapping. */
-    if (vcp->mapcnt == 0) {
-       vcp->execsOrWriters++;
-       vcp->opens++;
-       vcp->states |= CMAPPED;
+       /* Add an open reference on the first mapping. */
+       if (vcp->mapcnt == 0) {
+           vcp->execsOrWriters++;
+           vcp->opens++;
+           vcp->states |= CMAPPED;
+       }
+       ReleaseWriteLock(&vcp->lock);
+       vcp->mapcnt++;
     }
-    ReleaseWriteLock(&vcp->lock);
-    vcp->mapcnt++;
 
     AFS_GUNLOCK();
     crfree(credp);
@@ -364,7 +421,13 @@ int afs_linux_open(struct inode *ip, struct file *fp)
     cred_t *credp = crref();
 
     AFS_GLOCK();
+#ifdef AFS_LINUX24_ENV
+    lock_kernel();
+#endif
     code = afs_open((struct vcache**)&ip, fp->f_flags, credp);
+#ifdef AFS_LINUX24_ENV
+    unlock_kernel();
+#endif
     AFS_GUNLOCK();
 
     crfree(credp);
@@ -385,26 +448,42 @@ static int afs_linux_release(struct inode *ip, struct file *fp)
     struct vcache *vcp = (struct vcache*)ip;
 
     AFS_GLOCK();
+#ifdef AFS_LINUX24_ENV
+    lock_kernel();
+#endif
     if (vcp->flushcnt) {
        vcp->flushcnt--; /* protected by AFS global lock. */
     }
     else {
        code = afs_close(vcp, fp->f_flags, credp);
     }
+#ifdef AFS_LINUX24_ENV
+    unlock_kernel();
+#endif
     AFS_GUNLOCK();
 
     crfree(credp);
     return -code;
 }
 
+#if defined(AFS_LINUX24_ENV)
+static int afs_linux_fsync(struct file *fp, struct dentry *dp, int datasync)
+#else
 static int afs_linux_fsync(struct file *fp, struct dentry *dp)
+#endif
 {
     int code;
     struct inode *ip = FILE_INODE(fp);
     cred_t *credp = crref();
 
     AFS_GLOCK();
+#ifdef AFS_LINUX24_ENV
+    lock_kernel();
+#endif
     code = afs_fsync((struct vcache*)ip, credp);
+#ifdef AFS_LINUX24_ENV
+    unlock_kernel();
+#endif
     AFS_GUNLOCK();
     crfree(credp);
     return -code;
@@ -464,7 +543,11 @@ int afs_linux_flush(struct file *fp)
     cred_t *credp;
 
     /* Only do this on the last close of the file pointer. */
+#if defined(AFS_LINUX24_ENV)
+    if (atomic_read(&fp->f_count) > 1)
+#else
     if (fp->f_count > 1)
+#endif
        return 0;
 
     credp = crref();
@@ -479,13 +562,22 @@ int afs_linux_flush(struct file *fp)
 }
 
 /* Not allowed to directly read a directory. */
-int afs_linux_dir_read(struct file *fp, char *buf, size_t count, loff_t *ppos)
+ssize_t afs_linux_dir_read(struct file *fp, char *buf, size_t count, loff_t *ppos)
 {
     return -EISDIR;
 }
 
 
 
+#if defined(AFS_LINUX24_ENV)
+struct file_operations afs_dir_fops = {
+    read:      generic_read_dir,
+    readdir:   afs_linux_readdir,
+    ioctl:     afs_xioctl,
+    open:      afs_linux_open,
+    release:   afs_linux_release,
+};
+#else
 struct file_operations afs_dir_fops = {
     NULL,              /* afs_linux_lseek */
     afs_linux_dir_read,
@@ -503,7 +595,21 @@ struct file_operations afs_dir_fops = {
     NULL,              /* afs_linux_file_revalidate */
     afs_linux_lock,
 };
+#endif
 
+#if defined(AFS_LINUX24_ENV)
+struct file_operations afs_file_fops = {
+    read:      afs_linux_read,
+    write:     afs_linux_write,
+    ioctl:     afs_xioctl,
+    mmap:      afs_linux_mmap,
+    open:      afs_linux_open,
+    flush:     afs_linux_flush,
+    release:   afs_linux_release,
+    fsync:     afs_linux_fsync,
+    lock:      afs_linux_lock,
+};
+#else
 struct file_operations afs_file_fops = {
     NULL,              /* afs_linux_lseek */
     afs_linux_read,
@@ -521,6 +627,7 @@ struct file_operations afs_file_fops = {
     NULL,              /* afs_linux_file_revalidate */
     afs_linux_lock,
 };
+#endif
    
 
 /**********************************************************************
@@ -538,18 +645,18 @@ static int afs_linux_revalidate(struct dentry *dp)
     struct vcache *vcp = (struct vcache*)dp->d_inode;
 
     AFS_GLOCK();
-
-    /* If it's a negative dentry, then there's nothing to do. */
-    if (!vcp) {
-       AFS_GUNLOCK();
-       return 0;
-    }
+#ifdef AFS_LINUX24_ENV
+    lock_kernel();
+#endif
 
     /* Make this a fast path (no crref), since it's called so often. */
     if (vcp->states & CStatd) {
         if (*dp->d_name.name != '/' && vcp->mvstat == 2) /* root vnode */
            check_bad_parent(dp); /* check and correct mvid */
        vcache2inode(vcp);
+#ifdef AFS_LINUX24_ENV
+       unlock_kernel();
+#endif
        AFS_GUNLOCK();
        return 0;
     }
@@ -559,71 +666,39 @@ static int afs_linux_revalidate(struct dentry *dp)
     if (!code)
        code = afs_VerifyVCache(vcp, &treq);
 
+#ifdef AFS_LINUX24_ENV
+    unlock_kernel();
+#endif
     AFS_GUNLOCK();
     crfree(credp);
 
     return -code ;
 }
 
-/* Validate a dentry. Return 0 if unchanged, 1 if VFS layer should re-evaluate.
- * In kernels 2.2.10 and above, we are passed an additional flags var which
- * may have either the LOOKUP_FOLLOW OR LOOKUP_DIRECTORY set in which case
- * we are advised to follow the entry if it is a link or to make sure that 
- * it is a directory. But since the kernel itself checks these possibilities
- * later on, we shouldn't have to do it until later. Perhaps in the future..
- */
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,2,10)
 static int afs_linux_dentry_revalidate(struct dentry *dp, int flags)
 #else
 static int afs_linux_dentry_revalidate(struct dentry *dp)
 #endif
 {
-    int code;
-    cred_t *credp;
-    struct vrequest treq;
-    struct vcache *vcp = (struct vcache*)dp->d_inode;
-
-    AFS_GLOCK();
-
-    /* If it's a negative dentry, then there's nothing to do. */
-    if (!vcp) {
-       AFS_GUNLOCK();
-       return 0;
-    }
-
-    /* Make this a fast path (no crref), since it's called so often. */
-    if (vcp->states & CStatd) {
-        if (*dp->d_name.name != '/' && vcp->mvstat == 2) /* root vnode */
-           check_bad_parent(dp); /* check and correct mvid */
-       vcache2inode(vcp);
-       AFS_GUNLOCK();
-       return 0;
-    }
-
-    credp = crref();
-    code = afs_InitReq(&treq, credp);
-    if (!code)
-       code = afs_VerifyVCache(vcp, &treq);
-
-    AFS_GUNLOCK();
-    crfree(credp);
-
-    return 1;
+    /* Force revalidation as this may be a different client than the
+       one which caused an entry to get cached */
+    return 0;
 }
 
 /* afs_dentry_iput */
 static void afs_dentry_iput(struct dentry *dp, struct inode *ip)
 {
-    if (ip->i_count == 0 || ip->i_count & 0xffff0000) {
-       osi_Panic("Bad refCount %d on inode 0x%x\n",
-                 ip->i_count, ip);
-    }
-    ip->i_count --;
-    if (!ip->i_count) {
-       afs_delete_inode(ip);
-    }
+    osi_iput(ip);
 }
 
+#if defined(AFS_LINUX24_ENV)
+struct dentry_operations afs_dentry_operations = {
+       d_revalidate:   afs_linux_dentry_revalidate,
+       d_iput:         afs_dentry_iput,
+};
+struct dentry_operations *afs_dops = &afs_dentry_operations;
+#else
 struct dentry_operations afs_dentry_operations = {
        afs_linux_dentry_revalidate,    /* d_validate(struct dentry *) */
        NULL,                   /* d_hash */
@@ -633,6 +708,7 @@ struct dentry_operations afs_dentry_operations = {
        afs_dentry_iput         /* d_iput(struct dentry *, struct inode *) */
 };
 struct dentry_operations *afs_dops = &afs_dentry_operations;
+#endif
 
 /**********************************************************************
  * AFS Linux inode operations
@@ -665,12 +741,31 @@ int afs_linux_create(struct inode *dip, struct dentry *dp, int mode)
     if (!code) {
        vattr2inode(ip, &vattr);
        /* Reset ops if symlink or directory. */
+#if defined(AFS_LINUX24_ENV)
+       if (S_ISREG(ip->i_mode)) {
+           ip->i_op = &afs_file_iops;
+           ip->i_fop = &afs_file_fops;
+           ip->i_data.a_ops = &afs_file_aops;
+        } else if (S_ISDIR(ip->i_mode)) {
+           ip->i_op = &afs_dir_iops;
+           ip->i_fop = &afs_dir_fops;
+        } else if (S_ISLNK(ip->i_mode)) {
+           ip->i_op = &afs_symlink_iops;
+           ip->i_data.a_ops = &afs_symlink_aops;
+           ip->i_mapping = &ip->i_data;
+        } else
+           printk("afs_linux_create: FIXME\n");
+#else
        if (S_ISDIR(ip->i_mode))
            ip->i_op = &afs_dir_iops;
        else if (S_ISLNK(ip->i_mode))
            ip->i_op = &afs_symlink_iops;
+#endif
 
        dp->d_op = afs_dops;
+        /* This DV is probably wrong, unfortunately, Perhaps we should 
+           VerifyVCache the directory  */
+        dp->d_time=hgetlo(((struct vcache *)dip)->m.DataVersion);
        d_instantiate(dp, ip);
     }
 
@@ -696,11 +791,29 @@ int afs_linux_lookup(struct inode *dip, struct dentry *dp)
     if (vcp) {
        struct inode *ip = (struct inode*)vcp;
        /* Reset ops if symlink or directory. */
+#if defined(AFS_LINUX24_ENV)
+       if (S_ISREG(ip->i_mode)) {
+           ip->i_op = &afs_file_iops;
+           ip->i_fop = &afs_file_fops;
+           ip->i_data.a_ops = &afs_file_aops;
+        } else if (S_ISDIR(ip->i_mode)) {
+           ip->i_op = &afs_dir_iops;
+           ip->i_fop = &afs_dir_fops;
+        } else if (S_ISLNK(ip->i_mode)) {
+           ip->i_op = &afs_symlink_iops;
+           ip->i_data.a_ops = &afs_symlink_aops;
+           ip->i_mapping = &ip->i_data;
+       } else
+           printk("afs_linux_lookup: FIXME\n");
+#else
        if (S_ISDIR(ip->i_mode))
            ip->i_op = &afs_dir_iops;
        else if (S_ISLNK(ip->i_mode))
            ip->i_op = &afs_symlink_iops;
+#endif
     }
+    /* directory ought to be stat'd here.... */
+    dp->d_time=hgetlo(((struct vcache *)dip)->m.DataVersion);
     dp->d_op = afs_dops;
     d_add(dp, (struct inode*)vcp);
 
@@ -753,7 +866,11 @@ int afs_linux_unlink(struct inode *dip, struct dentry *dp)
     if (!list_empty(&dp->d_hash)) {
        d_drop(dp);
        /* Install a definite non-existence if we're the only user. */
+#if defined(AFS_LINUX24_ENV)
+       if (atomic_read(&dp->d_count) == 1)
+#else
        if (dp->d_count == 1)
+#endif
            putback = 1;
     }
 
@@ -762,9 +879,13 @@ int afs_linux_unlink(struct inode *dip, struct dentry *dp)
     AFS_GUNLOCK();
     if (!code) {
        d_delete(dp);
-       if (putback)
+       if (putback) {
+           /* This DV is probably wrong, unfortunately, Perhaps we should 
+              VerifyVCache the directory  */
+            dp->d_time=hgetlo(((struct vcache *)dip)->m.DataVersion);
            d_add(dp, NULL); /* means definitely does _not_ exist */
     }
+    }
     crfree(credp);
     return -code;
 }
@@ -807,7 +928,13 @@ int afs_linux_mkdir(struct inode *dip, struct dentry *dp, int mode)
 
     if (tvcp) {
        tvcp->v.v_op = &afs_dir_iops;
+#if defined(AFS_LINUX24_ENV)
+       tvcp->v.v_fop = &afs_dir_fops;
+#endif
        dp->d_op = afs_dops;
+        /* This DV is probably wrong, unfortunately, Perhaps we should 
+           VerifyVCache the directory  */
+        dp->d_time=hgetlo(((struct vcache *)dip)->m.DataVersion);
        d_instantiate(dp, (struct inode*)tvcp);
     }
     AFS_GUNLOCK();
@@ -824,12 +951,12 @@ int afs_linux_rmdir(struct inode *dip, struct dentry *dp)
     AFS_GLOCK();
     code = afs_rmdir((struct vcache*)dip, name, credp);
 
-    /* Linux likes to see ENOTDIR returned from an rmdir() syscall
+    /* Linux likes to see ENOTEMPTY returned from an rmdir() syscall
      * that failed because a directory is not empty. So, we map
-     * EEXIST to ENOTDIR on linux.
+     * EEXIST to ENOTEMPTY on linux.
      */
     if (code == EEXIST) {
-       code = ENOTDIR;
+       code = ENOTEMPTY;
     }
     
     if (!code) {
@@ -867,8 +994,13 @@ int afs_linux_rename(struct inode *oldip, struct dentry *olddp,
                      newname, credp);
     AFS_GUNLOCK();
 
-    if (!code)
+    if (!code) {
+        /* update time so it doesn't expire immediately */
+        /* This DV is probably wrong, unfortunately, Perhaps we should 
+           VerifyVCache the directory  */
+        newdp->d_time=hgetlo(((struct vcache *)newdp->d_parent->d_inode)->m.DataVersion);
        d_move(olddp, newdp);
+    }
 
     crfree(credp);
     return -code;
@@ -897,6 +1029,7 @@ static int afs_linux_ireadlink(struct inode *ip, char *target, int maxlen,
        return -code;
 }
 
+#if !defined(AFS_LINUX24_ENV)
 /* afs_linux_readlink 
  * Fill target (which is in user space) with contents of symlink.
  */
@@ -923,7 +1056,6 @@ struct dentry * afs_linux_follow_link(struct dentry *dp,
     char *name;
     struct dentry *res;
 
-
     AFS_GLOCK();
     name = osi_Alloc(PATH_MAX+1);
     if (!name) {
@@ -949,6 +1081,7 @@ struct dentry * afs_linux_follow_link(struct dentry *dp,
     AFS_GUNLOCK();
     return res;
 }
+#endif
 
 /* afs_linux_readpage
  * all reads come through here. A strategy-like read call.
@@ -957,7 +1090,13 @@ int afs_linux_readpage(struct file *fp, struct page *pp)
 {
     int code;
     cred_t *credp = crref();
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
+    char *address;
+    loff_t offset = pp->index << PAGE_CACHE_SHIFT;
+#else
     ulong address = afs_linux_page_address(pp);
+    loff_t offset = pageoff(pp);
+#endif
     uio_t tuio;
     struct iovec iovec;
     struct inode *ip = FILE_INODE(fp);
@@ -969,24 +1108,45 @@ int afs_linux_readpage(struct file *fp, struct page *pp)
               ICL_TYPE_POINTER, pp,
               ICL_TYPE_INT32, cnt,
               ICL_TYPE_INT32, 99999); /* not a possible code value */
+
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
+    address = kmap(pp);
+    ClearPageError(pp);
+
+    lock_kernel();
+#else
     atomic_add(1, &pp->count);
     set_bit(PG_locked, &pp->flags); /* other bits? See mm.h */
     clear_bit(PG_error, &pp->flags);
+#endif
 
-    setup_uio(&tuio, &iovec, (char*)address, pp->offset, PAGESIZE,
+    setup_uio(&tuio, &iovec, (char*)address, offset, PAGESIZE,
              UIO_READ, AFS_UIOSYS);
     code = afs_rdwr((struct vcache*)ip, &tuio, UIO_READ, 0, credp);
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
+    unlock_kernel();
+#endif
 
     if (!code) {
        if (tuio.uio_resid) /* zero remainder of page */
            memset((void*)(address+(PAGESIZE-tuio.uio_resid)), 0,
                   tuio.uio_resid);
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
+       flush_dcache_page(pp);
+       SetPageUptodate(pp);
+#else
        set_bit(PG_uptodate, &pp->flags);
+#endif
     }
 
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
+    kunmap(pp);
+    UnlockPage(pp);
+#else
     clear_bit(PG_locked, &pp->flags);
     wake_up(&pp->wait);
     free_page(address);
+#endif
 
     crfree(credp);
     afs_Trace4(afs_iclSetp, CM_TRACE_READPAGE,
@@ -998,10 +1158,40 @@ int afs_linux_readpage(struct file *fp, struct page *pp)
     return -code;
 }
 
-#ifdef NOTUSED
-/* afs_linux_writepage - is this used anywhere? */
-int afs_linux_writepage(struct inode *ip, struct page *) { return -EINVAL };
+#if defined(AFS_LINUX24_ENV)
+int afs_linux_writepage(struct page *pp)
+{
+    struct address_space *mapping = pp->mapping;
+    struct inode *inode;
+    unsigned long end_index;
+    unsigned offset = PAGE_CACHE_SIZE;
+    long status;
+    inode = (struct inode *) mapping->host;
+    end_index = inode->i_size >> PAGE_CACHE_SHIFT;
+
+    /* easy case */
+    if (pp->index < end_index)
+       goto do_it;
+    /* things got complicated... */
+    offset = inode->i_size & (PAGE_CACHE_SIZE-1);
+    /* OK, are we completely out? */
+    if (pp->index >= end_index+1 || !offset)
+       return -EIO;
+do_it:
+    AFS_GLOCK();
+    status = afs_linux_writepage_sync(inode, pp, 0, offset);
+    AFS_GUNLOCK();
+    SetPageUptodate(pp);
+    UnlockPage(pp);
+    if (status == offset)
+       return 0;
+    else
+       return status;
+}
+#endif
 
+#ifdef NOTUSED
 /* afs_linux_bmap - supports generic_readpage, but we roll our own. */
 int afs_linux_bmap(struct inode *ip, int) { return -EINVAL; }
 
@@ -1040,6 +1230,55 @@ int afs_linux_permission(struct inode *ip, int mode)
 int afs_linux_smap(struct inode *ip, int) { return -EINVAL; }
 #endif
 
+#if defined(AFS_LINUX24_ENV)
+int afs_linux_writepage_sync(struct inode *ip, struct page *pp,
+                        unsigned long offset,
+                        unsigned int count)
+{
+    struct vcache *vcp = (struct vcache *) ip;
+    char *buffer;
+    loff_t base;
+    int code = 0;
+    cred_t *credp;
+    uio_t tuio;
+    struct iovec iovec;
+    int f_flags = 0;
+
+    buffer = kmap(pp) + offset;
+    base = (pp->index << PAGE_CACHE_SHIFT) + offset;
+
+    credp = crref();
+    afs_Trace4(afs_iclSetp, CM_TRACE_UPDATEPAGE, ICL_TYPE_POINTER, vcp,
+              ICL_TYPE_POINTER, pp,
+              ICL_TYPE_INT32, atomic_read(&pp->count),
+              ICL_TYPE_INT32, 99999);
+    setup_uio(&tuio, &iovec, buffer, base, count, UIO_WRITE, AFS_UIOSYS);
+
+    code = afs_write(vcp, &tuio, f_flags, credp, 0);
+
+    vcache2inode(vcp);
+
+    code = code ? -code : count - tuio.uio_resid;
+    afs_Trace4(afs_iclSetp, CM_TRACE_UPDATEPAGE, ICL_TYPE_POINTER, vcp,
+              ICL_TYPE_POINTER, pp,
+              ICL_TYPE_INT32, atomic_read(&pp->count),
+              ICL_TYPE_INT32, code);
+
+    crfree(credp);
+    kunmap(pp);
+
+    return code;
+}
+
+static int
+afs_linux_updatepage(struct file *file, struct page *page, 
+                    unsigned long offset, unsigned int count)
+{
+    struct dentry *dentry = file->f_dentry;
+
+    return afs_linux_writepage_sync(dentry->d_inode, page, offset, count);
+}
+#else
 /* afs_linux_updatepage
  * What one would have thought was writepage - write dirty page to file.
  * Called from generic_file_write. buffer is still in user space. pagep
@@ -1083,9 +1322,48 @@ int afs_linux_updatepage(struct file *fp, struct page *pp,
     clear_bit(PG_locked, &pp->flags);
     return code;
 }
+#endif
+
+#if defined(AFS_LINUX24_ENV)
+static int afs_linux_commit_write(struct file *file, struct page *page, unsigned offset, unsigned to)
+{
+    int code;
+
+    AFS_GLOCK();
+    lock_kernel();
+    code = afs_linux_updatepage(file, page, offset, to-offset);
+    unlock_kernel();
+    AFS_GUNLOCK();
+    kunmap(page);
+
+    return code;
+}
 
+static int afs_linux_prepare_write(struct file *file, struct page *page,
+                                  unsigned from, unsigned to)
+{
+    kmap(page);
+    return 0;
+}
 
+extern int afs_notify_change(struct dentry *dp, struct iattr* iattrp);
+#endif
+
+#if defined(AFS_LINUX24_ENV)
+struct inode_operations afs_file_iops = {
+    revalidate:                afs_linux_revalidate,
+    setattr:           afs_notify_change,
+    permission:                afs_linux_permission,
+};
+struct address_space_operations afs_file_aops = {
+        readpage: afs_linux_readpage,
+        writepage: afs_linux_writepage,
+        commit_write: afs_linux_commit_write,
+        prepare_write: afs_linux_prepare_write,
+};
 
+struct inode_operations *afs_ops = &afs_file_iops;
+#else
 struct inode_operations afs_iops = {
     &afs_file_fops,    /* file operations */
     NULL,                      /* afs_linux_create */
@@ -1109,9 +1387,27 @@ struct inode_operations afs_iops = {
     afs_linux_revalidate,
 };
 
+struct inode_operations *afs_ops = &afs_iops;
+#endif
+
 /* Separate ops vector for directories. Linux 2.2 tests type of inode
  * by what sort of operation is allowed.....
  */
+#if defined(AFS_LINUX24_ENV)
+struct inode_operations afs_dir_iops = {
+    create:    afs_linux_create,
+    lookup:    afs_linux_lookup,
+    link:      afs_linux_link,
+    unlink:    afs_linux_unlink,
+    symlink:   afs_linux_symlink,
+    mkdir:     afs_linux_mkdir,
+    rmdir:     afs_linux_rmdir,
+    rename:    afs_linux_rename,
+    revalidate:        afs_linux_revalidate,
+    setattr:   afs_notify_change,
+    permission:        afs_linux_permission,
+};
+#else
 struct inode_operations afs_dir_iops = {
     &afs_dir_fops,     /* file operations for directories */
     afs_linux_create,
@@ -1134,12 +1430,53 @@ struct inode_operations afs_dir_iops = {
     NULL,              /* afs_linux_updatepage */
     afs_linux_revalidate,
 };
-
-struct inode_operations *afs_ops = &afs_iops;
+#endif
 
 /* We really need a separate symlink set of ops, since do_follow_link()
  * determines if it _is_ a link by checking if the follow_link op is set.
  */
+#if defined(AFS_LINUX24_ENV)
+static int afs_symlink_filler(struct file *file, struct page *page)
+{
+    struct inode *ip = (struct inode *) page->mapping->host;
+    char *p = (char *)kmap(page);
+    int code;
+
+    AFS_GLOCK();
+    lock_kernel();
+    code = afs_linux_ireadlink(ip, p, PAGE_SIZE, AFS_UIOSYS);
+
+    if (code<0)
+           goto fail;
+    p[code] = '\0';            /* null terminate? */
+    unlock_kernel();
+    AFS_GUNLOCK();
+
+    SetPageUptodate(page);
+    kunmap(page);
+    UnlockPage(page);
+    return 0;
+
+fail:
+    unlock_kernel();
+    AFS_GUNLOCK();
+
+    SetPageError(page);
+    kunmap(page);
+    UnlockPage(page);
+    return code;
+}
+
+struct address_space_operations afs_symlink_aops = {
+       readpage:       afs_symlink_filler
+};
+
+struct inode_operations afs_symlink_iops = {
+    readlink:          page_readlink,
+    follow_link:       page_follow_link,
+    setattr:           afs_notify_change,
+};
+#else
 struct inode_operations afs_symlink_iops = {
     NULL,              /* file operations */
     NULL,              /* create */
@@ -1162,3 +1499,4 @@ struct inode_operations afs_symlink_iops = {
     NULL,              /* updatepage */
     afs_linux_revalidate, /* tho the code appears to indicate not used? */
 };
+#endif