Received: by 2002:a05:6902:102b:0:0:0:0 with SMTP id x11csp1584818ybt; Thu, 2 Jul 2020 08:52:29 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwLP/sVrPyqCm72gkniHILvhRbG/GpyxchSudAV37Uz/Na7oschJVx2ZalJuDsXz0xCmuxg X-Received: by 2002:a17:906:2b52:: with SMTP id b18mr28846854ejg.158.1593705148890; Thu, 02 Jul 2020 08:52:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1593705148; cv=none; d=google.com; s=arc-20160816; b=OzJTqFs1jW48hFF6AdqOZ9bijNucIvQ059hA2XapeVitFrWn+qjivxcEVVKHVnoK2r zWdkbBlBpCcnHUPNsPeBp/qTEq7L91xrgQ2ElHCP0TVDnsG1b8xA8njqyed7NoQZFspS RoV/DT6Ug5YbJxc8D7sKySRAbX9goY829DElk9J/N1RubeEGp1vh3NomL5cuq/oQHqvH eWZnVvJqOXrk5Dmjki1cUUyZ8HalXW+ieCGf7BsZGJZpG8ZZPpEYLs1IpLYC4w3rWiYg iJWgwj6iAdYktbcegx0TuXHyywNiyHwx6rM8iAQRh05j4+/GKPrBkKnB0GNk1wAiXgDn 8t3Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=3VZY2zDbwBbzLtxgPU26SXjP7JkefPkdEkB2jqODvmQ=; b=NB95KgkFRU46DJtymLk0rwKOq/kvDq8TDpRhM6y4nm8S5TPXtqinzJN8nLSF6SGfob Hq3BsBkcfibP2/WJbcjC37f7T2vh8iDCzBT2zgHJDJBOSjIt8cE9ur3micIoA18OAZ93 7AEmXlT/XmUvmLQsGBDqNZ6HH7LNB8JwZ4jqUU+TCf2agtnAlaScdH4Fq4IjGs3FUHMb dgB/Bcm4+qmaQCb0Ogy89p06TBOS8SN3mTNHZNfQ4xARwvt0JIRpmDtvvXfZb/jaclcR KKo6gnon/3QSfdnqswX0V2CUDzHWh7jniNOC10Dvft1akTDbcTgG1tFIe3TCbF7yVqq2 Sssg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Af0DT3tX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id bv17si5990820ejb.263.2020.07.02.08.52.05; Thu, 02 Jul 2020 08:52:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=Af0DT3tX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726213AbgGBPvp (ORCPT + 99 others); Thu, 2 Jul 2020 11:51:45 -0400 Received: from us-smtp-2.mimecast.com ([207.211.31.81]:28353 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726187AbgGBPvp (ORCPT ); Thu, 2 Jul 2020 11:51:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1593705103; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=3VZY2zDbwBbzLtxgPU26SXjP7JkefPkdEkB2jqODvmQ=; b=Af0DT3tXG3+oqpR3j1ZsBx/Rtcv/mzOG5z7bJQQAew+6sMt7IqblPlKmiZmFDEqBzYgUMu N8R174HmmM3Lslmtgv4nX0fwv1T5SeTADarmWOLuHEgWjhiFpVql4I2GkZzYbqCiS0a6kS dWCdw75wP+/+6B5lt7t/iZI31tfcgTs= Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-463-P9_Ur8O7MH2efJ2D0UyRcw-1; Thu, 02 Jul 2020 11:51:37 -0400 X-MC-Unique: P9_Ur8O7MH2efJ2D0UyRcw-1 Received: by mail-qt1-f198.google.com with SMTP id d45so11770490qte.12 for ; Thu, 02 Jul 2020 08:51:37 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=3VZY2zDbwBbzLtxgPU26SXjP7JkefPkdEkB2jqODvmQ=; b=VruEBGA9FTGJSaX7BHXBgskmcFE50D13iYp2DHNBgWyZA5OAFvqShyJljbkGO2roUi NjWaiuYyDYiTaoOOFLFtgNFXP8XFFymj5Fs06xm4WyBe3i+ynjWvqHjTYhU4L3+148KV vlOqf392rRfA7ETqOECSBD5Q5BOc/fT+n1qyveLfhXBwI1zo/8xFHoJPFWTn7us6wYKx eyzf62hXkyRuFKvhcwznr7+eGfCJwnvnpoNs/+n8daxhNodKUs36CKPolk075+0KTXw8 AV9SlrQnrZEZSv5fOTbLOu6fvidoN2e4ijiz9Z5W9ToYT3Br4MgBacvIMEJU13Rc3t/p adkQ== X-Gm-Message-State: AOAM53211SdqQyNtVC29NE16AMh7AKqzpZJLLEvHuvf6ZBVlWFR1vJEe D3JRYst4YCmlsZ92iAb8U+hkyYZwLI8mc8G6UKAmlrrwl9ZbAfS1vUZR8MkkpE/jeHuFJKzJoLM 4mv41dElaE0Mtuia1PTRzoXYC X-Received: by 2002:a37:5bc4:: with SMTP id p187mr31620570qkb.166.1593705095717; Thu, 02 Jul 2020 08:51:35 -0700 (PDT) X-Received: by 2002:a37:5bc4:: with SMTP id p187mr31620555qkb.166.1593705095513; Thu, 02 Jul 2020 08:51:35 -0700 (PDT) Received: from xz-x1 ([2607:9880:19c0:32::2]) by smtp.gmail.com with ESMTPSA id x13sm8044078qts.57.2020.07.02.08.51.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 02 Jul 2020 08:51:33 -0700 (PDT) Date: Thu, 2 Jul 2020 11:51:32 -0400 From: Peter Xu To: Pekka Enberg Cc: "linux-mm@kvack.org" , LKML , Andrew Morton , Linus Torvalds , Gerald Schaefer , Andrea Arcangeli , Will Deacon , Michael Ellerman , Paul Walmsley , Palmer Dabbelt , Albert Ou , linux-riscv@lists.infradead.org Subject: Re: [PATCH 17/26] mm/riscv: Use general page fault accounting Message-ID: <20200702155132.GJ40675@xz-x1> References: <20200626223130.199227-1-peterx@redhat.com> <20200626223625.199813-1-peterx@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 01, 2020 at 02:46:24PM +0300, Pekka Enberg wrote: > Hi Peter, Hi, Pekka, > > On Sat, Jun 27, 2020 at 1:36 AM Peter Xu wrote: > > Use the general page fault accounting by passing regs into handle_mm_fault(). > > It naturally solve the issue of multiple page fault accounting when page fault > > retry happened. > > I sent a patch to fix up riscv page fault accounting some days ago: > > http://lists.infradead.org/pipermail/linux-riscv/2020-June/000775.html Yes, this is a valid fix too. > > However, your fix is obviously even better. For the generic and riscv parts: > > Reviewed-by: Pekka Enberg Thanks! -- Peter Xu