dread-do-validation-20041012
[openafs.git] / src / afs / LINUX / osi_vnodeops.c
index 0e0ec36..c09561a 100644 (file)
@@ -9,9 +9,7 @@
 
 /*
  * Linux specific vnodeops. Also includes the glue routines required to call
- * AFS vnodeops. The "NOTUSED" #define is used to indicate routines and
- * calling sequences present in an ops table that we don't actually use.
- * They are present solely for documentation purposes.
+ * AFS vnodeops.
  *
  * So far the only truly scary part is that Linux relies on the inode cache
  * to be up to date. Don't you dare break a callback and expect an fstat
@@ -31,6 +29,9 @@ RCSID
 #include "afs/afs_stats.h"
 #include "afs/afs_osidnlc.h"
 #include "h/mm.h"
+#ifdef HAVE_MM_INLINE_H
+#include "h/mm_inline.h"
+#endif
 #include "h/pagemap.h"
 #if defined(AFS_LINUX24_ENV)
 #include "h/smp_lock.h"
@@ -42,10 +43,13 @@ RCSID
 #define pageoff(pp) pp->offset
 #endif
 
+#if defined(AFS_LINUX26_ENV)
+#define UnlockPage(pp) unlock_page(pp)
+#endif
+
 extern struct vcache *afs_globalVp;
 extern afs_rwlock_t afs_xvcache;
 
-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;
@@ -55,13 +59,6 @@ extern struct inode_operations afs_dir_iops;
 extern struct inode_operations afs_symlink_iops;
 
 
-#ifdef NOTUSED
-static int
-afs_linux_lseek(struct inode *ip, struct file *fp, off_t, int)
-{
-}
-#endif
-
 static ssize_t
 afs_linux_read(struct file *fp, char *buf, size_t count, loff_t * offp)
 {
@@ -118,6 +115,7 @@ afs_linux_read(struct file *fp, char *buf, size_t count, loff_t * offp)
                *offp += count;
            }
          done:
+               ;
        } else {
 #endif /* AFS_64BIT_CLIENT */
            osi_FlushPages(vcp, credp); /* ensure stale pages are gone */
@@ -225,6 +223,7 @@ afs_linux_write(struct file *fp, const char *buf, size_t count, loff_t * offp)
                ReleaseWriteLock(&vcp->lock);
            }
          done:
+               ;
        } else {
 #endif /* AFS_64BIT_CLIENT */
            AFS_GUNLOCK();
@@ -274,40 +273,34 @@ afs_linux_readdir(struct file *fp, void *dirbuf, filldir_t filldir)
     cred_t *credp = crref();
     struct afs_fakestat_state fakestat;
 
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     AFS_GLOCK();
     AFS_STATCNT(afs_readdir);
 
     code = afs_InitReq(&treq, credp);
     crfree(credp);
-    if (code) {
-       AFS_GUNLOCK();
-       return -code;
-    }
+    if (code)
+       goto out1;
 
     afs_InitFakeStat(&fakestat);
     code = afs_EvalFakeStat(&avc, &fakestat, &treq);
-    if (code) {
-       afs_PutFakeStat(&fakestat);
-       AFS_GUNLOCK();
-       return -code;
-    }
+    if (code)
+       goto out;
 
     /* update the cache entry */
   tagain:
     code = afs_VerifyVCache(avc, &treq);
-    if (code) {
-       afs_PutFakeStat(&fakestat);
-       AFS_GUNLOCK();
-       return -code;
-    }
+    if (code)
+       goto out;
 
     /* get a reference to the entire directory */
     tdc = afs_GetDCache(avc, (afs_size_t) 0, &treq, &origOffset, &tlen, 1);
     len = tlen;
     if (!tdc) {
-       afs_PutFakeStat(&fakestat);
-       AFS_GUNLOCK();
-       return -ENOENT;
+       code = -ENOENT;
+       goto out;
     }
     ObtainReadLock(&avc->lock);
     ObtainReadLock(&tdc->lock);
@@ -338,13 +331,13 @@ afs_linux_readdir(struct file *fp, void *dirbuf, filldir_t filldir)
      * takes an offset in units of blobs, rather than bytes.
      */
     code = 0;
-    offset = (int)fp->f_pos;
+    offset = (int) fp->f_pos;
     while (1) {
-       dirpos = BlobScan(&tdc->f.inode, offset);
+       dirpos = BlobScan(&tdc->f, offset);
        if (!dirpos)
            break;
 
-       de = afs_dir_GetBlob(&tdc->f.inode, dirpos);
+       de = afs_dir_GetBlob(&tdc->f, dirpos);
        if (!de)
            break;
 
@@ -355,15 +348,15 @@ afs_linux_readdir(struct file *fp, void *dirbuf, filldir_t filldir)
        else {
            printf("afs_linux_readdir: afs_dir_GetBlob failed, null name (inode %x, dirpos %d)\n", 
                   &tdc->f.inode, dirpos);
-           DRelease(de, 0);
+           DRelease((struct buffer *) de, 0);
            afs_PutDCache(tdc);
            ReleaseReadLock(&avc->lock);
-           afs_PutFakeStat(&fakestat);
-           return -ENOENT;
+           code = -ENOENT;
+           goto out;
        }
 
        /* filldir returns -EINVAL when the buffer is full. */
-#if (defined(AFS_LINUX24_ENV) || defined(pgoff2loff)) && defined(DECLARE_FSTYPE)
+#if defined(AFS_LINUX26_ENV) || ((defined(AFS_LINUX24_ENV) || defined(pgoff2loff)) && defined(DECLARE_FSTYPE))
        {
            unsigned int type = DT_UNKNOWN;
            struct VenusFid afid;
@@ -399,7 +392,7 @@ afs_linux_readdir(struct file *fp, void *dirbuf, filldir_t filldir)
 #else
        code = (*filldir) (dirbuf, de->name, len, offset, ino);
 #endif
-       DRelease(de, 0);
+       DRelease((struct buffer *)de, 0);
        if (code)
            break;
        offset = dirpos + 1 + ((len + 16) >> 5);
@@ -412,14 +405,18 @@ afs_linux_readdir(struct file *fp, void *dirbuf, filldir_t filldir)
     ReleaseReadLock(&tdc->lock);
     afs_PutDCache(tdc);
     ReleaseReadLock(&avc->lock);
+    code = 0;
+
+out:
     afs_PutFakeStat(&fakestat);
+out1:
     AFS_GUNLOCK();
-    return 0;
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
+    return code;
 }
 
-#ifdef NOTUSED
-int afs_linux_select(struct inode *ip, struct file *fp, int, select_table *);
-#endif
 
 /* in afs_pioctl.c */
 extern int afs_xioctl(struct inode *ip, struct file *fp, unsigned int com,
@@ -470,8 +467,9 @@ afs_linux_vma_close(struct vm_area_struct *vmap)
                (void)afs_close(vcp, vmap->vm_file->f_flags, credp);
                /* only decrement the execsOrWriters flag if this is not a
                 * writable file. */
-               if (!(vmap->vm_file->f_flags & (FWRITE | FTRUNC)))
-                   vcp->execsOrWriters--;
+               if (!(vcp->states & CRO) )
+                   if (! (vmap->vm_file->f_flags & (FWRITE | FTRUNC)))
+                       vcp->execsOrWriters--;
                vcp->states &= ~CMAPPED;
                crfree(credp);
            } else if ((vmap->vm_file->f_flags & (FWRITE | FTRUNC)))
@@ -552,7 +550,8 @@ afs_linux_mmap(struct file *fp, struct vm_area_struct *vmap)
 
        /* Add an open reference on the first mapping. */
        if (vcp->mapcnt == 0) {
-           vcp->execsOrWriters++;
+           if (!(vcp->states & CRO))
+               vcp->execsOrWriters++;
            vcp->opens++;
            vcp->states |= CMAPPED;
        }
@@ -571,15 +570,15 @@ afs_linux_open(struct inode *ip, struct file *fp)
     int code;
     cred_t *credp = crref();
 
-    AFS_GLOCK();
 #ifdef AFS_LINUX24_ENV
     lock_kernel();
 #endif
+    AFS_GLOCK();
     code = afs_open((struct vcache **)&ip, fp->f_flags, credp);
+    AFS_GUNLOCK();
 #ifdef AFS_LINUX24_ENV
     unlock_kernel();
 #endif
-    AFS_GUNLOCK();
 
     crfree(credp);
     return -code;
@@ -599,19 +598,19 @@ afs_linux_release(struct inode *ip, struct file *fp)
     cred_t *credp = crref();
     struct vcache *vcp = ITOAFS(ip);
 
-    AFS_GLOCK();
 #ifdef AFS_LINUX24_ENV
     lock_kernel();
 #endif
+    AFS_GLOCK();
     if (vcp->flushcnt) {
        vcp->flushcnt--;        /* protected by AFS global lock. */
     } else {
        code = afs_close(vcp, fp->f_flags, credp);
     }
+    AFS_GUNLOCK();
 #ifdef AFS_LINUX24_ENV
     unlock_kernel();
 #endif
-    AFS_GUNLOCK();
 
     crfree(credp);
     return -code;
@@ -629,30 +628,20 @@ afs_linux_fsync(struct file *fp, struct dentry *dp)
     struct inode *ip = FILE_INODE(fp);
     cred_t *credp = crref();
 
-    AFS_GLOCK();
 #ifdef AFS_LINUX24_ENV
     lock_kernel();
 #endif
+    AFS_GLOCK();
     code = afs_fsync(ITOAFS(ip), credp);
+    AFS_GUNLOCK();
 #ifdef AFS_LINUX24_ENV
     unlock_kernel();
 #endif
-    AFS_GUNLOCK();
     crfree(credp);
     return -code;
 
 }
 
-#ifdef NOTUSED
-/* No support for async i/o */
-int afs_linux_fasync(struct inode *ip, struct file *fp, int);
-
-/* I don't think it will, at least not as can be detected here. */
-int afs_linux_check_media_change(kdev_t dev);
-
-/* Revalidate media and file system. */
-int afs_linux_file_revalidate(kdev_t dev);
-#endif /* NOTUSED */
 
 static int
 afs_linux_lock(struct file *fp, int cmd, struct file_lock *flp)
@@ -660,12 +649,7 @@ afs_linux_lock(struct file *fp, int cmd, struct file_lock *flp)
     int code = 0;
     struct vcache *vcp = ITOAFS(FILE_INODE(fp));
     cred_t *credp = crref();
-#ifdef AFS_LINUX24_ENV
-    struct flock64 flock;
-#else
-    struct flock flock;
-#endif
-
+    struct AFS_FLOCK flock;
     /* Convert to a lock format afs_lockctl understands. */
     memset((char *)&flock, 0, sizeof(flock));
     flock.l_type = flp->fl_type;
@@ -737,74 +721,42 @@ afs_linux_flush(struct file *fp)
     return -code;
 }
 
+#if !defined(AFS_LINUX24_ENV)
 /* Not allowed to directly read a directory. */
 ssize_t
 afs_linux_dir_read(struct file * fp, char *buf, size_t count, loff_t * ppos)
 {
     return -EISDIR;
 }
+#endif
 
 
 
-#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,
-};
+#if !defined(AFS_LINUX24_ENV)
+  .read =      afs_linux_dir_read,
+  .lock =      afs_linux_lock,
+  .fsync =     afs_linux_fsync,
 #else
-struct file_operations afs_dir_fops = {
-    NULL,                      /* afs_linux_lseek */
-    afs_linux_dir_read,
-    NULL,                      /* afs_linux_write */
-    afs_linux_readdir,
-    NULL,                      /* afs_linux_select */
-    afs_xioctl,                        /* close enough to use the ported AFS one */
-    NULL,                      /* afs_linux_mmap */
-    afs_linux_open,
-    NULL,                      /* afs_linux_flush */
-    afs_linux_release,
-    afs_linux_fsync,
-    NULL,                      /* afs_linux_fasync */
-    NULL,                      /* afs_linux_check_media_change */
-    NULL,                      /* afs_linux_file_revalidate */
-    afs_linux_lock,
-};
+  .read =      generic_read_dir,
 #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,
+  .readdir =   afs_linux_readdir,
+  .ioctl =     afs_xioctl,
+  .open =      afs_linux_open,
+  .release =   afs_linux_release,
 };
-#else
+
 struct file_operations afs_file_fops = {
-    NULL,                      /* afs_linux_lseek */
-    afs_linux_read,
-    afs_linux_write,
-    NULL,                      /* afs_linux_readdir */
-    NULL,                      /* afs_linux_select */
-    afs_xioctl,                        /* close enough to use the ported AFS one */
-    afs_linux_mmap,
-    afs_linux_open,
-    afs_linux_flush,
-    afs_linux_release,
-    afs_linux_fsync,
-    NULL,                      /* afs_linux_fasync */
-    NULL,                      /* afs_linux_check_media_change */
-    NULL,                      /* afs_linux_file_revalidate */
-    afs_linux_lock,
+  .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,
 };
-#endif
 
 
 /**********************************************************************
@@ -823,6 +775,9 @@ afs_linux_revalidate(struct dentry *dp)
     struct vcache *vcp = ITOAFS(dp->d_inode);
     struct vcache *rootvp = NULL;
 
+#ifdef AFS_LINUX24_ENV
+    lock_kernel();
+#endif
     AFS_GLOCK();
 
     if (afs_fakestat_enable && vcp->mvstat == 1 && vcp->mvid
@@ -831,9 +786,6 @@ afs_linux_revalidate(struct dentry *dp)
        rootvp = afs_FindVCache(vcp->mvid, 0, 0);
        ReleaseSharedLock(&afs_xvcache);
     }
-#ifdef AFS_LINUX24_ENV
-    lock_kernel();
-#endif
 
     /* Make this a fast path (no crref), since it's called so often. */
     if (vcp->states & CStatd) {
@@ -843,12 +795,12 @@ afs_linux_revalidate(struct dentry *dp)
            vcache2fakeinode(rootvp, vcp);
        else
            vcache2inode(vcp);
-#ifdef AFS_LINUX24_ENV
-       unlock_kernel();
-#endif
        if (rootvp)
            afs_PutVCache(rootvp);
        AFS_GUNLOCK();
+#ifdef AFS_LINUX24_ENV
+       unlock_kernel();
+#endif
        return 0;
     }
 
@@ -857,15 +809,25 @@ afs_linux_revalidate(struct dentry *dp)
     if (!code)
        code = afs_VerifyVCache(vcp, &treq);
 
+    AFS_GUNLOCK();
 #ifdef AFS_LINUX24_ENV
     unlock_kernel();
 #endif
-    AFS_GUNLOCK();
     crfree(credp);
 
     return -code;
 }
 
+#if defined(AFS_LINUX26_ENV)
+static int
+afs_linux_getattr(struct vfsmount *mnt, struct dentry *dentry, struct kstat *stat)
+{
+        int err = afs_linux_revalidate(dentry);
+        if (!err)
+                generic_fillattr(dentry->d_inode, stat);
+        return err;
+}
+#endif
 
 /* Validate a dentry. Return 1 if unchanged, 0 if VFS layer should re-evaluate.
  * In kernels 2.2.10 and above, we are passed an additional flags var which
@@ -888,13 +850,17 @@ afs_linux_dentry_revalidate(struct dentry *dp)
     struct vcache *lookupvcp = NULL;
     int code, bad_dentry = 1;
     struct sysname_info sysState;
-    struct vcache *vcp = ITOAFS(dp->d_inode);
-    struct vcache *parentvcp = ITOAFS(dp->d_parent->d_inode);
+    struct vcache *vcp, *parentvcp;
 
-    AFS_GLOCK();
+    sysState.allocked = 0;
+
+#ifdef AFS_LINUX24_ENV
     lock_kernel();
+#endif
+    AFS_GLOCK();
 
-    sysState.allocked = 0;
+    vcp = ITOAFS(dp->d_inode);
+    parentvcp = ITOAFS(dp->d_parent->d_inode);
 
     /* If it's a negative dentry, then there's nothing to do. */
     if (!vcp || !parentvcp)
@@ -946,78 +912,51 @@ afs_linux_dentry_revalidate(struct dentry *dp)
        osi_FreeLargeSpace(name);
 
     AFS_GUNLOCK();
-    crfree(credp);
 
     if (bad_dentry) {
        shrink_dcache_parent(dp);
        d_drop(dp);
     }
-    unlock_kernel();
-
-    return !bad_dentry;
-}
 
-#ifdef notdef
-#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)
+#ifdef AFS_LINUX24_ENV
+    unlock_kernel();
 #endif
-{
-    int code;
-    cred_t *credp;
-    struct vrequest treq;
-    struct inode *ip = AFSTOI(dp->d_inode);
-
-    unsigned long timeout = 3 * HZ;    /* 3 seconds */
-
-    if (!ip)
-       printk("negative dentry: %s\n", dp->d_name.name);
-
-    if (!(flags & LOOKUP_CONTINUE)) {
-       long diff = CURRENT_TIME - dp->d_parent->d_inode->i_mtime;
-
-       if (diff < 15 * 60)
-           timeout = 0;
-    }
-
-    if (time_after(jiffies, dp->d_time + timeout))
-       goto out_bad;
-
-  out_valid:
-    return 1;
+    crfree(credp);
 
-  out_bad:
-    return 0;
+    return !bad_dentry;
 }
-#endif
 
+#if !defined(AFS_LINUX26_ENV)
 /* afs_dentry_iput */
 static void
 afs_dentry_iput(struct dentry *dp, struct inode *ip)
 {
+    int isglock;
+
     if (ICL_SETACTIVE(afs_iclSetp)) {
-       AFS_GLOCK();
+       isglock = ISAFS_GLOCK();
+       if (!isglock) AFS_GLOCK();
        afs_Trace3(afs_iclSetp, CM_TRACE_DENTRYIPUT, ICL_TYPE_POINTER, ip,
                   ICL_TYPE_STRING, dp->d_parent->d_name.name,
                   ICL_TYPE_STRING, dp->d_name.name);
-       AFS_GUNLOCK();
+       if (!isglock) AFS_GUNLOCK();
     }
 
     osi_iput(ip);
 }
+#endif
 
 static int
 afs_dentry_delete(struct dentry *dp)
 {
+    int isglock;
     if (ICL_SETACTIVE(afs_iclSetp)) {
-       AFS_GLOCK();
+       isglock = ISAFS_GLOCK();
+       if (!isglock) AFS_GLOCK();
        afs_Trace3(afs_iclSetp, CM_TRACE_DENTRYDELETE, ICL_TYPE_POINTER,
                   dp->d_inode, ICL_TYPE_STRING, dp->d_parent->d_name.name,
                   ICL_TYPE_STRING, dp->d_name.name);
-       AFS_GUNLOCK();
+       if (!isglock) AFS_GUNLOCK();
     }
 
     if (dp->d_inode && (ITOAFS(dp->d_inode)->states & CUnlinked))
@@ -1026,24 +965,13 @@ afs_dentry_delete(struct dentry *dp)
     return 0;
 }
 
-#if defined(AFS_LINUX24_ENV)
-struct dentry_operations afs_dentry_operations = {
-  d_revalidate:afs_linux_dentry_revalidate,
-  d_iput:afs_dentry_iput,
-  d_delete:afs_dentry_delete,
-};
-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 */
-    NULL,                      /* d_compare */
-    afs_dentry_delete,         /* d_delete(struct dentry *) */
-    NULL,                      /* d_release(struct dentry *) */
-    afs_dentry_iput            /* d_iput(struct dentry *, struct inode *) */
-};
-struct dentry_operations *afs_dops = &afs_dentry_operations;
+  .d_revalidate =      afs_linux_dentry_revalidate,
+  .d_delete =          afs_dentry_delete,
+#if !defined(AFS_LINUX26_ENV)
+  .d_iput =            afs_dentry_iput,
 #endif
+};
 
 /**********************************************************************
  * AFS Linux inode operations
@@ -1070,6 +998,9 @@ afs_linux_create(struct inode *dip, struct dentry *dp, int mode)
     VATTR_NULL(&vattr);
     vattr.va_mode = mode;
 
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     AFS_GLOCK();
     code =
        afs_create(ITOAFS(dip), name, &vattr, NONEXCL, mode,
@@ -1099,12 +1030,15 @@ afs_linux_create(struct inode *dip, struct dentry *dp, int mode)
            ip->i_op = &afs_symlink_iops;
 #endif
 
-       dp->d_op = afs_dops;
+       dp->d_op = &afs_dentry_operations;
        dp->d_time = jiffies;
        d_instantiate(dp, ip);
     }
 
     AFS_GUNLOCK();
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
     crfree(credp);
     return -code;
 }
@@ -1122,9 +1056,14 @@ afs_linux_lookup(struct inode *dip, struct dentry *dp)
     cred_t *credp = crref();
     struct vcache *vcp = NULL;
     const char *comp = dp->d_name.name;
+
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     AFS_GLOCK();
     code = afs_lookup(ITOAFS(dip), comp, &vcp, credp);
-
+    AFS_GUNLOCK();
+    
     if (vcp) {
        struct inode *ip = AFSTOI(vcp);
        /* Reset ops if symlink or directory. */
@@ -1144,6 +1083,12 @@ afs_linux_lookup(struct inode *dip, struct dentry *dp)
            printk
                ("afs_linux_lookup: ip->i_mode 0x%x  dp->d_name.name %s  code %d\n",
                 ip->i_mode, dp->d_name.name, code);
+#ifdef STRUCT_INODE_HAS_I_SECURITY
+       if (ip->i_security == NULL) {
+           if (security_inode_alloc(ip))
+               panic("afs_linux_lookup: Cannot allocate inode security");
+       }
+#endif
 #else
        if (S_ISDIR(ip->i_mode))
            ip->i_op = &afs_dir_iops;
@@ -1152,10 +1097,12 @@ afs_linux_lookup(struct inode *dip, struct dentry *dp)
 #endif
     }
     dp->d_time = jiffies;
-    dp->d_op = afs_dops;
+    dp->d_op = &afs_dentry_operations;
     d_add(dp, AFSTOI(vcp));
 
-    AFS_GUNLOCK();
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
     crfree(credp);
 
     /* It's ok for the file to not be found. That's noted by the caller by
@@ -1201,11 +1148,17 @@ afs_linux_unlink(struct inode *dip, struct dentry *dp)
     cred_t *credp = crref();
     const char *name = dp->d_name.name;
 
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     AFS_GLOCK();
     code = afs_remove(ITOAFS(dip), name, credp);
     AFS_GUNLOCK();
     if (!code)
        d_drop(dp);
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
     crfree(credp);
     return -code;
 }
@@ -1241,23 +1194,29 @@ afs_linux_mkdir(struct inode *dip, struct dentry *dp, int mode)
     struct vattr vattr;
     const char *name = dp->d_name.name;
 
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     AFS_GLOCK();
     VATTR_NULL(&vattr);
     vattr.va_mask = ATTR_MODE;
     vattr.va_mode = mode;
     code = afs_mkdir(ITOAFS(dip), name, &vattr, &tvcp, credp);
+    AFS_GUNLOCK();
 
     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;
+       dp->d_op = &afs_dentry_operations;
        dp->d_time = jiffies;
        d_instantiate(dp, AFSTOI(tvcp));
     }
 
-    AFS_GUNLOCK();
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
     crfree(credp);
     return -code;
 }
@@ -1269,8 +1228,12 @@ afs_linux_rmdir(struct inode *dip, struct dentry *dp)
     cred_t *credp = crref();
     const char *name = dp->d_name.name;
 
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     AFS_GLOCK();
     code = afs_rmdir(ITOAFS(dip), name, credp);
+    AFS_GUNLOCK();
 
     /* Linux likes to see ENOTEMPTY returned from an rmdir() syscall
      * that failed because a directory is not empty. So, we map
@@ -1284,7 +1247,9 @@ afs_linux_rmdir(struct inode *dip, struct dentry *dp)
        d_drop(dp);
     }
 
-    AFS_GUNLOCK();
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
     crfree(credp);
     return -code;
 }
@@ -1300,17 +1265,25 @@ afs_linux_rename(struct inode *oldip, struct dentry *olddp,
     const char *oldname = olddp->d_name.name;
     const char *newname = newdp->d_name.name;
 
+#if defined(AFS_LINUX26_ENV)
+    lock_kernel();
+#endif
     /* Remove old and new entries from name hash. New one will change below.
      * While it's optimal to catch failures and re-insert newdp into hash,
      * it's also error prone and in that case we're already dealing with error
      * cases. Let another lookup put things right, if need be.
      */
-    if (!list_empty(&olddp->d_hash)) {
+#if defined(AFS_LINUX26_ENV)
+    if (!d_unhashed(olddp))
        d_drop(olddp);
-    }
-    if (!list_empty(&newdp->d_hash)) {
+    if (!d_unhashed(newdp))
        d_drop(newdp);
-    }
+#else
+    if (!list_empty(&olddp->d_hash))
+       d_drop(olddp);
+    if (!list_empty(&newdp->d_hash))
+       d_drop(newdp);
+#endif
     AFS_GLOCK();
     code = afs_rename(ITOAFS(oldip), oldname, ITOAFS(newip), newname, credp);
     AFS_GUNLOCK();
@@ -1321,6 +1294,10 @@ afs_linux_rename(struct inode *oldip, struct dentry *olddp,
        d_move(olddp, newdp);
     }
 
+#if defined(AFS_LINUX26_ENV)
+    unlock_kernel();
+#endif
+
     crfree(credp);
     return -code;
 }
@@ -1421,16 +1398,13 @@ afs_linux_readpage(struct file *fp, struct page *pp)
     uio_t tuio;
     struct iovec iovec;
     struct inode *ip = FILE_INODE(fp);
-    int cnt = atomic_read(&pp->count);
+    int cnt = page_count(pp);
     struct vcache *avc = ITOAFS(ip);
 
-    AFS_GLOCK();
-    afs_Trace4(afs_iclSetp, CM_TRACE_READPAGE, ICL_TYPE_POINTER, ip, 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 */
@@ -1439,8 +1413,17 @@ afs_linux_readpage(struct file *fp, struct page *pp)
 
     setup_uio(&tuio, &iovec, (char *)address, offset, PAGESIZE, UIO_READ,
              AFS_UIOSYS);
+#ifdef AFS_LINUX24_ENV
+    lock_kernel();
+#endif
+    AFS_GLOCK();
+    afs_Trace4(afs_iclSetp, CM_TRACE_READPAGE, ICL_TYPE_POINTER, ip, ICL_TYPE_POINTER, pp, ICL_TYPE_INT32, cnt, ICL_TYPE_INT32, 99999);        /* not a possible code value */
     code = afs_rdwr(avc, &tuio, UIO_READ, 0, credp);
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
+    afs_Trace4(afs_iclSetp, CM_TRACE_READPAGE, ICL_TYPE_POINTER, ip,
+              ICL_TYPE_POINTER, pp, ICL_TYPE_INT32, cnt, ICL_TYPE_INT32,
+              code);
+    AFS_GUNLOCK();
+#ifdef AFS_LINUX24_ENV
     unlock_kernel();
 #endif
 
@@ -1455,6 +1438,7 @@ afs_linux_readpage(struct file *fp, struct page *pp)
        set_bit(PG_uptodate, &pp->flags);
 #endif
     }
+
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,0)
     kunmap(pp);
     UnlockPage(pp);
@@ -1468,6 +1452,7 @@ afs_linux_readpage(struct file *fp, struct page *pp)
        struct dcache *tdc;
        struct vrequest treq;
 
+       AFS_GLOCK();
        code = afs_InitReq(&treq, credp);
        if (!code && !NBObtainWriteLock(&avc->lock, 534)) {
            tdc = afs_FindDCache(avc, offset);
@@ -1478,13 +1463,10 @@ afs_linux_readpage(struct file *fp, struct page *pp)
            }
            ReleaseWriteLock(&avc->lock);
        }
+       AFS_GUNLOCK();
     }
 
     crfree(credp);
-    afs_Trace4(afs_iclSetp, CM_TRACE_READPAGE, ICL_TYPE_POINTER, ip,
-              ICL_TYPE_POINTER, pp, ICL_TYPE_INT32, cnt, ICL_TYPE_INT32,
-              code);
-    AFS_GUNLOCK();
     return -code;
 }
 
@@ -1522,24 +1504,6 @@ afs_linux_writepage(struct page *pp)
 }
 #endif
 
-#ifdef NOTUSED
-/* afs_linux_bmap - supports generic_readpage, but we roll our own. */
-int
-afs_linux_bmap(struct inode *ip, int)
-{
-    return -EINVAL;
-}
-
-/* afs_linux_truncate
- * Handles discarding disk blocks if this were a device. ext2 indicates we
- * may need to zero partial last pages of memory mapped files.
- */
-void
-afs_linux_truncate(struct inode *ip)
-{
-}
-#endif
-
 /* afs_linux_permission
  * Check access rights - returns error if can't check or permission denied.
  */
@@ -1565,15 +1529,6 @@ afs_linux_permission(struct inode *ip, int mode)
 }
 
 
-#ifdef NOTUSED
-/* msdos sector mapping hack for memory mapping. */
-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,
@@ -1593,7 +1548,7 @@ afs_linux_writepage_sync(struct inode *ip, struct page *pp,
 
     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_POINTER, pp, ICL_TYPE_INT32, page_count(pp),
               ICL_TYPE_INT32, 99999);
 
     setup_uio(&tuio, &iovec, buffer, base, count, UIO_WRITE, AFS_UIOSYS);
@@ -1615,7 +1570,7 @@ afs_linux_writepage_sync(struct inode *ip, struct page *pp,
     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_POINTER, pp, ICL_TYPE_INT32, page_count(pp),
               ICL_TYPE_INT32, code);
 
     crfree(credp);
@@ -1654,7 +1609,7 @@ afs_linux_updatepage(struct file *fp, struct page *pp, unsigned long offset,
     credp = crref();
     AFS_GLOCK();
     afs_Trace4(afs_iclSetp, CM_TRACE_UPDATEPAGE, ICL_TYPE_POINTER, vcp,
-              ICL_TYPE_POINTER, pp, ICL_TYPE_INT32, atomic_read(&pp->count),
+              ICL_TYPE_POINTER, pp, ICL_TYPE_INT32, page_count(pp),
               ICL_TYPE_INT32, 99999);
     setup_uio(&tuio, &iovec, page_addr + offset,
              (afs_offs_t) (pageoff(pp) + offset), count, UIO_WRITE,
@@ -1666,7 +1621,7 @@ afs_linux_updatepage(struct file *fp, struct page *pp, unsigned long offset,
 
     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_POINTER, pp, ICL_TYPE_INT32, page_count(pp),
               ICL_TYPE_INT32, code);
 
     AFS_GUNLOCK();
@@ -1684,11 +1639,11 @@ afs_linux_commit_write(struct file *file, struct page *page, unsigned offset,
 {
     int code;
 
-    AFS_GLOCK();
     lock_kernel();
+    AFS_GLOCK();
     code = afs_linux_updatepage(file, page, offset, to - offset);
-    unlock_kernel();
     AFS_GUNLOCK();
+    unlock_kernel();
     kunmap(page);
 
     return code;
@@ -1705,88 +1660,58 @@ afs_linux_prepare_write(struct file *file, struct page *page, unsigned from,
 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;
+#if defined(AFS_LINUX26_ENV)
+  .permission =                afs_linux_permission,
+  .getattr =           afs_linux_getattr,
+  .setattr =           afs_notify_change,
+#elif defined(AFS_LINUX24_ENV)
+  .permission =                afs_linux_permission,
+  .revalidate =                afs_linux_revalidate,
+  .setattr =           afs_notify_change,
 #else
-struct inode_operations afs_iops = {
-    &afs_file_fops,            /* file operations */
-    NULL,                      /* afs_linux_create */
-    NULL,                      /* afs_linux_lookup */
-    NULL,                      /* afs_linux_link */
-    NULL,                      /* afs_linux_unlink */
-    NULL,                      /* afs_linux_symlink */
-    NULL,                      /* afs_linux_mkdir */
-    NULL,                      /* afs_linux_rmdir */
-    NULL,                      /* afs_linux_mknod */
-    NULL,                      /* afs_linux_rename */
-    NULL,                      /* afs_linux_readlink */
-    NULL,                      /* afs_linux_follow_link */
-    afs_linux_readpage,
-    NULL,                      /* afs_linux_writepage */
-    NULL,                      /* afs_linux_bmap */
-    NULL,                      /* afs_linux_truncate */
-    afs_linux_permission,
-    NULL,                      /* afs_linux_smap */
-    afs_linux_updatepage,
-    afs_linux_revalidate,
+  .default_file_ops =  &afs_file_fops,
+  .readpage =          afs_linux_readpage,
+  .revalidate =                afs_linux_revalidate,
+  .updatepage =                afs_linux_updatepage,
+#endif
 };
 
-struct inode_operations *afs_ops = &afs_iops;
+#if defined(AFS_LINUX24_ENV)
+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,
+};
 #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,
-};
+#if !defined(AFS_LINUX24_ENV)
+  .default_file_ops =  &afs_dir_fops,
 #else
-struct inode_operations afs_dir_iops = {
-    &afs_dir_fops,             /* file operations for directories */
-    afs_linux_create,
-    afs_linux_lookup,
-    afs_linux_link,
-    afs_linux_unlink,
-    afs_linux_symlink,
-    afs_linux_mkdir,
-    afs_linux_rmdir,
-    NULL,                      /* afs_linux_mknod */
-    afs_linux_rename,
-    NULL,                      /* afs_linux_readlink */
-    NULL,                      /* afs_linux_follow_link */
-    NULL,                      /* afs_linux_readpage */
-    NULL,                      /* afs_linux_writepage */
-    NULL,                      /* afs_linux_bmap */
-    NULL,                      /* afs_linux_truncate */
-    afs_linux_permission,
-    NULL,                      /* afs_linux_smap */
-    NULL,                      /* afs_linux_updatepage */
-    afs_linux_revalidate,
-};
+  .setattr =           afs_notify_change,
 #endif
+  .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,
+#if defined(AFS_LINUX26_ENV)
+  .getattr =           afs_linux_getattr,
+#else
+  .revalidate =                afs_linux_revalidate,
+#endif
+  .permission =                afs_linux_permission,
+};
 
 /* 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.
@@ -1799,15 +1724,15 @@ afs_symlink_filler(struct file *file, struct page *page)
     char *p = (char *)kmap(page);
     int code;
 
-    AFS_GLOCK();
     lock_kernel();
+    AFS_GLOCK();
     code = afs_linux_ireadlink(ip, p, PAGE_SIZE, AFS_UIOSYS);
+    AFS_GUNLOCK();
 
     if (code < 0)
        goto fail;
     p[code] = '\0';            /* null terminate? */
     unlock_kernel();
-    AFS_GUNLOCK();
 
     SetPageUptodate(page);
     kunmap(page);
@@ -1816,7 +1741,6 @@ afs_symlink_filler(struct file *file, struct page *page)
 
   fail:
     unlock_kernel();
-    AFS_GUNLOCK();
 
     SetPageError(page);
     kunmap(page);
@@ -1825,35 +1749,19 @@ afs_symlink_filler(struct file *file, struct page *page)
 }
 
 struct address_space_operations afs_symlink_aops = {
-  readpage:afs_symlink_filler
+  .readpage =  afs_symlink_filler
 };
+#endif
 
 struct inode_operations afs_symlink_iops = {
-  readlink:page_readlink,
-  follow_link:page_follow_link,
-  setattr:afs_notify_change,
-};
+#if defined(AFS_LINUX24_ENV)
+  .readlink =          page_readlink,
+  .follow_link =       page_follow_link,
+  .setattr =           afs_notify_change,
 #else
-struct inode_operations afs_symlink_iops = {
-    NULL,                      /* file operations */
-    NULL,                      /* create */
-    NULL,                      /* lookup */
-    NULL,                      /* link */
-    NULL,                      /* unlink */
-    NULL,                      /* symlink */
-    NULL,                      /* mkdir */
-    NULL,                      /* rmdir */
-    NULL,                      /* afs_linux_mknod */
-    NULL,                      /* rename */
-    afs_linux_readlink,
-    afs_linux_follow_link,
-    NULL,                      /* readpage */
-    NULL,                      /* afs_linux_writepage */
-    NULL,                      /* afs_linux_bmap */
-    NULL,                      /* afs_linux_truncate */
-    afs_linux_permission,      /* tho the code appears to indicate not used? */
-    NULL,                      /* afs_linux_smap */
-    NULL,                      /* updatepage */
-    afs_linux_revalidate,      /* tho the code appears to indicate not used? */
-};
+  .readlink =          afs_linux_readlink,
+  .follow_link =       afs_linux_follow_link,
+  .permission =                afs_linux_permission,
+  .revalidate =                afs_linux_revalidate,
 #endif
+};