xfs
[Top] [All Lists]

[XFS updates] XFS development tree branch, xfs-bulkstat-refactor-NEEDS-F

To: xfs@xxxxxxxxxxx
Subject: [XFS updates] XFS development tree branch, xfs-bulkstat-refactor-NEEDS-FIXING, created. xfs-for-linus-3.16-rc1-13116-g1e451aa
From: xfs@xxxxxxxxxxx
Date: Tue, 29 Jul 2014 18:21:43 -0500 (CDT)
Delivered-to: xfs@xxxxxxxxxxx
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, xfs-bulkstat-refactor-NEEDS-FIXING has been created
        at  1e451aad70915a33111dfcce2c2e1e4582b462a3 (commit)

- Log -----------------------------------------------------------------
commit 1e451aad70915a33111dfcce2c2e1e4582b462a3
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 18:45:28 2014 +1000

    xfs: introduce xfs_bulkstat_ag_ichunk
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    Introduce xfs_bulkstat_ag_ichunk() to process inodes in chunk with a
    pointer to a formatter function that will iget the inode and fill in
    the appropriate structure.
    
    Refactor xfs_bulkstat() with it.
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit f3d1e587437b784635459dcfccaedd17149282cc
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 18:42:21 2014 +1000

    xfs: introduce xfs_bulkstat_grab_ichunk
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    Introduce xfs_bulkstat_grab_ichunk() to look up an inode chunk in where
    the given inode resides, then grab the record.  Update the data for the
    pointed-to record if the inode was not the last in the chunk and there
    are some left allocated, return the grabbed inode count on success.
    
    Refactor xfs_bulkstat() with it.
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 4b8fdfecd84528e044a6ca32242de641b203995b
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 18:41:18 2014 +1000

    xfs: introduce xfs_bulkstat_ichunk_ra
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    Introduce xfs_bulkstat_ichunk_ra() to loop over all clusters in the
    next inode chunk, then performs readahead if there are any allocated
    inodes in that cluster.
    
    Refactor xfs_bulkstat() with it.
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Christoph Hellwig <hch@xxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit d4c27348751bffeb57d87fc631f76716e3036c6d
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 18:40:43 2014 +1000

    xfs: fix error handling at xfs_bulkstat
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    We should not ignore the btree operation errors at xfs_bulkstat() but
    to propagate them if any. This patch fix two places in this function
    and the remaining things will be fixed with code refactoring thereafter.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 296dfd7fdba3a8b702d9cb47afc9061f52e18656
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 18:40:26 2014 +1000

    xfs: remove redundant user buffer count checks at xfs_bulkstat
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    Remove the redundant user buffer and count checks as it has already
    been validated at xfs_ioc_bulkstat().
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit c7cb51dcb0a38624d42eeabb38502fa54a4d774b
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 12:18:47 2014 +1000

    xfs: fix error handling at xfs_inumbers
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    To fetch the file system number tables, we currently just ignore the
    errors and proceed to loop over the next AG or bump agino to the next
    chunk in case of btree operations failed, that is not properly because
    those errors might hint us potential file system problems.
    
    This patch rework xfs_inumbers() to handle the btree operation errors
    as well as the loop conditions.
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 549fa00679dd14beaaa685486549c637e8e02182
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 12:11:47 2014 +1000

    xfs: consolidate xfs_inumbers
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    Consolidate xfs_inumbers() to make the formatter function return correct
    error and make the source code looks a bit neat.
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit d716f8eedb143387afd35d337327090b8348159e
Author: Christoph Hellwig <hch@xxxxxx>
Date:   Thu Jul 24 12:07:15 2014 +1000

    xfs: remove xfs_bulkstat_single
    
    From: Christoph Hellwig <hch@xxxxxx>
    
    xfs_bukstat_one doesn't have any failure case that would go away when
    called through xfs_bulkstat, so remove the fallback and the now unessecary
    xfs_bulkstat_single function.
    
    Signed-off-by: Christoph Hellwig <hch@xxxxxx>
    Reviewed-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

commit 8fe657760d5c8372cd4cf022c4cdb9307a3af995
Author: Jie Liu <jeff.liu@xxxxxxxxxx>
Date:   Thu Jul 24 11:33:28 2014 +1000

    xfs: remove redundant stat assignment in xfs_bulkstat_one_int
    
    From: Jie Liu <jeff.liu@xxxxxxxxxx>
    
    Remove the redundant BULKSTAT_RV_NOTHING assignment in case of call
    xfs_iget() failed at xfs_bulkstat_one_int().
    
    Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx>
    Reviewed-by: Christoph Hellwig <hch@xxxxxx>
    Signed-off-by: Dave Chinner <david@xxxxxxxxxxxxx>

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


hooks/post-receive
-- 
XFS development tree

<Prev in Thread] Current Thread [Next in Thread>
  • [XFS updates] XFS development tree branch, xfs-bulkstat-refactor-NEEDS-FIXING, created. xfs-for-linus-3.16-rc1-13116-g1e451aa, xfs <=