xfs
[Top] [All Lists]

[XFS updates] XFS development tree branch, for-linus, updated. v2.6.28-r

To: xfs@xxxxxxxxxxx
Subject: [XFS updates] XFS development tree branch, for-linus, updated. v2.6.28-rc3-11553-gb6e3222
From: xfs@xxxxxxxxxxx
Date: Sun, 18 Jan 2009 22:05:17 -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, for-linus has been updated
  b6e3222 [XFS] Remove the rest of the macro-to-function indirections.
  b828d8c xfs: sanity check attr fork size
  4973914 xfs: fix bad_features2 fixups for the root filesystem
  5aa2dc0 xfs: add a lock class for group/project dquots
  4f2d4ac xfs: lockdep annotations for xfs_dqlock2
  080dda7 xfs: add a separate lock class for the per-mount list of dquots
  62e194e xfs: use mnt_want_write in compat_attrmulti ioctl
  ab596ad xfs: fix dentry aliasing issues in open_by_handle
      from  f3b8436ad9a8ad36b3c9fa1fe030c7f38e5d3d0b (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 b6e3222732a3551e786aa47b90a8eab2a517711c
Author: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date:   Wed Jan 14 23:22:07 2009 -0600

    [XFS] Remove the rest of the macro-to-function indirections.
    
    Remove the last of the macros-defined-to-static-functions.
    
    Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
    Reviewed-by: Christoph Hellwig <hch@xxxxxxxxxxxxx>
    Signed-off-by: Lachlan McIlroy <lachlan@xxxxxxx>

commit b828d8c33867dd6479644c06500975570bfd525c
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:04:16 2009 +0100

    xfs: sanity check attr fork size
    
    Recently we have quite a few kerneloops reports about dereferencing a NULL
    if_data in the attribute fork.  From looking over the code this can only
    happen if we pass a 0 size argument to xfs_iformat_local.  This implies some
    sort of corruption and in fact the only mailinglist report about this from
    earlier this year was after a powerfail presumably on a system with write
    cache and without barriers.
    
    Add a quick sanity check for the attr fork size in xfs_iformat to catch
    these early and without an oops.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 49739140e57a65114d9e1976c4c158d2145595fb
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:04:07 2009 +0100

    xfs: fix bad_features2 fixups for the root filesystem
    
    Currently the bad_features2 fixup and the alignment updates in the 
superblock
    are skipped if we mount a filesystem read-only.  But for the root filesystem
    the typical case is to mount read-only first and only later remount 
writeable
    so we'll never perform this update at all.  It's not a big problem but means
    the logs of people needing the fixup get spammed at every boot because they
    never happen on disk.
    
    Reported-by: Arkadiusz Miskiewicz <arekm@xxxxxxxx>
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 5aa2dc0a0697c762874241fa9ddbecd2d878b934
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:03:25 2009 +0100

    xfs: add a lock class for group/project dquots
    
    We can have both a user and a group/project dquot locked at the same time,
    as long as the user dquot is locked first.  Tell lockdep about that fact
    by making the group/project dquots a different lock class.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 4f2d4ac6e5eb7d72e8df7f3fbf67a78dab8b91cf
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:03:19 2009 +0100

    xfs: lockdep annotations for xfs_dqlock2
    
    xfs_dqlock2 locks two xfs_dquots, which is fine as it always locks the
    dquot with the lower id first.  Use mutex_lock_nested to tell lockdep
    about this fact.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 080dda7f5e8e8df95bcd17a5345c276e365a2054
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:03:11 2009 +0100

    xfs: add a separate lock class for the per-mount list of dquots
    
    We can have both a a quota hash chain and the per-mount list locked at
    the same time.  But given that both use the same struct dqhash as list
    head we have to tell lockdep that they are different lock classes.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 62e194ecdaf8a1935991c1f8704886328d96a391
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:03:03 2009 +0100

    xfs: use mnt_want_write in compat_attrmulti ioctl
    
    The compat version of the attrmulti ioctl needs to ask for and then
    later release write access to the mount just like the native version,
    otherwise we could potentially write to read-only mounts.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit ab596ad8972f314ace538799734c7e1bdd1da2ff
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Mon Jan 19 02:02:57 2009 +0100

    xfs: fix dentry aliasing issues in open_by_handle
    
    Open by handle just grabs an inode by handle and then creates itself
    a dentry for it.  While this works for regular files it is horribly
    broken for directories, where the VFS locking relies on the fact that
    there is only just one single dentry for a given inode, and that
    these are always connected to the root of the filesystem so that
    it's locking algorithms work (see Documentations/filesystems/Locking)
    
    Remove all the existing open by handle code and replace it with a small
    wrapper around the exportfs code which deals with all these issues.
    At the same time we also make the checks for a valid handle strict
    enough to reject all not perfectly well formed handles - given that
    we never hand out others that's okay and simplifies the code.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Dave Chinner <david@xxxxxxxxxxxxx>

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

Summary of changes:
 fs/xfs/Kconfig                 |    1 +
 fs/xfs/linux-2.6/xfs_ioctl.c   |  305 ++++++++++++++++++----------------------
 fs/xfs/linux-2.6/xfs_ioctl.h   |   15 ++-
 fs/xfs/linux-2.6/xfs_ioctl32.c |  184 ++++++++-----------------
 fs/xfs/linux-2.6/xfs_super.c   |   17 ++-
 fs/xfs/quota/xfs_dquot.c       |   38 ++++--
 fs/xfs/quota/xfs_dquot.h       |   10 ++
 fs/xfs/quota/xfs_qm.c          |    9 ++
 fs/xfs/xfs_ag.h                |    6 +-
 fs/xfs/xfs_alloc_btree.c       |    2 +-
 fs/xfs/xfs_attr.c              |   26 ++--
 fs/xfs/xfs_bmap.c              |  166 +++++++++++-----------
 fs/xfs/xfs_bmap.h              |    2 -
 fs/xfs/xfs_bmap_btree.c        |   10 +-
 fs/xfs/xfs_bmap_btree.h        |    4 -
 fs/xfs/xfs_btree.c             |    6 +-
 fs/xfs/xfs_da_btree.c          |    8 +-
 fs/xfs/xfs_ialloc.c            |    6 +-
 fs/xfs/xfs_ialloc.h            |    2 -
 fs/xfs/xfs_ialloc_btree.h      |    1 -
 fs/xfs/xfs_inode.c             |   19 ++-
 fs/xfs/xfs_inode_item.h        |    4 -
 fs/xfs/xfs_iomap.c             |   10 +-
 fs/xfs/xfs_itable.c            |    6 +-
 fs/xfs/xfs_mount.c             |   26 ++--
 fs/xfs/xfs_mount.h             |    9 +-
 fs/xfs/xfs_rename.c            |    2 +-
 fs/xfs/xfs_rtalloc.c           |    2 +-
 fs/xfs/xfs_rw.h                |    1 -
 fs/xfs/xfs_sb.h                |    2 +-
 fs/xfs/xfs_vnodeops.c          |   20 ++--
 31 files changed, 436 insertions(+), 483 deletions(-)


hooks/post-receive
-- 
XFS development tree

<Prev in Thread] Current Thread [Next in Thread>
  • [XFS updates] XFS development tree branch, for-linus, updated. v2.6.28-rc3-11553-gb6e3222, xfs <=