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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id D264DEB64DD for ; Thu, 20 Jul 2023 09:31:50 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 11CE82800E6; Thu, 20 Jul 2023 05:31:50 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0A4D928004C; Thu, 20 Jul 2023 05:31:50 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E88302800E6; Thu, 20 Jul 2023 05:31:49 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id D62E328004C for ; Thu, 20 Jul 2023 05:31:49 -0400 (EDT) Received: from smtpin03.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id A108912025C for ; Thu, 20 Jul 2023 09:31:49 +0000 (UTC) X-FDA: 81031473138.03.82419B8 Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by imf11.hostedemail.com (Postfix) with ESMTP id A72814001C for ; Thu, 20 Jul 2023 09:31:45 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf11.hostedemail.com: domain of zhangpeng362@huawei.com designates 45.249.212.187 as permitted sender) smtp.mailfrom=zhangpeng362@huawei.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1689845507; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=9+h35ugE219PU5zxwYcn3qiRUgt6fkBWKlWba61e3l4=; b=qN9jNCqBGLDDpKT7cewNyBIicrTpt8drj1lsHjC7HYiJwvt4xjvqA38C5QwG/i4g/JpoiJ yHUmRGNQRiGi+ygiS0KujZS4Kd1llSx829yGUpBjncy6fhjvwCfaugrIYwPYGrL30tGJcf cP6LZSR1QyoalKL2HO6Qzx0UvDr2V6I= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=none; dmarc=pass (policy=quarantine) header.from=huawei.com; spf=pass (imf11.hostedemail.com: domain of zhangpeng362@huawei.com designates 45.249.212.187 as permitted sender) smtp.mailfrom=zhangpeng362@huawei.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1689845507; a=rsa-sha256; cv=none; b=pf2bE69TqHz22M1Te3Fwp00Q0ZdtczCIsugeUuG7H6EpaMa7gunLxEDdXG7FwcUrlNUR9k xr07YQdrIR03crnXBuex+kf/8mmoVpeFkbxpordCbJPyExGxy27x0VwnsIA3vRH5Nmal5F i1jX1tSx5yMYaA+2x1pD2o6SKc6al90= Received: from kwepemm600020.china.huawei.com (unknown [172.30.72.55]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4R66nF0vkCztQvm; Thu, 20 Jul 2023 17:28:33 +0800 (CST) Received: from [10.174.179.160] (10.174.179.160) by kwepemm600020.china.huawei.com (7.193.23.147) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.27; Thu, 20 Jul 2023 17:31:40 +0800 Message-ID: <2f844f7f-f671-f046-03e3-4e7d28cdcbc2@huawei.com> Date: Thu, 20 Jul 2023 17:31:39 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 From: "zhangpeng (AS)" Subject: Re: [PATCH v2 3/9] mm/page_io: convert bio_first_page_all() to bio_first_folio_all() To: Matthew Wilcox CC: , , , , , References: <20230719095848.3422629-1-zhangpeng362@huawei.com> <20230719095848.3422629-4-zhangpeng362@huawei.com> Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.174.179.160] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To kwepemm600020.china.huawei.com (7.193.23.147) X-CFilter-Loop: Reflected X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: A72814001C X-Stat-Signature: scciudd5gswgtrwcg9udhrdwkg53ueb5 X-HE-Tag: 1689845505-736823 X-HE-Meta: U2FsdGVkX18tgnSqxbfegW+bwyygO8n8OfKIPYLMbYSZGDXk//tAR4NIKBXGnk+rKlh8EeN9jQxZIpPTf08kPDtE819Q7C3yJxhvNoQvZAPlg7CeKK6GN7v8jmofUHZA2P8aqrELRROD/JNyX+I9LfGy4WuxWbu7p5zR57s8kBRkey2X65Bb+GwAtryAOYJFbmpe9Gqa14ovkotUeL//DX5teEkKb5xF/t3/Rw966AN1j1FdFjb5DpFRARyBPxRTDULjIB6M5q5xjeIF8YOdrRSz0nqv/79XRajGVvq7tYrLTASsMB0dcuxI9xiM0AIlyaS2F3ZSeB+uQYv24psm7aMOo4lBEdKRKOulBxZ+h96hpu9yRJSHduwDFBBNANPNPikFMpgekG3mrQ+7TxcVeeYWQYDo+7LXu0uIz0wegPG3HZv4zpKsaCJNyUjrzD73TGGY+rfXR7jQFTFBgag5SqtlJnH5kSfZGrSQ5HtD3mAmnJOja2cpNqINgK0Lx9ej5emB+IDkg486nJyCsMM0O6ldi5fTzfbpTv5PdtDgNEgdloxFFacY0X4fhUzeO4bKY5WQHBUc7Hd/1cqPJIdJLD6S3CUARoNNRzVlnW0UA6sEDS+MxbYTL7+FaQe5oNFkKs9WD8qe4mOoarKSKHC74YRcBHnR61bHW2GWACPC3gDUjPgug1asDerToE6ppxZTdpOb687ggmpgYayMwYHXt68/EOpLnj2yjseIM4+X9TtMnhijF8oIMQR3hP4aO+1J3qq7ydb/Z3F+ltaTx+jdsKZVqdCW9fN50cD3cCXarObD6GOj1UNTBIZfplHs/tjCACqbI/6WJfxYzm4QjdUcBJhXEXrPaKpPJvz3SygiR7B3HlzZPVlIclKa72rrY+zTYitfWqwagCk5PnrWKcobfxv8jv1bTiQUIWLhGTkuj0Hw/jI+S/FbAoQO8l+dyJB0wDlMuMxIYbqjg/1B7zg ZdPDcMrO 4NkkaNvvpyehy9SAb6pMxPdvT3Ka0HdvOMaFBP5gEQ4Oqqz2qbLYYV/0lHVb7+/aYXKjBOiHLJyTtA9E78a++CWmxx4G6pcnk+P+OgTv009FO/3G9CXEJM8LYlzDLJCT9dLldLyty26sus7V9pRT76MxOWHa/9LB8dOT6YaDtmxK0VckPEKapHs19qARxtMzt9o4gucPZJKqyCyd6N6Gl02TpEo4Z+yis70tCqKSnX94w53IIrWc0sA/pO75uuZdv8XtKwA+T9GuRpZE= 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: On 2023/7/19 22:17, Matthew Wilcox wrote: > On Wed, Jul 19, 2023 at 05:58:42PM +0800, Peng Zhang wrote: >> From: ZhangPeng >> >> Convert bio_first_page_all() to bio_first_folio_all() to return a >> folio, which makes it easier to use. > This wasn't what I was suggesting. Indeed, this may introduce bugs. > I was suggesting adding bio_first_folio_all() so that it can be used > by code that knows this is what it wants. Sorry for my misunderstanding, I will fix it in the next version. >> Suggested-by: Matthew Wilcox (Oracle) >> Signed-off-by: ZhangPeng >> --- >> Documentation/block/biovecs.rst | 2 +- >> drivers/block/drbd/drbd_bitmap.c | 2 +- >> fs/f2fs/data.c | 2 +- >> include/linux/bio.h | 4 ++-- >> kernel/power/swap.c | 2 +- >> mm/page_io.c | 4 ++-- >> 6 files changed, 8 insertions(+), 8 deletions(-) >> >> diff --git a/Documentation/block/biovecs.rst b/Documentation/block/biovecs.rst >> index ddb867e0185b..258f6faed333 100644 >> --- a/Documentation/block/biovecs.rst >> +++ b/Documentation/block/biovecs.rst >> @@ -133,7 +133,7 @@ Usage of helpers: >> bio_for_each_segment_all() >> bio_for_each_bvec_all() >> bio_first_bvec_all() >> - bio_first_page_all() >> + bio_first_folio_all() >> bio_last_bvec_all() >> >> * The following helpers iterate over single-page segment. The passed 'struct >> diff --git a/drivers/block/drbd/drbd_bitmap.c b/drivers/block/drbd/drbd_bitmap.c >> index 85ca000a0564..ce1a07ba85ff 100644 >> --- a/drivers/block/drbd/drbd_bitmap.c >> +++ b/drivers/block/drbd/drbd_bitmap.c >> @@ -939,7 +939,7 @@ static void drbd_bm_endio(struct bio *bio) >> struct drbd_bm_aio_ctx *ctx = bio->bi_private; >> struct drbd_device *device = ctx->device; >> struct drbd_bitmap *b = device->bitmap; >> - unsigned int idx = bm_page_to_idx(bio_first_page_all(bio)); >> + unsigned int idx = bm_page_to_idx(&bio_first_folio_all(bio)->page); >> >> if ((ctx->flags & BM_AIO_COPY_PAGES) == 0 && >> !bm_test_page_unchanged(b->bm_pages[idx])) >> diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c >> index 5882afe71d82..6706858de984 100644 >> --- a/fs/f2fs/data.c >> +++ b/fs/f2fs/data.c >> @@ -288,7 +288,7 @@ static void f2fs_post_read_work(struct work_struct *work) >> >> static void f2fs_read_end_io(struct bio *bio) >> { >> - struct f2fs_sb_info *sbi = F2FS_P_SB(bio_first_page_all(bio)); >> + struct f2fs_sb_info *sbi = F2FS_P_SB(&bio_first_folio_all(bio)->page); >> struct bio_post_read_ctx *ctx; >> bool intask = in_task(); >> >> diff --git a/include/linux/bio.h b/include/linux/bio.h >> index c4f5b5228105..7d2979d24530 100644 >> --- a/include/linux/bio.h >> +++ b/include/linux/bio.h >> @@ -248,9 +248,9 @@ static inline struct bio_vec *bio_first_bvec_all(struct bio *bio) >> return bio->bi_io_vec; >> } >> >> -static inline struct page *bio_first_page_all(struct bio *bio) >> +static inline struct folio *bio_first_folio_all(struct bio *bio) >> { >> - return bio_first_bvec_all(bio)->bv_page; >> + return page_folio(bio_first_bvec_all(bio)->bv_page); >> } >> >> static inline struct bio_vec *bio_last_bvec_all(struct bio *bio) >> diff --git a/kernel/power/swap.c b/kernel/power/swap.c >> index f6ebcd00c410..2796ce48ca4b 100644 >> --- a/kernel/power/swap.c >> +++ b/kernel/power/swap.c >> @@ -247,7 +247,7 @@ static void hib_finish_batch(struct hib_bio_batch *hb) >> static void hib_end_io(struct bio *bio) >> { >> struct hib_bio_batch *hb = bio->bi_private; >> - struct page *page = bio_first_page_all(bio); >> + struct page *page = &bio_first_folio_all(bio)->page; >> >> if (bio->bi_status) { >> pr_alert("Read-error on swap-device (%u:%u:%Lu)\n", >> diff --git a/mm/page_io.c b/mm/page_io.c >> index 5ddb5d9c5013..e21d570c873d 100644 >> --- a/mm/page_io.c >> +++ b/mm/page_io.c >> @@ -29,7 +29,7 @@ >> >> static void __end_swap_bio_write(struct bio *bio) >> { >> - struct page *page = bio_first_page_all(bio); >> + struct page *page = &bio_first_folio_all(bio)->page; >> >> if (bio->bi_status) { >> /* >> @@ -57,7 +57,7 @@ static void end_swap_bio_write(struct bio *bio) >> >> static void __end_swap_bio_read(struct bio *bio) >> { >> - struct page *page = bio_first_page_all(bio); >> + struct page *page = &bio_first_folio_all(bio)->page; >> >> if (bio->bi_status) { >> pr_alert_ratelimited("Read-error on swap-device (%u:%u:%llu)\n", >> -- >> 2.25.1 >> -- Best Regards, Peng