xfs
[Top] [All Lists]

[XFS updates] XFS development tree branch, master, updated. v2.6.28-rc3-

To: xfs@xxxxxxxxxxx
Subject: [XFS updates] XFS development tree branch, master, updated. v2.6.28-rc3-1102-g5a8d0f3
From: xfs@xxxxxxxxxxx
Date: Wed, 3 Dec 2008 22:40:12 -0600
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "XFS development tree".

The branch, master has been updated
  5a8d0f3 move inode tracing out of xfs_vnode.
  25e41b3 move vn_iowait / vn_iowake into xfs_aops.c
  583fa58 kill vn_ioerror
  f95099b kill xfs_unmount_flush
  e57481d no explicit xfs_iflush for special inodes during unmount
  070c461 use xfs_trans_ijoin in xfs_trans_iget
  b56757b remove leftovers of shared read-only support
  e88f11a remove unused m_inode_quiesce member from struct xfs_mount
  6bd16ff kill dead inode flags
  5efcbb8 cleanup xfs_sb.h feature flag helpers
  df6771b kill dead quota flags
  63ad2a5 remove dead code from sv_t implementation
  39e2def reduce l_icloglock roundtrips
  d9424b3 stop using igrab in xfs_vn_link
  5d765b9 kill xfs_buf_iostart
  5cafdeb cleanup the inode reclaim path
  ccd0be6 remove unused prototypes for xfs_ihash_init / xfs_ihash_free
  73e6335 remove unused behvavior cruft in xfs_super.h
  2234d54 remove useless mnt_want_write call in xfs_write
      from  ddcd856d81861a523d79d077facd875da1f66792 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5a8d0f3c7af801c7263fbba39952504d6fc7ff60
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:40 2008 +0100

    move inode tracing out of xfs_vnode.
    
    Move the inode tracing into xfs_iget.c / xfs_inode.h and kill xfs_vnode.c
    now that it's empty.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 25e41b3d521f52771354a718042a753a3e77df0a
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:39 2008 +0100

    move vn_iowait / vn_iowake into xfs_aops.c
    
    The whole machinery to wait on I/O completion is related to the I/O path
    and should be there instead of in xfs_vnode.c.  Also give the functions
    more descriptive names.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 583fa586f0e4a8222dd091ce971b85c1364f3d92
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:38 2008 +0100

    kill vn_ioerror
    
    There's just one caller of this helper, and it's much cleaner to just merge
    the xfs_do_force_shutdown call into it.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit f95099ba5ae06b96a9c17ef93cc655f686d79077
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:37 2008 +0100

    kill xfs_unmount_flush
    
    There's almost nothing left in this function, instead remove the IRELE
    on the real times inodes and the call to XFS_QM_UNMOUNT into xfs_unmountfs.
    
    For the regular unmount case that means it now also happenes after dmapi
    notification, but otherwise there is no difference in behaviour.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit e57481dc269cd3773b22f53bfb869308780a7bf1
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:36 2008 +0100

    no explicit xfs_iflush for special inodes during unmount
    
    Currently we explicitly call xfs_iflush on the quota, real-time and root
    inodes from xfs_unmount_flush.  But we just called xfs_sync_inodes with
    SYNC_ATTR and do an XFS_bflush aka xfs_flush_buftarg to make sure all inodes
    are on disk already, so there is no need for these special cases.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 070c4616ec62fc207e2aeef9d0f28af294c651d0
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:35 2008 +0100

    use xfs_trans_ijoin in xfs_trans_iget
    
    Use xfs_trans_ijoin in xfs_trans_iget in case we need to join an inode into
    a transaction instead of opencoding it.  Based on a discussion with and an
    incomplete patch from Niv Sardi.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit b56757becf8bc62292263a24a23cf55edb4be55f
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:34 2008 +0100

    remove leftovers of shared read-only support
    
    We never supported shared read-only filesystems, so remove the dead
    code left over from IRIX for it.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit e88f11abe09d14718b82a991db118c5e485aa897
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:33 2008 +0100

    remove unused m_inode_quiesce member from struct xfs_mount
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 6bd16ff27060819d16b3e7abe59b6644b349aea3
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:32 2008 +0100

    kill dead inode flags
    
    There are a few inode flags around that aren't used anywhere, so remove
    them.  Also update xfsidbg to display all used inode flags correctly.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 5efcbb853bc2f051d720a191268f8dd901fea9c2
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:31 2008 +0100

    cleanup xfs_sb.h feature flag helpers
    
    The various inlines in xfs_sb.h that deal with the superblock version
    and fature flags were converted from macros a while ago, and this
    show by the odd coding style full of useless braces and backslashes
    and the avoidance of conditionals.
    
    Clean these up to look like normal C code.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Donald Douwsma <donaldd@xxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit df6771bde14551eceeacf331666a92735e0773ac
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:30 2008 +0100

    kill dead quota flags
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 63ad2a5c4cf37e3242142eee8a8dcd4a8515302e
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:29 2008 +0100

    remove dead code from sv_t implementation
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 39e2defe73106ca2e1c85e5286038a0a13f49513
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:28 2008 +0100

    reduce l_icloglock roundtrips
    
    All but one caller of xlog_state_want_sync drop and re-acquire
    l_icloglock around the call to it, just so that xlog_state_want_sync can
    acquire and drop it.
    
    Move all lock operation out of l_icloglock and assert that the lock is
    held when it is called.
    
    Note that it would make sense to extende this scheme to
    xlog_state_release_iclog, but the locking in there is more complicated
    and we'd like to keep the atomic_dec_and_lock optmization for those
    callers not having l_icloglock yet.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit d9424b3c4a1e96f87c6cfd4d8dd2f8d9bbb4dcc5
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:27 2008 +0100

    stop using igrab in xfs_vn_link
    
    ->link is guranteed to get an already reference inode passed so we
    can do a simple increment of i_count instead of using igrab and thus
    avoid banging on the global inode_lock.  This is what most filesystems
    already do.
    
    Also move the increment after the call to xfs_link to simplify error
    handling.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 5d765b976c3a41faf9a73718fb8cc5833990a8ef
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:26 2008 +0100

    kill xfs_buf_iostart
    
    xfs_buf_iostart is a "shared" helper for xfs_buf_read_flags,
    xfs_bawrite, and xfs_bdwrite - except that there isn't much shared
    code but rather special cases for each caller.
    
    So remove this function and move the functionality to the caller.
    xfs_bawrite and xfs_bdwrite are now big enough to be moved out of
    line and the xfs_buf_read_flags is moved into a new helper called
    _xfs_buf_read.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 5cafdeb2891a415a5dbf0ad80f0afedf8369e6bb
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:25 2008 +0100

    cleanup the inode reclaim path
    
    Merge xfs_iextract and xfs_idestroy into xfs_ireclaim as they are never
    called individually.  Also rewrite most comments in this area as they
    were severly out of date.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit ccd0be6cfc6943c4e0b3e3cdb598e0b7354a2d78
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:24 2008 +0100

    remove unused prototypes for xfs_ihash_init / xfs_ihash_free
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 73e6335c14209e508bec8ca7985d1fbde183bd1f
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:23 2008 +0100

    remove unused behvavior cruft in xfs_super.h
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

commit 2234d54d3d855d6ffae88a24772a9389d6755e0c
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Wed Dec 3 12:20:22 2008 +0100

    remove useless mnt_want_write call in xfs_write
    
    When mnt_want_write was introduced a call to it was added around
    xfs_ichgtime, but there is no need for this because a file can't be open
    read/write on a r/o mount, and a mount can't degrade r/o while we still
    have files open for writing.  As the mnt_want_write changes were never
    merged into the CVS tree this patch is for mainline only.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>
    Signed-off-by: Niv Sardi <xaiki@xxxxxxx>

-----------------------------------------------------------------------

Summary of changes:
 fs/xfs/Makefile              |    2 -
 fs/xfs/linux-2.6/sv.h        |   22 +----
 fs/xfs/linux-2.6/xfs_aops.c  |   64 ++++++++++++---
 fs/xfs/linux-2.6/xfs_aops.h  |    3 +
 fs/xfs/linux-2.6/xfs_buf.c   |   84 +++++++++++---------
 fs/xfs/linux-2.6/xfs_buf.h   |   22 +-----
 fs/xfs/linux-2.6/xfs_file.c  |   10 +--
 fs/xfs/linux-2.6/xfs_iops.c  |   14 +---
 fs/xfs/linux-2.6/xfs_lrw.c   |   10 +--
 fs/xfs/linux-2.6/xfs_super.c |   37 +---------
 fs/xfs/linux-2.6/xfs_super.h |   12 ---
 fs/xfs/linux-2.6/xfs_sync.c  |    2 +-
 fs/xfs/linux-2.6/xfs_vnode.c |  132 -------------------------------
 fs/xfs/linux-2.6/xfs_vnode.h |   64 ---------------
 fs/xfs/quota/xfs_qm.c        |   44 ++---------
 fs/xfs/quota/xfs_qm.h        |    2 +-
 fs/xfs/xfs_iget.c            |  177 +++++++++++++++++++++++++++++++----------
 fs/xfs/xfs_inode.c           |   78 +------------------
 fs/xfs/xfs_inode.h           |   66 ++++++++++++----
 fs/xfs/xfs_log.c             |   13 ++--
 fs/xfs/xfs_mount.c           |   36 +++------
 fs/xfs/xfs_mount.h           |    7 +--
 fs/xfs/xfs_quota.h           |    8 +-
 fs/xfs/xfs_sb.h              |  166 ++++++++++++++++++++-------------------
 fs/xfs/xfs_trans_inode.c     |   30 +-------
 fs/xfs/xfs_vfsops.c          |  141 ---------------------------------
 fs/xfs/xfs_vnodeops.c        |    7 +-
 27 files changed, 424 insertions(+), 829 deletions(-)
 delete mode 100644 fs/xfs/linux-2.6/xfs_vnode.c
 delete mode 100644 fs/xfs/xfs_vfsops.c


hooks/post-receive
-- 
XFS development tree

<Prev in Thread] Current Thread [Next in Thread>
  • [XFS updates] XFS development tree branch, master, updated. v2.6.28-rc3-1102-g5a8d0f3, xfs <=