From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54390 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727979AbgKIB1b (ORCPT ); Sun, 8 Nov 2020 20:27:31 -0500 Subject: Re: [PATCH 00/19] Add generic user_landing tracking References: <20201108051730.2042693-1-dima@arista.com> From: Dmitry Safonov <0x7f454c46@gmail.com> Message-ID: <9f416ebd-2535-1b57-7033-e1755e906743@gmail.com> Date: Mon, 9 Nov 2020 01:27:27 +0000 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit List-ID: To: Andy Lutomirski , Dmitry Safonov Cc: LKML , Alexander Viro , Andrew Morton , Arnd Bergmann , Borislav Petkov , Catalin Marinas , Christophe Leroy , Guo Ren , "H. Peter Anvin" , Ingo Molnar , Oleg Nesterov , Russell King , Thomas Bogendoerfer , Thomas Gleixner , Vincenzo Frascino , Will Deacon , X86 ML , linux-arm-kernel , Albert Ou , "David S. Miller" , Palmer Dabbelt , Paul Walmsley , Linux FS Devel , Christian Borntraeger , Heiko Carstens , Vasily Gorbik , linux-s390 , sparclinux , "open list:MIPS" On 11/8/20 7:07 PM, Andy Lutomirski wrote: > On Sat, Nov 7, 2020 at 9:17 PM Dmitry Safonov wrote: >> >> Started from discussion [1], where was noted that currently a couple of >> architectures support mremap() for vdso/sigpage, but not munmap(). >> If an application maps something on the ex-place of vdso/sigpage, >> later after processing signal it will land there (good luck!) >> >> Patches set is based on linux-next (next-20201106) and it depends on >> changes in x86/cleanups (those reclaim TIF_IA32/TIF_X32) and also >> on my changes in akpm (fixing several mremap() issues). >> >> Logically, the patches set divides on: >> - patch 1: cleanup for patches in x86/cleanups >> - patches 2-11: cleanups for arch_setup_additional_pages() > > I like these cleanups, although I think you should stop using terms > like "new-born". A task being exec'd is not newborn at all -- it's in > the middle of a transformation. Thank you for looking at them, Andy :-) Yeah, somehow I thought about new-execed process as a new-born binary. I'll try to improve changelogs in v2. Thanks, Dmitry