xfs
[Top] [All Lists]

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

To: xfs@xxxxxxxxxxx
Subject: [XFS updates] XFS development tree branch, for-linus, updated. v2.6.30-rc4-23627-gda15007
From: xfs@xxxxxxxxxxx
Date: Fri, 30 Oct 2009 13:43:18 -0500
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
  da15007 xfs: fix xfs_quota remove error
  f66e825 xfs: free temporary cursor in xfs_dialloc
      from  055bdbf7f36954893cbf1c86ea00f4af157e83b8 (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 da15007f696491f063018c46b70918e107f6edc9
Author: Ryota Yamauchi <r-yamauchi@xxxxxxxxxxxxx>
Date:   Fri Oct 30 09:27:44 2009 +0100

    xfs: fix xfs_quota remove error
    
    The xfs_quota returns ENOSYS when remove command is executed.
    Reproducable with following steps.
    
        # mount -t xfs -o uquota /dev/sda7 /mnt/mp1
        # xfs_quota -x -c off -c remove
        XFS_QUOTARM: Function not implemented.
    
    The remove command is allowed during quotaoff, but xfs_fs_set_xstate()
    checks whether quota is running, and it leads to ENOSYS.
    
    To solve this problem, add a check for X_QUOTARM.
    
    Signed-off-by: Ryota Yamauchi <r-yamauchi@xxxxxxxxxxxxx>
    Signed-off-by: Utako Kusaka <u-kusaka@xxxxxxxxxxxxx>
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Signed-off-by: Alex Elder <aelder@xxxxxxx>

commit f66e8253daa61642f7dc374a94b5672f5d67b920
Author: Eric Sandeen <sandeen@xxxxxxxxxxx>
Date:   Fri Oct 30 09:27:07 2009 +0100

    xfs: free temporary cursor in xfs_dialloc
    
    Commit bd169565993b39b9b4b102cdac8b13e0a259ce2f seems
    to have a slight regression where this code path:
    
        if (!--searchdistance) {
            /*
             * Not in range - save last search
             * location and allocate a new inode
             */
            ...
            goto newino;
        }
    
    doesn't free the temporary cursor (tcur) that got dup'd in
    this function.
    
    This leaks an item in the xfs_btree_cur zone, and it's caught
    on module unload:
    
    ===========================================================
    BUG xfs_btree_cur: Objects remaining on kmem_cache_close()
    -----------------------------------------------------------
    
    It seems like maybe a single free at the end of the function might
    be cleaner, but for now put a del_cursor right in this code block
    similar to the handling in the rest of the function.
    
    Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxxx>
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Signed-off-by: Alex Elder <aelder@xxxxxxx>

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

Summary of changes:
 fs/xfs/linux-2.6/xfs_quotaops.c |    2 +-
 fs/xfs/xfs_ialloc.c             |    1 +
 2 files changed, 2 insertions(+), 1 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.30-rc4-23627-gda15007, xfs <=