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 23E80C433EF for ; Tue, 10 May 2022 16:42:45 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 683BC6B0071; Tue, 10 May 2022 12:42:45 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 633CE6B0073; Tue, 10 May 2022 12:42:45 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4D50C6B0074; Tue, 10 May 2022 12:42:45 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 3A51D6B0071 for ; Tue, 10 May 2022 12:42:45 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 0729431106 for ; Tue, 10 May 2022 16:42:45 +0000 (UTC) X-FDA: 79450402290.23.271AE9F Received: from mail-yb1-f181.google.com (mail-yb1-f181.google.com [209.85.219.181]) by imf30.hostedemail.com (Postfix) with ESMTP id 173D1800A6 for ; Tue, 10 May 2022 16:42:26 +0000 (UTC) Received: by mail-yb1-f181.google.com with SMTP id m128so31735860ybm.5 for ; Tue, 10 May 2022 09:42:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=CyIHSpV7urkNC/m60X3OuzStVpyeIryc1aK1Punw4D8=; b=V92PI4bmOdix/67urkG91/TJAjUY3Ex/AdHQvLC+n6j+EjO3mVD7v3Mz4eGUlGoEJ7 zRNK7bexwodR/5e4GXaB5rQtzm4pkmocwRan7CNTEzgNkJXNdLAHUYpfx9bDBRNojZEj 1aHaCp25650Pvre4A5jQlG2Ku68l073Z6vB/qyS6TUMrMEakIwuxYunG9ihkOykkv5aD n1Ui5FW3lbLiZagup2m7ABMIdbLVCmw7qTRG8mK3vm/NXR/wtXg2LAeBCA3FWtYN/fx6 1PHDacN8QP0kfjppDBfC0ZgdsgdLqP288ie4lkFHXo0SLOZWiBUol1lgo7C3v2kNkaIE 4Vcw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=CyIHSpV7urkNC/m60X3OuzStVpyeIryc1aK1Punw4D8=; b=LgSXsoujTJVHGVypZYWPueXcyNxq15D7uBdv0gWnyWvBlQ4KQiFyhlkqsNUUouAnOK UY4vq6BumVAPU717gnYTN7yrn52gZGxem6JNCzhNJLLchirZLt+uX7+8Y/kcGBqGcsI8 ZWqUAji0eHW1WaAWFpWB8a75ncZDZ5lSw5Uqbmlla/PESCqcZJRRKROFY4YX+gL8iM1K S7bj9LUlZvK3Fo6M/hJrWCfaACvHqFyAteSjfYxPZbTC5f3jhw3AzbGm471N5yVocC6V iOzp0qaYZupUfSiae2M9y34hX0edT2xQ//ECBQZm/9Ulq6emigw8+dC+FiwtpNUg31qI 8cEg== X-Gm-Message-State: AOAM531B1YX+1fMgKmojxdyF9GBXxyJ+9jF5ochMoVy6/0igsMwz76Bg 1sqxKscr+5W1r08wFlGyI/69LFr6xf7cEa8Z9v6PQg== X-Google-Smtp-Source: ABdhPJxZ/+tgVgvg8fQm/jcM9aB8pFitGhgZM6GXVcw+NtkFAeAj8nhv59Uys3MXLYyvAYh+f32/Oi2k53WS8hYVTeY= X-Received: by 2002:a25:d55:0:b0:648:7008:61e0 with SMTP id 82-20020a250d55000000b00648700861e0mr18424242ybn.282.1652200963633; Tue, 10 May 2022 09:42:43 -0700 (PDT) MIME-Version: 1.0 References: <20220510030014.3842475-1-surenb@google.com> <04858a5d-98c8-69be-025f-214e4b10d502@linuxfoundation.org> In-Reply-To: From: Suren Baghdasaryan Date: Tue, 10 May 2022 09:42:32 -0700 Message-ID: Subject: Re: [PATCH 1/3] selftests: vm: add process_mrelease tests To: Shuah Khan Cc: Andrew Morton , Michal Hocko , David Rientjes , Matthew Wilcox , Johannes Weiner , Roman Gushchin , Minchan Kim , "Kirill A. Shutemov" , Andrea Arcangeli , Christian Brauner , Christoph Hellwig , Oleg Nesterov , David Hildenbrand , Jann Horn , Shakeel Butt , Peter Xu , John Hubbard , shuah@kernel.org, LKML , linux-mm , linux-kselftest@vger.kernel.org, kernel-team Content-Type: text/plain; charset="UTF-8" X-Stat-Signature: xo59emukbz56y34fse6oaa8qsu6gnit6 Authentication-Results: imf30.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=V92PI4bm; spf=pass (imf30.hostedemail.com: domain of surenb@google.com designates 209.85.219.181 as permitted sender) smtp.mailfrom=surenb@google.com; dmarc=pass (policy=reject) header.from=google.com X-Rspam-User: X-Rspamd-Server: rspam01 X-Rspamd-Queue-Id: 173D1800A6 X-HE-Tag: 1652200946-300462 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 Tue, May 10, 2022 at 9:36 AM Shuah Khan wrote: > > On 5/10/22 10:29 AM, Suren Baghdasaryan wrote: > > On Tue, May 10, 2022 at 8:43 AM Shuah Khan wrote: > >> > >> On 5/9/22 9:00 PM, Suren Baghdasaryan wrote: > >>> Introduce process_mrelease syscall sanity tests. They include tests of > >>> invalid pidfd and flags inputs, attempting to call process_mrelease > >>> with a live process and a valid usage of process_mrelease. Because > >>> process_mrelease has to be used against a process with a pending SIGKILL, > >>> it's possible that the process exits before process_mrelease gets called. > >>> In such cases we retry the test with a victim that allocates twice more > >>> memory up to 1GB. This would require the victim process to spend more > >>> time during exit and process_mrelease has a better chance of catching > >>> the process before it exits. > >>> > >> > >> +1 on Mike's comments on improving the change log. List what is getting > >> tested as opposed to describing the test code. > > > > I'll try to improve the description but IMHO it does describe what > > it's testing - the process_mrelease syscall with valid and invalid > > inputs. I could omit the implementation details if that helps. > > > >> > >>> Signed-off-by: Suren Baghdasaryan > >>> --- > >>> tools/testing/selftests/vm/Makefile | 1 + > >>> tools/testing/selftests/vm/mrelease_test.c | 176 +++++++++++++++++++++ > >>> tools/testing/selftests/vm/run_vmtests.sh | 16 ++ > >>> 3 files changed, 193 insertions(+) > >>> create mode 100644 tools/testing/selftests/vm/mrelease_test.c > >> > >> Please update .gitignore with the new executable. > > > > Ack. > > > >> > >>> > >>> diff --git a/tools/testing/selftests/vm/Makefile b/tools/testing/selftests/vm/Makefile > >>> index 04a49e876a46..733fccbff0ef 100644 > >>> --- a/tools/testing/selftests/vm/Makefile > >>> +++ b/tools/testing/selftests/vm/Makefile > >>> @@ -43,6 +43,7 @@ TEST_GEN_FILES += map_populate > >>> TEST_GEN_FILES += memfd_secret > >>> TEST_GEN_FILES += mlock-random-test > >>> TEST_GEN_FILES += mlock2-tests > >>> +TEST_GEN_FILES += mrelease_test > >>> TEST_GEN_FILES += mremap_dontunmap > >>> TEST_GEN_FILES += mremap_test > >>> TEST_GEN_FILES += on-fault-limit > >>> diff --git a/tools/testing/selftests/vm/mrelease_test.c b/tools/testing/selftests/vm/mrelease_test.c > >>> new file mode 100644 > >>> index 000000000000..a61061bf8433 > >>> --- /dev/null > >>> +++ b/tools/testing/selftests/vm/mrelease_test.c > >>> @@ -0,0 +1,176 @@ > >>> +// SPDX-License-Identifier: GPL-2.0 > >>> +/* > >>> + * Copyright 2022 Google LLC > >>> + */ > >>> +#define _GNU_SOURCE > >>> +#include > >>> +#include > >>> +#include > >>> +#include > >>> +#include > >>> + > >>> +#include "util.h" > >>> + > >>> +static inline int pidfd_open(pid_t pid, unsigned int flags) > >>> +{ > >>> +#ifdef __NR_pidfd_open > >>> + return syscall(__NR_pidfd_open, pid, flags); > >>> +#else > >>> + errno = ENOSYS; > >> > >> This isn't an error - this would be skip because this syscall > >> isn't supported. > > > > Ack. > > > >> > >>> + return -1; > >>> +#endif > >> > >> Key off of syscall return instead of these ifdefs - same comment > >> on all of the ifdefs > > > > Ack. I was using some other test as an example but I guess that was > > not a good model. > > > >> > >>> +} > >>> + > >> > >> I am not seeing any reason for breaking this code up have a separate > >> routine for pidfd_open(). > > > > I'm a bit unclear what you mean. Do you mean that userspace headers > > should already define pidfd_open() and I don't need to define it? > > > > Do you need pidfd_open() or can this be part of main? Without the ifdefs, > it is really a one line code. Ah, I see. I think it's cleaner that way but I'll make them one-line inline functions. > > thanks, > -- Shuah