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=-9.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham 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 363C1C4332D for ; Fri, 20 Mar 2020 14:22:38 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id D543220776 for ; Fri, 20 Mar 2020 14:22:37 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="K4+K0d80" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D543220776 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 1BF106B000D; Fri, 20 Mar 2020 10:22:36 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0ABF96B0008; Fri, 20 Mar 2020 10:22:35 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id DA3516B000A; Fri, 20 Mar 2020 10:22:35 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0179.hostedemail.com [216.40.44.179]) by kanga.kvack.org (Postfix) with ESMTP id AB06A6B0003 for ; Fri, 20 Mar 2020 10:22:35 -0400 (EDT) Received: from smtpin28.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id 66A8C181AEF09 for ; Fri, 20 Mar 2020 14:22:35 +0000 (UTC) X-FDA: 76615956270.28.floor79_16893672d6b0c X-HE-Tag: floor79_16893672d6b0c X-Filterd-Recvd-Size: 12829 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) by imf28.hostedemail.com (Postfix) with ESMTP for ; Fri, 20 Mar 2020 14:22:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20170209; h=Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From:Sender :Reply-To:Content-Type:Content-ID:Content-Description; bh=PaU0E8Bve7jBa01t57OFuZRPqHzqqGYR2616on4MjoM=; b=K4+K0d800Iz9CFlyqexo1H0pYr Rm0Rz5imDr4KgDHIu4L1c0SirCRlMWCJP+mE59wJeKIVjDr27Si4EmMgRcPVLyKrLmSZlOTGqa7e/ tFUVg7pLHAyWMuaDdqZyqwMPA6Jn3ToRyV8GXRYNV2yeXb30GmE9+bmrXFHpgPwC0Kzb1s93uevV1 LOdkTsKkLS8ovcCr/zrSQtAroRFMU8m2bdDVrmcnay3Idq7+x0jyjwXxR/zokkp2hcDah0ahsdAhC Ibs6ZFEpy9RVr0/rtnmnK1GyOe1jLH7SRbc7hXGJI9uoozaf3LBlxJLkc16dnpf7sHGgpsXtHwgHB d9z4V1PA==; Received: from willy by bombadil.infradead.org with local (Exim 4.92.3 #3 (Red Hat Linux)) id 1jFIXh-0000hR-Ad; Fri, 20 Mar 2020 14:22:33 +0000 From: Matthew Wilcox To: Andrew Morton Cc: "Matthew Wilcox (Oracle)" , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-erofs@lists.ozlabs.org, linux-ext4@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, cluster-devel@redhat.com, ocfs2-devel@oss.oracle.com, linux-xfs@vger.kernel.org, Zi Yan , William Kucharski Subject: [PATCH v9 07/25] mm: Rename various 'offset' parameters to 'index' Date: Fri, 20 Mar 2020 07:22:13 -0700 Message-Id: <20200320142231.2402-8-willy@infradead.org> X-Mailer: git-send-email 2.21.1 In-Reply-To: <20200320142231.2402-1-willy@infradead.org> References: <20200320142231.2402-1-willy@infradead.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: From: "Matthew Wilcox (Oracle)" The word 'offset' is used ambiguously to mean 'byte offset within a page', 'byte offset from the start of the file' and 'page offset from the start of the file'. Use 'index' to mean 'page offset from the start of the file' throughout the readahead code. Signed-off-by: Matthew Wilcox (Oracle) Reviewed-by: Zi Yan Reviewed-by: William Kucharski --- mm/readahead.c | 86 ++++++++++++++++++++++++-------------------------- 1 file changed, 42 insertions(+), 44 deletions(-) diff --git a/mm/readahead.c b/mm/readahead.c index 9d9aa4ffc7d4..8a65d6bd97e0 100644 --- a/mm/readahead.c +++ b/mm/readahead.c @@ -156,7 +156,7 @@ static void read_pages(struct readahead_control *rac,= struct list_head *pages, * We really don't want to intermingle reads and writes like that. */ void __do_page_cache_readahead(struct address_space *mapping, - struct file *filp, pgoff_t offset, unsigned long nr_to_read, + struct file *filp, pgoff_t index, unsigned long nr_to_read, unsigned long lookahead_size) { struct inode *inode =3D mapping->host; @@ -180,7 +180,7 @@ void __do_page_cache_readahead(struct address_space *= mapping, * Preallocate as many pages as we will need. */ for (page_idx =3D 0; page_idx < nr_to_read; page_idx++) { - pgoff_t page_offset =3D offset + page_idx; + pgoff_t page_offset =3D index + page_idx; =20 if (page_offset > end_index) break; @@ -219,7 +219,7 @@ void __do_page_cache_readahead(struct address_space *= mapping, * memory at once. */ void force_page_cache_readahead(struct address_space *mapping, - struct file *filp, pgoff_t offset, unsigned long nr_to_read) + struct file *filp, pgoff_t index, unsigned long nr_to_read) { struct backing_dev_info *bdi =3D inode_to_bdi(mapping->host); struct file_ra_state *ra =3D &filp->f_ra; @@ -239,9 +239,9 @@ void force_page_cache_readahead(struct address_space = *mapping, =20 if (this_chunk > nr_to_read) this_chunk =3D nr_to_read; - __do_page_cache_readahead(mapping, filp, offset, this_chunk, 0); + __do_page_cache_readahead(mapping, filp, index, this_chunk, 0); =20 - offset +=3D this_chunk; + index +=3D this_chunk; nr_to_read -=3D this_chunk; } } @@ -322,21 +322,21 @@ static unsigned long get_next_ra_size(struct file_r= a_state *ra, */ =20 /* - * Count contiguously cached pages from @offset-1 to @offset-@max, + * Count contiguously cached pages from @index-1 to @index-@max, * this count is a conservative estimation of * - length of the sequential read sequence, or * - thrashing threshold in memory tight systems */ static pgoff_t count_history_pages(struct address_space *mapping, - pgoff_t offset, unsigned long max) + pgoff_t index, unsigned long max) { pgoff_t head; =20 rcu_read_lock(); - head =3D page_cache_prev_miss(mapping, offset - 1, max); + head =3D page_cache_prev_miss(mapping, index - 1, max); rcu_read_unlock(); =20 - return offset - 1 - head; + return index - 1 - head; } =20 /* @@ -344,13 +344,13 @@ static pgoff_t count_history_pages(struct address_s= pace *mapping, */ static int try_context_readahead(struct address_space *mapping, struct file_ra_state *ra, - pgoff_t offset, + pgoff_t index, unsigned long req_size, unsigned long max) { pgoff_t size; =20 - size =3D count_history_pages(mapping, offset, max); + size =3D count_history_pages(mapping, index, max); =20 /* * not enough history pages: @@ -363,10 +363,10 @@ static int try_context_readahead(struct address_spa= ce *mapping, * starts from beginning of file: * it is a strong indication of long-run stream (or whole-file-read) */ - if (size >=3D offset) + if (size >=3D index) size *=3D 2; =20 - ra->start =3D offset; + ra->start =3D index; ra->size =3D min(size + req_size, max); ra->async_size =3D 1; =20 @@ -378,13 +378,13 @@ static int try_context_readahead(struct address_spa= ce *mapping, */ static void ondemand_readahead(struct address_space *mapping, struct file_ra_state *ra, struct file *filp, - bool hit_readahead_marker, pgoff_t offset, + bool hit_readahead_marker, pgoff_t index, unsigned long req_size) { struct backing_dev_info *bdi =3D inode_to_bdi(mapping->host); unsigned long max_pages =3D ra->ra_pages; unsigned long add_pages; - pgoff_t prev_offset; + pgoff_t prev_index; =20 /* * If the request exceeds the readahead window, allow the read to @@ -396,15 +396,15 @@ static void ondemand_readahead(struct address_space= *mapping, /* * start of file */ - if (!offset) + if (!index) goto initial_readahead; =20 /* - * It's the expected callback offset, assume sequential access. + * It's the expected callback index, assume sequential access. * Ramp up sizes, and push forward the readahead window. */ - if ((offset =3D=3D (ra->start + ra->size - ra->async_size) || - offset =3D=3D (ra->start + ra->size))) { + if ((index =3D=3D (ra->start + ra->size - ra->async_size) || + index =3D=3D (ra->start + ra->size))) { ra->start +=3D ra->size; ra->size =3D get_next_ra_size(ra, max_pages); ra->async_size =3D ra->size; @@ -421,14 +421,14 @@ static void ondemand_readahead(struct address_space= *mapping, pgoff_t start; =20 rcu_read_lock(); - start =3D page_cache_next_miss(mapping, offset + 1, max_pages); + start =3D page_cache_next_miss(mapping, index + 1, max_pages); rcu_read_unlock(); =20 - if (!start || start - offset > max_pages) + if (!start || start - index > max_pages) return; =20 ra->start =3D start; - ra->size =3D start - offset; /* old async_size */ + ra->size =3D start - index; /* old async_size */ ra->size +=3D req_size; ra->size =3D get_next_ra_size(ra, max_pages); ra->async_size =3D ra->size; @@ -443,29 +443,29 @@ static void ondemand_readahead(struct address_space= *mapping, =20 /* * sequential cache miss - * trivial case: (offset - prev_offset) =3D=3D 1 - * unaligned reads: (offset - prev_offset) =3D=3D 0 + * trivial case: (index - prev_index) =3D=3D 1 + * unaligned reads: (index - prev_index) =3D=3D 0 */ - prev_offset =3D (unsigned long long)ra->prev_pos >> PAGE_SHIFT; - if (offset - prev_offset <=3D 1UL) + prev_index =3D (unsigned long long)ra->prev_pos >> PAGE_SHIFT; + if (index - prev_index <=3D 1UL) goto initial_readahead; =20 /* * Query the page cache and look for the traces(cached history pages) * that a sequential stream would leave behind. */ - if (try_context_readahead(mapping, ra, offset, req_size, max_pages)) + if (try_context_readahead(mapping, ra, index, req_size, max_pages)) goto readit; =20 /* * standalone, small random read * Read as is, and do not pollute the readahead state. */ - __do_page_cache_readahead(mapping, filp, offset, req_size, 0); + __do_page_cache_readahead(mapping, filp, index, req_size, 0); return; =20 initial_readahead: - ra->start =3D offset; + ra->start =3D index; ra->size =3D get_init_ra_size(req_size, max_pages); ra->async_size =3D ra->size > req_size ? ra->size - req_size : ra->size= ; =20 @@ -476,7 +476,7 @@ static void ondemand_readahead(struct address_space *= mapping, * the resulted next readahead window into the current one. * Take care of maximum IO pages as above. */ - if (offset =3D=3D ra->start && ra->size =3D=3D ra->async_size) { + if (index =3D=3D ra->start && ra->size =3D=3D ra->async_size) { add_pages =3D get_next_ra_size(ra, max_pages); if (ra->size + add_pages <=3D max_pages) { ra->async_size =3D add_pages; @@ -495,9 +495,8 @@ static void ondemand_readahead(struct address_space *= mapping, * @mapping: address_space which holds the pagecache and I/O vectors * @ra: file_ra_state which holds the readahead state * @filp: passed on to ->readpage() and ->readpages() - * @offset: start offset into @mapping, in pagecache page-sized units - * @req_size: hint: total size of the read which the caller is performin= g in - * pagecache pages + * @index: Index of first page to be read. + * @req_count: Total number of pages being read by the caller. * * page_cache_sync_readahead() should be called when a cache miss happen= ed: * it will submit the read. The readahead logic may decide to piggyback= more @@ -506,7 +505,7 @@ static void ondemand_readahead(struct address_space *= mapping, */ void page_cache_sync_readahead(struct address_space *mapping, struct file_ra_state *ra, struct file *filp, - pgoff_t offset, unsigned long req_size) + pgoff_t index, unsigned long req_count) { /* no read-ahead */ if (!ra->ra_pages) @@ -517,12 +516,12 @@ void page_cache_sync_readahead(struct address_space= *mapping, =20 /* be dumb */ if (filp && (filp->f_mode & FMODE_RANDOM)) { - force_page_cache_readahead(mapping, filp, offset, req_size); + force_page_cache_readahead(mapping, filp, index, req_count); return; } =20 /* do read-ahead */ - ondemand_readahead(mapping, ra, filp, false, offset, req_size); + ondemand_readahead(mapping, ra, filp, false, index, req_count); } EXPORT_SYMBOL_GPL(page_cache_sync_readahead); =20 @@ -531,21 +530,20 @@ EXPORT_SYMBOL_GPL(page_cache_sync_readahead); * @mapping: address_space which holds the pagecache and I/O vectors * @ra: file_ra_state which holds the readahead state * @filp: passed on to ->readpage() and ->readpages() - * @page: the page at @offset which has the PG_readahead flag set - * @offset: start offset into @mapping, in pagecache page-sized units - * @req_size: hint: total size of the read which the caller is performin= g in - * pagecache pages + * @page: The page at @index which triggered the readahead call. + * @index: Index of first page to be read. + * @req_count: Total number of pages being read by the caller. * * page_cache_async_readahead() should be called when a page is used whi= ch - * has the PG_readahead flag; this is a marker to suggest that the appli= cation + * is marked as PageReadahead; this is a marker to suggest that the appl= ication * has used up enough of the readahead window that we should start pulli= ng in * more pages. */ void page_cache_async_readahead(struct address_space *mapping, struct file_ra_state *ra, struct file *filp, - struct page *page, pgoff_t offset, - unsigned long req_size) + struct page *page, pgoff_t index, + unsigned long req_count) { /* no read-ahead */ if (!ra->ra_pages) @@ -569,7 +567,7 @@ page_cache_async_readahead(struct address_space *mapp= ing, return; =20 /* do read-ahead */ - ondemand_readahead(mapping, ra, filp, true, offset, req_size); + ondemand_readahead(mapping, ra, filp, true, index, req_count); } EXPORT_SYMBOL_GPL(page_cache_async_readahead); =20 --=20 2.25.1