Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1564629

Re: [PATCH 1/2] nilfs2: use i_blocksize()

Path csiph.com!news.redatomik.org!aioe.org!news.mixmin.net!news.unit0.net!news.panservice.it!diesel.cu.mi.it!bofh.it!news.nic.it!robomod
From Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Newsgroups linux.kernel
Subject Re: [PATCH 1/2] nilfs2: use i_blocksize()
Date Mon, 23 Jan 2017 02:30:01 +0100
Message-ID <t2BCh-8lT-3@gated-at.bofh.it> (permalink)
References <t1Iwa-8q4-11@gated-at.bofh.it>
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:message-id:to:cc:subject:from:in-reply-to:references :mime-version:content-transfer-encoding; bh=OgCwEUstFX8EXYeqeZbF3acQ1IaxOo4q0LqBpjDdrPE=; b=LDpCEofFja+1pFgNuHsv3CFXHaiz6Vlmgk5Q8lLsrFQcLQQWgPELOEYwc6d9NSYHoi 4gm9X+utICSfhlr51qwxzkITqJtnjeIZaCFgwhbV89OEnwievtiviop3JUtjIn+TDsr7 ncqsTqPZhaU84BxqXIHG8wf+2Q5QjKZYnHl9n3IduZj0wVgQwwPLAgKyL+rXqBhocpC0 IOW/YI8h2WAixGR9+jMrejI8szBdb1KzkAA0220GpiXwXH3402rwRj/ihN0NzHI7SA41 pAhF3ApG8WkRT1LR/RmkEloNfPPKUSmhImS/bKe5PXVZ9qwwdu/XKhoBTKL2/7LW9MF9 J2GA==
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:message-id:to:cc:subject:from :in-reply-to:references:mime-version:content-transfer-encoding; bh=OgCwEUstFX8EXYeqeZbF3acQ1IaxOo4q0LqBpjDdrPE=; b=OhFhig68wcQngrH2OD1Ea9itlihRu58DgOfn5s21QaGWhGj2IX6vJIr2dJ1SBda3pa 88Z8gvZUni9Ny0pmTfN/Qdai0tsB8YDg7jdQ08vahiQ2EEInkXjDwkPBbXmnWonjdJ0E sLKEEUASbimVUsn5z2L3ZsKYJoif/pja4rSK4pI2Bx/huiLhZptA6GtACex+UW2hu75L ZOeuWFa265lNcxej6/hy4rXilE3Ys5POZpEnMoMM92+ANk2n7L8uZetQxNc0LzgLTU4N J1fXR1P9Nsi6lEMEzdUSfM99pwJi5XSTrgJtls8YEGD+o81T+vtMR+TzIYGBG8XSXS9P EGjg==
X-Gm-Message-State AIkVDXJlhU7QfBJXegTX3ebB2rztzRrt3INUHMzO4IwLYQCqYEvUmPfSWK8e5qTE5Rkn2g==
X-Received by 10.99.158.18 with SMTP id s18mr29762459pgd.88.1485134940303; Sun, 22 Jan 2017 17:29:00 -0800 (PST)
X-Mailer Mew version 6.6 on Emacs 24.3 / Mule 6.0 (HANACHIRUSATO)
MIME-Version 1.0
Content-Type Text/Plain; charset=us-ascii
Content-Transfer-Encoding 7bit
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 50
Organization linux.* mail to news gateway
X-Original-Cc linux-nilfs@vger.kernel.org, linux-kernel@vger.kernel.org
X-Original-Date Mon, 23 Jan 2017 10:28:56 +0900 (JST)
X-Original-Message-ID <20170123.102856.1031133549844815930.konishi.ryusuke@lab.ntt.co.jp>
X-Original-References <ebd4d1fb0aada6ed0bb1d2de21230c3b1fb148a9.1484896146.git.geliangtang@gmail.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1564629

Show key headers only | View raw


On Fri, 20 Jan 2017 22:34:57 +0800, Geliang Tang wrote:
> Since i_blocksize() helper has been defined in fs.h, use it instead
> of open-coding.
> 
> Signed-off-by: Geliang Tang <geliangtang@gmail.com>

i_blocksize() doesn't exist in the mainline kernel.

This needs another patch titled "fs: add i_blocksize()" which I found
in the latest -mm tree.

I will send this to the -mm tree along with this dependency.

Regards,
Ryusuke Konishi

> ---
>  fs/nilfs2/alloc.c | 2 +-
>  fs/nilfs2/btree.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/nilfs2/alloc.c b/fs/nilfs2/alloc.c
> index 2c90e28..03b8ba9 100644
> --- a/fs/nilfs2/alloc.c
> +++ b/fs/nilfs2/alloc.c
> @@ -34,7 +34,7 @@
>  static inline unsigned long
>  nilfs_palloc_groups_per_desc_block(const struct inode *inode)
>  {
> -	return (1UL << inode->i_blkbits) /
> +	return i_blocksize(inode) /
>  		sizeof(struct nilfs_palloc_group_desc);
>  }
>  
> diff --git a/fs/nilfs2/btree.c b/fs/nilfs2/btree.c
> index 2e315f9..d75238a 100644
> --- a/fs/nilfs2/btree.c
> +++ b/fs/nilfs2/btree.c
> @@ -119,7 +119,7 @@ nilfs_btree_node_set_nchildren(struct nilfs_btree_node *node, int nchildren)
>  
>  static int nilfs_btree_node_size(const struct nilfs_bmap *btree)
>  {
> -	return 1 << btree->b_inode->i_blkbits;
> +	return i_blocksize(btree->b_inode);
>  }
>  
>  static int nilfs_btree_nchildren_per_block(const struct nilfs_bmap *btree)
> -- 
> 2.9.3
> 

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread


Thread

[PATCH 1/2] nilfs2: use i_blocksize() Geliang Tang <geliangtang@gmail.com> - 2017-01-20 15:40 +0100
  Re: [PATCH 1/2] nilfs2: use i_blocksize() kbuild test robot <lkp@intel.com> - 2017-01-21 06:40 +0100
  Re: [PATCH 1/2] nilfs2: use i_blocksize() Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> - 2017-01-23 02:30 +0100

csiph-web