From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 66538C47420 for ; Tue, 22 Sep 2020 08:50:39 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 02C15221EB for ; Tue, 22 Sep 2020 08:50:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="WuYdDyoX" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 02C15221EB Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=suse.cz Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=LGWHgV0Gi8ftwahUDVXtKrd2CaKLe/6E74td0Tk0muQ=; b=WuYdDyoX6SWsvv7mvda/4eWJl xXgfC/VcStqSi6wSU/gSP888/UR9dhtgqZpsH4ka6UjfgEKvRbx4ctdZk063GcHlj6eBsMaTcc48k 1+4Xep1LasKwMinh7D4nJ5V9FbIny6wJGLycAmilKt7sMogqsryrG7QP6cPsHumnQ280KAcEBWS1e n8MjfXHYnrXM3p0z9Hq280HqwlwupnHM/zRxsPmOpQNdn3vfvrElLIEKvkkO0XpX6wH+OcjsLJxOr AXEyPdfZVHN3bx4A3OAYkGjOyI7ng6ZJ77ZvZQTaNsMrAZNFtGnNySbDKbVQ/RLSAZFQIpa/Dpk0J DAQkM19Zw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kKdzs-0001e6-45; Tue, 22 Sep 2020 08:50:00 +0000 Received: from mx2.suse.de ([195.135.220.15]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kKdzo-0001dF-NZ for linux-mtd@lists.infradead.org; Tue, 22 Sep 2020 08:49:57 +0000 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id 01CA2AF16; Tue, 22 Sep 2020 08:50:32 +0000 (UTC) Received: by quack2.suse.cz (Postfix, from userid 1000) id EA6C01E12E3; Tue, 22 Sep 2020 10:49:54 +0200 (CEST) Date: Tue, 22 Sep 2020 10:49:54 +0200 From: Jan Kara To: Christoph Hellwig Subject: Re: [PATCH 05/13] bdi: initialize ->ra_pages and ->io_pages in bdi_init Message-ID: <20200922084954.GC16464@quack2.suse.cz> References: <20200921080734.452759-1-hch@lst.de> <20200921080734.452759-6-hch@lst.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200921080734.452759-6-hch@lst.de> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200922_044957_122477_20D70605 X-CRM114-Status: GOOD ( 27.38 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jens Axboe , linux-raid@vger.kernel.org, Hans de Goede , Justin Sanders , Minchan Kim , Richard Weinberger , linux-bcache@vger.kernel.org, Coly Li , linux-block@vger.kernel.org, Song Liu , dm-devel@redhat.com, linux-mtd@lists.infradead.org, Johannes Thumshirn , linux-fsdevel@vger.kernel.org, cgroups@vger.kernel.org, David Sterba , linux-mm@kvack.org, linux-kernel@vger.kernel.org, drbd-dev@lists.linbit.com Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org On Mon 21-09-20 10:07:26, Christoph Hellwig wrote: > Set up a readahead size by default, as very few users have a good > reason to change it. > > Signed-off-by: Christoph Hellwig > Acked-by: David Sterba [btrfs] > Acked-by: Richard Weinberger [ubifs, mtd] The patch looks good to me. You can add: Reviewed-by: Jan Kara I'd just prefer if the changelog explicitely mentioned that this patch results in enabling readahead for coda, ecryptfs, and orangefs... Just in case someone bisects some issue down to this patch :). Honza > --- > block/blk-core.c | 2 -- > drivers/mtd/mtdcore.c | 2 ++ > fs/9p/vfs_super.c | 6 ++++-- > fs/afs/super.c | 1 - > fs/btrfs/disk-io.c | 1 - > fs/fuse/inode.c | 1 - > fs/nfs/super.c | 9 +-------- > fs/ubifs/super.c | 2 ++ > fs/vboxsf/super.c | 2 ++ > mm/backing-dev.c | 2 ++ > 10 files changed, 13 insertions(+), 15 deletions(-) > > diff --git a/block/blk-core.c b/block/blk-core.c > index ca3f0f00c9435f..865d39e5be2b28 100644 > --- a/block/blk-core.c > +++ b/block/blk-core.c > @@ -538,8 +538,6 @@ struct request_queue *blk_alloc_queue(int node_id) > if (!q->stats) > goto fail_stats; > > - q->backing_dev_info->ra_pages = VM_READAHEAD_PAGES; > - q->backing_dev_info->io_pages = VM_READAHEAD_PAGES; > q->backing_dev_info->capabilities = BDI_CAP_CGROUP_WRITEBACK; > q->node = node_id; > > diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c > index 7d930569a7dfb7..b5e5d3140f578e 100644 > --- a/drivers/mtd/mtdcore.c > +++ b/drivers/mtd/mtdcore.c > @@ -2196,6 +2196,8 @@ static struct backing_dev_info * __init mtd_bdi_init(char *name) > bdi = bdi_alloc(NUMA_NO_NODE); > if (!bdi) > return ERR_PTR(-ENOMEM); > + bdi->ra_pages = 0; > + bdi->io_pages = 0; > > /* > * We put '-0' suffix to the name to get the same name format as we > diff --git a/fs/9p/vfs_super.c b/fs/9p/vfs_super.c > index 74df32be4c6a52..e34fa20acf612e 100644 > --- a/fs/9p/vfs_super.c > +++ b/fs/9p/vfs_super.c > @@ -80,8 +80,10 @@ v9fs_fill_super(struct super_block *sb, struct v9fs_session_info *v9ses, > if (ret) > return ret; > > - if (v9ses->cache) > - sb->s_bdi->ra_pages = VM_READAHEAD_PAGES; > + if (!v9ses->cache) { > + sb->s_bdi->ra_pages = 0; > + sb->s_bdi->io_pages = 0; > + } > > sb->s_flags |= SB_ACTIVE | SB_DIRSYNC; > if (!v9ses->cache) > diff --git a/fs/afs/super.c b/fs/afs/super.c > index b552357b1d1379..3a40ee752c1e3f 100644 > --- a/fs/afs/super.c > +++ b/fs/afs/super.c > @@ -456,7 +456,6 @@ static int afs_fill_super(struct super_block *sb, struct afs_fs_context *ctx) > ret = super_setup_bdi(sb); > if (ret) > return ret; > - sb->s_bdi->ra_pages = VM_READAHEAD_PAGES; > > /* allocate the root inode and dentry */ > if (as->dyn_root) { > diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c > index f6bba7eb1fa171..047934cea25efa 100644 > --- a/fs/btrfs/disk-io.c > +++ b/fs/btrfs/disk-io.c > @@ -3092,7 +3092,6 @@ int __cold open_ctree(struct super_block *sb, struct btrfs_fs_devices *fs_device > } > > sb->s_bdi->capabilities |= BDI_CAP_CGROUP_WRITEBACK; > - sb->s_bdi->ra_pages = VM_READAHEAD_PAGES; > sb->s_bdi->ra_pages *= btrfs_super_num_devices(disk_super); > sb->s_bdi->ra_pages = max(sb->s_bdi->ra_pages, SZ_4M / PAGE_SIZE); > > diff --git a/fs/fuse/inode.c b/fs/fuse/inode.c > index bba747520e9b08..17b00670fb539e 100644 > --- a/fs/fuse/inode.c > +++ b/fs/fuse/inode.c > @@ -1049,7 +1049,6 @@ static int fuse_bdi_init(struct fuse_conn *fc, struct super_block *sb) > if (err) > return err; > > - sb->s_bdi->ra_pages = VM_READAHEAD_PAGES; > /* fuse does it's own writeback accounting */ > sb->s_bdi->capabilities = BDI_CAP_NO_ACCT_WB | BDI_CAP_STRICTLIMIT; > > diff --git a/fs/nfs/super.c b/fs/nfs/super.c > index 7a70287f21a2c1..f943e37853fa25 100644 > --- a/fs/nfs/super.c > +++ b/fs/nfs/super.c > @@ -1200,13 +1200,6 @@ static void nfs_get_cache_cookie(struct super_block *sb, > } > #endif > > -static void nfs_set_readahead(struct backing_dev_info *bdi, > - unsigned long iomax_pages) > -{ > - bdi->ra_pages = VM_READAHEAD_PAGES; > - bdi->io_pages = iomax_pages; > -} > - > int nfs_get_tree_common(struct fs_context *fc) > { > struct nfs_fs_context *ctx = nfs_fc2context(fc); > @@ -1251,7 +1244,7 @@ int nfs_get_tree_common(struct fs_context *fc) > MINOR(server->s_dev)); > if (error) > goto error_splat_super; > - nfs_set_readahead(s->s_bdi, server->rpages); > + s->s_bdi->io_pages = server->rpages; > server->super = s; > } > > diff --git a/fs/ubifs/super.c b/fs/ubifs/super.c > index a2420c900275a8..fbddb2a1c03f5e 100644 > --- a/fs/ubifs/super.c > +++ b/fs/ubifs/super.c > @@ -2177,6 +2177,8 @@ static int ubifs_fill_super(struct super_block *sb, void *data, int silent) > c->vi.vol_id); > if (err) > goto out_close; > + sb->s_bdi->ra_pages = 0; > + sb->s_bdi->io_pages = 0; > > sb->s_fs_info = c; > sb->s_magic = UBIFS_SUPER_MAGIC; > diff --git a/fs/vboxsf/super.c b/fs/vboxsf/super.c > index 8fe03b4a0d2b03..8e3792177a8523 100644 > --- a/fs/vboxsf/super.c > +++ b/fs/vboxsf/super.c > @@ -167,6 +167,8 @@ static int vboxsf_fill_super(struct super_block *sb, struct fs_context *fc) > err = super_setup_bdi_name(sb, "vboxsf-%d", sbi->bdi_id); > if (err) > goto fail_free; > + sb->s_bdi->ra_pages = 0; > + sb->s_bdi->io_pages = 0; > > /* Turn source into a shfl_string and map the folder */ > size = strlen(fc->source) + 1; > diff --git a/mm/backing-dev.c b/mm/backing-dev.c > index 8e8b00627bb2d8..2dac3be6127127 100644 > --- a/mm/backing-dev.c > +++ b/mm/backing-dev.c > @@ -746,6 +746,8 @@ struct backing_dev_info *bdi_alloc(int node_id) > kfree(bdi); > return NULL; > } > + bdi->ra_pages = VM_READAHEAD_PAGES; > + bdi->io_pages = VM_READAHEAD_PAGES; > return bdi; > } > EXPORT_SYMBOL(bdi_alloc); > -- > 2.28.0 > -- Jan Kara SUSE Labs, CR ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/