On 5.07.2018 14:50, Nikolay Borisov wrote:
> Commit 5d23515be669 ("btrfs: Move qgroup rescan on quota enable to
> btrfs_quota_enable") not only resulted in an easier to follow code but
> it also introduced a subtle bug. It changed the timing when the initial
> transaction rescan was happening - before the commit it would happen
> after transaction commit had occured but after the commit it might happen
> before the transaction was committed. This results in failure to
> correctly rescan the quota since there could be data which is still not
> committed on disk.
>
> This patch aims to fix this by movign the transaction creation/commit
> inside btrfs_quota_enable, which allows to schedule the quota commit
> after the transaction has been committed.
>
> Fixes: 5d23515be669 ("btrfs: Move qgroup rescan on quota enable to
> btrfs_quota_enable")
> Reported-by: Misono Tomohiro <misono.tomoh...@jp.fujitsu.com>
> Link: https://marc.info/?l=linux-btrfs&m=152999289017582
> Signed-off-by: Nikolay Borisov <nbori...@suse.com>
> ---
>
> Hello,
>
> One open-ended question this patch puts is how many items should be reserved
> to guarantee successful qgroup disable. Before this patch quota disable was
> always using the arbitrary reservation of 2 items. Since this patch makes the
> the transaction for enable/disable independent we have a chance to actually
> set appropriate values for both transactions. In any case this patch doesn't
> make things worse than before.
>
> fs/btrfs/ioctl.c | 15 ++--------
> fs/btrfs/qgroup.c | 86
> ++++++++++++++++++++++++++++++++++++++++++++++---------
> fs/btrfs/qgroup.h | 6 ++--
> 3 files changed, 76 insertions(+), 31 deletions(-)
>
Changes since v2:
* Actually implemented proper transaction unwiding in both functions.
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html