From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cn.fujitsu.com ([59.151.112.132]:50200 "EHLO heian.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752805AbbCDBuj (ORCPT ); Tue, 3 Mar 2015 20:50:39 -0500 Message-ID: <54F66418.3030102@cn.fujitsu.com> Date: Wed, 4 Mar 2015 09:47:04 +0800 From: Dongsheng Yang MIME-Version: 1.0 To: Josef Bacik CC: Subject: Re: [PATCH 0/4] Btrfs: qgroup: part-2: bug fixes for qgroup reservation. References: <1423563828-9053-1-git-send-email-yangds.fnst@cn.fujitsu.com> In-Reply-To: <1423563828-9053-1-git-send-email-yangds.fnst@cn.fujitsu.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Sender: linux-btrfs-owner@vger.kernel.org List-ID: On 02/10/2015 06:23 PM, Dongsheng Yang wrote: > Hi all, > This is part2 for qgroup. It is based on > [PATCH 0/9] Btrfs: qgroup: part-1: bug fixes. > > [1/4] is reviewed-by Josef. > > [2/4] - [4/4] are trying to solve a problem in qgroup reservation. > It would be weired I introduce a may_use in 2/4 and delete it in 4/4. > > Yes, I am glad to rebase and squash them, here I am keeping this history > of the problem for reviewing. I have explained the problem and sent the > [2/4] out. Josef give me a looks good at that time. But in the later, I > found another problem and solved it with [4/4]. So they are trying to > solve different problems at different timing. When the all patches are > reviewed by you guys, I am okey to squash them if you request. Hi, any comment on this patchset? Thanx > > Thanx. > > Dongsheng Yang (4): > Btrfs: qgroup: free reserved in exceeding quota. > Btrfs: qgroup: Introduce a may_use to account > space_info->bytes_may_use. > Btrfs: qgroup, Account data space in more proper timings. > btrfs: qgroup: do a reservation in a higher level. > > fs/btrfs/ctree.h | 2 +- > fs/btrfs/extent-tree.c | 25 +++++++++++-------------- > fs/btrfs/file.c | 13 ++----------- > fs/btrfs/inode.c | 3 ++- > fs/btrfs/qgroup.c | 10 ++++------ > fs/btrfs/relocation.c | 2 +- > 6 files changed, 21 insertions(+), 34 deletions(-) >