Received: by 2002:a05:7412:3290:b0:fa:6e18:a558 with SMTP id ev16csp472763rdb; Fri, 26 Jan 2024 00:53:53 -0800 (PST) X-Google-Smtp-Source: AGHT+IHb4mLBITbOumjt+Rq+igEY3iub4dfNIkVisXmYU2BMg1YpAOWo+/4p5J+alXyuVwKELnrD X-Received: by 2002:a05:622a:587:b0:42a:70d2:45f2 with SMTP id c7-20020a05622a058700b0042a70d245f2mr1204742qtb.120.1706259233439; Fri, 26 Jan 2024 00:53:53 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706259233; cv=pass; d=google.com; s=arc-20160816; b=HoNLrmfSLE9ntD1OhtFxeYKJ1E909plous7XtFoQenRmzpC90NEKl/MsL+PGma6Pnp 8N858acyDDtkY7SUDaSP+fo/WD+muVlAOyXNwvKREWHL7vxTbASDTMaXUc04Le3fJNF8 H0Ksw+q3jxLljDlgN3CWukl1mG1TmYxauc9cZaUwC2Kx8V55tOIDrWjFfjlj579ZGK+h 3wRGDPBrPmIiP1Z8mhB3qXRm/WeYwe/Me+pMEK7n2ZDubur6N9KiWnPB02RxfNXMW997 hnrobRn/eF5rBHeeYEZJSYoFlCccPFFHpLXn/GPrZFWXKePCL6ZxwX9PuG/sYEc60cda yj3g== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=Xb6T+Rm32ycbDoAgSUxsRyhNv4ZvA5Ga3kFWEo8h53w=; fh=rSNjgtPymQy9ASTO3QBXWkr7O6ZtII5a7aOnrjx/E0E=; b=NoYf4ZC3KXa0lIWg9sA4Me4sGtlOLva5Jl43yUPVVCYHDoww94Qwn51tqGJrzB4wJM 9NuHuHZaVmCfO+MhB4QH4GI8lfmKC+ru5SePLiFNUpkO4+UKKhTfkvs7GktAyRR7KlSm gryQmq3O1mdjadzBWqtvsvGkY+EwLcMXNh8zeCv/6yPHxgNYgFLYhJ9xo/XXzcHMImY0 Z+PAbnIgsjuHgeQEO7uwnZIIpQay0BMBb4DvLRYuzhbVmqI855Q++vVUmx5qKzCdZy/P 6RmuEA1i7exBHQpXql2BE48PvDp6qVyjoUU/bmATopUUGmdkJXma2HlOENECU035Bv/Z ziKg== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=Ya96dtn2; arc=pass (i=1 spf=pass spfdomain=chromium.org dkim=pass dkdomain=chromium.org dmarc=pass fromdomain=chromium.org); spf=pass (google.com: domain of linux-kernel+bounces-39682-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-39682-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id o6-20020ac85a46000000b0042a3d046c39si789174qta.61.2024.01.26.00.53.53 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 26 Jan 2024 00:53:53 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-39682-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=Ya96dtn2; arc=pass (i=1 spf=pass spfdomain=chromium.org dkim=pass dkdomain=chromium.org dmarc=pass fromdomain=chromium.org); spf=pass (google.com: domain of linux-kernel+bounces-39682-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-39682-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 3452A1C24E78 for ; Fri, 26 Jan 2024 08:53:53 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C5C1822313; Fri, 26 Jan 2024 07:00:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="Ya96dtn2" Received: from mail-oo1-f50.google.com (mail-oo1-f50.google.com [209.85.161.50]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B2C0818035 for ; Fri, 26 Jan 2024 07:00:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.161.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706252426; cv=none; b=g3+WECB49pbcOW0SnrWqW2umww9y2pPg5ScGC9s1feZ72OYGd/dBK5a8kuqhcV9XfnJrhVJNGHRUl2RQM2ChMdp76Hxpj3YyZz+rF4scHgA2U4yNg9A46HCZxSAlHwvUVrriqTThkSHXCEv5i+o+GnrgJHyZy2HatvkE7cA4mTM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706252426; c=relaxed/simple; bh=h9Xbnp41TcXaVOuPJbaZAjj1zavL9QQaF5Q3FFE2lsU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=r493ZtKIVmG7ANKY+N2X8xnS/zeAfFTrELVf0fZcq2JxEKdN2D6wcazjCdiCf5Tt4GT8E8jp4/+0AFQJzNrgfuWTlK93sHUpk3BS8NsAUh1umtoxD2yZQKSrYAVdIJgVleN7beck9KXF2AWpeqgWTS14xaeJiV1fgmrkrNF4kcw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=Ya96dtn2; arc=none smtp.client-ip=209.85.161.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Received: by mail-oo1-f50.google.com with SMTP id 006d021491bc7-5957ede4deaso70760eaf.1 for ; Thu, 25 Jan 2024 23:00:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1706252422; x=1706857222; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=Xb6T+Rm32ycbDoAgSUxsRyhNv4ZvA5Ga3kFWEo8h53w=; b=Ya96dtn2QiWJSzZ5EXqnwXOK7Qi5tzLnXc8nBBN2FdQ7sKTgMYB/xfFZp/JepgqYMf rRTy1fIu1fi4Y0c0yNrzc3IUkt7CDDvZJQnUdTe4mlT5AxKGMol6TQRU8w/+v6gpG+xR BpDzKWGUpgJNL/sRF7Wbt/ZzObAeoF1vkLMC8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706252422; x=1706857222; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Xb6T+Rm32ycbDoAgSUxsRyhNv4ZvA5Ga3kFWEo8h53w=; b=DiRsihn/n/q0kFZgFdN3dKK/0dRRFjkdbP9asYs7M15nrbPJQfzITGwPdErDp1v+YH Pla2pYYLV/+m9MPLqe3GJCjfaBpTcN3XvLysaCdhfUXbTUlWGmtyq6Erf2odHRW2Wd7V ZUVJCatBRb4rGiZbYmm6s626rrh184DFEsEdHKBtgYFv1SqIL8+WRV4RCnpZkOR46NxK PghewUltbzfvIlhvk3Jkezh3fDZcQkPcVpOLe4ZOzm3dP4Icaxl0BoaYPScZ15fKwYKx m6Zc309YKpbYhJI4lue/MkG5v6USTLLa58zG9b9auC9XlzldrBsP7CjejNVWAqkm7Fl+ TpFQ== X-Gm-Message-State: AOJu0YzA9jH3q6vUytRZxDVl8nzARVFs6dKdmMjbp84+qP3WjakI2DEE jVQV0ZQLliwWVSpAO3x4IGhKAUK5SQXNXS1pXeyZh+MNOqGs+6YHiDEaBQ9cGw== X-Received: by 2002:a05:6358:7253:b0:176:91cd:1a94 with SMTP id i19-20020a056358725300b0017691cd1a94mr1356572rwa.33.1706252421480; Thu, 25 Jan 2024 23:00:21 -0800 (PST) Received: from google.com ([2401:fa00:8f:203:c4a9:6453:1d67:60e8]) by smtp.gmail.com with ESMTPSA id e12-20020aa78c4c000000b006dde0df3a7esm531772pfd.101.2024.01.25.23.00.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 25 Jan 2024 23:00:21 -0800 (PST) Date: Fri, 26 Jan 2024 16:00:17 +0900 From: Sergey Senozhatsky To: Andrew Morton Cc: Sergey Senozhatsky , Jiexun Wang , linux-kernel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH] mm/madvise: don't forget to leave lazy MMU mode in madvise_cold_or_pageout_pte_range() Message-ID: <20240126070017.GB329333@google.com> References: <20230921174101.8e6271422a857af5414ce0a0@linux-foundation.org> <20240126032608.355899-1-senozhatsky@chromium.org> <20240125225336.6a444c01d9d9812a23a6890b@linux-foundation.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240125225336.6a444c01d9d9812a23a6890b@linux-foundation.org> On (24/01/25 22:53), Andrew Morton wrote: > Date: Thu, 25 Jan 2024 22:53:36 -0800 > From: Andrew Morton > To: Sergey Senozhatsky > Cc: Jiexun Wang , linux-kernel@vger.kernel.org, > linux-mm@kvack.org > Subject: Re: [PATCH] mm/madvise: don't forget to leave lazy MMU mode in > madvise_cold_or_pageout_pte_range() > Message-Id: <20240125225336.6a444c01d9d9812a23a6890b@linux-foundation.org> > X-Mailer: Sylpheed 3.8.0beta1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) > > On Fri, 26 Jan 2024 12:25:48 +0900 Sergey Senozhatsky wrote: > > > We need to leave lazy MMU mode before unlocking. It depends on the arch, as far as I understand it. We can enter lazy MMU mode (on each goto restart) more times than leave it, and, for isntance, on powerpc that means that we can preempt_disable() more times than preempt_enable(). That's how enter/leave lazy MMU mode is implemented there: static inline void arch_enter_lazy_mmu_mode(void) { struct ppc64_tlb_batch *batch; if (radix_enabled()) return; /* * apply_to_page_range can call us this preempt enabled when * operating on kernel page tables. */ preempt_disable(); batch = this_cpu_ptr(&ppc64_tlb_batch); batch->active = 1; } static inline void arch_leave_lazy_mmu_mode(void) { struct ppc64_tlb_batch *batch; if (radix_enabled()) return; batch = this_cpu_ptr(&ppc64_tlb_batch); if (batch->index) __flush_tlb_pending(batch); batch->active = 0; preempt_enable(); } > What might be the userspace-visible effects of this? > > > Fixes: b2f557a21bc8 ("mm/madvise: add cond_resched() in madvise_cold_or_pageout_pte_range()" > > Signed-off-by: Sergey Senozhatsky > > I'll add a cc:stable. Thanks.