From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934233AbcIGBiy (ORCPT ); Tue, 6 Sep 2016 21:38:54 -0400 Received: from mail-pa0-f66.google.com ([209.85.220.66]:33503 "EHLO mail-pa0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932133AbcIGBiw (ORCPT ); Tue, 6 Sep 2016 21:38:52 -0400 Date: Wed, 7 Sep 2016 09:38:44 +0800 From: Sean Fu To: Qu Wenruo Cc: Zhao Lei , dsterba@suse.com, clm@fb.com, anand.jain@oracle.com, fdmanana@suse.com, linux-btrfs@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Btrfs: remove unnecessary code of chunk_root assignment in btrfs_read_chunk_tree. Message-ID: <20160907013843.GA3907@linux-zmni.apac.novell.com> Mail-Followup-To: Qu Wenruo , Zhao Lei , dsterba@suse.com, clm@fb.com, anand.jain@oracle.com, fdmanana@suse.com, linux-btrfs@vger.kernel.org, linux-kernel@vger.kernel.org References: <1472990010-10707-1-git-send-email-fxinrong@gmail.com> <002501d20713$7ee67050$7cb350f0$@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 05, 2016 at 03:56:41PM +0800, Qu Wenruo wrote: > > > 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? Sorry for late reply. chunk_root is processed in btrfs_read_chunk_tree. Why should we pass fs_info directly to btrfs_read_chunk_tree? Could you give me more detail? Many thanks > > 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 > > > >