From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754953AbcIEH4v (ORCPT ); Mon, 5 Sep 2016 03:56:51 -0400 Received: from cn.fujitsu.com ([59.151.112.132]:18637 "EHLO heian.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1754323AbcIEH4s (ORCPT ); Mon, 5 Sep 2016 03:56:48 -0400 X-IronPort-AV: E=Sophos;i="5.22,518,1449504000"; d="scan'208";a="10605659" Subject: Re: [PATCH] Btrfs: remove unnecessary code of chunk_root assignment in btrfs_read_chunk_tree. To: Zhao Lei , "'Sean Fu'" , References: <1472990010-10707-1-git-send-email-fxinrong@gmail.com> <002501d20713$7ee67050$7cb350f0$@cn.fujitsu.com> CC: , , , , From: Qu Wenruo Message-ID: Date: Mon, 5 Sep 2016 15:56:41 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: <002501d20713$7ee67050$7cb350f0$@cn.fujitsu.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.167.226.34] X-yoursite-MailScanner-ID: 61CF24042423.AA494 X-yoursite-MailScanner: Found to be clean X-yoursite-MailScanner-From: quwenruo@cn.fujitsu.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At 09/05/2016 09:19 AM, Zhao Lei wrote: > Hi, Sean Fu > >> From: Sean Fu [mailto:fxinrong@gmail.com] >> Sent: Sunday, September 04, 2016 7:54 PM >> To: dsterba@suse.com >> Cc: clm@fb.com; anand.jain@oracle.com; fdmanana@suse.com; >> zhaolei@cn.fujitsu.com; linux-btrfs@vger.kernel.org; >> linux-kernel@vger.kernel.org; Sean Fu >> Subject: [PATCH] Btrfs: remove unnecessary code of chunk_root assignment in >> btrfs_read_chunk_tree. >> >> The input argument root is already set with "fs_info->chunk_root". >> "chunk_root = fs_info->chunk_root = btrfs_alloc_root(fs_info)" in caller >> "open_ctree". >> “root->fs_info = fs_info” in "btrfs_alloc_root". >> > The root argument of this function means "any root". > And the function is designed getting chunk root from > "any root" in head. > > Since there is only one caller of this function, > and the caller always send chunk_root as root argument in > current code, we can remove above conversion, > and I suggest renaming root to chunk_root to make it clear, > something like: > > - btrfs_read_chunk_tree(struct btrfs_root *root) > + btrfs_read_chunk_tree(struct btrfs_root *chunk_root) Since root is only used to get fs_info->chunk_root, why not use fs_info directly? Thanks, Qu > > Thanks > Zhaolei > >> Signed-off-by: Sean Fu >> --- >> fs/btrfs/volumes.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c >> index 366b335..384a6d2 100644 >> --- a/fs/btrfs/volumes.c >> +++ b/fs/btrfs/volumes.c >> @@ -6600,8 +6600,6 @@ int btrfs_read_chunk_tree(struct btrfs_root *root) >> int ret; >> int slot; >> >> - root = root->fs_info->chunk_root; >> - >> path = btrfs_alloc_path(); >> if (!path) >> return -ENOMEM; >> -- >> 2.6.2 >> > > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >