Re: [PATCH 9/10] xfs: implement bulkstat per allocation group

[Jeff Liu <jeff.liu@xxxxxxxxxx>]

On 2013å12æ31æ 02:18, Mark Tinguely wrote:
> On 12/28/13 05:22, Jeff Liu wrote:
>> From: Jie Liu<jeff.liu@xxxxxxxxxx>
>>
>> With xfs_perag_bulkstat(), we can return stat information in bulk by
>> inode as per the given allocation group number and start inode number
>> in it via a new ioctl(2) in the future.
>>
>> Refactor xfs_bulkstat() with it.
>>
>> Signed-off-by: Jie Liu<jeff.liu@xxxxxxxxxx>
>> ---
> 
> 
> causes xfs/183 to assert for me:
> 
> XFS: Assertion failed: fs_is_ok, file: /root/xfs/fs/xfs/xfs_itable.c,
> line: 409
Oops!! I guess the assertion is failed at:

+                       error = xfs_inobt_lookup(cur, 0, XFS_LOOKUP_GE, &i);
+                       if (error)
+                               break;
+                       XFS_WANT_CORRUPTED_GOTO(i == 1, error0);

Which means I have to take care of those non-primary AGs on an new initialized
empty-filesystem as there has no inode chunk has been allocated...

Thanks,
-Jeff