]> git.saurik.com Git - apple/xnu.git/blobdiff - bsd/hfs/hfs_readwrite.c
xnu-344.32.tar.gz
[apple/xnu.git] / bsd / hfs / hfs_readwrite.c
index 825ec1ca55976a6144e1589fb3feea05ea17adc8..e7fba04137416943204f3ba9c723f0c69dc798a3 100644 (file)
@@ -1,5 +1,5 @@
 /*
 /*
- * Copyright (c) 2000 Apple Computer, Inc. All rights reserved.
+ * Copyright (c) 2000-2002 Apple Computer, Inc. All rights reserved.
  *
  * @APPLE_LICENSE_HEADER_START@
  * 
  *
  * @APPLE_LICENSE_HEADER_START@
  * 
  */
 /*     @(#)hfs_readwrite.c     1.0
  *
  */
 /*     @(#)hfs_readwrite.c     1.0
  *
- *     (c) 1990, 1992 NeXT Computer, Inc.  All Rights Reserved
- *     (c) 1998       Apple Computer, Inc.  All Rights Reserved
+ *     (c) 1998-2001 Apple Computer, Inc.  All Rights Reserved
  *     
  *     
- *
  *     hfs_readwrite.c -- vnode operations to deal with reading and writing files.
  *
  *     hfs_readwrite.c -- vnode operations to deal with reading and writing files.
  *
- *     MODIFICATION HISTORY:
- *      9-Nov-1999     Scott Roberts   hfs_allocate now returns sizes based on allocation block boundaries (#2398794)
- *      3-Feb-1999     Pat Dirks               Merged in Joe's change to hfs_truncate to skip vinvalbuf if LEOF isn't changing (#2302796)
- *                                                             Removed superfluous (and potentially dangerous) second call to vinvalbuf() in hfs_truncate.
- *      2-Dec-1998     Pat Dirks               Added support for read/write bootstrap ioctls.
- *     10-Nov-1998     Pat Dirks               Changed read/write/truncate logic to optimize block sizes for first extents of a file.
- *                              Changed hfs_strategy to correct I/O sizes from cluser code I/O requests in light of
- *                              different block sizing.  Changed bexpand to handle RELEASE_BUFFER flag.
- *     22-Sep-1998     Don Brady               Changed truncate zero-fill to use bwrite after several bawrites have been queued.
- *     11-Sep-1998     Pat Dirks               Fixed buffering logic to not rely on B_CACHE, which is set for empty buffers that
- *                                                             have been pre-read by cluster_read (use b_validend > 0 instead).
- *  27-Aug-1998        Pat Dirks               Changed hfs_truncate to use cluster_write in place of bawrite where possible.
- *     25-Aug-1998     Pat Dirks               Changed hfs_write to do small device-block aligned writes into buffers without doing
- *                                                             read-ahead of the buffer.  Added bexpand to deal with incomplete [dirty] buffers.
- *                                                             Fixed can_cluster macro to use MAXPHYSIO instead of MAXBSIZE.
- *     19-Aug-1998     Don Brady               Remove optimization in hfs_truncate that prevented extra physical blocks from
- *                                                             being truncated (radar #2265750). Also set fcb->fcbEOF before calling vinvalbuf.
- *      7-Jul-1998     Pat Dirks               Added code to honor IO_NOZEROFILL in hfs_truncate.
- *     16-Jul-1998     Don Brady               In hfs_bmap use MAXPHYSIO instead of MAXBSIZE when calling MapFileBlockC (radar #2263753).
- *     16-Jul-1998     Don Brady               Fix error handling in hfs_allocate (radar #2252265).
- *     04-Jul-1998     chw                             Synchronized options in hfs_allocate with flags in call to ExtendFileC
- *     25-Jun-1998     Don Brady               Add missing blockNo incrementing to zero fill loop in hfs_truncate.
- *     22-Jun-1998     Don Brady               Add bp = NULL assignment after brelse in hfs_read.
- *      4-Jun-1998     Pat Dirks               Split off from hfs_vnodeops.c
  */
 
 #include <sys/param.h>
  */
 
 #include <sys/param.h>
@@ -61,7 +35,6 @@
 #include <sys/stat.h>
 #include <sys/buf.h>
 #include <sys/proc.h>
 #include <sys/stat.h>
 #include <sys/buf.h>
 #include <sys/proc.h>
-//#include <mach/machine/vm_types.h>
 #include <sys/vnode.h>
 #include <sys/uio.h>
 
 #include <sys/vnode.h>
 #include <sys/uio.h>
 
 #include <sys/kdebug.h>
 
 #include       "hfs.h"
 #include <sys/kdebug.h>
 
 #include       "hfs.h"
-#include       "hfs_dbg.h"
 #include       "hfs_endian.h"
 #include       "hfs_endian.h"
+#include       "hfs_quota.h"
 #include       "hfscommon/headers/FileMgrInternal.h"
 #include       "hfscommon/headers/BTreesInternal.h"
 #include       "hfscommon/headers/FileMgrInternal.h"
 #include       "hfscommon/headers/BTreesInternal.h"
+#include       "hfs_cnode.h"
+#include       "hfs_dbg.h"
 
 
+extern int overflow_extents(struct filefork *fp);
 
 #define can_cluster(size) ((((size & (4096-1))) == 0) && (size <= (MAXPHYSIO/2)))
 
 
 #define can_cluster(size) ((((size & (4096-1))) == 0) && (size <= (MAXPHYSIO/2)))
 
@@ -87,13 +63,6 @@ enum {
 
 extern u_int32_t GetLogicalBlockSize(struct vnode *vp);
 
 
 extern u_int32_t GetLogicalBlockSize(struct vnode *vp);
 
-#if DBG_VOP_TEST_LOCKS
-extern void DbgVopTest(int maxSlots, int retval, VopDbgStoreRec *VopDbgStore, char *funcname);
-#endif
-
-#if HFS_DIAGNOSTIC
-void debug_check_blocksizes(struct vnode *vp);
-#endif
 
 /*****************************************************************************
 *
 
 /*****************************************************************************
 *
@@ -114,123 +83,83 @@ void debug_check_blocksizes(struct vnode *vp);
 
 int
 hfs_read(ap)
 
 int
 hfs_read(ap)
-struct vop_read_args /* {
-    struct vnode *a_vp;
-    struct uio *a_uio;
-    int a_ioflag;
-    struct ucred *a_cred;
-} */ *ap;
+       struct vop_read_args /* {
+               struct vnode *a_vp;
+               struct uio *a_uio;
+               int a_ioflag;
+               struct ucred *a_cred;
+       } */ *ap;
 {
 {
-    register struct vnode      *vp;
-    struct hfsnode                     *hp;
-    register struct uio        *uio;
-    struct buf                                 *bp;
-    daddr_t                            logBlockNo;
-    u_long                                     fragSize, moveSize, startOffset, ioxfersize;
-    int                                                devBlockSize = 0;
-    off_t                                      bytesRemaining;
-    int                                        retval;
-    u_short                            mode;
-    FCB                                                *fcb;
-
-    DBG_FUNC_NAME("hfs_read");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-
-    vp = ap->a_vp;
-    hp = VTOH(vp);
-    fcb = HTOFCB(hp);
-    mode = hp->h_meta->h_mode;
-    uio = ap->a_uio;
-
-#if HFS_DIAGNOSTIC
-    if (uio->uio_rw != UIO_READ)
-        panic("%s: mode", funcname);
-#endif
-
-    /* Can only read files */
-    if (ap->a_vp->v_type != VREG && ap->a_vp->v_type != VLNK) {
-        DBG_VOP_LOCKS_TEST(EISDIR);
-        return (EISDIR);
-    }
-    DBG_RW(("\tfile size Ox%X\n", (u_int)fcb->fcbEOF));
-    DBG_RW(("\tstarting at offset Ox%X of file, length Ox%X\n", (u_int)uio->uio_offset, (u_int)uio->uio_resid));
-
-#if HFS_DIAGNOSTIC
-    debug_check_blocksizes(vp);
-#endif
-
-    /*
-     * If they didn't ask for any data, then we are done.
-     */
-    if (uio->uio_resid == 0) {
-        DBG_VOP_LOCKS_TEST(E_NONE);
-        return (E_NONE);
-    }
-
-    /* cant read from a negative offset */
-    if (uio->uio_offset < 0) {
-        DBG_VOP_LOCKS_TEST(EINVAL);
-        return (EINVAL);
-    }
-
-    if (uio->uio_offset > fcb->fcbEOF) {
-        if ( (!ISHFSPLUS(VTOVCB(vp))) && (uio->uio_offset > (off_t)MAXHFSFILESIZE))
-            retval = EFBIG;
-        else
-            retval = E_NONE;
-
-        DBG_VOP_LOCKS_TEST(retval);
-        return (retval);
-    }
+       register struct uio *uio = ap->a_uio;
+       register struct vnode *vp = ap->a_vp;
+       struct cnode *cp;
+       struct filefork *fp;
+       struct buf *bp;
+       daddr_t logBlockNo;
+       u_long fragSize, moveSize, startOffset, ioxfersize;
+       int devBlockSize = 0;
+       off_t bytesRemaining;
+       int retval = 0;
+       off_t filesize;
+       off_t filebytes;
+
+       /* Preflight checks */
+       if (vp->v_type != VREG && vp->v_type != VLNK)
+               return (EISDIR);        /* HFS can only read files */
+       if (uio->uio_resid == 0)
+               return (0);             /* Nothing left to do */
+       if (uio->uio_offset < 0)
+               return (EINVAL);        /* cant read from a negative offset */
+
+       cp = VTOC(vp);
+       fp = VTOF(vp);
+       filesize = fp->ff_size;
+       filebytes = (off_t)fp->ff_blocks * (off_t)VTOVCB(vp)->blockSize;
+       if (uio->uio_offset > filesize) {
+               if ((!ISHFSPLUS(VTOVCB(vp))) && (uio->uio_offset > (off_t)MAXHFSFILESIZE))
+                       return (EFBIG);
+               else
+                       return (0);
+       }
 
 
-    VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
+       VOP_DEVBLOCKSIZE(cp->c_devvp, &devBlockSize);
 
 
-    KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 12)) | DBG_FUNC_START,
-                 (int)uio->uio_offset, uio->uio_resid, (int)fcb->fcbEOF,  (int)fcb->fcbPLen, 0);
+       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 12)) | DBG_FUNC_START,
+               (int)uio->uio_offset, uio->uio_resid, (int)filesize, (int)filebytes, 0);
 
 
-    if (UBCISVALID(vp))
-        retval = cluster_read(vp, uio, (off_t)fcb->fcbEOF, devBlockSize, 0);
-    else {
+       if (UBCISVALID(vp)) {
+               retval = cluster_read(vp, uio, filesize, devBlockSize, 0);
+       } else {
 
 
-        for (retval = 0, bp = NULL; uio->uio_resid > 0; bp = NULL) {
+               for (retval = 0, bp = NULL; uio->uio_resid > 0; bp = NULL) {
 
 
-            if ((bytesRemaining = (fcb->fcbEOF - uio->uio_offset)) <= 0)
+            if ((bytesRemaining = (filesize - uio->uio_offset)) <= 0)
                 break;
 
             logBlockNo  = (daddr_t)(uio->uio_offset / PAGE_SIZE_64);
             startOffset = (u_long) (uio->uio_offset & PAGE_MASK_64);
             fragSize    = PAGE_SIZE;
 
                 break;
 
             logBlockNo  = (daddr_t)(uio->uio_offset / PAGE_SIZE_64);
             startOffset = (u_long) (uio->uio_offset & PAGE_MASK_64);
             fragSize    = PAGE_SIZE;
 
-            if (((logBlockNo * PAGE_SIZE) + fragSize) < fcb->fcbEOF)
+            if (((logBlockNo * PAGE_SIZE) + fragSize) < filesize)
                 ioxfersize = fragSize;
             else {
                 ioxfersize = fragSize;
             else {
-                ioxfersize = fcb->fcbEOF - (logBlockNo * PAGE_SIZE);
+                ioxfersize = filesize - (logBlockNo * PAGE_SIZE);
                 ioxfersize = (ioxfersize + (devBlockSize - 1)) & ~(devBlockSize - 1);
             }
                 ioxfersize = (ioxfersize + (devBlockSize - 1)) & ~(devBlockSize - 1);
             }
-            DBG_RW(("\tat logBlockNo Ox%X, with Ox%lX left to read\n", logBlockNo, (UInt32)uio->uio_resid));
-            moveSize = ioxfersize;
-            DBG_RW(("\tmoveSize = Ox%lX; ioxfersize = Ox%lX; startOffset = Ox%lX.\n",
-                    moveSize, ioxfersize, startOffset));
-            DBG_ASSERT(moveSize >= startOffset);
-            moveSize -= startOffset;
+               moveSize = ioxfersize;
+               moveSize -= startOffset;
 
             if (bytesRemaining < moveSize)
                 moveSize = bytesRemaining;
 
             if (uio->uio_resid < moveSize) {
                 moveSize = uio->uio_resid;
 
             if (bytesRemaining < moveSize)
                 moveSize = bytesRemaining;
 
             if (uio->uio_resid < moveSize) {
                 moveSize = uio->uio_resid;
-                DBG_RW(("\treducing moveSize to Ox%lX (uio->uio_resid).\n", moveSize));
             };
             if (moveSize == 0) {
                 break;
             };
 
             };
             if (moveSize == 0) {
                 break;
             };
 
-            DBG_RW(("\tat logBlockNo Ox%X, extent of Ox%lX, xfer of Ox%lX; moveSize = Ox%lX\n", logBlockNo, fragSize, ioxfersize, moveSize));
-
-            if (( uio->uio_offset + fragSize) >= fcb->fcbEOF) {
+            if (( uio->uio_offset + fragSize) >= filesize) {
                 retval = bread(vp, logBlockNo, ioxfersize, NOCRED, &bp);
 
             } else if (logBlockNo - 1 == vp->v_lastr && !(vp->v_flag & VRAOFF)) {
                 retval = bread(vp, logBlockNo, ioxfersize, NOCRED, &bp);
 
             } else if (logBlockNo - 1 == vp->v_lastr && !(vp->v_flag & VRAOFF)) {
@@ -238,10 +167,10 @@ struct vop_read_args /* {
                 int nextsize;
 
                 if (((nextLogBlockNo * PAGE_SIZE) +
                 int nextsize;
 
                 if (((nextLogBlockNo * PAGE_SIZE) +
-                     (daddr_t)fragSize) < fcb->fcbEOF)
+                     (daddr_t)fragSize) < filesize)
                     nextsize = fragSize;
                 else {
                     nextsize = fragSize;
                 else {
-                    nextsize = fcb->fcbEOF - (nextLogBlockNo * PAGE_SIZE);
+                    nextsize = filesize - (nextLogBlockNo * PAGE_SIZE);
                     nextsize = (nextsize + (devBlockSize - 1)) & ~(devBlockSize - 1);
                 }
                 retval = breadn(vp, logBlockNo, ioxfersize, &nextLogBlockNo, &nextsize, 1, NOCRED, &bp);
                     nextsize = (nextsize + (devBlockSize - 1)) & ~(devBlockSize - 1);
                 }
                 retval = breadn(vp, logBlockNo, ioxfersize, &nextLogBlockNo, &nextsize, 1, NOCRED, &bp);
@@ -275,41 +204,29 @@ struct vop_read_args /* {
             if ((startOffset + moveSize) > bp->b_bcount)
                 panic("hfs_read: bad startOffset or moveSize\n");
 
             if ((startOffset + moveSize) > bp->b_bcount)
                 panic("hfs_read: bad startOffset or moveSize\n");
 
-            DBG_RW(("\tcopying Ox%lX bytes from %lX; resid = Ox%lX...\n", moveSize, (char *)bp->b_data + startOffset, bp->b_resid));
-
             if ((retval = uiomove((caddr_t)bp->b_data + startOffset, (int)moveSize, uio)))
                 break;
 
             if ((retval = uiomove((caddr_t)bp->b_data + startOffset, (int)moveSize, uio)))
                 break;
 
-            if (S_ISREG(mode) &&
-                (((startOffset + moveSize) == fragSize) || (uio->uio_offset == fcb->fcbEOF))) {
+            if (S_ISREG(cp->c_mode) &&
+                (((startOffset + moveSize) == fragSize) || (uio->uio_offset == filesize))) {
                 bp->b_flags |= B_AGE;
             };
 
                 bp->b_flags |= B_AGE;
             };
 
-            DBG_ASSERT(bp->b_bcount == bp->b_validend);
-
             brelse(bp);
             /* Start of loop resets bp to NULL before reaching outside this block... */
         }
 
             brelse(bp);
             /* Start of loop resets bp to NULL before reaching outside this block... */
         }
 
-        if (bp != NULL) {
-            DBG_ASSERT(bp->b_bcount == bp->b_validend);
-            brelse(bp);
-        };
-    }
-
-    if (HTOVCB(hp)->vcbSigWord == kHFSPlusSigWord)
-        hp->h_nodeflags |= IN_ACCESS;
-
-    DBG_VOP_LOCKS_TEST(retval);
+               if (bp != NULL) {
+                       brelse(bp);
+               }
+       }
 
 
-    #if HFS_DIAGNOSTIC
-        debug_check_blocksizes(vp);
-    #endif
+       cp->c_flag |= C_ACCESS;
 
 
-    KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 12)) | DBG_FUNC_END,
-                (int)uio->uio_offset, uio->uio_resid, (int)fcb->fcbEOF,  (int)fcb->fcbPLen, 0);
+       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 12)) | DBG_FUNC_END,
+               (int)uio->uio_offset, uio->uio_resid, (int)filesize,  (int)filebytes, 0);
 
 
-    return (retval);
+       return (retval);
 }
 
 /*
 }
 
 /*
@@ -325,144 +242,163 @@ struct vop_read_args /* {
      */
 int
 hfs_write(ap)
      */
 int
 hfs_write(ap)
-struct vop_write_args /* {
-    struct vnode *a_vp;
-    struct uio *a_uio;
-    int a_ioflag;
-    struct ucred *a_cred;
-} */ *ap;
+       struct vop_write_args /* {
+               struct vnode *a_vp;
+               struct uio *a_uio;
+               int a_ioflag;
+               struct ucred *a_cred;
+       } */ *ap;
 {
 {
-    struct hfsnode             *hp = VTOH(ap->a_vp);
-    struct uio                         *uio = ap->a_uio;
-    struct vnode               *vp = ap->a_vp ;
-    struct vnode               *dev;
-    struct buf                         *bp;
-    struct proc                *p, *cp;
-    struct timeval tv;
-    FCB                                        *fcb = HTOFCB(hp);
-    ExtendedVCB                        *vcb = HTOVCB(hp);
+       struct vnode *vp = ap->a_vp;
+       struct uio *uio = ap->a_uio;
+       struct cnode *cp;
+       struct filefork *fp;
+       struct buf *bp;
+       struct proc *p;
+       struct timeval tv;
+       ExtendedVCB *vcb;
     int                                        devBlockSize = 0;
     daddr_t                    logBlockNo;
     long                               fragSize;
     off_t                              origFileSize, currOffset, writelimit, bytesToAdd;
     off_t                              actualBytesAdded;
     u_long                             blkoffset, resid, xfersize, clearSize;
     int                                        devBlockSize = 0;
     daddr_t                    logBlockNo;
     long                               fragSize;
     off_t                              origFileSize, currOffset, writelimit, bytesToAdd;
     off_t                              actualBytesAdded;
     u_long                             blkoffset, resid, xfersize, clearSize;
-    int                                        flags, ioflag;
+    int                                        eflags, ioflag;
     int                                retval;
     int                                retval;
-    DBG_FUNC_NAME("hfs_write");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-    DBG_RW(("\thfsnode 0x%x (%s)\n", (u_int)hp, H_NAME(hp)));
-    DBG_RW(("\tstarting at offset Ox%lX of file, length Ox%lX\n", (UInt32)uio->uio_offset, (UInt32)uio->uio_resid));
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
+       off_t filebytes;
+       u_long fileblocks;
+       struct hfsmount *hfsmp;
+       int started_tr = 0, grabbed_lock = 0;
 
 
-    dev = hp->h_meta->h_devvp;
+       ioflag = ap->a_ioflag;
 
 
-#if HFS_DIAGNOSTIC
-    debug_check_blocksizes(vp);
-#endif
-
-    if (uio->uio_offset < 0) {
-        DBG_VOP_LOCKS_TEST(EINVAL);
-        return (EINVAL);
-    }
-
-    if (uio->uio_resid == 0) {
-        DBG_VOP_LOCKS_TEST(E_NONE);
-        return (E_NONE);
-    }
-
-    if (ap->a_vp->v_type != VREG && ap->a_vp->v_type != VLNK) {                /* Can only write files */
-        DBG_VOP_LOCKS_TEST(EISDIR);
-        return (EISDIR);
-    };
-
-#if HFS_DIAGNOSTIC
-       if (uio->uio_rw != UIO_WRITE)
-               panic("%s: mode", funcname);
-#endif
-
-    ioflag = ap->a_ioflag;
-    uio = ap->a_uio;
-    vp = ap->a_vp;
-
-    if (ioflag & IO_APPEND) uio->uio_offset = fcb->fcbEOF;
-    if ((hp->h_meta->h_pflags & APPEND) && uio->uio_offset != fcb->fcbEOF)
-       return (EPERM);
+       if (uio->uio_offset < 0)
+               return (EINVAL);
+       if (uio->uio_resid == 0)
+               return (E_NONE);
+       if (vp->v_type != VREG && vp->v_type != VLNK)
+               return (EISDIR);        /* Can only write files */
+
+       cp = VTOC(vp);
+       fp = VTOF(vp);
+       vcb = VTOVCB(vp);
+       fileblocks = fp->ff_blocks;
+       filebytes = (off_t)fileblocks * (off_t)vcb->blockSize;
+
+       if (ioflag & IO_APPEND)
+               uio->uio_offset = fp->ff_size;
+       if ((cp->c_flags & APPEND) && uio->uio_offset != fp->ff_size)
+               return (EPERM);
+
+       // XXXdbg - don't allow modification of the journal or journal_info_block
+       if (VTOHFS(vp)->jnl && cp->c_datafork) {
+               struct HFSPlusExtentDescriptor *extd;
+
+               extd = &cp->c_datafork->ff_data.cf_extents[0];
+               if (extd->startBlock == VTOVCB(vp)->vcbJinfoBlock || extd->startBlock == VTOHFS(vp)->jnl_start) {
+                       return EPERM;
+               }
+       }
 
        writelimit = uio->uio_offset + uio->uio_resid;
 
 
        writelimit = uio->uio_offset + uio->uio_resid;
 
-    /*
-    * Maybe this should be above the vnode op call, but so long as
-    * file servers have no limits, I don't think it matters.
-    */
-    p = uio->uio_procp;
-    if (vp->v_type == VREG && p &&
-        writelimit > p->p_rlimit[RLIMIT_FSIZE].rlim_cur) {
-        psignal(p, SIGXFSZ);
-        return (EFBIG);
-    };
-    VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
-
-    resid = uio->uio_resid;
-    origFileSize = fcb->fcbEOF;
-    flags = ioflag & IO_SYNC ? B_SYNC : 0;
+       /*
+        * Maybe this should be above the vnode op call, but so long as
+        * file servers have no limits, I don't think it matters.
+        */
+       p = uio->uio_procp;
+       if (vp->v_type == VREG && p &&
+           writelimit > p->p_rlimit[RLIMIT_FSIZE].rlim_cur) {
+               psignal(p, SIGXFSZ);
+               return (EFBIG);
+       }
+       p = current_proc();
 
 
-    DBG_RW(("\tLEOF is 0x%lX, PEOF is 0x%lX.\n", fcb->fcbEOF, fcb->fcbPLen));
+       VOP_DEVBLOCKSIZE(cp->c_devvp, &devBlockSize);
 
 
-    /*
-    NOTE:      In the following loop there are two positions tracked:
-    currOffset is the current I/O starting offset.  currOffset is never >LEOF; the
-    LEOF is nudged along with currOffset as data is zeroed or written.
-    uio->uio_offset is the start of the current I/O operation.  It may be arbitrarily
-    beyond currOffset.
+       resid = uio->uio_resid;
+       origFileSize = fp->ff_size;
+       eflags = kEFDeferMask;  /* defer file block allocations */
+       filebytes = (off_t)fp->ff_blocks * (off_t)vcb->blockSize;
 
 
-    The following is true at all times:
+       /*
+        * NOTE: In the following loop there are two positions tracked:
+        * currOffset is the current I/O starting offset.  currOffset
+        * is never >LEOF; the LEOF is nudged along with currOffset as
+        * data is zeroed or written. uio->uio_offset is the start of
+        * the current I/O operation.  It may be arbitrarily beyond
+        * currOffset.
+        *
+        * The following is true at all times:
+        *   currOffset <= LEOF <= uio->uio_offset <= writelimit
+        */
+       currOffset = MIN(uio->uio_offset, fp->ff_size);
 
 
-    currOffset <= LEOF <= uio->uio_offset <= writelimit
-    */
-    currOffset = MIN(uio->uio_offset, fcb->fcbEOF);
+       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 0)) | DBG_FUNC_START,
+               (int)uio->uio_offset, uio->uio_resid, (int)fp->ff_size, (int)filebytes, 0);
+       retval = 0;
 
 
-    DBG_RW(("\tstarting I/O loop at 0x%lX.\n", (u_long)currOffset));
+       /* Now test if we need to extend the file */
+       /* Doing so will adjust the filebytes for us */
 
 
-    cp = current_proc();
+#if QUOTA
+       if(writelimit > filebytes) {
+               bytesToAdd = writelimit - filebytes;
 
 
-    KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 0)) | DBG_FUNC_START,
-                (int)uio->uio_offset, uio->uio_resid, (int)fcb->fcbEOF,  (int)fcb->fcbPLen, 0);
-    retval = 0;
+               retval = hfs_chkdq(cp, (int64_t)(roundup(bytesToAdd, vcb->blockSize)), 
+                                  ap->a_cred, 0);
+               if (retval)
+                       return (retval);
+       }
+#endif /* QUOTA */
 
 
-    /* Now test if we need to extend the file */
-    /* Doing so will adjust the fcbPLen for us */
+       hfsmp = VTOHFS(vp);
+       if (writelimit > filebytes) {
+               hfs_global_shared_lock_acquire(hfsmp);
+               grabbed_lock = 1;
+       }
+       if (hfsmp->jnl && (writelimit > filebytes)) {
+               if (journal_start_transaction(hfsmp->jnl) != 0) {
+                       hfs_global_shared_lock_release(hfsmp);
+                       return EINVAL;
+               }
+               started_tr = 1;
+       }
 
 
-    while (writelimit > (off_t)fcb->fcbPLen) {
+       while (writelimit > filebytes) {
        
        
-        bytesToAdd = writelimit - fcb->fcbPLen;
-        DBG_RW(("\textending file by 0x%lX bytes; 0x%lX blocks free",
-                (unsigned long)bytesToAdd, (unsigned long)vcb->freeBlocks));
-
-        /* lock extents b-tree (also protects volume bitmap) */
-        retval = hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_EXCLUSIVE, cp);
-        if (retval != E_NONE)
-            break;
-
-        retval = MacToVFSError(
-                            ExtendFileC (vcb,
-                                            fcb,
-                                            bytesToAdd,
-                                            0,
-                                            kEFContigBit,
-                                            &actualBytesAdded));
-
-        (void) hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_RELEASE, cp);
-        DBG_VOP_CONT(("\tactual bytes added = 0x%lX bytes, retval = %d...\n", actualBytesAdded, retval));
-        if ((actualBytesAdded == 0) && (retval == E_NONE)) retval = ENOSPC;
-        if (retval != E_NONE) break;
-
-        KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 0)) | DBG_FUNC_NONE,
-                    (int)uio->uio_offset, uio->uio_resid, (int)fcb->fcbEOF,  (int)fcb->fcbPLen, 0);
-    };
+               bytesToAdd = writelimit - filebytes;
+               if (suser(ap->a_cred, NULL) != 0)
+                       eflags |= kEFReserveMask;
+
+               /* lock extents b-tree (also protects volume bitmap) */
+               retval = hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_EXCLUSIVE, current_proc());
+               if (retval != E_NONE)
+                       break;
+
+               retval = MacToVFSError(ExtendFileC (vcb, (FCB*)fp, bytesToAdd,
+                               0, eflags, &actualBytesAdded));
+
+               (void) hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_RELEASE, p);
+               if ((actualBytesAdded == 0) && (retval == E_NONE))
+                       retval = ENOSPC;
+               if (retval != E_NONE)
+                       break;
+               filebytes = (off_t)fp->ff_blocks * (off_t)vcb->blockSize;
+               KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 0)) | DBG_FUNC_NONE,
+                       (int)uio->uio_offset, uio->uio_resid, (int)fp->ff_size,  (int)filebytes, 0);
+       }
+
+       // XXXdbg
+       if (started_tr) {
+               hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+               journal_end_transaction(hfsmp->jnl);
+               started_tr = 0;
+       }
+       if (grabbed_lock) {
+               hfs_global_shared_lock_release(hfsmp);
+               grabbed_lock = 0;
+       }
 
        if (UBCISVALID(vp) && retval == E_NONE) {
                off_t filesize;
 
        if (UBCISVALID(vp) && retval == E_NONE) {
                off_t filesize;
@@ -474,27 +410,26 @@ struct vop_write_args /* {
                int lflag;
                struct rl_entry *invalid_range;
 
                int lflag;
                struct rl_entry *invalid_range;
 
-               if (writelimit > fcb->fcbEOF)
+               if (writelimit > fp->ff_size)
                        filesize = writelimit;
                else
                        filesize = writelimit;
                else
-                       filesize = fcb->fcbEOF;
+                       filesize = fp->ff_size;
 
                lflag = (ioflag & IO_SYNC);
 
 
                lflag = (ioflag & IO_SYNC);
 
-               if (uio->uio_offset <= fcb->fcbEOF) {
+               if (uio->uio_offset <= fp->ff_size) {
                        zero_off = uio->uio_offset & ~PAGE_MASK_64;
                        
                        /* Check to see whether the area between the zero_offset and the start
                           of the transfer to see whether is invalid and should be zero-filled
                           as part of the transfer:
                         */
                        zero_off = uio->uio_offset & ~PAGE_MASK_64;
                        
                        /* Check to see whether the area between the zero_offset and the start
                           of the transfer to see whether is invalid and should be zero-filled
                           as part of the transfer:
                         */
-                       if (rl_scan(&hp->h_invalidranges, zero_off, uio->uio_offset - 1, &invalid_range) != RL_NOOVERLAP) {
+                       if (rl_scan(&fp->ff_invalidranges, zero_off, uio->uio_offset - 1, &invalid_range) != RL_NOOVERLAP)
                                lflag |= IO_HEADZEROFILL;
                                lflag |= IO_HEADZEROFILL;
-                       };
                } else {
                } else {
-                       off_t eof_page_base = fcb->fcbEOF & ~PAGE_MASK_64;
+                       off_t eof_page_base = fp->ff_size & ~PAGE_MASK_64;
                        
                        
-                       /* The bytes between fcb->fcbEOF and uio->uio_offset must never be
+                       /* The bytes between fp->ff_size and uio->uio_offset must never be
                           read without being zeroed.  The current last block is filled with zeroes
                           if it holds valid data but in all cases merely do a little bookkeeping
                           to track the area from the end of the current last page to the start of
                           read without being zeroed.  The current last block is filled with zeroes
                           if it holds valid data but in all cases merely do a little bookkeeping
                           to track the area from the end of the current last page to the start of
@@ -506,14 +441,14 @@ struct vop_write_args /* {
                           may be past the start of the write, in which case the zeroing
                           will be handled by the cluser_write of the actual data.
                         */
                           may be past the start of the write, in which case the zeroing
                           will be handled by the cluser_write of the actual data.
                         */
-                       inval_start = (fcb->fcbEOF + (PAGE_SIZE_64 - 1)) & ~PAGE_MASK_64;
+                       inval_start = (fp->ff_size + (PAGE_SIZE_64 - 1)) & ~PAGE_MASK_64;
                        inval_end = uio->uio_offset & ~PAGE_MASK_64;
                        inval_end = uio->uio_offset & ~PAGE_MASK_64;
-                       zero_off = fcb->fcbEOF;
+                       zero_off = fp->ff_size;
                        
                        
-                       if ((fcb->fcbEOF & PAGE_MASK_64) &&
-                               (rl_scan(&hp->h_invalidranges,
+                       if ((fp->ff_size & PAGE_MASK_64) &&
+                               (rl_scan(&fp->ff_invalidranges,
                                                        eof_page_base,
                                                        eof_page_base,
-                                                       fcb->fcbEOF - 1,
+                                                       fp->ff_size - 1,
                                                        &invalid_range) != RL_NOOVERLAP)) {
                                /* The page containing the EOF is not valid, so the
                                   entire page must be made inaccessible now.  If the write
                                                        &invalid_range) != RL_NOOVERLAP)) {
                                /* The page containing the EOF is not valid, so the
                                   entire page must be made inaccessible now.  If the write
@@ -538,14 +473,17 @@ struct vop_write_args /* {
                                           and the actual write will start on a page past inval_end.  Now's the last
                                           chance to zero-fill the page containing the EOF:
                                         */
                                           and the actual write will start on a page past inval_end.  Now's the last
                                           chance to zero-fill the page containing the EOF:
                                         */
-                                       retval = cluster_write(vp, (struct uio *) 0, fcb->fcbEOF, inval_start,
-                                                                                       zero_off, (off_t)0, devBlockSize, lflag | IO_HEADZEROFILL);
+                                       retval = cluster_write(vp, (struct uio *) 0,
+                                                       fp->ff_size, inval_start,
+                                                       zero_off, (off_t)0, devBlockSize,
+                                                       lflag | IO_HEADZEROFILL | IO_NOZERODIRTY);
                                        if (retval) goto ioerr_exit;
                                };
                                
                                /* Mark the remaining area of the newly allocated space as invalid: */
                                        if (retval) goto ioerr_exit;
                                };
                                
                                /* Mark the remaining area of the newly allocated space as invalid: */
-                               rl_add(inval_start, inval_end - 1 , &hp->h_invalidranges);
-                               zero_off = fcb->fcbEOF = inval_end;
+                               rl_add(inval_start, inval_end - 1 , &fp->ff_invalidranges);
+                               cp->c_zftimeout = time.tv_sec + ZFTIMELIMIT;
+                               zero_off = fp->ff_size = inval_end;
                        };
                        
                        if (uio->uio_offset > zero_off) lflag |= IO_HEADZEROFILL;
                        };
                        
                        if (uio->uio_offset > zero_off) lflag |= IO_HEADZEROFILL;
@@ -557,7 +495,7 @@ struct vop_write_args /* {
                tail_off = (writelimit + (PAGE_SIZE_64 - 1)) & ~PAGE_MASK_64;
                if (tail_off > filesize) tail_off = filesize;
                if (tail_off > writelimit) {
                tail_off = (writelimit + (PAGE_SIZE_64 - 1)) & ~PAGE_MASK_64;
                if (tail_off > filesize) tail_off = filesize;
                if (tail_off > writelimit) {
-                       if (rl_scan(&hp->h_invalidranges, writelimit, tail_off - 1, &invalid_range) != RL_NOOVERLAP) {
+                       if (rl_scan(&fp->ff_invalidranges, writelimit, tail_off - 1, &invalid_range) != RL_NOOVERLAP) {
                                lflag |= IO_TAILZEROFILL;
                        };
                };
                                lflag |= IO_TAILZEROFILL;
                        };
                };
@@ -574,164 +512,144 @@ struct vop_write_args /* {
                 */
                io_start = (lflag & IO_HEADZEROFILL) ? zero_off : uio->uio_offset;
                io_end = (lflag & IO_TAILZEROFILL) ? tail_off : writelimit;
                 */
                io_start = (lflag & IO_HEADZEROFILL) ? zero_off : uio->uio_offset;
                io_end = (lflag & IO_TAILZEROFILL) ? tail_off : writelimit;
-               if (io_start < fcb->fcbEOF) {
-                       rl_remove(io_start, io_end - 1, &hp->h_invalidranges);
+               if (io_start < fp->ff_size) {
+                       rl_remove(io_start, io_end - 1, &fp->ff_invalidranges);
                };
                };
-               retval = cluster_write(vp, uio, fcb->fcbEOF, filesize, zero_off, tail_off, devBlockSize, lflag);
+               retval = cluster_write(vp, uio, fp->ff_size, filesize, zero_off,
+                               tail_off, devBlockSize, lflag | IO_NOZERODIRTY);
                                
                                
-               if (uio->uio_offset > fcb->fcbEOF) {
-                       fcb->fcbEOF = uio->uio_offset;
+               if (uio->uio_offset > fp->ff_size) {
+                       fp->ff_size = uio->uio_offset;
 
 
-                       ubc_setsize(vp, (off_t)fcb->fcbEOF);       /* XXX check errors */
+                       ubc_setsize(vp, fp->ff_size);       /* XXX check errors */
                }
                }
-               if (resid > uio->uio_resid) hp->h_nodeflags |= IN_CHANGE | IN_UPDATE;
-
-    } else {
-
-        while (retval == E_NONE && uio->uio_resid > 0) {
-            logBlockNo = currOffset / PAGE_SIZE;
-            blkoffset  = currOffset & PAGE_MASK;
-
-            if (((off_t)(fcb->fcbPLen) - currOffset) < PAGE_SIZE_64)
-                fragSize = (off_t)(fcb->fcbPLen) - ((off_t)logBlockNo * PAGE_SIZE_64);
-            else
-                fragSize = PAGE_SIZE;
-            xfersize = fragSize - blkoffset;
-
-            DBG_RW(("\tcurrOffset = Ox%lX, logBlockNo = Ox%X, blkoffset = Ox%lX, xfersize = Ox%lX, fragSize = Ox%lX.\n",
-                    (unsigned long)currOffset, logBlockNo, blkoffset, xfersize, fragSize));
-
-            /* Make any adjustments for boundary conditions */
-            if (currOffset + (off_t)xfersize > writelimit) {
-                xfersize = writelimit - currOffset;
-                DBG_RW(("\ttrimming xfersize to 0x%lX to match writelimit (uio_resid)...\n", xfersize));
-            };
-
-            /*
-            * There is no need to read into bp if:
-            * We start on a block boundary and will overwrite the whole block
-            *
-            *                                          OR
-            */
-            if ((blkoffset == 0) && (xfersize >= fragSize)) {
-                DBG_RW(("\tRequesting %ld-byte block Ox%lX w/o read...\n", fragSize, (long)logBlockNo));
-
-                bp = getblk(vp, logBlockNo, fragSize, 0, 0, BLK_READ);
-                retval = 0;
-
-                if (bp->b_blkno == -1) {
-                    brelse(bp);
-                    retval = EIO;              /* XXX */
-                    break;
-                }
-            } else {
-
-                if (currOffset == fcb->fcbEOF && blkoffset == 0) {
-                    bp = getblk(vp, logBlockNo, fragSize, 0, 0, BLK_READ);
-                    retval = 0;
-
-                    if (bp->b_blkno == -1) {
-                        brelse(bp);
-                        retval = EIO;          /* XXX */
-                        break;
-                    }
-
-                } else {
-                    /*
-                    * This I/O transfer is not sufficiently aligned, so read the affected block into a buffer:
-                    */
-                    DBG_VOP(("\tRequesting block Ox%X, size = 0x%08lX...\n", logBlockNo, fragSize));
-                    retval = bread(vp, logBlockNo, fragSize, ap->a_cred, &bp);
-
-                    if (retval != E_NONE) {
-                        if (bp)
-                            brelse(bp);
-                        break;
-                    }
-                }
-            }
-
-            /* See if we are starting to write within file boundaries:
-                If not, then we need to present a "hole" for the area between
-                the current EOF and the start of the current I/O operation:
-
-                Note that currOffset is only less than uio_offset if uio_offset > LEOF...
-                */
-            if (uio->uio_offset > currOffset) {
-                clearSize = MIN(uio->uio_offset - currOffset, xfersize);
-                DBG_RW(("\tzeroing Ox%lX bytes Ox%lX bytes into block Ox%X...\n", clearSize, blkoffset, logBlockNo));
-                bzero(bp->b_data + blkoffset, clearSize);
-                currOffset += clearSize;
-                blkoffset += clearSize;
-                xfersize -= clearSize;
-            };
-
-            if (xfersize > 0) {
-                DBG_RW(("\tCopying Ox%lX bytes Ox%lX bytes into block Ox%X... ioflag == 0x%X\n",
-                        xfersize, blkoffset, logBlockNo, ioflag));
-                retval = uiomove((caddr_t)bp->b_data + blkoffset, (int)xfersize, uio);
-                currOffset += xfersize;
-            };
-            DBG_ASSERT((bp->b_bcount % devBlockSize) == 0);
-
-            if (ioflag & IO_SYNC) {
-                (void)VOP_BWRITE(bp);
-                //DBG_RW(("\tissuing bwrite\n"));
-            } else if ((xfersize + blkoffset) == fragSize) {
-                //DBG_RW(("\tissuing bawrite\n"));
-                bp->b_flags |= B_AGE;
-                bawrite(bp);
-            } else {
-                //DBG_RW(("\tissuing bdwrite\n"));
-                bdwrite(bp);
-            };
-
-            /* Update the EOF if we just extended the file
-                (the PEOF has already been moved out and the block mapping table has been updated): */
-            if (currOffset > fcb->fcbEOF) {
-                DBG_VOP(("\textending EOF to 0x%lX...\n", (UInt32)fcb->fcbEOF));
-                fcb->fcbEOF = currOffset;
-
-                if (UBCISVALID(vp))
-                    ubc_setsize(vp, (off_t)fcb->fcbEOF); /* XXX check errors */
-            };
-
-            if (retval || (resid == 0))
-                break;
-            hp->h_nodeflags |= IN_CHANGE | IN_UPDATE;
-        };
-    };
+               if (resid > uio->uio_resid)
+                       cp->c_flag |= C_CHANGE | C_UPDATE;
+       } else {
+               while (retval == E_NONE && uio->uio_resid > 0) {
+                       logBlockNo = currOffset / PAGE_SIZE;
+                       blkoffset  = currOffset & PAGE_MASK;
+
+                       if ((filebytes - currOffset) < PAGE_SIZE_64)
+                               fragSize = filebytes - ((off_t)logBlockNo * PAGE_SIZE_64);
+                       else
+                               fragSize = PAGE_SIZE;
+                       xfersize = fragSize - blkoffset;
+       
+                       /* Make any adjustments for boundary conditions */
+                       if (currOffset + (off_t)xfersize > writelimit)
+                               xfersize = writelimit - currOffset;
+        
+                       /*
+                        * There is no need to read into bp if:
+                        * We start on a block boundary and will overwrite the whole block
+                        *
+                        *                                              OR
+                        */
+                       if ((blkoffset == 0) && (xfersize >= fragSize)) {
+                               bp = getblk(vp, logBlockNo, fragSize, 0, 0, BLK_READ);
+                               retval = 0;
+       
+                               if (bp->b_blkno == -1) {
+                                       brelse(bp);
+                                       retval = EIO;           /* XXX */
+                                       break;
+                               }
+                       } else {
+       
+                               if (currOffset == fp->ff_size && blkoffset == 0) {
+                                       bp = getblk(vp, logBlockNo, fragSize, 0, 0, BLK_READ);
+                                       retval = 0;
+                                       if (bp->b_blkno == -1) {
+                                               brelse(bp);
+                                               retval = EIO;           /* XXX */
+                                               break;
+                                       }
+                               } else {
+                                       /*
+                                        * This I/O transfer is not sufficiently aligned,
+                                        * so read the affected block into a buffer:
+                                        */
+                                       retval = bread(vp, logBlockNo, fragSize, ap->a_cred, &bp);
+                                       if (retval != E_NONE) {
+                                               if (bp)
+                                               brelse(bp);
+                                               break;
+                                       }
+                               }
+                       }
+       
+                       /* See if we are starting to write within file boundaries:
+                        * If not, then we need to present a "hole" for the area
+                        * between the current EOF and the start of the current
+                        * I/O operation:
+                        *
+                        * Note that currOffset is only less than uio_offset if
+                        * uio_offset > LEOF...
+                        */
+                       if (uio->uio_offset > currOffset) {
+                               clearSize = MIN(uio->uio_offset - currOffset, xfersize);
+                               bzero(bp->b_data + blkoffset, clearSize);
+                               currOffset += clearSize;
+                               blkoffset += clearSize;
+                               xfersize -= clearSize;
+                       }
+       
+                       if (xfersize > 0) {
+                               retval = uiomove((caddr_t)bp->b_data + blkoffset, (int)xfersize, uio);
+                               currOffset += xfersize;
+                       }
+       
+                       if (ioflag & IO_SYNC) {
+                               (void)VOP_BWRITE(bp);
+                       } else if ((xfersize + blkoffset) == fragSize) {
+                               bp->b_flags |= B_AGE;
+                               bawrite(bp);
+                       } else {
+                               bdwrite(bp);
+                       }
+       
+                       /* Update the EOF if we just extended the file
+                        * (the PEOF has already been moved out and the
+                        * block mapping table has been updated):
+                        */
+                       if (currOffset > fp->ff_size) {
+                               fp->ff_size = currOffset;
+                               if (UBCISVALID(vp))
+                                       ubc_setsize(vp, fp->ff_size); /* XXX check errors */
+                       }
+                       if (retval || (resid == 0))
+                               break;
+                       cp->c_flag |= C_CHANGE | C_UPDATE;
+               } /* endwhile */
+       }
 
 ioerr_exit:
 
 ioerr_exit:
-    /*
+       /*
         * If we successfully wrote any data, and we are not the superuser
         * If we successfully wrote any data, and we are not the superuser
-     * we clear the setuid and setgid bits as a precaution against
-     * tampering.
-     */
-    if (resid > uio->uio_resid && ap->a_cred && ap->a_cred->cr_uid != 0)
-    hp->h_meta->h_mode &= ~(ISUID | ISGID);
-
-    if (retval) {
-        if (ioflag & IO_UNIT) {
-            (void)VOP_TRUNCATE(vp, origFileSize,
-                            ioflag & IO_SYNC, ap->a_cred, uio->uio_procp);
-            uio->uio_offset -= resid - uio->uio_resid;
-            uio->uio_resid = resid;
-        }
-    } else if (resid > uio->uio_resid && (ioflag & IO_SYNC)) {
-        tv = time;
-        retval = VOP_UPDATE(vp, &tv, &tv, 1);
-    }
-
-    #if HFS_DIAGNOSTIC
-    debug_check_blocksizes(vp);
-    #endif
+        * we clear the setuid and setgid bits as a precaution against
+        * tampering.
+        */
+       if (resid > uio->uio_resid && ap->a_cred && ap->a_cred->cr_uid != 0)
+               cp->c_mode &= ~(S_ISUID | S_ISGID);
+
+       if (retval) {
+               if (ioflag & IO_UNIT) {
+                       (void)VOP_TRUNCATE(vp, origFileSize,
+                               ioflag & IO_SYNC, ap->a_cred, uio->uio_procp);
+                       uio->uio_offset -= resid - uio->uio_resid;
+                       uio->uio_resid = resid;
+                       filebytes = (off_t)fp->ff_blocks * (off_t)vcb->blockSize;
+               }
+       } else if (resid > uio->uio_resid && (ioflag & IO_SYNC)) {
+               tv = time;
+               retval = VOP_UPDATE(vp, &tv, &tv, 1);
+       }
 
 
-    KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 0)) | DBG_FUNC_END,
-                (int)uio->uio_offset, uio->uio_resid, (int)fcb->fcbEOF, (int)fcb->fcbPLen, 0);
+       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 0)) | DBG_FUNC_END,
+               (int)uio->uio_offset, uio->uio_resid, (int)fp->ff_size, (int)filebytes, 0);
 
 
-    DBG_VOP_LOCKS_TEST(retval);
-    return (retval);
+       return (retval);
 }
 
 
 }
 
 
@@ -753,61 +671,55 @@ ioerr_exit:
 /* ARGSUSED */
 int
 hfs_ioctl(ap)
 /* ARGSUSED */
 int
 hfs_ioctl(ap)
-struct vop_ioctl_args /* {
-    struct vnode *a_vp;
-    int  a_command;
-    caddr_t  a_data;
-    int  a_fflag;
-    struct ucred *a_cred;
-    struct proc *a_p;
-} */ *ap;
+       struct vop_ioctl_args /* {
+               struct vnode *a_vp;
+               int  a_command;
+               caddr_t  a_data;
+               int  a_fflag;
+               struct ucred *a_cred;
+               struct proc *a_p;
+       } */ *ap;
 {
 {
-    DBG_FUNC_NAME("hfs_ioctl");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_UNLOCKED, VOPDBG_UNLOCKED, VOPDBG_UNLOCKED, VOPDBG_POS);
-
-    switch (ap->a_command) {
-       
-        case 1:
-    {   register struct hfsnode *hp;
-            register struct vnode *vp;
-       register struct radvisory *ra;
-       FCB *fcb;
-       int devBlockSize = 0;
-       int error;
-
-       vp = ap->a_vp;
-
-       VOP_LEASE(vp, ap->a_p, ap->a_cred, LEASE_READ);
-       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, ap->a_p);
-
-       ra = (struct radvisory *)(ap->a_data);
-       hp = VTOH(vp);
+       switch (ap->a_command) {
+       case 1: {
+               register struct cnode *cp;
+               register struct vnode *vp;
+               register struct radvisory *ra;
+               struct filefork *fp;
+               int devBlockSize = 0;
+               int error;
+
+               vp = ap->a_vp;
+
+               if (vp->v_type != VREG)
+                       return EINVAL;
+               VOP_LEASE(vp, ap->a_p, ap->a_cred, LEASE_READ);
+               error = vn_lock(vp, LK_EXCLUSIVE | LK_RETRY, ap->a_p);
+               if (error)
+                       return (error);
+
+               ra = (struct radvisory *)(ap->a_data);
+               cp = VTOC(vp);
+               fp = VTOF(vp);
+
+               if (ra->ra_offset >= fp->ff_size) {
+                       VOP_UNLOCK(vp, 0, ap->a_p);
+                       return (EFBIG);
+               }
+               VOP_DEVBLOCKSIZE(cp->c_devvp, &devBlockSize);
 
 
-       fcb = HTOFCB(hp);
+               error = advisory_read(vp, fp->ff_size, ra->ra_offset, ra->ra_count, devBlockSize);
+               VOP_UNLOCK(vp, 0, ap->a_p);
 
 
-       if (ra->ra_offset >= fcb->fcbEOF) {
-           VOP_UNLOCK(vp, 0, ap->a_p);
-           DBG_VOP_LOCKS_TEST(EFBIG);
-           return (EFBIG);
+               return (error);
        }
        }
-       VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
-
-       error = advisory_read(vp, fcb->fcbEOF, ra->ra_offset, ra->ra_count, devBlockSize);
-       VOP_UNLOCK(vp, 0, ap->a_p);
-
-       DBG_VOP_LOCKS_TEST(error);
-       return (error);
-            }
 
         case 2: /* F_READBOOTBLOCKS */
         case 3: /* F_WRITEBOOTBLOCKS */
             {
            struct vnode *vp = ap->a_vp;
 
         case 2: /* F_READBOOTBLOCKS */
         case 3: /* F_WRITEBOOTBLOCKS */
             {
            struct vnode *vp = ap->a_vp;
-           struct hfsnode *hp = VTOH(vp);
+           struct vnode *devvp = NULL;
            struct fbootstraptransfer *btd = (struct fbootstraptransfer *)ap->a_data;
            int devBlockSize;
            int error;
            struct fbootstraptransfer *btd = (struct fbootstraptransfer *)ap->a_data;
            int devBlockSize;
            int error;
@@ -821,6 +733,7 @@ struct vop_ioctl_args /* {
             if ((vp->v_flag & VROOT) == 0) return EINVAL;
             if (btd->fbt_offset + btd->fbt_length > 1024) return EINVAL;
            
             if ((vp->v_flag & VROOT) == 0) return EINVAL;
             if (btd->fbt_offset + btd->fbt_length > 1024) return EINVAL;
            
+           devvp = VTOHFS(vp)->hfs_devvp;
            aiov.iov_base = btd->fbt_buffer;
            aiov.iov_len = btd->fbt_length;
            
            aiov.iov_base = btd->fbt_buffer;
            aiov.iov_len = btd->fbt_length;
            
@@ -832,11 +745,11 @@ struct vop_ioctl_args /* {
            auio.uio_rw = (ap->a_command == 3) ? UIO_WRITE : UIO_READ; /* F_WRITEBOOTSTRAP / F_READBOOTSTRAP */
            auio.uio_procp = ap->a_p;
 
            auio.uio_rw = (ap->a_command == 3) ? UIO_WRITE : UIO_READ; /* F_WRITEBOOTSTRAP / F_READBOOTSTRAP */
            auio.uio_procp = ap->a_p;
 
-           VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
+           VOP_DEVBLOCKSIZE(devvp, &devBlockSize);
 
            while (auio.uio_resid > 0) {
              blockNumber = auio.uio_offset / devBlockSize;
 
            while (auio.uio_resid > 0) {
              blockNumber = auio.uio_offset / devBlockSize;
-             error = bread(hp->h_meta->h_devvp, blockNumber, devBlockSize, ap->a_cred, &bp);
+             error = bread(devvp, blockNumber, devBlockSize, ap->a_cred, &bp);
              if (error) {
                   if (bp) brelse(bp);
                   return error;
              if (error) {
                   if (bp) brelse(bp);
                   return error;
@@ -866,7 +779,6 @@ struct vop_ioctl_args /* {
             }
 
         default:
             }
 
         default:
-            DBG_VOP_LOCKS_TEST(ENOTTY);
             return (ENOTTY);
     }
 
             return (ENOTTY);
     }
 
@@ -877,104 +789,21 @@ struct vop_ioctl_args /* {
 /* ARGSUSED */
 int
 hfs_select(ap)
 /* ARGSUSED */
 int
 hfs_select(ap)
-struct vop_select_args /* {
-    struct vnode *a_vp;
-    int  a_which;
-    int  a_fflags;
-    struct ucred *a_cred;
-       void *a_wql;
-    struct proc *a_p;
-} */ *ap;
-{
-    DBG_FUNC_NAME("hfs_select");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_POS);
-
-    /*
-     * We should really check to see if I/O is possible.
-     */
-    DBG_VOP_LOCKS_TEST(1);
-    return (1);
-}
-
-
-
-/*
- * Mmap a file
- *
- * NB Currently unsupported.
-# XXX - not used
-#
- vop_mmap {
-     IN struct vnode *vp;
-     IN int fflags;
-     IN struct ucred *cred;
-     IN struct proc *p;
-
-     */
-
-/* ARGSUSED */
-
-int
-hfs_mmap(ap)
-struct vop_mmap_args /* {
-    struct vnode *a_vp;
-    int  a_fflags;
-    struct ucred *a_cred;
-    struct proc *a_p;
-} */ *ap;
-{
-    DBG_FUNC_NAME("hfs_mmap");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_POS);
-
-    DBG_VOP_LOCKS_TEST(EINVAL);
-    return (EINVAL);
-}
-
-
-
-/*
- * Seek on a file
- *
- * Nothing to do, so just return.
-# XXX - not used
-# Needs work: Is newoff right?  What's it mean?
-#
- vop_seek {
-     IN struct vnode *vp;
-     IN off_t oldoff;
-     IN off_t newoff;
-     IN struct ucred *cred;
-     */
-/* ARGSUSED */
-int
-hfs_seek(ap)
-struct vop_seek_args /* {
-    struct vnode *a_vp;
-    off_t  a_oldoff;
-    off_t  a_newoff;
-    struct ucred *a_cred;
-} */ *ap;
+       struct vop_select_args /* {
+               struct vnode *a_vp;
+               int  a_which;
+               int  a_fflags;
+               struct ucred *a_cred;
+               void *a_wql;
+               struct proc *a_p;
+       } */ *ap;
 {
 {
-    DBG_FUNC_NAME("hfs_seek");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_POS);
-
-    DBG_VOP_LOCKS_TEST(E_NONE);
-    return (E_NONE);
+       /*
+        * We should really check to see if I/O is possible.
+        */
+       return (1);
 }
 
 }
 
-
 /*
  * Bmap converts a the logical block number of a file to its physical block
  * number on the disk.
 /*
  * Bmap converts a the logical block number of a file to its physical block
  * number on the disk.
@@ -1005,17 +834,19 @@ struct vop_seek_args /* {
 
 int
 hfs_bmap(ap)
 
 int
 hfs_bmap(ap)
-struct vop_bmap_args /* {
-    struct vnode *a_vp;
-    daddr_t a_bn;
-    struct vnode **a_vpp;
-    daddr_t *a_bnp;
-    int *a_runp;
-} */ *ap;
+       struct vop_bmap_args /* {
+               struct vnode *a_vp;
+               daddr_t a_bn;
+               struct vnode **a_vpp;
+               daddr_t *a_bnp;
+               int *a_runp;
+       } */ *ap;
 {
 {
-    struct hfsnode             *hp = VTOH(ap->a_vp);
-    struct hfsmount    *hfsmp = VTOHFS(ap->a_vp);
-    int                                        retval = E_NONE;
+       struct vnode *vp = ap->a_vp;
+       struct cnode *cp = VTOC(vp);
+       struct filefork *fp = VTOF(vp);
+       struct hfsmount *hfsmp = VTOHFS(vp);
+   int                                 retval = E_NONE;
     daddr_t                            logBlockSize;
     size_t                             bytesContAvail = 0;
     off_t blockposition;
     daddr_t                            logBlockSize;
     size_t                             bytesContAvail = 0;
     off_t blockposition;
@@ -1024,46 +855,33 @@ struct vop_bmap_args /* {
     struct rl_entry *invalid_range;
     enum rl_overlaptype overlaptype;
 
     struct rl_entry *invalid_range;
     enum rl_overlaptype overlaptype;
 
-#define DEBUG_BMAP 0
-#if DEBUG_BMAP
-    DBG_FUNC_NAME("hfs_bmap");
-    DBG_VOP_LOCKS_DECL(2);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-    if (ap->a_vpp != NULL) {
-        DBG_VOP_LOCKS_INIT(1,*ap->a_vpp, VOPDBG_IGNORE, VOPDBG_UNLOCKED, VOPDBG_IGNORE, VOPDBG_POS);
-    } else {
-        DBG_VOP_LOCKS_INIT(1,NULL, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_IGNORE, VOPDBG_POS);
-       };
-#endif
-
-       DBG_IO(("\tMapped blk %d --> ", ap->a_bn));
-    /*
-     * Check for underlying vnode requests and ensure that logical
-     * to physical mapping is requested.
-     */
-    if (ap->a_vpp != NULL)
-        *ap->a_vpp = VTOH(ap->a_vp)->h_meta->h_devvp;
-    if (ap->a_bnp == NULL)
-        return (0);
-
-    logBlockSize = GetLogicalBlockSize(ap->a_vp);
-    blockposition = (off_t)(ap->a_bn * logBlockSize);
-        
-    lockExtBtree = hasOverflowExtents(hp);
-    if (lockExtBtree)
-    {
-        p = current_proc();
-        retval = hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_EXCLUSIVE | LK_CANRECURSE, p);
-        if (retval)
-            return (retval);
-    }
+       /*
+        * Check for underlying vnode requests and ensure that logical
+        * to physical mapping is requested.
+        */
+       if (ap->a_vpp != NULL)
+               *ap->a_vpp = cp->c_devvp;
+       if (ap->a_bnp == NULL)
+               return (0);
+
+       /* Only clustered I/O should have delayed allocations. */
+       DBG_ASSERT(fp->ff_unallocblocks == 0);
+
+       logBlockSize = GetLogicalBlockSize(vp);
+       blockposition = (off_t)ap->a_bn * (off_t)logBlockSize;
+
+       lockExtBtree = overflow_extents(fp);
+       if (lockExtBtree) {
+               p = current_proc();
+               retval = hfs_metafilelocking(hfsmp, kHFSExtentsFileID,
+                               LK_EXCLUSIVE | LK_CANRECURSE, p);
+               if (retval)
+                       return (retval);
+       }
 
 
-    retval = MacToVFSError(
+       retval = MacToVFSError(
                             MapFileBlockC (HFSTOVCB(hfsmp),
                             MapFileBlockC (HFSTOVCB(hfsmp),
-                                            HTOFCB(hp),
+                                            (FCB*)fp,
                                             MAXPHYSIO,
                                             blockposition,
                                             ap->a_bnp,
                                             MAXPHYSIO,
                                             blockposition,
                                             ap->a_bnp,
@@ -1073,7 +891,7 @@ struct vop_bmap_args /* {
 
     if (retval == E_NONE) {
         /* Adjust the mapping information for invalid file ranges: */
 
     if (retval == E_NONE) {
         /* Adjust the mapping information for invalid file ranges: */
-        overlaptype = rl_scan(&hp->h_invalidranges,
+        overlaptype = rl_scan(&fp->ff_invalidranges,
                             blockposition,
                             blockposition + MAXPHYSIO - 1,
                             &invalid_range);
                             blockposition,
                             blockposition + MAXPHYSIO - 1,
                             &invalid_range);
@@ -1093,7 +911,7 @@ struct vop_bmap_args /* {
                     if (invalid_range->rl_start == blockposition) {
                        /* There's actually no valid information to be had starting here: */
                        *ap->a_bnp = (daddr_t)-1;
                     if (invalid_range->rl_start == blockposition) {
                        /* There's actually no valid information to be had starting here: */
                        *ap->a_bnp = (daddr_t)-1;
-                                               if ((HTOFCB(hp)->fcbEOF > (invalid_range->rl_end + 1)) &&
+                                               if ((fp->ff_size > (invalid_range->rl_end + 1)) &&
                                                        (invalid_range->rl_end + 1 - blockposition < bytesContAvail)) {
                                bytesContAvail = invalid_range->rl_end + 1 - blockposition;
                        };
                                                        (invalid_range->rl_end + 1 - blockposition < bytesContAvail)) {
                                bytesContAvail = invalid_range->rl_end + 1 - blockposition;
                        };
@@ -1116,21 +934,6 @@ struct vop_bmap_args /* {
         };
     };
 
         };
     };
 
-    DBG_IO(("%d:%d.\n", *ap->a_bnp, (bytesContAvail < logBlockSize) ? 0 : (bytesContAvail / logBlockSize) - 1));
-
-#if DEBUG_BMAP
-
-    DBG_VOP_LOCKS_TEST(retval);
-#endif
-
-    if (ap->a_runp) {
-        DBG_ASSERT((*ap->a_runp * logBlockSize) < bytesContAvail);                                                     /* At least *ap->a_runp blocks left and ... */
-        if (can_cluster(logBlockSize)) {
-            DBG_ASSERT(bytesContAvail - (*ap->a_runp * logBlockSize) < (2*logBlockSize));      /* ... at most 1 logical block accounted for by current block */
-                                                                                            /* ... plus some sub-logical block sized piece */
-        };
-    };
-
     return (retval);
 }
 
     return (retval);
 }
 
@@ -1138,11 +941,11 @@ struct vop_bmap_args /* {
 
 int
 hfs_blktooff(ap)
 
 int
 hfs_blktooff(ap)
-struct vop_blktooff_args /* {
-    struct vnode *a_vp;
-    daddr_t a_lblkno;  
-    off_t *a_offset;
-} */ *ap;
+       struct vop_blktooff_args /* {
+               struct vnode *a_vp;
+               daddr_t a_lblkno;  
+               off_t *a_offset;
+       } */ *ap;
 {      
        if (ap->a_vp == NULL)
                return (EINVAL);
 {      
        if (ap->a_vp == NULL)
                return (EINVAL);
@@ -1153,14 +956,12 @@ struct vop_blktooff_args /* {
 
 int
 hfs_offtoblk(ap)
 
 int
 hfs_offtoblk(ap)
-struct vop_offtoblk_args /* {
-       struct vnode *a_vp;
-       off_t a_offset;    
-       daddr_t *a_lblkno;
-} */ *ap;
+       struct vop_offtoblk_args /* {
+               struct vnode *a_vp;
+               off_t a_offset;    
+               daddr_t *a_lblkno;
+       } */ *ap;
 {      
 {      
-       long lbsize, boff;
-
        if (ap->a_vp == NULL)
                return (EINVAL);
        *ap->a_lblkno = ap->a_offset / PAGE_SIZE_64;
        if (ap->a_vp == NULL)
                return (EINVAL);
        *ap->a_lblkno = ap->a_offset / PAGE_SIZE_64;
@@ -1170,65 +971,159 @@ struct vop_offtoblk_args /* {
 
 int
 hfs_cmap(ap)
 
 int
 hfs_cmap(ap)
-struct vop_cmap_args /* {
-       struct vnode *a_vp;
-       off_t a_foffset;    
-       size_t a_size;
-       daddr_t *a_bpn;
-       size_t *a_run;
-       void *a_poff;
-} */ *ap;
+       struct vop_cmap_args /* {
+               struct vnode *a_vp;
+               off_t a_foffset;    
+               size_t a_size;
+               daddr_t *a_bpn;
+               size_t *a_run;
+               void *a_poff;
+       } */ *ap;
 {
 {
-    struct hfsnode     *hp = VTOH(ap->a_vp);
-    struct hfsmount    *hfsmp = VTOHFS(ap->a_vp);
-    FCB                                        *fcb = HTOFCB(hp);
+    struct hfsmount *hfsmp = VTOHFS(ap->a_vp);
+    struct filefork *fp = VTOF(ap->a_vp);
     size_t                             bytesContAvail = 0;
     int                        retval = E_NONE;
     size_t                             bytesContAvail = 0;
     int                        retval = E_NONE;
-    int                                        lockExtBtree;
+    int lockExtBtree = 0;
     struct proc                *p = NULL;
     struct rl_entry *invalid_range;
     enum rl_overlaptype overlaptype;
     struct proc                *p = NULL;
     struct rl_entry *invalid_range;
     enum rl_overlaptype overlaptype;
-    off_t limit;
+    int started_tr = 0, grabbed_lock = 0;
 
 
-#define DEBUG_CMAP 0
-#if DEBUG_CMAP
-    DBG_FUNC_NAME("hfs_cmap");
-    DBG_VOP_LOCKS_DECL(2);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);
+       /*
+        * Check for underlying vnode requests and ensure that logical
+        * to physical mapping is requested.
+        */
+       if (ap->a_bpn == NULL)
+               return (0);
+
+       p = current_proc();
+  retry:
+       if (fp->ff_unallocblocks) {
+               lockExtBtree = 1;
+
+               // XXXdbg
+               hfs_global_shared_lock_acquire(hfsmp);
+               grabbed_lock = 1;
+
+               if (hfsmp->jnl) {
+                       if (journal_start_transaction(hfsmp->jnl) != 0) {
+                               hfs_global_shared_lock_release(hfsmp);
+                               return EINVAL;
+                       } else {
+                               started_tr = 1;
+                       }
+               } 
 
 
-    DBG_VOP_LOCKS_INIT(0, ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-#endif
+               if (retval = hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_EXCLUSIVE | LK_CANRECURSE, p)) {
+                       if (started_tr) {
+                               journal_end_transaction(hfsmp->jnl);
+                       }
+                       if (grabbed_lock) {
+                               hfs_global_shared_lock_release(hfsmp);
+                       }
+                       return (retval);
+               }
+       } else if (overflow_extents(fp)) {
+               lockExtBtree = 1;
+               if (retval = hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_EXCLUSIVE | LK_CANRECURSE, p)) {
+                       return retval;
+               }
+       }
 
 
-    DBG_IO(("\tMapped offset %qx --> ", ap->a_foffset));
-    /*
-     * Check for underlying vnode requests and ensure that logical
-     * to physical mapping is requested.
-     */
-    if (ap->a_bpn == NULL) {
-        return (0);
-    };
+       /*
+        * Check for any delayed allocations.
+        */
+       if (fp->ff_unallocblocks) {
+               SInt64 reqbytes, actbytes;
+
+               //
+               // Make sure we have a transaction.  It's possible
+               // that we came in and fp->ff_unallocblocks was zero
+               // but during the time we blocked acquiring the extents
+               // btree, ff_unallocblocks became non-zero and so we
+               // will need to start a transaction.
+               //
+               if (hfsmp->jnl && started_tr == 0) {
+                   if (lockExtBtree) {
+                       (void) hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_RELEASE, p);
+                       lockExtBtree = 0;
+                   }
+    
+                   goto retry;
+               }
 
 
-    if (lockExtBtree = hasOverflowExtents(hp))
-    {
-        p = current_proc();
-        if (retval = hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_EXCLUSIVE | LK_CANRECURSE, p)) {
-            return (retval);
-        };
-    }
-    retval = MacToVFSError(
+               reqbytes = (SInt64)fp->ff_unallocblocks *
+                            (SInt64)HFSTOVCB(hfsmp)->blockSize;
+               /*
+                * Release the blocks on loan and aquire some real ones.
+                * Note that we can race someone else for these blocks
+                * (and lose) so cmap needs to handle a failure here.
+                * Currently this race can't occur because all allocations
+                * are protected by an exclusive lock on the  Extents
+                * Overflow file.
+                */
+               HFSTOVCB(hfsmp)->loanedBlocks -= fp->ff_unallocblocks;
+               FTOC(fp)->c_blocks            -= fp->ff_unallocblocks;
+               fp->ff_blocks                 -= fp->ff_unallocblocks;
+               fp->ff_unallocblocks           = 0;
+
+               while (retval == 0 && reqbytes > 0) {
+                       retval = MacToVFSError(ExtendFileC(HFSTOVCB(hfsmp),
+                                       (FCB*)fp, reqbytes, 0,
+                                       kEFAllMask | kEFNoClumpMask, &actbytes));
+                       if (retval == 0 && actbytes == 0)
+                               retval = ENOSPC;
+
+                       if (retval) {
+                               fp->ff_unallocblocks =
+                                       reqbytes / HFSTOVCB(hfsmp)->blockSize;
+                               HFSTOVCB(hfsmp)->loanedBlocks += fp->ff_unallocblocks;
+                               FTOC(fp)->c_blocks            += fp->ff_unallocblocks;
+                               fp->ff_blocks                 += fp->ff_unallocblocks;
+                       }
+                       reqbytes -= actbytes;
+               }
+
+               if (retval) {
+                       (void) hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_RELEASE, p);
+                       if (started_tr) {
+                               hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+                               journal_end_transaction(hfsmp->jnl);
+                       }
+                       if (grabbed_lock) {
+                               hfs_global_shared_lock_release(hfsmp);
+                       }
+                       return (retval);
+               }
+               VTOC(ap->a_vp)->c_flag |= C_MODIFIED;
+       }
+
+       retval = MacToVFSError(
                           MapFileBlockC (HFSTOVCB(hfsmp),
                           MapFileBlockC (HFSTOVCB(hfsmp),
-                                         fcb,
+                                         (FCB *)fp,
                                          ap->a_size,
                                          ap->a_foffset,
                                          ap->a_bpn,
                                          &bytesContAvail));
 
                                          ap->a_size,
                                          ap->a_foffset,
                                          ap->a_bpn,
                                          &bytesContAvail));
 
-    if (lockExtBtree) (void) hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_RELEASE, p);
+       if (lockExtBtree)
+               (void) hfs_metafilelocking(hfsmp, kHFSExtentsFileID, LK_RELEASE, p);
 
 
+       // XXXdbg
+       if (started_tr) {
+               hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+               journal_end_transaction(hfsmp->jnl);
+               started_tr = 0;
+       }
+       if (grabbed_lock) {
+               hfs_global_shared_lock_release(hfsmp);
+               grabbed_lock = 0;
+       }
+                       
     if (retval == E_NONE) {
         /* Adjust the mapping information for invalid file ranges: */
     if (retval == E_NONE) {
         /* Adjust the mapping information for invalid file ranges: */
-        overlaptype = rl_scan(&hp->h_invalidranges,
+        overlaptype = rl_scan(&fp->ff_invalidranges,
                             ap->a_foffset,
                             ap->a_foffset + (off_t)bytesContAvail - 1,
                             &invalid_range);
                             ap->a_foffset,
                             ap->a_foffset + (off_t)bytesContAvail - 1,
                             &invalid_range);
@@ -1245,7 +1140,7 @@ struct vop_cmap_args /* {
                        (i.e. there's no valid bytes between the end of this range
                        and the file's EOF):
                      */
                        (i.e. there's no valid bytes between the end of this range
                        and the file's EOF):
                      */
-                    if ((fcb->fcbEOF > (invalid_range->rl_end + 1)) &&
+                    if ((fp->ff_size > (invalid_range->rl_end + 1)) &&
                                        (invalid_range->rl_end + 1 - ap->a_foffset < bytesContAvail)) {
                        bytesContAvail = invalid_range->rl_end + 1 - ap->a_foffset;
                     };
                                        (invalid_range->rl_end + 1 - ap->a_foffset < bytesContAvail)) {
                        bytesContAvail = invalid_range->rl_end + 1 - ap->a_foffset;
                     };
@@ -1257,7 +1152,7 @@ struct vop_cmap_args /* {
                     if (invalid_range->rl_start == ap->a_foffset) {
                        /* There's actually no valid information to be had starting here: */
                        *ap->a_bpn = (daddr_t)-1;
                     if (invalid_range->rl_start == ap->a_foffset) {
                        /* There's actually no valid information to be had starting here: */
                        *ap->a_bpn = (daddr_t)-1;
-                                               if ((fcb->fcbEOF > (invalid_range->rl_end + 1)) &&
+                                               if ((fp->ff_size > (invalid_range->rl_end + 1)) &&
                                                        (invalid_range->rl_end + 1 - ap->a_foffset < bytesContAvail)) {
                                bytesContAvail = invalid_range->rl_end + 1 - ap->a_foffset;
                        };
                                                        (invalid_range->rl_end + 1 - ap->a_foffset < bytesContAvail)) {
                                bytesContAvail = invalid_range->rl_end + 1 - ap->a_foffset;
                        };
@@ -1272,19 +1167,105 @@ struct vop_cmap_args /* {
         if (ap->a_run) *ap->a_run = bytesContAvail;
     };
 
         if (ap->a_run) *ap->a_run = bytesContAvail;
     };
 
-    if (ap->a_poff) *(int *)ap->a_poff = 0;
-
-    DBG_IO(("%d:%d.\n", *ap->a_bpn, bytesContAvail));
+       if (ap->a_poff)
+               *(int *)ap->a_poff = 0;
 
 
-#if DEBUG_BMAP
+       return (retval);
+}
 
 
-    DBG_VOP_LOCKS_TEST(retval);
-#endif
 
 
-    return (retval);
+/*
+ * Read or write a buffer that is not contiguous on disk.  We loop over
+ * each device block, copying to or from caller's buffer.
+ *
+ * We could be a bit more efficient by transferring as much data as is
+ * contiguous.  But since this routine should rarely be called, and that
+ * would be more complicated; best to keep it simple.
+ */
+static int
+hfs_strategy_fragmented(struct buf *bp)
+{
+       register struct vnode *vp = bp->b_vp;
+       register struct cnode *cp = VTOC(vp);
+       register struct vnode *devvp = cp->c_devvp;
+       caddr_t ioaddr;         /* Address of fragment within bp  */
+       struct buf *frag = NULL; /* For reading or writing a single block */
+       int retval = 0;
+       long remaining;         /* Bytes (in bp) left to transfer */
+       off_t offset;           /* Logical offset of current fragment in vp */
+       u_long block_size;      /* Size of one device block (and one I/O) */
+       
+       /* Make sure we redo this mapping for the next I/O */
+       bp->b_blkno = bp->b_lblkno;
+       
+       /* Set up the logical position and number of bytes to read/write */
+       offset = (off_t) bp->b_lblkno * (off_t) GetLogicalBlockSize(vp);
+       block_size = VTOHFS(vp)->hfs_phys_block_size;
+       
+       /* Get an empty buffer to do the deblocking */
+       frag = geteblk(block_size);
+       if (ISSET(bp->b_flags, B_READ))
+               SET(frag->b_flags, B_READ);
+
+       for (ioaddr = bp->b_data, remaining = bp->b_bcount; remaining != 0;
+           ioaddr += block_size, offset += block_size,
+           remaining -= block_size) {
+               frag->b_resid = frag->b_bcount;
+               CLR(frag->b_flags, B_DONE);
+
+               /* Map the current position to a physical block number */
+               retval = VOP_CMAP(vp, offset, block_size, &frag->b_lblkno,
+                   NULL, NULL);
+               if (retval != 0)
+                       break;
 
 
+               /*
+                * Did we try to read a hole?
+                * (Should never happen for metadata!)
+                */
+               if ((long)frag->b_lblkno == -1) {
+                       bzero(ioaddr, block_size);
+                       continue;
+               }
+               
+               /* If writing, copy before I/O */
+               if (!ISSET(bp->b_flags, B_READ))
+                       bcopy(ioaddr, frag->b_data, block_size);
+
+               /* Call the device to do the I/O and wait for it */
+               frag->b_blkno = frag->b_lblkno;
+               frag->b_vp = devvp;  /* Used to dispatch via VOP_STRATEGY */
+               frag->b_dev = devvp->v_rdev;
+               retval = VOP_STRATEGY(frag);
+               frag->b_vp = NULL;
+               if (retval != 0)
+                       break;
+               retval = biowait(frag);
+               if (retval != 0)
+                       break;
+               
+               /* If reading, copy after the I/O */
+               if (ISSET(bp->b_flags, B_READ))
+                       bcopy(frag->b_data, ioaddr, block_size);
+       }
+       
+       frag->b_vp = NULL;
+       //
+       // XXXdbg - in the case that this is a meta-data block, it won't affect
+       //          the journal because this bp is for a physical disk block,
+       //          not a logical block that is part of the catalog or extents
+       //          files.
+       SET(frag->b_flags, B_INVAL);
+       brelse(frag);
+       
+       if ((bp->b_error = retval) != 0)
+               SET(bp->b_flags, B_ERROR);
+       
+       biodone(bp);    /* This I/O is now complete */
+       return retval;
 }
 
 }
 
+
 /*
  * Calculate the logical to physical mapping if not done already,
  * then call the device strategy routine.
 /*
  * Calculate the logical to physical mapping if not done already,
  * then call the device strategy routine.
@@ -1294,106 +1275,89 @@ struct vop_cmap_args /* {
     */
 int
 hfs_strategy(ap)
     */
 int
 hfs_strategy(ap)
-struct vop_strategy_args /* {
-    struct buf *a_bp;
-} */ *ap;
+       struct vop_strategy_args /* {
+               struct buf *a_bp;
+       } */ *ap;
 {
 {
-    register struct buf *bp = ap->a_bp;
-    register struct vnode *vp = bp->b_vp;
-    register struct hfsnode *hp;
-    int retval = 0;
-
-       DBG_FUNC_NAME("hfs_strategy");
-
-//     DBG_VOP_PRINT_FUNCNAME();DBG_VOP_CONT(("\n"));
-
-    hp = VTOH(vp);
-
-    if ( !(bp->b_flags & B_VECTORLIST)) {
-
-        if (vp->v_type == VBLK || vp->v_type == VCHR)
-           panic("hfs_strategy: device vnode passed!");
+       register struct buf *bp = ap->a_bp;
+       register struct vnode *vp = bp->b_vp;
+       register struct cnode *cp = VTOC(vp);
+       int retval = 0;
+       off_t offset;
+       size_t bytes_contig;
+       
+       if ( !(bp->b_flags & B_VECTORLIST)) {
+               if (vp->v_type == VBLK || vp->v_type == VCHR)
+                       panic("hfs_strategy: device vnode passed!");
 
 
-       if (bp->b_flags & B_PAGELIST) {
-           /*
-            * if we have a page list associated with this bp,
-            * then go through cluster_bp since it knows how to 
-            * deal with a page request that might span non-contiguous
-            * physical blocks on the disk...
-            */
-           retval = cluster_bp(bp);
-           vp = hp->h_meta->h_devvp;
-           bp->b_dev = vp->v_rdev;
+               if (bp->b_flags & B_PAGELIST) {
+                       /*
+                        * If we have a page list associated with this bp,
+                        * then go through cluster_bp since it knows how to 
+                        * deal with a page request that might span non-
+                        * contiguous physical blocks on the disk...
+                        */
+                       retval = cluster_bp(bp);
+                       vp = cp->c_devvp;
+                       bp->b_dev = vp->v_rdev;
 
 
-           return (retval);
-       }
-       /*
-        * If we don't already know the filesystem relative block number
-        * then get it using VOP_BMAP().  If VOP_BMAP() returns the block
-        * number as -1 then we've got a hole in the file.  Although HFS
-         * filesystems don't create files with holes, invalidating of
-         * subranges of the file (lazy zero filling) may create such a
-         * situation.
-        */
-       if (bp->b_blkno == bp->b_lblkno) {
-           if ((retval = VOP_BMAP(vp, bp->b_lblkno, NULL, &bp->b_blkno, NULL))) {
-               bp->b_error = retval;
-               bp->b_flags |= B_ERROR;
-               biodone(bp);
-               return (retval);
-           }
-           if ((long)bp->b_blkno == -1)
-               clrbuf(bp);
-       }
-       if ((long)bp->b_blkno == -1) {
-           biodone(bp);
-           return (0);
+                       return (retval);
+               }
+               
+               /*
+                * If we don't already know the filesystem relative block
+                * number then get it using VOP_BMAP().  If VOP_BMAP()
+                * returns the block number as -1 then we've got a hole in
+                * the file.  Although HFS filesystems don't create files with
+                * holes, invalidating of subranges of the file (lazy zero
+                * filling) may create such a situation.
+                */
+               if (bp->b_blkno == bp->b_lblkno) {
+                       offset = (off_t) bp->b_lblkno *
+                           (off_t) GetLogicalBlockSize(vp);
+
+                       if ((retval = VOP_CMAP(vp, offset, bp->b_bcount,
+                           &bp->b_blkno, &bytes_contig, NULL))) {
+                               bp->b_error = retval;
+                               bp->b_flags |= B_ERROR;
+                               biodone(bp);
+                               return (retval);
+                       }
+                       if (bytes_contig < bp->b_bcount)
+                       {
+                               /*
+                                * We were asked to read a block that wasn't
+                                * contiguous, so we have to read each of the
+                                * pieces and copy them into the buffer.
+                                * Since ordinary file I/O goes through
+                                * cluster_io (which won't ask us for
+                                * discontiguous data), this is probably an
+                                * attempt to read or write metadata.
+                                */
+                               return hfs_strategy_fragmented(bp);
+                       }
+                       if ((long)bp->b_blkno == -1)
+                               clrbuf(bp);
+               }
+               if ((long)bp->b_blkno == -1) {
+                       biodone(bp);
+                       return (0);
+               }
+               if (bp->b_validend == 0) {
+                       /*
+                        * Record the exact size of the I/O transfer about to
+                        * be made:
+                        */
+                       bp->b_validend = bp->b_bcount;
+               }
        }
        }
-       if (bp->b_validend == 0) {
-           /* Record the exact size of the I/O transfer about to be made: */
-           DBG_ASSERT(bp->b_validoff == 0);
-           bp->b_validend = bp->b_bcount;
-           DBG_ASSERT(bp->b_dirtyoff == 0);
-       };
-    }
-    vp = hp->h_meta->h_devvp;
-    bp->b_dev = vp->v_rdev;
-    DBG_IO(("\t\t>>>%s: continuing w/ vp: 0x%x with logBlk Ox%X and phyBlk Ox%X\n", funcname, (u_int)vp, bp->b_lblkno, bp->b_blkno));
+       vp = cp->c_devvp;
+       bp->b_dev = vp->v_rdev;
 
 
-    return VOCALL (vp->v_op, VOFFSET(vop_strategy), ap);
+       return VOCALL (vp->v_op, VOFFSET(vop_strategy), ap);
 }
 
 
 }
 
 
-/*
-#% reallocblks vp      L L L
-#
- vop_reallocblks {
-     IN struct vnode *vp;
-     IN struct cluster_save *buflist;
-
-     */
-
-int
-hfs_reallocblks(ap)
-struct vop_reallocblks_args /* {
-    struct vnode *a_vp;
-    struct cluster_save *a_buflist;
-} */ *ap;
-{
-    DBG_FUNC_NAME("hfs_reallocblks");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-
-    /* Currently no support for clustering */          /* XXX */
-    DBG_VOP_LOCKS_TEST(ENOSPC);
-    return (ENOSPC);
-}
-
-
-
 /*
 #
 #% truncate    vp      L L L
 /*
 #
 #% truncate    vp      L L L
@@ -1405,275 +1369,307 @@ vop_truncate {
     IN struct ucred *cred;
     IN struct proc *p;
 };
     IN struct ucred *cred;
     IN struct proc *p;
 };
- * Truncate the hfsnode hp to at most length size, freeing (or adding) the
+ * Truncate a cnode to at most length size, freeing (or adding) the
  * disk blocks.
  */
 int hfs_truncate(ap)
  * disk blocks.
  */
 int hfs_truncate(ap)
-    struct vop_truncate_args /* {
-        struct vnode *a_vp;
-        off_t a_length;
-        int a_flags;
-        struct ucred *a_cred;
-        struct proc *a_p;
-    } */ *ap;
+       struct vop_truncate_args /* {
+               struct vnode *a_vp;
+               off_t a_length;
+               int a_flags;
+               struct ucred *a_cred;
+               struct proc *a_p;
+       } */ *ap;
 {
 {
-    register struct vnode *vp = ap->a_vp;
-    register struct hfsnode *hp = VTOH(vp);
-    off_t length = ap->a_length;
-    long vflags;
-    struct timeval tv;
-    int retval;
-    FCB *fcb;
-    off_t bytesToAdd;
-    off_t actualBytesAdded;
-    DBG_FUNC_NAME("hfs_truncate");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-
-#if HFS_DIAGNOSTIC
-    debug_check_blocksizes(ap->a_vp);
-#endif
+       register struct vnode *vp = ap->a_vp;
+       register struct cnode *cp = VTOC(vp);
+       struct filefork *fp = VTOF(vp);
+       off_t length;
+       long vflags;
+       struct timeval tv;
+       int retval;
+       off_t bytesToAdd;
+       off_t actualBytesAdded;
+       off_t filebytes;
+       u_long fileblocks;
+       int blksize;
+       struct hfsmount *hfsmp;
+
+       if (vp->v_type != VREG && vp->v_type != VLNK)
+               return (EISDIR);        /* cannot truncate an HFS directory! */
+
+       length = ap->a_length;
+       blksize = VTOVCB(vp)->blockSize;
+       fileblocks = fp->ff_blocks;
+       filebytes = (off_t)fileblocks * (off_t)blksize;
+
+       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_START,
+                (int)length, (int)fp->ff_size, (int)filebytes, 0, 0);
+
+       if (length < 0)
+               return (EINVAL);
 
 
-    fcb = HTOFCB(hp);
+       if ((!ISHFSPLUS(VTOVCB(vp))) && (length > (off_t)MAXHFSFILESIZE))
+               return (EFBIG);
 
 
-    KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_START,
-                (int)length, fcb->fcbEOF, fcb->fcbPLen, 0, 0);
+       hfsmp = VTOHFS(vp);
 
 
-    if (length < 0) {
-        DBG_VOP_LOCKS_TEST(EINVAL);
-        return (EINVAL);
-    }
+       tv = time;
+       retval = E_NONE;
 
 
-    if ((!ISHFSPLUS(VTOVCB(vp))) && (length > (off_t)MAXHFSFILESIZE)) {        
-        DBG_VOP_LOCKS_TEST(EFBIG);
-       return (EFBIG);
-    }
-
-    if (vp->v_type != VREG && vp->v_type != VLNK) {            
-        DBG_VOP_LOCKS_TEST(EISDIR);
-        return (EISDIR);               /* hfs doesn't support truncating of directories */
-    }
+       /* 
+        * We cannot just check if fp->ff_size == length (as an optimization)
+        * since there may be extra physical blocks that also need truncation.
+        */
+#if QUOTA
+       if (retval = hfs_getinoquota(cp))
+               return(retval);
+#endif /* QUOTA */
 
 
-    tv = time;
-    retval = E_NONE;
-       
-    DBG_RW(("%s: truncate from Ox%lX to Ox%X bytes\n", funcname, fcb->fcbPLen, length));
+       /*
+        * Lengthen the size of the file. We must ensure that the
+        * last byte of the file is allocated. Since the smallest
+        * value of ff_size is 0, length will be at least 1.
+        */
+       if (length > fp->ff_size) {
+#if QUOTA
+               retval = hfs_chkdq(cp, (int64_t)(roundup(length - filebytes, blksize)),
+                               ap->a_cred, 0);
+               if (retval)
+                       goto Err_Exit;
+#endif /* QUOTA */
+               /*
+                * If we don't have enough physical space then
+                * we need to extend the physical size.
+                */
+               if (length > filebytes) {
+                       int eflags;
+
+                       /* All or nothing and don't round up to clumpsize. */
+                       eflags = kEFAllMask | kEFNoClumpMask;
+
+                       if (suser(ap->a_cred, NULL) != 0)
+                               eflags |= kEFReserveMask;  /* keep a reserve */
+
+                       // XXXdbg
+                       hfs_global_shared_lock_acquire(hfsmp);
+                       if (hfsmp->jnl) {
+                               if (journal_start_transaction(hfsmp->jnl) != 0) {
+                                       retval = EINVAL;
+                                       goto Err_Exit;
+                               }
+                       }
 
 
-    /* 
-     * we cannot just check if fcb->fcbEOF == length (as an optimization)
-     * since there may be extra physical blocks that also need truncation
-     */
+                       /* lock extents b-tree (also protects volume bitmap) */
+                       retval = hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
+                       if (retval) {
+                               if (hfsmp->jnl) {
+                                       journal_end_transaction(hfsmp->jnl);
+                               } 
+                               hfs_global_shared_lock_release(hfsmp);
 
 
-    /*
-     * Lengthen the size of the file. We must ensure that the
-     * last byte of the file is allocated. Since the smallest
-     * value of fcbEOF is 0, length will be at least 1.
-     */
-    if (length > fcb->fcbEOF) {
-               off_t filePosition;
-               daddr_t logBlockNo;
-               long logBlockSize;
-               long blkOffset;
-               off_t bytestoclear;
-               int blockZeroCount;
-               struct buf *bp=NULL;
+                               goto Err_Exit;
+                       }
 
 
-       /*
-        * If we don't have enough physical space then
-        * we need to extend the physical size.
-        */
-       if (length > fcb->fcbPLen) {
-           /* lock extents b-tree (also protects volume bitmap) */
-           retval = hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
-           if (retval)
-               goto Err_Exit;
-
-           while ((length > fcb->fcbPLen) && (retval == E_NONE)) {
-               bytesToAdd = length - fcb->fcbPLen;
-               retval = MacToVFSError(
-                                       ExtendFileC (HTOVCB(hp),
-                                                    fcb,
+                       while ((length > filebytes) && (retval == E_NONE)) {
+                               bytesToAdd = length - filebytes;
+                               retval = MacToVFSError(ExtendFileC(VTOVCB(vp),
+                                                    (FCB*)fp,
                                                     bytesToAdd,
                                                     0,
                                                     bytesToAdd,
                                                     0,
-                                                    kEFAllMask,        /* allocate all requested bytes or none */
+                                                    eflags,
                                                     &actualBytesAdded));
 
                                                     &actualBytesAdded));
 
-               if (actualBytesAdded == 0 && retval == E_NONE) {
-                   if (length > fcb->fcbPLen)
-                       length = fcb->fcbPLen;
-                   break;
+                               filebytes = (off_t)fp->ff_blocks * (off_t)blksize;
+                               if (actualBytesAdded == 0 && retval == E_NONE) {
+                                       if (length > filebytes)
+                                               length = filebytes;
+                                       break;
+                               }
+                       } /* endwhile */
+
+                       (void) hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
+
+                       // XXXdbg
+                       if (hfsmp->jnl) {
+                               hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+                               journal_end_transaction(hfsmp->jnl);
+                       } 
+                       hfs_global_shared_lock_release(hfsmp);
+
+                       if (retval)
+                               goto Err_Exit;
+
+                       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_NONE,
+                               (int)length, (int)fp->ff_size, (int)filebytes, 0, 0);
                }
                }
-           } 
-           (void) hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
-           if (retval)
-               goto Err_Exit;
-
-           DBG_ASSERT(length <= fcb->fcbPLen);
-           KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_NONE,
-                        (int)length, fcb->fcbEOF, fcb->fcbPLen, 0, 0);
-       }
  
  
-       if (! (ap->a_flags & IO_NOZEROFILL)) {
-
-           if (UBCISVALID(vp) && retval == E_NONE) {
-                       struct rl_entry *invalid_range;
-               int devBlockSize;
-                       off_t zero_limit;
+               if (!(ap->a_flags & IO_NOZEROFILL)) {
+                       if (UBCINFOEXISTS(vp) && retval == E_NONE) {
+                               struct rl_entry *invalid_range;
+                               int devBlockSize;
+                               off_t zero_limit;
                        
                        
-                       zero_limit = (fcb->fcbEOF + (PAGE_SIZE_64 - 1)) & ~PAGE_MASK_64;
-                       if (length < zero_limit) zero_limit = length;
-
-                       if (length > fcb->fcbEOF) {
-                               /* Extending the file: time to fill out the current last page w. zeroes? */
-                               if ((fcb->fcbEOF & PAGE_MASK_64) &&
-                                       (rl_scan(&hp->h_invalidranges,
-                                                        fcb->fcbEOF & ~PAGE_MASK_64,
-                                                        fcb->fcbEOF - 1,
-                                                        &invalid_range) == RL_NOOVERLAP)) {
+                               zero_limit = (fp->ff_size + (PAGE_SIZE_64 - 1)) & ~PAGE_MASK_64;
+                               if (length < zero_limit) zero_limit = length;
+
+                               if (length > fp->ff_size) {
+                                       /* Extending the file: time to fill out the current last page w. zeroes? */
+                                       if ((fp->ff_size & PAGE_MASK_64) &&
+                                           (rl_scan(&fp->ff_invalidranges, fp->ff_size & ~PAGE_MASK_64,
+                                           fp->ff_size - 1, &invalid_range) == RL_NOOVERLAP)) {
                                                
                                                /* There's some valid data at the start of the (current) last page
                                                   of the file, so zero out the remainder of that page to ensure the
                                                   entire page contains valid data.  Since there is no invalid range
                                                   possible past the (current) eof, there's no need to remove anything
                                                   from the invalid range list before calling cluster_write():                                           */
                                                
                                                /* There's some valid data at the start of the (current) last page
                                                   of the file, so zero out the remainder of that page to ensure the
                                                   entire page contains valid data.  Since there is no invalid range
                                                   possible past the (current) eof, there's no need to remove anything
                                                   from the invalid range list before calling cluster_write():                                           */
-                                               VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
-                                               retval = cluster_write(vp, (struct uio *) 0, fcb->fcbEOF, zero_limit,
-                                                                                               fcb->fcbEOF, (off_t)0, devBlockSize, (ap->a_flags & IO_SYNC) | IO_HEADZEROFILL);
+                                               VOP_DEVBLOCKSIZE(cp->c_devvp, &devBlockSize);
+                                               retval = cluster_write(vp, (struct uio *) 0, fp->ff_size, zero_limit,
+                                                               fp->ff_size, (off_t)0, devBlockSize,
+                                                               (ap->a_flags & IO_SYNC) | IO_HEADZEROFILL | IO_NOZERODIRTY);
                                                if (retval) goto Err_Exit;
                                                
                                                /* Merely invalidate the remaining area, if necessary: */
                                                if (retval) goto Err_Exit;
                                                
                                                /* Merely invalidate the remaining area, if necessary: */
-                                               if (length > zero_limit) rl_add(zero_limit, length - 1, &hp->h_invalidranges);
-                               } else {
+                                               if (length > zero_limit) {
+                                                       rl_add(zero_limit, length - 1, &fp->ff_invalidranges);
+                                                       cp->c_zftimeout = time.tv_sec + ZFTIMELIMIT;
+                                               }
+                                       } else {
                                        /* The page containing the (current) eof is invalid: just add the
                                           remainder of the page to the invalid list, along with the area
                                           being newly allocated:
                                         */
                                        /* The page containing the (current) eof is invalid: just add the
                                           remainder of the page to the invalid list, along with the area
                                           being newly allocated:
                                         */
-                                       rl_add(fcb->fcbEOF, length - 1, &hp->h_invalidranges);
-                               };
-                       }
-           } else {
+                                       rl_add(fp->ff_size, length - 1, &fp->ff_invalidranges);
+                                       cp->c_zftimeout = time.tv_sec + ZFTIMELIMIT;
+                                       };
+                               }
+                       } else {
+                                       panic("hfs_truncate: invoked on non-UBC object?!");
+                       };
+               }
+               cp->c_flag |= C_UPDATE;
+               fp->ff_size = length;
 
 
-#if 0
-                   /*
-                    * zero out any new logical space...
-                    */
-                   bytestoclear = length - fcb->fcbEOF;
-                   filePosition = fcb->fcbEOF;
+               if (UBCISVALID(vp))
+                       ubc_setsize(vp, fp->ff_size);   /* XXX check errors */
 
 
-                   while (bytestoclear > 0) {
-                       logBlockNo   = (daddr_t)(filePosition / PAGE_SIZE_64);
-                       blkOffset    = (long)(filePosition & PAGE_MASK_64);  
+       } else { /* Shorten the size of the file */
 
 
-                       if (((off_t)(fcb->fcbPLen) - ((off_t)logBlockNo * (off_t)PAGE_SIZE)) < PAGE_SIZE_64)
-                           logBlockSize = (off_t)(fcb->fcbPLen) - ((off_t)logBlockNo * PAGE_SIZE_64);
-                       else
-                           logBlockSize = PAGE_SIZE;
-                       
-                       if (logBlockSize < blkOffset)
-                           panic("hfs_truncate: bad logBlockSize computed\n");
-                               
-                       blockZeroCount = MIN(bytestoclear, logBlockSize - blkOffset);
+               if (fp->ff_size > length) {
+                       /*
+                        * Any buffers that are past the truncation point need to be
+                        * invalidated (to maintain buffer cache consistency).  For
+                        * simplicity, we invalidate all the buffers by calling vinvalbuf.
+                        */
+                       if (UBCISVALID(vp))
+                               ubc_setsize(vp, length); /* XXX check errors */
 
 
-                       if (blkOffset == 0 && ((bytestoclear >= logBlockSize) || filePosition >= fcb->fcbEOF)) {
-                           bp = getblk(vp, logBlockNo, logBlockSize, 0, 0, BLK_WRITE);
-                           retval = 0;
+                       vflags = ((length > 0) ? V_SAVE : 0)  | V_SAVEMETA;     
+                       retval = vinvalbuf(vp, vflags, ap->a_cred, ap->a_p, 0, 0);
+           
+                       /* Any space previously marked as invalid is now irrelevant: */
+                       rl_remove(length, fp->ff_size - 1, &fp->ff_invalidranges);
+               }
 
 
-                       } else {
-                           retval = bread(vp, logBlockNo, logBlockSize, ap->a_cred, &bp);
-                           if (retval) {
-                               brelse(bp);
+               /* 
+                * Account for any unmapped blocks. Note that the new
+                * file length can still end up with unmapped blocks.
+                */
+               if (fp->ff_unallocblocks > 0) {
+                       u_int32_t finalblks;
+
+                       /* lock extents b-tree */
+                       retval = hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID,
+                                       LK_EXCLUSIVE, ap->a_p);
+                       if (retval)
                                goto Err_Exit;
                                goto Err_Exit;
-                           }
-                       }
-                       bzero((char *)bp->b_data + blkOffset, blockZeroCount);
-                                       
-                       bp->b_flags |= B_DIRTY | B_AGE;
-
-                       if (ap->a_flags & IO_SYNC)
-                           VOP_BWRITE(bp);
-                       else if (logBlockNo % 32)
-                           bawrite(bp);
-                       else
-                           VOP_BWRITE(bp);     /* wait after we issue 32 requests */
 
 
-                       bytestoclear -= blockZeroCount;
-                       filePosition += blockZeroCount;
-                   }
-#else
-                       panic("hfs_truncate: invoked on non-UBC object?!");
-#endif
-           };
-       }
-       fcb->fcbEOF = length;
+                       VTOVCB(vp)->loanedBlocks -= fp->ff_unallocblocks;
+                       cp->c_blocks             -= fp->ff_unallocblocks;
+                       fp->ff_blocks            -= fp->ff_unallocblocks;
+                       fp->ff_unallocblocks      = 0;
+
+                       finalblks = (length + blksize - 1) / blksize;
+                       if (finalblks > fp->ff_blocks) {
+                               /* calculate required unmapped blocks */
+                               fp->ff_unallocblocks      = finalblks - fp->ff_blocks;
+                               VTOVCB(vp)->loanedBlocks += fp->ff_unallocblocks;
+                               cp->c_blocks             += fp->ff_unallocblocks;
+                               fp->ff_blocks            += fp->ff_unallocblocks;
+                       }
+                       (void) hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID,
+                                       LK_RELEASE, ap->a_p);
+               }
 
 
-       if (UBCISVALID(vp))
-               ubc_setsize(vp, (off_t)fcb->fcbEOF); /* XXX check errors */
+               /*
+                * For a TBE process the deallocation of the file blocks is
+                * delayed until the file is closed.  And hfs_close calls
+                * truncate with the IO_NDELAY flag set.  So when IO_NDELAY
+                * isn't set, we make sure this isn't a TBE process.
+                */
+               if ((ap->a_flags & IO_NDELAY) || (!ISSET(ap->a_p->p_flag, P_TBE))) {
+#if QUOTA
+                 off_t savedbytes = ((off_t)fp->ff_blocks * (off_t)blksize);
+#endif /* QUOTA */
+                 // XXXdbg
+                 hfs_global_shared_lock_acquire(hfsmp);
+                       if (hfsmp->jnl) {
+                               if (journal_start_transaction(hfsmp->jnl) != 0) {
+                                       retval = EINVAL;
+                                       goto Err_Exit;
+                               }
+                       }
 
 
-    } else { /* Shorten the size of the file */
+                       /* lock extents b-tree (also protects volume bitmap) */
+                       retval = hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
+                       if (retval) {
+                               if (hfsmp->jnl) {
+                                       journal_end_transaction(hfsmp->jnl);
+                               }
+                               hfs_global_shared_lock_release(hfsmp);
+                               goto Err_Exit;
+                       }
+                       
+                       if (fp->ff_unallocblocks == 0)
+                               retval = MacToVFSError(TruncateFileC(VTOVCB(vp),
+                                               (FCB*)fp, length, false));
 
 
-        if (fcb->fcbEOF > length) {
-           /*
-            * Any buffers that are past the truncation point need to be
-            * invalidated (to maintain buffer cache consistency).  For
-            * simplicity, we invalidate all the buffers by calling vinvalbuf.
-            */
-           if (UBCISVALID(vp))
-               ubc_setsize(vp, (off_t)length); /* XXX check errors */
+                       (void) hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
 
 
-           vflags = ((length > 0) ? V_SAVE : 0)  | V_SAVEMETA; 
-           retval = vinvalbuf(vp, vflags, ap->a_cred, ap->a_p, 0, 0);
-           
-           /* Any space previously marked as invalid is now irrelevant: */
-           rl_remove(length, fcb->fcbEOF - 1, &hp->h_invalidranges);
-       }
+                       // XXXdbg
+                       if (hfsmp->jnl) {
+                               hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+                               journal_end_transaction(hfsmp->jnl);
+                       }
+                       hfs_global_shared_lock_release(hfsmp);
 
 
-       /*
-        * For a TBE process the deallocation of the file blocks is
-        * delayed until the file is closed.  And hfs_close calls
-        * truncate with the IO_NDELAY flag set.  So when IO_NDELAY
-        * isn't set, we make sure this isn't a TBE process.
-        */
-       if ((ap->a_flags & IO_NDELAY) || (!ISSET(ap->a_p->p_flag, P_TBE))) {
-
-           /* lock extents b-tree (also protects volume bitmap) */
-           retval = hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
-           if (retval)
-               goto Err_Exit;
-           retval = MacToVFSError(
-                               TruncateFileC(  
-                                             HTOVCB(hp),
-                                             fcb,
-                                             length,
-                                             false));
-           (void) hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
-           if (retval)
-               goto Err_Exit;
+                       filebytes = (off_t)fp->ff_blocks * (off_t)blksize;
+                       if (retval)
+                               goto Err_Exit;
+#if QUOTA
+                       /* These are bytesreleased */
+                       (void) hfs_chkdq(cp, (int64_t)-(savedbytes - filebytes), NOCRED, 0);
+#endif /* QUOTA */
+               }
+               /* Only set update flag if the logical length changes */
+               if (fp->ff_size != length)
+                       cp->c_flag |= C_UPDATE;
+               fp->ff_size = length;
        }
        }
-       fcb->fcbEOF = length;
-
-       if (fcb->fcbFlags & fcbModifiedMask)
-           hp->h_nodeflags |= IN_MODIFIED;
-    }
-    hp->h_nodeflags |= IN_CHANGE | IN_UPDATE;
-    retval = VOP_UPDATE(vp, &tv, &tv, MNT_WAIT);
-    if (retval) {
-        DBG_ERR(("Could not update truncate"));
+       cp->c_flag |= C_CHANGE;
+       retval = VOP_UPDATE(vp, &tv, &tv, MNT_WAIT);
+       if (retval) {
                KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_NONE,
                     -1, -1, -1, retval, 0);
                KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_NONE,
                     -1, -1, -1, retval, 0);
-    }
-Err_Exit:;
+       }
 
 
-#if HFS_DIAGNOSTIC
-    debug_check_blocksizes(ap->a_vp);
-#endif
+Err_Exit:
 
 
-    KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_END,
-                (int)length, fcb->fcbEOF, fcb->fcbPLen, retval, 0);
+       KERNEL_DEBUG((FSDBG_CODE(DBG_FSRW, 7)) | DBG_FUNC_END,
+                (int)length, (int)fp->ff_size, (int)filebytes, retval, 0);
 
 
-    DBG_VOP_LOCKS_TEST(retval);
-    return (retval);
+       return (retval);
 }
 
 
 }
 
 
@@ -1691,131 +1687,148 @@ vop_allocate {
        IN struct ucred *cred;
        IN struct proc *p;
 };
        IN struct ucred *cred;
        IN struct proc *p;
 };
- * allocate the hfsnode hp to at most length size
+ * allocate a cnode to at most length size
  */
 int hfs_allocate(ap)
  */
 int hfs_allocate(ap)
-    struct vop_allocate_args /* {
-        struct vnode *a_vp;
-        off_t a_length;
-        u_int32_t  a_flags;
-        off_t *a_bytesallocated;
-        off_t a_offset;
-        struct ucred *a_cred;
-        struct proc *a_p;
-    } */ *ap;
+       struct vop_allocate_args /* {
+               struct vnode *a_vp;
+               off_t a_length;
+               u_int32_t  a_flags;
+               off_t *a_bytesallocated;
+               off_t a_offset;
+               struct ucred *a_cred;
+               struct proc *a_p;
+       } */ *ap;
 {
 {
-    register struct vnode *vp = ap->a_vp;
-    register struct hfsnode *hp = VTOH(vp);
-    off_t      length = ap->a_length;
-    off_t      startingPEOF;
-    off_t      moreBytesRequested;
-    off_t      actualBytesAdded;
-    long vflags;
-    struct timeval tv;
-    int retval, retval2;
-    FCB *fcb;
-    UInt32 blockHint;
-    UInt32 extendFlags =0;   /* For call to ExtendFileC */
-    DBG_FUNC_NAME("hfs_allocate");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(ap->a_vp);DBG_VOP_CONT(("\n"));
-    DBG_VOP_LOCKS_INIT(0,ap->a_vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-
-    /* Set the number of bytes allocated to 0 so that the caller will know that we
-       did nothing.  ExtendFileC will fill this in for us if we actually allocate space */
-
-    *(ap->a_bytesallocated) = 0; 
-    fcb = HTOFCB(hp);
-
-    /* Now for some error checking */
-
-    if (length < (off_t)0) {
-        DBG_VOP_LOCKS_TEST(EINVAL);
-        return (EINVAL);
-    }
-
-    if (vp->v_type != VREG && vp->v_type != VLNK) {
-        DBG_VOP_LOCKS_TEST(EISDIR);
-        return (EISDIR);        /* hfs doesn't support truncating of directories */
-    }
-
-    if ((ap->a_flags & ALLOCATEFROMVOL) && (length <= fcb->fcbPLen))
-        return (EINVAL);
+       struct vnode *vp = ap->a_vp;
+       struct cnode *cp = VTOC(vp);
+       struct filefork *fp = VTOF(vp);
+       off_t length = ap->a_length;
+       off_t startingPEOF;
+       off_t moreBytesRequested;
+       off_t actualBytesAdded;
+       off_t filebytes;
+       u_long fileblocks;
+       long vflags;
+       struct timeval tv;
+       int retval, retval2;
+       UInt32 blockHint;
+       UInt32 extendFlags =0;   /* For call to ExtendFileC */
+       struct hfsmount *hfsmp;
+
+       hfsmp = VTOHFS(vp);
+
+       *(ap->a_bytesallocated) = 0;
+       fileblocks = fp->ff_blocks;
+       filebytes = (off_t)fileblocks * (off_t)VTOVCB(vp)->blockSize;
+
+       if (length < (off_t)0)
+               return (EINVAL);
+       if (vp->v_type != VREG && vp->v_type != VLNK)
+               return (EISDIR);
+       if ((ap->a_flags & ALLOCATEFROMVOL) && (length <= filebytes))
+               return (EINVAL);
 
 
-    /* Fill in the flags word for the call to Extend the file */
+       /* Fill in the flags word for the call to Extend the file */
 
 
-       if (ap->a_flags & ALLOCATECONTIG) {
+       if (ap->a_flags & ALLOCATECONTIG) 
                extendFlags |= kEFContigMask;
                extendFlags |= kEFContigMask;
-       }
 
 
-    if (ap->a_flags & ALLOCATEALL) {
+       if (ap->a_flags & ALLOCATEALL)
                extendFlags |= kEFAllMask;
                extendFlags |= kEFAllMask;
-       }
 
 
-    tv = time;
-    retval = E_NONE;
-    blockHint = 0;
-    startingPEOF = fcb->fcbPLen;
+       if (suser(ap->a_cred, NULL) != 0)
+               extendFlags |= kEFReserveMask;
 
 
-    if (ap->a_flags & ALLOCATEFROMPEOF) {
-               length += fcb->fcbPLen;
-       }
+       tv = time;
+       retval = E_NONE;
+       blockHint = 0;
+       startingPEOF = filebytes;
 
 
-       if (ap->a_flags & ALLOCATEFROMVOL)
-               blockHint = ap->a_offset / HTOVCB(hp)->blockSize;
+       if (ap->a_flags & ALLOCATEFROMPEOF)
+               length += filebytes;
+       else if (ap->a_flags & ALLOCATEFROMVOL)
+               blockHint = ap->a_offset / VTOVCB(vp)->blockSize;
 
 
-    /* If no changes are necesary, then we're done */
-    if (fcb->fcbPLen == length)
-       goto Std_Exit;
+       /* If no changes are necesary, then we're done */
+       if (filebytes == length)
+               goto Std_Exit;
 
 
-    /*
-    * Lengthen the size of the file. We must ensure that the
-    * last byte of the file is allocated. Since the smallest
-    * value of fcbPLen is 0, length will be at least 1.
-    */
-    if (length > fcb->fcbPLen) {
-               moreBytesRequested = length - fcb->fcbPLen;
+       /*
+        * Lengthen the size of the file. We must ensure that the
+        * last byte of the file is allocated. Since the smallest
+        * value of filebytes is 0, length will be at least 1.
+        */
+       if (length > filebytes) {
+               moreBytesRequested = length - filebytes;
                
                
+#if QUOTA
+               retval = hfs_chkdq(cp,
+                               (int64_t)(roundup(moreBytesRequested, VTOVCB(vp)->blockSize)), 
+                               ap->a_cred, 0);
+               if (retval)
+                       return (retval);
+
+#endif /* QUOTA */
+               // XXXdbg
+               hfs_global_shared_lock_acquire(hfsmp);
+               if (hfsmp->jnl) {
+                       if (journal_start_transaction(hfsmp->jnl) != 0) {
+                               retval = EINVAL;
+                               goto Err_Exit;
+                       }
+               }
+
                /* lock extents b-tree (also protects volume bitmap) */
                /* lock extents b-tree (also protects volume bitmap) */
-               retval = hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
-               if (retval) goto Err_Exit;
+               retval = hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
+               if (retval) {
+                       if (hfsmp->jnl) {
+                               journal_end_transaction(hfsmp->jnl);
+                       }
+                       hfs_global_shared_lock_release(hfsmp);
+                       goto Err_Exit;
+               }
 
 
-               retval = MacToVFSError(
-                                                               ExtendFileC(HTOVCB(hp),
-                                                                                       fcb,
-                                                                                       moreBytesRequested,
-                                                                                       blockHint,
-                                                                                       extendFlags,
-                                                                                       &actualBytesAdded));
+               retval = MacToVFSError(ExtendFileC(VTOVCB(vp),
+                                               (FCB*)fp,
+                                               moreBytesRequested,
+                                               blockHint,
+                                               extendFlags,
+                                               &actualBytesAdded));
 
                *(ap->a_bytesallocated) = actualBytesAdded;
 
                *(ap->a_bytesallocated) = actualBytesAdded;
+               filebytes = (off_t)fp->ff_blocks * (off_t)VTOVCB(vp)->blockSize;
 
 
-               (void) hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
+               (void) hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
 
 
-               DBG_ASSERT(length <= fcb->fcbPLen);
+               // XXXdbg
+               if (hfsmp->jnl) {
+                       hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+                       journal_end_transaction(hfsmp->jnl);
+               }
+               hfs_global_shared_lock_release(hfsmp);
 
                /*
                 * if we get an error and no changes were made then exit
                 * otherwise we must do the VOP_UPDATE to reflect the changes
                 */
 
                /*
                 * if we get an error and no changes were made then exit
                 * otherwise we must do the VOP_UPDATE to reflect the changes
                 */
-        if (retval && (startingPEOF == fcb->fcbPLen)) goto Err_Exit;
+               if (retval && (startingPEOF == filebytes))
+                       goto Err_Exit;
         
         
-        /*
-         * Adjust actualBytesAdded to be allocation block aligned, not
-         * clump size aligned.
-         * NOTE: So what we are reporting does not affect reality
-         * until the file is closed, when we truncate the file to allocation
-         * block size.
-         */
-
+               /*
+                * Adjust actualBytesAdded to be allocation block aligned, not
+                * clump size aligned.
+                * NOTE: So what we are reporting does not affect reality
+                * until the file is closed, when we truncate the file to allocation
+                * block size.
+                */
                if ((actualBytesAdded != 0) && (moreBytesRequested < actualBytesAdded))
                        *(ap->a_bytesallocated) =
                                roundup(moreBytesRequested, (off_t)VTOVCB(vp)->blockSize);
 
                if ((actualBytesAdded != 0) && (moreBytesRequested < actualBytesAdded))
                        *(ap->a_bytesallocated) =
                                roundup(moreBytesRequested, (off_t)VTOVCB(vp)->blockSize);
 
-    } else { /* Shorten the size of the file */
+       } else { /* Shorten the size of the file */
 
 
-       if (fcb->fcbEOF > length) {
+               if (fp->ff_size > length) {
                        /*
                         * Any buffers that are past the truncation point need to be
                         * invalidated (to maintain buffer cache consistency).  For
                        /*
                         * Any buffers that are past the truncation point need to be
                         * invalidated (to maintain buffer cache consistency).  For
@@ -1825,51 +1838,74 @@ int hfs_allocate(ap)
                        (void) vinvalbuf(vp, vflags, ap->a_cred, ap->a_p, 0, 0);
                }
 
                        (void) vinvalbuf(vp, vflags, ap->a_cred, ap->a_p, 0, 0);
                }
 
-       /* lock extents b-tree (also protects volume bitmap) */
-        retval = hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
-        if (retval) goto Err_Exit;
+               // XXXdbg
+               hfs_global_shared_lock_acquire(hfsmp);
+               if (hfsmp->jnl) {
+                       if (journal_start_transaction(hfsmp->jnl) != 0) {
+                               retval = EINVAL;
+                               goto Err_Exit;
+                       }
+               }
+
+               /* lock extents b-tree (also protects volume bitmap) */
+               retval = hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_EXCLUSIVE, ap->a_p);
+               if (retval) {
+                       if (hfsmp->jnl) {
+                               journal_end_transaction(hfsmp->jnl);
+                       }
+                       hfs_global_shared_lock_release(hfsmp);
 
 
-        retval = MacToVFSError(
+                       goto Err_Exit;
+               }                       
+
+               retval = MacToVFSError(
                             TruncateFileC(
                             TruncateFileC(
-                                            HTOVCB(hp),
-                                            fcb,
+                                            VTOVCB(vp),
+                                            (FCB*)fp,
                                             length,
                                             false));
                                             length,
                                             false));
-        (void) hfs_metafilelocking(HTOHFS(hp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
+               (void) hfs_metafilelocking(VTOHFS(vp), kHFSExtentsFileID, LK_RELEASE, ap->a_p);
+               filebytes = (off_t)fp->ff_blocks * (off_t)VTOVCB(vp)->blockSize;
+
+               if (hfsmp->jnl) {
+                       hfs_flushvolumeheader(hfsmp, MNT_NOWAIT, 0);
+                       journal_end_transaction(hfsmp->jnl);
+               }
+               hfs_global_shared_lock_release(hfsmp);
+               
 
                /*
                 * if we get an error and no changes were made then exit
                 * otherwise we must do the VOP_UPDATE to reflect the changes
                 */
 
                /*
                 * if we get an error and no changes were made then exit
                 * otherwise we must do the VOP_UPDATE to reflect the changes
                 */
-               if (retval && (startingPEOF == fcb->fcbPLen)) goto Err_Exit;
-        if (fcb->fcbFlags & fcbModifiedMask)
-           hp->h_nodeflags |= IN_MODIFIED;
-
-        DBG_ASSERT(length <= fcb->fcbPLen)  // DEBUG DEBUG DEBUG DEBUG DEBUG DEBUG DEBUG
+               if (retval && (startingPEOF == filebytes)) goto Err_Exit;
+#if QUOTA
+               /* These are  bytesreleased */
+               (void) hfs_chkdq(cp, (int64_t)-((startingPEOF - filebytes)), NOCRED,0);
+#endif /* QUOTA */
 
 
-        if (fcb->fcbEOF > fcb->fcbPLen) {
-                       fcb->fcbEOF = fcb->fcbPLen;
+               if (fp->ff_size > filebytes) {
+                       fp->ff_size = filebytes;
 
                        if (UBCISVALID(vp))
 
                        if (UBCISVALID(vp))
-                               ubc_setsize(vp, (off_t)fcb->fcbEOF); /* XXX check errors */
-        }
-    }
+                               ubc_setsize(vp, fp->ff_size); /* XXX check errors */
+               }
+       }
 
 Std_Exit:
 
 Std_Exit:
-    hp->h_nodeflags |= IN_CHANGE | IN_UPDATE;
+       cp->c_flag |= C_CHANGE | C_UPDATE;
        retval2 = VOP_UPDATE(vp, &tv, &tv, MNT_WAIT);
 
        retval2 = VOP_UPDATE(vp, &tv, &tv, MNT_WAIT);
 
-    if (retval == 0) retval = retval2;
-
+       if (retval == 0)
+               retval = retval2;
 Err_Exit:
 Err_Exit:
-    DBG_VOP_LOCKS_TEST(retval);
-    return (retval);
+       return (retval);
 }
 
 
 }
 
 
-
-
-/* pagein for HFS filesystem, similar to hfs_read(), but without cluster_read() */
+/*
+ * pagein for HFS filesystem
+ */
 int
 hfs_pagein(ap)
        struct vop_pagein_args /* {
 int
 hfs_pagein(ap)
        struct vop_pagein_args /* {
@@ -1882,44 +1918,19 @@ hfs_pagein(ap)
                int           a_flags
        } */ *ap;
 {
                int           a_flags
        } */ *ap;
 {
-    register struct vnode *vp;
-    struct hfsnode       *hp;
-    FCB                          *fcb;
-    int                                devBlockSize = 0;
-    int                   retval;
-
-    DBG_FUNC_NAME("hfs_pagein");
-    DBG_VOP_LOCKS_DECL(1);
-    DBG_VOP_PRINT_FUNCNAME();
-    DBG_VOP_PRINT_VNODE_INFO(vp);DBG_VOP_CONT(("\n"));
-    DBG_VOP_LOCKS_INIT(0,vp, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-
-    vp  = ap->a_vp;
-    hp  = VTOH(vp);
-    fcb = HTOFCB(hp);
-
-    if (vp->v_type != VREG && vp->v_type != VLNK)
-       panic("hfs_pagein: vp not UBC type\n");
-
-    DBG_VOP(("\tfile size Ox%X\n", (u_int)fcb->fcbEOF));
-    DBG_VOP(("\tstarting at offset Ox%X of file, length Ox%X\n", (u_int)ap->a_f_offset, (u_int)ap->a_size));
-
-#if HFS_DIAGNOSTIC
-    debug_check_blocksizes(vp);
-#endif
-
-    VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
+       register struct vnode *vp = ap->a_vp;
+       int devBlockSize = 0;
+       int error;
 
 
-    retval = cluster_pagein(vp, ap->a_pl, ap->a_pl_offset, ap->a_f_offset,
-                        ap->a_size, (off_t)fcb->fcbEOF, devBlockSize,
-                        ap->a_flags);
+       if (vp->v_type != VREG && vp->v_type != VLNK)
+               panic("hfs_pagein: vp not UBC type\n");
 
 
-#if HFS_DIAGNOSTIC
-    debug_check_blocksizes(vp);
-#endif
-    DBG_VOP_LOCKS_TEST(retval);
+       VOP_DEVBLOCKSIZE(VTOC(vp)->c_devvp, &devBlockSize);
 
 
-    return (retval);
+       error = cluster_pagein(vp, ap->a_pl, ap->a_pl_offset, ap->a_f_offset,
+                               ap->a_size, (off_t)VTOF(vp)->ff_size, devBlockSize,
+                               ap->a_flags);
+       return (error);
 }
 
 /* 
 }
 
 /* 
@@ -1937,43 +1948,28 @@ hfs_pageout(ap)
           int           a_flags
        } */ *ap;
 {
           int           a_flags
        } */ *ap;
 {
-       struct vnode    *vp = ap->a_vp;
-       struct hfsnode  *hp =  VTOH(vp);
-       FCB             *fcb = HTOFCB(hp);
-       int              retval;
-       int              devBlockSize = 0;
-       off_t            end_of_range;
-
-       DBG_FUNC_NAME("hfs_pageout");
-       DBG_VOP_LOCKS_DECL(1);
-       DBG_VOP_PRINT_FUNCNAME();
-       DBG_VOP_PRINT_VNODE_INFO(vp);DBG_VOP_CONT(("\n"));
-       DBG_VOP(("\thfsnode 0x%x (%s)\n", (u_int)hp, H_NAME(hp)));
-       DBG_VOP(("\tstarting at offset Ox%lX of file, length Ox%lX\n", 
-               (UInt32)ap->a_f_offset, (UInt32)ap->a_size));
-
-       DBG_VOP_LOCKS_INIT(0, vp, VOPDBG_LOCKED, 
-               VOPDBG_LOCKED, VOPDBG_LOCKED, VOPDBG_POS);
-
-#if HFS_DIAGNOSTIC
-       debug_check_blocksizes(vp);
-#endif
+       struct vnode *vp = ap->a_vp;
+       struct cnode *cp = VTOC(vp);
+       struct filefork *fp = VTOF(vp);
+       int retval;
+       int devBlockSize = 0;
+       off_t end_of_range;
+       off_t filesize;
 
        if (UBCINVALID(vp))
                panic("hfs_pageout: Not a  VREG: vp=%x", vp);
 
 
        if (UBCINVALID(vp))
                panic("hfs_pageout: Not a  VREG: vp=%x", vp);
 
-       VOP_DEVBLOCKSIZE(hp->h_meta->h_devvp, &devBlockSize);
-
+       VOP_DEVBLOCKSIZE(cp->c_devvp, &devBlockSize);
+       filesize = fp->ff_size;
        end_of_range = ap->a_f_offset + ap->a_size - 1;
 
        end_of_range = ap->a_f_offset + ap->a_size - 1;
 
-       if (end_of_range >= (off_t)fcb->fcbEOF)
-               end_of_range = (off_t)(fcb->fcbEOF - 1);
-
-       if (ap->a_f_offset < (off_t)fcb->fcbEOF)
-               rl_remove(ap->a_f_offset, end_of_range, &hp->h_invalidranges);
+       if (end_of_range >= filesize)
+               end_of_range = (off_t)(filesize - 1);
+       if (ap->a_f_offset < filesize)
+               rl_remove(ap->a_f_offset, end_of_range, &fp->ff_invalidranges);
 
        retval = cluster_pageout(vp, ap->a_pl, ap->a_pl_offset, ap->a_f_offset, ap->a_size,
 
        retval = cluster_pageout(vp, ap->a_pl, ap->a_pl_offset, ap->a_f_offset, ap->a_size,
-                                (off_t)fcb->fcbEOF, devBlockSize, ap->a_flags);
+                                filesize, devBlockSize, ap->a_flags);
 
        /*
         * If we successfully wrote any data, and we are not the superuser
 
        /*
         * If we successfully wrote any data, and we are not the superuser
@@ -1981,13 +1977,8 @@ hfs_pageout(ap)
         * tampering.
         */
        if (retval == 0 && ap->a_cred && ap->a_cred->cr_uid != 0)
         * tampering.
         */
        if (retval == 0 && ap->a_cred && ap->a_cred->cr_uid != 0)
-               hp->h_meta->h_mode &= ~(ISUID | ISGID);
+               cp->c_mode &= ~(S_ISUID | S_ISGID);
 
 
-#if HFS_DIAGNOSTIC
-       debug_check_blocksizes(vp);
-#endif
-
-       DBG_VOP_LOCKS_TEST(retval);
        return (retval);
 }
 
        return (retval);
 }
 
@@ -1999,39 +1990,46 @@ hfs_pageout(ap)
  */
 int
 hfs_bwrite(ap)
  */
 int
 hfs_bwrite(ap)
-struct vop_bwrite_args /* {
-    struct buf *a_bp;
-} */ *ap;
+       struct vop_bwrite_args /* {
+               struct buf *a_bp;
+       } */ *ap;
 {
 {
-    register struct buf *bp = ap->a_bp;
-    register struct vnode *vp = bp->b_vp;
-    BlockDescriptor block;
-    int retval = 0;
-
-       DBG_FUNC_NAME("hfs_bwrite");
-
+       int retval = 0;
+       register struct buf *bp = ap->a_bp;
+       register struct vnode *vp = bp->b_vp;
 #if BYTE_ORDER == LITTLE_ENDIAN
 #if BYTE_ORDER == LITTLE_ENDIAN
-    /* Trap B-Tree writes */
-    if ((H_FILEID(VTOH(vp)) == kHFSExtentsFileID) ||
-        (H_FILEID(VTOH(vp)) == kHFSCatalogFileID)) {
-
-        /* Swap if the B-Tree node is in native byte order */
-        if (((UInt16 *)((char *)bp->b_data + bp->b_bcount - 2))[0] == 0x000e) {
-            /* Prepare the block pointer */
-            block.blockHeader = bp;
-            block.buffer = bp->b_data;
-            block.blockReadFromDisk = (bp->b_flags & B_CACHE) == 0;    /* not found in cache ==> came from disk */
-            block.blockSize = bp->b_bcount;
+       BlockDescriptor block;
+
+       /* Trap B-Tree writes */
+       if ((VTOC(vp)->c_fileid == kHFSExtentsFileID) ||
+           (VTOC(vp)->c_fileid == kHFSCatalogFileID)) {
+
+               /* Swap if the B-Tree node is in native byte order */
+               if (((UInt16 *)((char *)bp->b_data + bp->b_bcount - 2))[0] == 0x000e) {
+                       /* Prepare the block pointer */
+                       block.blockHeader = bp;
+                       block.buffer = bp->b_data;
+                       /* not found in cache ==> came from disk */
+                       block.blockReadFromDisk = (bp->b_flags & B_CACHE) == 0;
+                       block.blockSize = bp->b_bcount;
     
     
-            /* Endian un-swap B-Tree node */
-            SWAP_BT_NODE (&block, ISHFSPLUS (VTOVCB(vp)), H_FILEID(VTOH(vp)), 1);
-        }
+                       /* Endian un-swap B-Tree node */
+                       SWAP_BT_NODE (&block, ISHFSPLUS (VTOVCB(vp)), VTOC(vp)->c_fileid, 1);
+               }
 
 
-        /* We don't check to make sure that it's 0x0e00 because it could be all zeros */
-    }
+               /* We don't check to make sure that it's 0x0e00 because it could be all zeros */
+       }
 #endif
 #endif
+       /* This buffer shouldn't be locked anymore but if it is clear it */
+       if (ISSET(bp->b_flags, B_LOCKED)) {
+           // XXXdbg
+           if (VTOHFS(vp)->jnl) {
+                       panic("hfs: CLEARING the lock bit on bp 0x%x\n", bp);
+           }
+               CLR(bp->b_flags, B_LOCKED);
+               printf("hfs_bwrite: called with lock bit set\n");
+       }
+       retval = vn_bwrite (ap);
 
 
-    retval = vn_bwrite (ap);
-
-    return (retval);
+       return (retval);
 }
 }