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 D3183C43334 for ; Mon, 20 Jun 2022 13:38:53 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 5991B8E0001; Mon, 20 Jun 2022 09:38:53 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 548116B0073; Mon, 20 Jun 2022 09:38:53 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 411998E0001; Mon, 20 Jun 2022 09:38:53 -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 31B636B0071 for ; Mon, 20 Jun 2022 09:38:53 -0400 (EDT) Received: from smtpin09.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 036B920467 for ; Mon, 20 Jun 2022 13:38:52 +0000 (UTC) X-FDA: 79598719746.09.6B98B0A Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by imf12.hostedemail.com (Postfix) with ESMTP id 2AF32400AC for ; Mon, 20 Jun 2022 13:38:51 +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 ams.source.kernel.org (Postfix) with ESMTPS id 3A7FFB80E7A; Mon, 20 Jun 2022 13:38:50 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7D789C3411B; Mon, 20 Jun 2022 13:38:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1655732329; bh=jHqZRV1TS/Tlj00IluoHA/Rfwsw8Kz/nXudckFV2IqI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=rQdL/NqIPEfC2WJ+2EP8kli161WNZOA9NBTZwlaQrgfdFU9AROa2X/LShwGuOkKDf wr5ohLPE23ieLcnPpnZ8YPi3aOEnAjAh0tuVz/qE91zDdd4w6N0vcWU8Bk8f5bh8fe GOFV1gZ3OVroMh4jLSLyXTntolrD7zr3DSe53yKY= Date: Mon, 20 Jun 2022 14:54:42 +0200 From: Greg KH To: Zi Yan Cc: Xianting Tian , akpm@linux-foundation.org, stable@vger.kernel.org, guoren@kernel.org, huanyi.xj@alibaba-inc.com, guohanjun@huawei.com, zjb194813@alibaba-inc.com, tianhu.hh@alibaba-inc.com, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 5.15] mm: validate buddy page before using Message-ID: References: <20220616161746.3565225-1-xianting.tian@linux.alibaba.com> <20220616161746.3565225-6-xianting.tian@linux.alibaba.com> <35bd7396-f5aa-e154-9495-0a36fc6f6a33@linux.alibaba.com> <8b16a502-5ad5-1efb-0d84-ed0a8ae63c0e@linux.alibaba.com> <3371C275-E45D-445F-838E-D43C60BCD750@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <3371C275-E45D-445F-838E-D43C60BCD750@nvidia.com> ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1655732332; a=rsa-sha256; cv=none; b=0GaB+3/CBCZ5S8L2UgooUWqM41CTTfY1E0pSvG6YXhCfTRdA5WJAU1YSxw6lSejsBAMAh0 OV8pSZRVGNvqlUxw3AAM4JLDu59Li0HfijLVabkEKYzEvcvnHFoZwO77MtW3JWDi7U5AG+ i0/3aUANx6GCMeDluDZ++EBmftlBVCE= ARC-Authentication-Results: i=1; imf12.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b="rQdL/NqI"; spf=pass (imf12.hostedemail.com: domain of gregkh@linuxfoundation.org designates 145.40.68.75 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org; dmarc=pass (policy=none) header.from=linuxfoundation.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1655732332; 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=uVGHzcOQwXSoxQ+vQzabUmbbg6RcnhU70G1Yu0vGMe4=; b=gfXuDK5D0XTm06rPz1khCjIMHbbTOOmn9iQoxPgu4qrH2i2DNq4wMjCgRIiBul0OFcNtjl EOxobZbLR6uIk+2VX6t/YF4gTF61xZ3RwdFyB6hg3PPcguXRjmWXn37xRvgU7+Dgiqu8KW l5p2rF/SHuX4FlAleR62dbzX3GNXdx8= X-Stat-Signature: w5ksodm1cipihxfn4tf58dqfhd119crb X-Rspamd-Queue-Id: 2AF32400AC Authentication-Results: imf12.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b="rQdL/NqI"; spf=pass (imf12.hostedemail.com: domain of gregkh@linuxfoundation.org designates 145.40.68.75 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org; dmarc=pass (policy=none) header.from=linuxfoundation.org X-Rspamd-Server: rspam07 X-Rspam-User: X-HE-Tag: 1655732331-359604 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, Jun 20, 2022 at 08:45:13AM -0400, Zi Yan wrote: > On 20 Jun 2022, at 8:24, Greg KH wrote: > > > On Mon, Jun 20, 2022 at 08:18:40PM +0800, Xianting Tian wrote: > >> > >> 在 2022/6/20 下午8:06, Greg KH 写道: > >>> On Mon, Jun 20, 2022 at 07:57:05PM +0800, Xianting Tian wrote: > >>>> 在 2022/6/20 下午7:42, Greg KH 写道: > >>>>> On Mon, Jun 20, 2022 at 06:54:44PM +0800, Xianting Tian wrote: > >>>>>> 在 2022/6/20 下午6:17, Greg KH 写道: > >>>>>>> On Fri, Jun 17, 2022 at 12:17:45AM +0800, Xianting Tian wrote: > >>>>>>>> Commit 787af64d05cd ("mm: page_alloc: validate buddy before check its migratetype.") > >>>>>>>> fixes a bug in 1dd214b8f21c and there is a similar bug in d9dddbf55667 that > >>>>>>>> can be fixed in a similar way too. > >>>>>>>> > >>>>>>>> In unset_migratetype_isolate(), we also need the fix, so move page_is_buddy() > >>>>>>>> from mm/page_alloc.c to mm/internal.h > >>>>>>>> > >>>>>>>> In addition, for RISC-V arch the first 2MB RAM could be reserved for opensbi, > >>>>>>>> so it would have pfn_base=512 and mem_map began with 512th PFN when > >>>>>>>> CONFIG_FLATMEM=y. > >>>>>>>> But __find_buddy_pfn algorithm thinks the start pfn 0, it could get 0 pfn or > >>>>>>>> less than the pfn_base value. We need page_is_buddy() to verify the buddy to > >>>>>>>> prevent accessing an invalid buddy. > >>>>>>>> > >>>>>>>> Fixes: d9dddbf55667 ("mm/page_alloc: prevent merging between isolated and other pageblocks") > >>>>>>>> Cc: stable@vger.kernel.org > >>>>>>>> Reported-by: zjb194813@alibaba-inc.com > >>>>>>>> Reported-by: tianhu.hh@alibaba-inc.com > >>>>>>>> Signed-off-by: Xianting Tian > >>>>>>>> --- > >>>>>>>> mm/internal.h | 34 ++++++++++++++++++++++++++++++++++ > >>>>>>>> mm/page_alloc.c | 37 +++---------------------------------- > >>>>>>>> mm/page_isolation.c | 3 ++- > >>>>>>>> 3 files changed, 39 insertions(+), 35 deletions(-) > >>>>>>> What is the commit id of this in Linus's tree? > >>>>>> It is also this one， > >>>>>> > >>>>>> commit 787af64d05cd528aac9ad16752d11bb1c6061bb9 > >>>>>> Author: Zi Yan > >>>>>> Date:   Wed Mar 30 15:45:43 2022 -0700 > >>>>>> > >>>>>>     mm: page_alloc: validate buddy before check its migratetype. > >>>>>> > >>>>>>     Whenever a buddy page is found, page_is_buddy() should be called to > >>>>>>     check its validity.  Add the missing check during pageblock merge check. > >>>>>> > >>>>>>     Fixes: 1dd214b8f21c ("mm: page_alloc: avoid merging non-fallbackable > >>>>>> pageblocks with others") > >>>>>>     Link: > >>>>>> https://lore.kernel.org/all/20220330154208.71aca532@gandalf.local.home/ > >>>>>>     Reported-and-tested-by: Steven Rostedt > >>>>>>     Signed-off-by: Zi Yan > >>>>>>     Signed-off-by: Linus Torvalds > >>>>> This commit looks nothing like what you posted here. > >>>>> > >>>>> Why the vast difference with no explaination as to why these are so > >>>>> different from the other backports you provided here? Also why is the > >>>>> subject lines changed? > >>>> Yes, the changes of 5.15 are not same with others branches, because we need > >>>> additional fix for 5.15, > >>>> > >>>> You can check it in the thread: > >>>> > >>>> https://lore.kernel.org/linux-mm/435B45C3-E6A5-43B2-A5A2-318C748691FC@nvidia.com/ > >>>> > >>>> Right. But pfn_valid_within() was removed since 5.15. So your fix is > >>>> required for kernels between 5.15 and 5.17 (inclusive). > >>> What is "your fix" here? > >>> > >>> This change differs a lot from what is in Linus's tree now, so this all > >>> needs to be resend and fixed up as I mention above if we are going to be > >>> able to take this. As-is, it's all not correct so are dropped. > >> > >> I think, for branches except 5.15,  you can just backport Zi Yan's commit > >> 787af64d05cd in Linus tree. I won't send more patches further, > > > > So just for 5.18? I am confused. > > > >> For 5.15, because it need additional fix except commit 787af64d05cd,  I will > >> send a new patch as your comments. > >> > >> Is it ok for you? > > > > No, please send fixed up patches for all branches you want them applied > > to as I do not understand what to do here at all, sorry. > > Hi Greg, > > The fixes sent by Xianting do not exist in Linus’s tree, since the bug is > fixed by another commit, which was not intended to fix the bug from the commit > d9dddbf55667. These fixes only target the stable branches. Then that all needs to be documented very very very well as to why we can't just take the commit that is in Linus's tree. Why can't we take that commit instead? thanks, greg k-h