From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161107AbdAJNOi (ORCPT ); Tue, 10 Jan 2017 08:14:38 -0500 Received: from exchange.catalysts.cc ([81.10.172.115]:9311 "EHLO mail01-vie.catalysts.cc" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S967277AbdAJNOg (ORCPT ); Tue, 10 Jan 2017 08:14:36 -0500 X-Greylist: delayed 902 seconds by postgrey-1.27 at vger.kernel.org; Tue, 10 Jan 2017 08:14:36 EST From: Andreas Gerstmayr To: CC: , Andreas Gerstmayr , "Yan, Zheng" , Sage Weil , Ilya Dryomov , Subject: [PATCH v3] ceph: set io_pages bdi hint Date: Tue, 10 Jan 2017 13:56:42 +0100 Message-ID: <1484053078-4490-1-git-send-email-andreas.gerstmayr@catalysts.cc> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <7C9D4085-7D14-4BFB-B675-8FE3C661B1C3@redhat.com> References: <7C9D4085-7D14-4BFB-B675-8FE3C661B1C3@redhat.com> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [172.22.50.12] X-ClientProxiedBy: mail01.internal.catalysts.cc (172.20.20.10) To mail01.internal.catalysts.cc (172.20.20.10) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch sets the io_pages bdi hint based on the rsize mount option. Without this patch large buffered reads (request size > max readahead) are processed sequentially in chunks of the readahead size (i.e. read requests are sent out up to the readahead size, then the do_generic_file_read() function waits until the first page is received). With this patch read requests are sent out at once up to the size specified in the rsize mount option (default: 64 MB). Signed-off-by: Andreas Gerstmayr --- Thanks for your input. Changes in v3: - set default rsize to 64 MB - sanity check of the rsize mount option (Note: This patch depends on kernel version 4.10-rc1) fs/ceph/super.c | 8 ++++++++ fs/ceph/super.h | 4 ++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/fs/ceph/super.c b/fs/ceph/super.c index 6bd20d7..a0a0b6d 100644 --- a/fs/ceph/super.c +++ b/fs/ceph/super.c @@ -952,6 +952,14 @@ static int ceph_register_bdi(struct super_block *sb, fsc->backing_dev_info.ra_pages = VM_MAX_READAHEAD * 1024 / PAGE_SIZE; + if (fsc->mount_options->rsize > fsc->mount_options->rasize && + fsc->mount_options->rsize >= PAGE_SIZE) + fsc->backing_dev_info.io_pages = + (fsc->mount_options->rsize + PAGE_SIZE - 1) + >> PAGE_SHIFT; + else if (fsc->mount_options->rsize == 0) + fsc->backing_dev_info.io_pages = ULONG_MAX; + err = bdi_register(&fsc->backing_dev_info, NULL, "ceph-%ld", atomic_long_inc_return(&bdi_seq)); if (!err) diff --git a/fs/ceph/super.h b/fs/ceph/super.h index 3373b61..88b2e6e 100644 --- a/fs/ceph/super.h +++ b/fs/ceph/super.h @@ -45,8 +45,8 @@ #define ceph_test_mount_opt(fsc, opt) \ (!!((fsc)->mount_options->flags & CEPH_MOUNT_OPT_##opt)) -#define CEPH_RSIZE_DEFAULT 0 /* max read size */ -#define CEPH_RASIZE_DEFAULT (8192*1024) /* readahead */ +#define CEPH_RSIZE_DEFAULT (64*1024*1024) /* max read size */ +#define CEPH_RASIZE_DEFAULT (8192*1024) /* max readahead */ #define CEPH_MAX_READDIR_DEFAULT 1024 #define CEPH_MAX_READDIR_BYTES_DEFAULT (512*1024) #define CEPH_SNAPDIRNAME_DEFAULT ".snap" -- 1.8.3.1