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 A71FAC433EF for ; Mon, 14 Mar 2022 23:45:14 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1C3A16B0071; Mon, 14 Mar 2022 19:45:14 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 1749F8D0002; Mon, 14 Mar 2022 19:45:14 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 013568D0001; Mon, 14 Mar 2022 19:45:13 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (relay.hostedemail.com [64.99.140.28]) by kanga.kvack.org (Postfix) with ESMTP id E25B56B0071 for ; Mon, 14 Mar 2022 19:45:13 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id BB4C220856 for ; Mon, 14 Mar 2022 23:45:13 +0000 (UTC) X-FDA: 79244625306.02.B786B2A Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf12.hostedemail.com (Postfix) with ESMTP id 47AE240008 for ; Mon, 14 Mar 2022 23:45:13 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 63BBE61480; Mon, 14 Mar 2022 23:45:12 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 77506C340E9; Mon, 14 Mar 2022 23:45:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linux-foundation.org; s=korg; t=1647301511; bh=Ju1DAPlgw5FMTla0jTg/jmHE1lxJHJq05Tdwhdan2zw=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=mOxloHx09cAsu6h0FKCFCiYZ5LpSNzc3AOEJI2K7Vmi8GcHtOOB9OWytTJCfQXMYL +jcxJFn60wOin2XjByEPaMvqw3VIYp9Hk+pSuJaYOJoyg6TKZ+byey1CwtrMkk5QgW 38ZGi7NjNUsz4mCXhYjRJtw2+D/WLeJKVBf2a1Ac= Date: Mon, 14 Mar 2022 16:45:10 -0700 From: Andrew Morton To: Miaohe Lin Cc: HORIGUCHI =?UTF-8?B?TkFPWUE=?=(=?UTF-8?B?5aCA5Y+jIOebtOS5nw==?=) , "tony.luck@intel.com" , "bp@alien8.de" , "mike.kravetz@oracle.com" , "shy828301@gmail.com" , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" , "linux-edac@vger.kernel.org" Subject: Re: [PATCH v2 2/3] mm/memory-failure.c: avoid calling invalidate_inode_page() with unexpected pages Message-Id: <20220314164510.acf6157930122583808375e1@linux-foundation.org> In-Reply-To: <219aeec6-5ff6-5101-8192-13b9f761e7c9@huawei.com> References: <20220312074613.4798-1-linmiaohe@huawei.com> <20220312074613.4798-3-linmiaohe@huawei.com> <20220313234157.GB3010057@hori.linux.bs1.fc.nec.co.jp> <8aa7cdd9-8104-2fea-879d-61519f6489d1@huawei.com> <20220314025034.GA3061370@hori.linux.bs1.fc.nec.co.jp> <219aeec6-5ff6-5101-8192-13b9f761e7c9@huawei.com> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.33; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: 47AE240008 X-Stat-Signature: mwx8oue6grfqbxsn6q16ipomj6agftwa Authentication-Results: imf12.hostedemail.com; dkim=pass header.d=linux-foundation.org header.s=korg header.b=mOxloHx0; spf=pass (imf12.hostedemail.com: domain of akpm@linux-foundation.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=akpm@linux-foundation.org; dmarc=none X-Rspam-User: X-HE-Tag: 1647301513-188280 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 Mon, 14 Mar 2022 10:59:40 +0800 Miaohe Lin wrote: > On 2022/3/14 10:50, HORIGUCHI NAOYA(堀口 直也) wrote: > > On Mon, Mar 14, 2022 at 09:58:49AM +0800, Miaohe Lin wrote: > >> On 2022/3/14 7:41, HORIGUCHI NAOYA(堀口 直也) wrote: > >>> On Sat, Mar 12, 2022 at 03:46:12PM +0800, Miaohe Lin wrote: > >>>> Since commit 042c4f32323b ("mm/truncate: Inline invalidate_complete_page() > >>> > >>> This commit ID does not exist in mainline (or in the latest mmotm?), > >>> so you can't use it in patch description. Could you update this part? > >>> > >> > >> This commit is in the mmotm but not in mainline yet: > >> > >> commit 042c4f32323beb28146c658202d3e69899e4f245 > >> Author: Matthew Wilcox (Oracle) > >> Date: Sat Feb 12 15:27:42 2022 -0500 > >> > >> mm/truncate: Inline invalidate_complete_page() into its one caller > >> > >> invalidate_inode_page() is the only caller of invalidate_complete_page() > >> and inlining it reveals that the first check is unnecessary (because we > >> hold the page locked, and we just retrieved the mapping from the page). > >> Actually, it does make a difference, in that tail pages no longer fail > >> at this check, so it's now possible to remove a tail page from a mapping. > >> > >> Signed-off-by: Matthew Wilcox (Oracle) > >> Reviewed-by: John Hubbard > >> Reviewed-by: Christoph Hellwig > >> > >> Am I "not" supposed to use this commit id as it's not "stable" now? > > > > No, it's not stable yet. In whatever way you get the above commit (I guess > > you get it from https://github.com/hnaz/linux-mm), all acked mm-related > > patches are sent to Linus by Andrew *by email*, so the eventual commit IDs > > should be determined when they are applied to mainline. > > > > Many thanks for your explanation. (I get this commit id from linux-next tree: > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git) > So I should remember always to get the commit id from mainline. It's likely that this commit ID will be the same once Matthew's patch goes into mainline. But this is why we include the patch title ("mm/truncate: Inline ...") when identifying commits. Sometimes stuff happens...