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 EC0DAC05027 for ; Sat, 18 Feb 2023 01:17:04 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 745136B0083; Fri, 17 Feb 2023 20:17:04 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 6F58A6B0085; Fri, 17 Feb 2023 20:17:04 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5BD3A6B0087; Fri, 17 Feb 2023 20:17:04 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 488706B0083 for ; Fri, 17 Feb 2023 20:17:04 -0500 (EST) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 1518FC0858 for ; Sat, 18 Feb 2023 01:17:04 +0000 (UTC) X-FDA: 80478648768.09.71F7C73 Received: from mail1.bemta34.messagelabs.com (mail1.bemta34.messagelabs.com [195.245.231.3]) by imf24.hostedemail.com (Postfix) with ESMTP id 5D933180010 for ; Sat, 18 Feb 2023 01:17:00 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=fujitsu.com header.s=170520fj header.b=AJbD5xni; spf=pass (imf24.hostedemail.com: domain of ruansy.fnst@fujitsu.com designates 195.245.231.3 as permitted sender) smtp.mailfrom=ruansy.fnst@fujitsu.com; dmarc=pass (policy=none) header.from=fujitsu.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1676683020; 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:dkim-signature; bh=EarUJXJHeKc/uIrwmeqmpWyHrRMI7AcdO5uvPtkXwtM=; b=ETb9GJAB2AwTuLsyGpQq/2pAFso0wt75AjlH9iCtRc0pVE3Kx9AWlJ6NaE6QhdN+58zAh+ 8X+K9qoaaXMJ9aVmBt8s0O+GgVZIiNwzTNZuf8Gct6JnyVYaR6E/rfvdrg5EGGlDZqsDGC tyLfmuJpzI5U8S89PSl53J+fJqRzR1Q= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=pass header.d=fujitsu.com header.s=170520fj header.b=AJbD5xni; spf=pass (imf24.hostedemail.com: domain of ruansy.fnst@fujitsu.com designates 195.245.231.3 as permitted sender) smtp.mailfrom=ruansy.fnst@fujitsu.com; dmarc=pass (policy=none) header.from=fujitsu.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1676683020; a=rsa-sha256; cv=none; b=TSWDdA8ud26Q4aYvz8SOYHrZuND1g2Ke2OQATUOr2CJuRhYUj6M0cQjohfJZlqa1htCOvH 5ZTA5rKChM52La77nAq2ZTfns0W7thqVi0jrnTAUuj7x9CdKCh6TzjL7UWlZwO6fMCvV9e d6sMS13+LpqFNJryxsR8r1dDBvYGYuc= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fujitsu.com; s=170520fj; t=1676683019; i=@fujitsu.com; bh=EarUJXJHeKc/uIrwmeqmpWyHrRMI7AcdO5uvPtkXwtM=; h=Message-ID:Date:MIME-Version:Subject:To:CC:References:From: In-Reply-To:Content-Type:Content-Transfer-Encoding; b=AJbD5xnirMYHgnmv0bUZg1KcyvYCo5v0rLrglNS6oAO51KHq1o2OX+7+ah/9TfemF zA1GTjy216Vn0FHe8tpCx0iy8B3CrpDi0323ZNSu6WhkmRFnMSJqiiPvt+JCg+94fp 2ZcY9pCNQuTj7swZmRYyQqx4b8sps6uBdiGDcVsWsW4QpLT0aghj6I4z/TvP3A3UkN xg/Zm5JW+q0XhBXu/jE0uxRuBHTHbOaeUtWWlHIAGtGRF2IEzoiedGZIqVK4Tysi6p E+hcAaO1Eq608yqzoZTN2xL6IDx13WuuPOfM3DGWT3L8uSo8DyUdVlQD8qIwburBal syL72CUr8mm2g== X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrKKsWRWlGSWpSXmKPExsViZ8ORpMuh/iH Z4MEDVos569ewWUyfeoHRYsuxe4wWl5/wWZyesIjJYvfrm2wWe/aeZLG4t+Y/q8WuPzvYLVb+ +MNq8fvHHDYHbo9TiyQ8Nq/Q8li85yWTx6ZVnWwemz5NYvc4MeM3i8eLzTMZPT4+vcXi8XmTX ABnFGtmXlJ+RQJrRuvmqcwF/VwV1598YWxgPMvRxcjFISSwkVHi4dWFjBDOUiaJC3c+MkM4Wx klbn4+DJTh5OAVsJN4dGUhE4jNIqAqsff2J6i4oMTJmU9YQGxRgWSJY+db2UBsYQFHiV3H2li 7GDk4RAQ0JN5sMQKZySzQwiRxfNIHqG3LGSWufr4ENpRNQEfiwoK/rCA2p4CJxN6Hv8AGMQtY SCx+c5AdwpaXaN46mxnElhBQkLgxaRULhF0p0frhF5StJnH13CbmCYxCs5DcNwvJqFlIRi1gZ F7FaFqcWlSWWqRrrJdUlJmeUZKbmJmjl1ilm6iXWqpbnlpcomukl1herJdaXKxXXJmbnJOil5 dasokRGJspxWrCOxi/9P7VO8QoycGkJMr7edv7ZCG+pPyUyozE4oz4otKc1OJDjDIcHEoSvPc UPyQLCRalpqdWpGXmANMETFqCg0dJhHe5DFCat7ggMbc4Mx0idYpRUUqcV0wNKCEAksgozYNr g6WmS4yyUsK8jAwMDEI8BalFuZklqPKvGMU5GJWEeaOUgabwZOaVwE1/BbSYCWjxAua3IItLE hFSUg1MobfmRjtl6ss5zLo9Nyc0cu633adu3+GYHGJUwTq5f/65e9IyihYMD9h9VGKM/gUr1J U8aj34IXzyqyjDvwYOi2bf1j9zmaNP7scqxYeRLAWFIR4/0vN9luXEmTVO//7QL+TsjV2rrps +uX/80uzzEr8kBU7cWa054XT1/NC6Wdf+8Ly1ts3dzS9456HtrI5TQl/eP1pypPiSFkvpM61S 0Yq359inztoSzfaozU5mX2fxoezd/w4sZNuvvN35D+/bmJueWs5JBV57zxRmBNktvsUU1xRdf aPu//aiWBkTO2sriairP++Xv3jhq/Wx9dmXeWf/3c2cMbn3x7Yt2zMfXt6y41irmqRV4MPd7y acFFRiKc5INNRiLipOBAC6nf7+yAMAAA== X-Env-Sender: ruansy.fnst@fujitsu.com X-Msg-Ref: server-6.tower-565.messagelabs.com!1676683015!365172!1 X-Originating-IP: [62.60.8.98] X-SYMC-ESS-Client-Auth: outbound-route-from=pass X-StarScan-Received: X-StarScan-Version: 9.102.2; banners=-,-,- X-VirusChecked: Checked Received: (qmail 21012 invoked from network); 18 Feb 2023 01:16:56 -0000 Received: from unknown (HELO n03ukasimr03.n03.fujitsu.local) (62.60.8.98) by server-6.tower-565.messagelabs.com with ECDHE-RSA-AES256-GCM-SHA384 encrypted SMTP; 18 Feb 2023 01:16:56 -0000 Received: from n03ukasimr03.n03.fujitsu.local (localhost [127.0.0.1]) by n03ukasimr03.n03.fujitsu.local (Postfix) with ESMTP id B8BF01B1; Sat, 18 Feb 2023 01:16:55 +0000 (GMT) Received: from R01UKEXCASM223.r01.fujitsu.local (R01UKEXCASM223 [10.182.185.121]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by n03ukasimr03.n03.fujitsu.local (Postfix) with ESMTPS id AA0611AC; Sat, 18 Feb 2023 01:16:55 +0000 (GMT) Received: from [10.167.201.2] (10.167.201.2) by R01UKEXCASM223.r01.fujitsu.local (10.182.185.121) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Sat, 18 Feb 2023 01:16:50 +0000 Message-ID: Date: Sat, 18 Feb 2023 09:16:43 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 Subject: Re: [PATCH v10 2/3] fs: introduce super_drop_pagecache() To: Matthew Wilcox CC: , , , , , , , , , References: <1676645312-13-1-git-send-email-ruansy.fnst@fujitsu.com> <1676645312-13-3-git-send-email-ruansy.fnst@fujitsu.com> From: Shiyang Ruan In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.167.201.2] X-ClientProxiedBy: G08CNEXCHPEKD07.g08.fujitsu.local (10.167.33.80) To R01UKEXCASM223.r01.fujitsu.local (10.182.185.121) X-Virus-Scanned: ClamAV using ClamSMTP X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 5D933180010 X-Rspam-User: X-Stat-Signature: bjnn8mrd6frz9oytuo1u4b4mw3dw4irw X-HE-Tag: 1676683020-185086 X-HE-Meta: U2FsdGVkX1/Gn0GmvHuSRq+51rc8ofWSnzcOHzKWmckiBT7W322cE1IcRJaIJrfZs7TGEntOU/ncDNCgKLgTPzbfit7LCAHDF10dN81l02ActSg73P5sgczu6VMMtCoDNSPzGsETMvT7wca+/TE0334U+CVCu9UkFFCMu1igSNup3HV5tyKji43wSP8zUWCs7MZR90gwnazerkNt5b5B5eRNwMUPyNnIaBzsY5jWySTEEqoPPHvBIJ39yRMGsJ4Voj8RG29qQwJnUvSliO9zBvKjiMLVMcpeCs+0sYbl6/lJmrMuOOucgxXcKUibWlkWPV/oxJyXIV3B1idnXVhUO8mF5MOycOBmLiVavPhbDqPTyQLNlOcItSKD21MTUgINdQSk3/j3GRUBVmmW0C48cqATo2YDynbndKaw9TBncteT3h1EiACQzmo0WaarHv8dCZMaXgSCJdc8XlJHfkHoxVKGJUj3P0OTXZ5gZ23rbp6iKeisdr+CUuwYukm37h5jjPyVaDoazAdSbaWekTuUZDHIGKXQ4JARnSybxJbEuX0avjigDyL/AV9t1+cSXTMMAJh9dQflfRW6OkzHIyQMfhBf7vI03NWp5CqN3M4knjFLNBK6KgrdR4wJVWw0GqhgPeR7WThTY1dtzMXeR2HXRzVOSvGS8vC4LE8Km/xQD1seHCHBt0mp/FuqjIy6RuNXx0n0mQc/Qih5u1i0jO1z16Dit2TnuPfJjRLJdN086JsU1DVqEHlbfGyuSy/LTOLm5dFps/sHGFkIR+rpSE+Ima9PjNyEZF2EsrVzJsy84OErZbwvzcWANWYojYREZ4dPSAvuTMf8R/ZN/68FEyDzT2LF+y0azPnw3rmMFlM+40UbKXpFh+zP7VdLtO1HnMJS5GmDgNlzTdr8OXes1HI+cR5d4aETfwKcOJiT3uYbtz+Hkiuwz3RUVYZZDuGodmrBk82KKxbl8ENTEGFpdd2 tsuCJ7mA k6CNAZnpzWG4x41CPMbGyOeGjxLkr6t30AmMJHuRKfzgkJcybc6ng1GOBhmlTLGhinmcFUksInjMo/EU6Z8rd0ecaEOeDig5xth/xsBB0wTmpg5+BW8ljuealxXEUFK4JmHkg17V7bmzYJTn6sYH0BdiTJ1R8LH1RBL76aF42KFiECqQjmXgcdG/0hgY8/R5+CnSZ0Nrwlsz3iZ+NIA1prRvOe64wBrztflmM1qnmHtBzaqvo+luBkexnPHwxKUsWG/PNa4sCvC8ubc4= 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: 在 2023/2/18 0:14, Matthew Wilcox 写道: > On Fri, Feb 17, 2023 at 02:48:31PM +0000, Shiyang Ruan wrote: >> - invalidate_mapping_pages(inode->i_mapping, 0, -1); >> - iput(toput_inode); >> - toput_inode = inode; >> - >> - cond_resched(); >> - spin_lock(&sb->s_inode_list_lock); >> - } >> - spin_unlock(&sb->s_inode_list_lock); >> - iput(toput_inode); >> + super_drop_pagecache(sb, invalidate_inode_pages); > > I thought I explained last time that you can do this with > invalidate_mapping_pages() / invalidate_inode_pages2_range() ? > Then you don't need to introduce invalidate_inode_pages(). > >> +void super_drop_pagecache(struct super_block *sb, >> + int (*invalidator)(struct address_space *)) > > void super_drop_pagecache(struct super_block *sb, > int (*invalidate)(struct address_space *, pgoff_t, pgoff_t)) > >> + invalidator(inode->i_mapping); > > invalidate(inode->i_mapping, 0, -1) > > ... then all the changes to mm/truncate.c and filemap.h go away. Yes, I tried as you suggested, but I found that they don't have same type of return value. int invalidate_inode_pages2_range(struct address_space *mapping, pgoff_t start, pgoff_t end); unsigned long invalidate_mapping_pages(struct address_space *mapping, pgoff_t start, pgoff_t end); -- Thanks, Ruan.