xfs
[Top] [All Lists]

[XFS updates] XFS development tree branch, master, updated. v2.6.37-rc4-

To: xfs@xxxxxxxxxxx
Subject: [XFS updates] XFS development tree branch, master, updated. v2.6.37-rc4-9187-geeb2036
From: xfs@xxxxxxxxxxx
Date: Tue, 1 Mar 2011 22:42:20 -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
  eeb2036 xfs: zero proper structure size for geometry calls
  20ad9ea xfs: enable delaylog by default
  ec3ba85 xfs: more sensible inode refcounting for ialloc
  1050c71 xfs: stop using xfs_trans_iget in the RT allocator
      from  5d15765594eeb5d82c5630b3428ea0ac4f7d3c31 (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 eeb2036b8a148629b762ae6d85cff0be8106f081
Author: Alex Elder <aelder@xxxxxxx>
Date:   Tue Mar 1 17:50:00 2011 +0000

    xfs: zero proper structure size for geometry calls
    
    Commit 493f3358cb289ccf716c5a14fa5bb52ab75943e5 added this call to
    xfs_fs_geometry() in order to avoid passing kernel stack data back
    to user space:
    
    +       memset(geo, 0, sizeof(*geo));
    
    Unfortunately, one of the callers of that function passes the
    address of a smaller data type, cast to fit the type that
    xfs_fs_geometry() requires.  As a result, this can happen:
    
    Kernel panic - not syncing: stack-protector: Kernel stack is corrupted
    in: f87aca93
    
    Pid: 262, comm: xfs_fsr Not tainted 2.6.38-rc6-493f3358cb2+ #1
    Call Trace:
    
    [<c12991ac>] ? panic+0x50/0x150
    [<c102ed71>] ? __stack_chk_fail+0x10/0x18
    [<f87aca93>] ? xfs_ioc_fsgeometry_v1+0x56/0x5d [xfs]
    
    Fix this by fixing that one caller to pass the right type and then
    copy out the subset it is interested in.
    
    Note: This patch is an alternative to one originally proposed by
    Eric Sandeen.
    
    Reported-by: Jeffrey Hundstad <jeffrey.hundstad@xxxxxxxx>
    Signed-off-by: Alex Elder <aelder@xxxxxxx>
    Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx>
    Tested-by: Jeffrey Hundstad <jeffrey.hundstad@xxxxxxxx>

commit 20ad9ea9becd34a3c16252ca9d815f2c74f8f30f
Author: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:   Sun Feb 13 12:06:34 2011 +0000

    xfs: enable delaylog by default
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Alex Elder <aelder@xxxxxxx>

commit ec3ba85f4083d10e32fe58b46db02d78ef71f6b8
Author: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:   Sun Feb 13 13:26:42 2011 +0000

    xfs: more sensible inode refcounting for ialloc
    
    Currently we return iodes from xfs_ialloc with just a single reference held.
    But we need two references, as one is dropped during transaction commit and
    the second needs to be transfered to the VFS.  Change xfs_ialloc to use
    xfs_iget plus xfs_trans_ijoin_ref to grab two references to the inode,
    and remove the now superflous IHOLD calls from all callers.  This also
    greatly simplifies the error handling in xfs_create and also allow to remove
    xfs_trans_iget as no other callers are left.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Alex Elder <aelder@xxxxxxx>

commit 1050c71e2925ab0cb025e4c89e08b15529a1ee36
Author: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:   Sun Feb 13 13:25:31 2011 +0000

    xfs: stop using xfs_trans_iget in the RT allocator
    
    During mount we establish references to the RT inodes, which we keep for
    the lifetime of the filesystem.  Instead of using xfs_trans_iget to grab
    additional references when adding RT inodes to transactions use the
    combination of xfs_ilock and xfs_trans_ijoin_ref, which archives the same
    end result with less overhead.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Alex Elder <aelder@xxxxxxx>

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

Summary of changes:
 fs/xfs/linux-2.6/xfs_ioctl.c |   11 +++++--
 fs/xfs/linux-2.6/xfs_super.c |    1 +
 fs/xfs/quota/xfs_qm.c        |    7 -----
 fs/xfs/xfs_bmap.c            |    8 +----
 fs/xfs/xfs_inode.c           |    5 ++-
 fs/xfs/xfs_rtalloc.c         |   54 ++++++++++++++----------------------
 fs/xfs/xfs_trans.h           |    2 -
 fs/xfs/xfs_trans_inode.c     |   22 ---------------
 fs/xfs/xfs_vnodeops.c        |   61 +++++++++++-------------------------------
 9 files changed, 51 insertions(+), 120 deletions(-)


hooks/post-receive
-- 
XFS development tree

<Prev in Thread] Current Thread [Next in Thread>
  • [XFS updates] XFS development tree branch, master, updated. v2.6.37-rc4-9187-geeb2036, xfs <=