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=-16.0 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED 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 E8BB8C433DB for ; Mon, 8 Feb 2021 19:49:02 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 6BB0F64E6E for ; Mon, 8 Feb 2021 19:49:02 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6BB0F64E6E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linux-foundation.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 16E8F6B006C; Mon, 8 Feb 2021 14:49:01 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 11FA76B006E; Mon, 8 Feb 2021 14:49:01 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 00BF46B0070; Mon, 8 Feb 2021 14:49:00 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0035.hostedemail.com [216.40.44.35]) by kanga.kvack.org (Postfix) with ESMTP id DE3E46B006C for ; Mon, 8 Feb 2021 14:49:00 -0500 (EST) Received: from smtpin12.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 9B15318590BAD for ; Mon, 8 Feb 2021 19:49:00 +0000 (UTC) X-FDA: 77796138840.12.dog07_2d09aed27601 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin12.hostedemail.com (Postfix) with ESMTP id 7C8131836ED17 for ; Mon, 8 Feb 2021 19:49:00 +0000 (UTC) X-HE-Tag: dog07_2d09aed27601 X-Filterd-Recvd-Size: 4013 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf38.hostedemail.com (Postfix) with ESMTP for ; Mon, 8 Feb 2021 19:48:59 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 92C1164E56; Mon, 8 Feb 2021 19:48:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linux-foundation.org; s=korg; t=1612813738; bh=fiMaPfVra/2zEjkMsZvR8Iyt6gmFiqnTTP4gAMQuARo=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=MTPknMTmNhE9DO36kKtRa7zJ2V5noBJBWB2U8EeHHO6MU76BZaAVYfbCCxMvHgNBe D9LD+e3ehRFEhc2ZrcI+BLv11Pnzl/dtrwDbc4XJxBZvLi6h/tH+YlaYJANtx/m19P UJTMWWsq4MxVBfXOB4lIOm65+CDZrCHF8dJc4vfc= Date: Mon, 8 Feb 2021 11:48:56 -0800 From: Andrew Morton To: Christophe Leroy Cc: Thomas Bogendoerfer , Huang Pei , ambrosehua@gmail.com, Bibo Mao , linux-mips@vger.kernel.org, linux-arch@vger.kernel.org, linux-mm@kvack.org, Jiaxun Yang , Paul Burton , Li Xuefeng , Yang Tiezhu , Gao Juxin , Fuxin Zhang , Huacai Chen , Nicholas Piggin Subject: Re: [PATCH] MIPS: make userspace mapping young by default Message-Id: <20210208114856.e8062823b2e84e1adb1d59bb@linux-foundation.org> In-Reply-To: <30b3fcb5-a60d-228f-15d2-cd182953de45@csgroup.eu> References: <20210204013942.8398-1-huangpei@loongson.cn> <20210204152239.GA14292@alpha.franken.de> <20210205154105.32bb13df439aa49b7fc167e7@linux-foundation.org> <30b3fcb5-a60d-228f-15d2-cd182953de45@csgroup.eu> X-Mailer: Sylpheed 3.5.1 (GTK+ 2.24.31; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 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: On Mon, 8 Feb 2021 18:44:22 +0100 Christophe Leroy wrote: >=20 >=20 > Le 06/02/2021 =E0 00:41, Andrew Morton a =E9crit=A0: > > On Thu, 4 Feb 2021 16:22:39 +0100 Thomas Bogendoerfer wrote: > >=20 > >> On Thu, Feb 04, 2021 at 09:39:42AM +0800, Huang Pei wrote: > >>> MIPS page fault path(except huge page) takes 3 exceptions (1 TLB Miss > >>> + 2 TLB Invalid), butthe second TLB Invalid exception is just > >>> triggered by __update_tlb from do_page_fault writing tlb without > >>> _PAGE_VALID set. With this patch, user space mapping prot is made > >>> young by default (with both _PAGE_VALID and _PAGE_YOUNG set), > >>> and it only take 1 TLB Miss + 1 TLB Invalid exception > >>> > >>> Remove pte_sw_mkyoung without polluting MM code and make page fault > >>> delay of MIPS on par with other architecture > >>> > >>> Signed-off-by: Huang Pei > >>> --- > >>> arch/mips/mm/cache.c | 30 ++++++++++++++++-------------- > >>> include/linux/pgtable.h | 8 -------- > >>> mm/memory.c | 3 --- > >>> 3 files changed, 16 insertions(+), 25 deletions(-) > >> > >> Acked-by: Thomas Bogendoerfer > >> > >> Andrew, can you take this patch through your tree ? > >=20 > > Sure. I'll drop Christophe's "mm/memory.c: remove pte_sw_mkyoung()" > > (https://lkml.kernel.org/r/f302ef92c48d1f08a0459aaee1c568ca11213814.161= 2345700.git.christophe.leroy@csgroup.eu) > > in favour of this one. > >=20 >=20 > Pitty. My patch was improving page faults on powerpc/32. How does it do that? By running pte_mkyoung() for powerpc32? Such a change is still valid, isn't it? > That one is only addressing MIPS. It cleans up core code nicely, by removing a MIPS wart. We can still add a ppc32 wart? >=20 > Any plan to take the series from Nick=20 > https://patchwork.kernel.org/project/linux-mm/list/?series=3D404539 ? I expect so. After -rc1, if the churn is settling down and reviewers are happy enough.