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 90874C3DA64 for ; Tue, 6 Aug 2024 12:03:52 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 06A886B0083; Tue, 6 Aug 2024 08:03:52 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 01A176B0085; Tue, 6 Aug 2024 08:03:51 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E24C06B0088; Tue, 6 Aug 2024 08:03:51 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id C124C6B0083 for ; Tue, 6 Aug 2024 08:03:51 -0400 (EDT) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 3CB6E80B7B for ; Tue, 6 Aug 2024 12:03:51 +0000 (UTC) X-FDA: 82421686662.01.0D1FE44 Received: from mail.ozlabs.org (gandalf.ozlabs.org [150.107.74.76]) by imf29.hostedemail.com (Postfix) with ESMTP id 7D24D12003D for ; Tue, 6 Aug 2024 12:03:48 +0000 (UTC) Authentication-Results: imf29.hostedemail.com; dkim=pass header.d=ellerman.id.au header.s=201909 header.b=hqPX5xuL; dmarc=none; spf=pass (imf29.hostedemail.com: domain of mpe@ellerman.id.au designates 150.107.74.76 as permitted sender) smtp.mailfrom=mpe@ellerman.id.au ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1722945759; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=TUApeyrXViRHjJ8RTPlVas4Td/RDYBhcdV8x/1z+BkM=; b=lIAuvsFsO0fQw9pGUKygLieYPz/laMkjxfqY1GTwzmi08ilsTW915vd9qtVPv632qvc3mu LxEU2GmNO7OPDvH2qO5xggxhnCfFiFgc8U6I3zN3z/4Iwu3FuFOs41xchzDEIesQ9Uauna x97MP5Zl+DhPA4r1qHAl9tPO+xGhNzY= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1722945759; a=rsa-sha256; cv=none; b=y9ng3rg8oVnTn/nOFfe0wkC9jAcKLhQwwKPmDMHZQ31qj0hhHfNqkS5nP3VzMYuUt+ZThz U0NZE0tVc29wfr3Atv0nElejEmkB75844W18daC6jW+CTBEtXafVgJAA725k5AdpVujjIT n3UBQVFr7UEWj4IcaCfYBZBZnsRxFLs= ARC-Authentication-Results: i=1; imf29.hostedemail.com; dkim=pass header.d=ellerman.id.au header.s=201909 header.b=hqPX5xuL; dmarc=none; spf=pass (imf29.hostedemail.com: domain of mpe@ellerman.id.au designates 150.107.74.76 as permitted sender) smtp.mailfrom=mpe@ellerman.id.au DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ellerman.id.au; s=201909; t=1722945825; bh=TUApeyrXViRHjJ8RTPlVas4Td/RDYBhcdV8x/1z+BkM=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=hqPX5xuL0Qux3mxKtPhI558jYrGFOmn3b6YIvND+qIAtlK1XmZ4kYcSu1oTMsmUCZ NSAabITVcmoXnWTK0cJ1+NiAhk7ZildLoUbhip8cmDOUWXgsn+lg1ZXImoUxy9vcwP X8u1wYq6guDP/73gxcQvHkCnxGcqB+/V3qPgTQk5cOk1ynctxPx9mWPlXMycktiWC7 q6jipj144CqjgC2VvJ2hf/F2DhgXarYcJmN0SnQDKtDAoSfNiUfDhRHlKIonXDH0Ic ACeC4e7iO95Aw11p28Heu1QUzCTTn3juA0wcuIQgJpFvcsQEFdGxE0YcUEqe6EIftG gOno6hu9Gj71w== Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mail.ozlabs.org (Postfix) with ESMTPSA id 4WdX5W4tWXz4w2J; Tue, 6 Aug 2024 22:03:43 +1000 (AEST) From: Michael Ellerman To: Linus Torvalds Cc: Jeff Xu , Nicholas Piggin , Christophe Leroy , Pedro Falcato , kernel test robot , Jeff Xu , oe-lkp@lists.linux.dev, lkp@intel.com, linux-kernel@vger.kernel.org, Andrew Morton , Kees Cook , "Liam R. Howlett" , Dave Hansen , Greg Kroah-Hartman , Guenter Roeck , Jann Horn , Jonathan Corbet , Jorge Lucangeli Obes , Matthew Wilcox , Muhammad Usama Anjum , Stephen =?utf-8?Q?R=C3=B6ttger?= , Suren Baghdasaryan , Amer Al Shanawany , Javier Carrasco , Shuah Khan , linux-api@vger.kernel.org, linux-mm@kvack.org, ying.huang@intel.com, feng.tang@intel.com, fengwei.yin@intel.com Subject: Re: [linus:master] [mseal] 8be7258aad: stress-ng.pagemove.page_remaps_per_sec -4.4% regression In-Reply-To: References: <202408041602.caa0372-oliver.sang@intel.com> <87o766iehy.fsf@mail.lhotse> Date: Tue, 06 Aug 2024 22:03:42 +1000 Message-ID: <87bk25j1sx.fsf@mail.lhotse> MIME-Version: 1.0 Content-Type: text/plain X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 7D24D12003D X-Stat-Signature: kp8wdiersa9gyoai47g8q8g4jpyfjn6a X-Rspam-User: X-HE-Tag: 1722945828-658793 X-HE-Meta: U2FsdGVkX18dNFRtacDw2WJsxRI+M6chPtFwcpbSqiLd0rNdRmmnQLuF4yGPFP40JPtxE7eC2zAYHtjnt+rxI5bRuzzOd6Ir3pbaxbBGsRyQ9/x55NSj+OCgdK0/5fhz58+QY3UWsWNdnmAl3SxWvmHJTJsemJYK3H/ydj4oudFRYqqYGcFXhJ/kfqMLtcWiEC11poZqO4OkffUC4ouHMwF52+W02RFLtHoZzb+dlF4MkVlsZluJfpd+HB6jNhuHCd+14v3LGomhBhO7W2Xv5AjVAmX/+aBCFS3W+f42QH2aEtnd25R1V3JsiKNmRqxEe7ECBKa6eh02WBmjsRMZ0Lnn4K/ssS9XpzC9HhcfePTYkIZrhCEEo0E0/yC58KhQh4VjBrOpYVXklM6NfhSpip1tD5rT7qrlsDJOYB4pqoxJKm/UgD66PocZwuXisB73krcbH8SMuQd9XJf5SjIsuBNM/+PIUbY6L7x78shHOZ369H/s1KvOJCThd8ZQRsZPgDmtHF0jt2cndXM6lFdjHafn9bmx5B83A9Gu7M7s67p869hhlklMCwb92fb/bUT9IhTWZ/G6DA0SzKpYD6NAhY4ODjIItaZRqDgyhIQXV8NtamFdunPANhHL2BwPbvB4fNcUWNDvHJQ3VbZik9k36kHH45r5BJp9kuPSM05vMGknpnemVEpjAKF5N+AeZDEcPIUmz8X85fI7V+UI9w9rDAukheBU/4d597R9o5aM4oK0QEsK6NImLhQ5kLkm39H4GrlQXaJOPeZfD8raj0wha714xJ/0a/7yLeC8REecvGvfNh4Pe6PG+LSXo+/aztKgfM0NCl1jrANytMs7q2j1jZd3MsLAPFD7Li5VkYC6KuY9GpYu13vc7tHluzWPzM8YHLkjd8qDEJf3uNSeF4BK1FaPisL28J2/SXnnMZfXj4tWAnvETkc5W+sGVkoeZ/jEwdGiusabS84ZvQA8sup dRbW2KGs 9CoXHPT5Z0Km0Xr6VNw93A1/QdqbUk5eNJB28qVoRUQF+DQdbxocbBp5y7EKZf4W9nimRy0WAazu02IgPXqqJtojHq1ZX7QvsJN9OhZ33sNj5ReCHC+r/HOmRarHqmxzVYSbbKe+uc2QCWe8uo2fphZFQ86jjSF1Gxu3jDxMV3h3j9L78kqaxm6gDYvFYLTFx5PNsHqu3HvdEv/v+0YveQ52sV204HGiHmseGPCQ2HRw5+Tf2RotUP6XkhrXa2fOfw+WwZ5rusgRw+8ufZKTpizXFW8E2YyPl/eLr7UCiD2FZuQS/uxEE1+YhTS1UmmvZ9qrSQNbW0YFqBmCXe2aYHIvBxw== 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: List-Subscribe: List-Unsubscribe: Linus Torvalds writes: > On Mon, 5 Aug 2024 at 19:14, Michael Ellerman wrote: >> >> Needs a slight tweak to compile, vvar_close() needs to return void. > > Ack, shows just how untested it was. > >> And should probably be renamed vdso_close(). > > .. and that was due to the initial confusion that I then fixed, but > didn't fix the naming. Ack. > So yes, those fixes look ObviouslyCorrect(tm). Needs another slight tweak to work correctly. Diff below. With that our sigreturn_vdso selftest passes, and the CRIU vdso tests pass also. So LGTM. I'm not sure of the urgency on this, do you want to apply it directly? If so feel free to add my tested-by/sob etc. Or should I turn it into a series and post it? cheers diff --git a/arch/powerpc/kernel/vdso.c b/arch/powerpc/kernel/vdso.c index 431b46976db8..ed5ac4af4d83 100644 --- a/arch/powerpc/kernel/vdso.c +++ b/arch/powerpc/kernel/vdso.c @@ -85,6 +85,15 @@ static void vdso_close(const struct vm_special_mapping *sm, struct vm_area_struct *vma) { struct mm_struct *mm = vma->vm_mm; + + /* + * close() is called for munmap() but also for mremap(). In the mremap() + * case the vdso pointer has already been updated by the mremap() hook + * above, so it must not be set to NULL here. + */ + if (vma->vm_start != (unsigned long)mm->context.vdso) + return; + mm->context.vdso = NULL; }