Re: [PATCH] btrfs: qgroup: Try our best to delete qgroup relations

[Qu Wenruo <quwenruo.btrfs@gmx.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1669 bytes --]



On 2019/8/6 上午2:13, David Sterba wrote:
> On Sat, Aug 03, 2019 at 02:45:59PM +0800, Qu Wenruo wrote:
>> When we try to delete qgroups, we're pretty cautious, we make sure both
>> qgroups exist and there is a relationship between them, then try to
>> delete the relation.
>>
>> This behavior is OK, but the problem is we need to two relation items,
>> and if we failed the first item deletion, we error out, leaving the
>> other relation item in qgroup tree.
>>
>> Sometimes the error from del_qgroup_relation_item() could just be
>> -ENOENT, thus we can ignore that error and continue without any problem.
>>
>> Further more, such cautious behavior makes qgroup relation deletion
>> impossible for orphan relation items.
>>
>> This patch will enhance __del_qgroup_relation():
>> - If both qgroups and their relation items exist
>>   Go the regular deletion routine and update their accounting if needed.
>>
>> - If any qgroup or relation item doesn't exist
>>   Then we still try to delete the orphan items anyway, but don't trigger
>>   the accounting update.
>>
>> By this, we try our best to remove relation items, and can handle orphan
>> relation items properly, while still keep the existing behavior for good
>> qgroup tree.
>>
>> Reported-by: Andrei Borzenkov <arvidjaar@gmail.com>
>> Signed-off-by: Qu Wenruo <wqu@suse.com>
> 
> Adding this to misc-next, please send a fstests testcase, thanks.
> 
That's the problem, I haven't found how the orphan relationship item is
left over.

Latest kernel will already delete both of them if nothing wrong happened.

So no idea how to write one test case.

Thanks,
Qu


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]