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 1E4D3C76196 for ; Tue, 11 Apr 2023 10:39:51 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 903CA900004; Tue, 11 Apr 2023 06:39:50 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 8B36C900002; Tue, 11 Apr 2023 06:39:50 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 77B23900004; Tue, 11 Apr 2023 06:39:50 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 66342900002 for ; Tue, 11 Apr 2023 06:39:50 -0400 (EDT) Received: from smtpin20.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 3AF8F80CA7 for ; Tue, 11 Apr 2023 10:39:50 +0000 (UTC) X-FDA: 80668764540.20.E04FDD0 Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by imf23.hostedemail.com (Postfix) with ESMTP id EB2D6140023 for ; Tue, 11 Apr 2023 10:39:47 +0000 (UTC) Authentication-Results: imf23.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=pjGUPIdQ; spf=pass (imf23.hostedemail.com: domain of rppt@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=rppt@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1681209588; 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=rqZDGds3qvhbK9cbbIfFzw9OPSONeRTt2V1GwH7loY0=; b=lnmoXT6c2KHuBNBP5rRd2D+f7kqzAgQmGzLWu0TDnPxpBBn9923hhcCcp8QRK3ipjMK5ny 7EkKwxAHQYKVOKCxZhz74ScAQNhOAuPJ31bVz6hN11t24eLCxmMXrZAcnjcXD7bmyYPnKb iz5NeEfptVtDnDNoTKEopaInShccd4c= ARC-Authentication-Results: i=1; imf23.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=pjGUPIdQ; spf=pass (imf23.hostedemail.com: domain of rppt@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=rppt@linux.ibm.com; dmarc=pass (policy=none) header.from=ibm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1681209588; a=rsa-sha256; cv=none; b=ax2zjLMMT3vadRpA60LLkP83Gq3oEbQdb0jL+CU1OE40Mnds2hUUs8NfP1kCGxWJRGiZs+ 0orHTtFtNLzREby1fTSe6INyfnk8ppui9fntAgRewXrjW3qVORlpqs4jAfCrpZ/bLbDtOl VsmGAYbPEg+nY9Tk3S2UFiIMQvGv7CA= Received: from pps.filterd (m0098419.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 33BAa01p024612; Tue, 11 Apr 2023 10:39:46 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=date : from : to : cc : subject : message-id : references : mime-version : content-type : in-reply-to; s=pp1; bh=rqZDGds3qvhbK9cbbIfFzw9OPSONeRTt2V1GwH7loY0=; b=pjGUPIdQjLXxR6R8PqYVozbn7cBB7Ef6vHLxEeEcaw4Tp39TGWw4EWS2ovgBOd+vwaGB 6TkPpX41t4BqTwrjEomjfn/A9gf/4DQDT8n5EHsO5Ax8negJOPoNToYbGftKl7jqvkVT TkVzaUA3hBxnYqtDunL/x8s/XW+l5BGDzhQMu8p9Z3HKObldWtn6EytdQJp//+RRPxG4 4Dyg8NWxmvw5LStajAUgi/1+X7ohTtydf8egaD1lQRZoFUGacp/OkWmasHp+HbSOaRe5 A+uhQFNYaN3ZDGbRMR3imeay10LpbGwqLtGA7eRrONF8kl3Nib46a/NWM2J61mTXIFsM FA== Received: from pps.reinject (localhost [127.0.0.1]) by mx0b-001b2d01.pphosted.com (PPS) with ESMTPS id 3pw5xg02a3-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 11 Apr 2023 10:39:46 +0000 Received: from m0098419.ppops.net (m0098419.ppops.net [127.0.0.1]) by pps.reinject (8.17.1.5/8.17.1.5) with ESMTP id 33BAbJlT030341; Tue, 11 Apr 2023 10:39:46 GMT Received: from ppma02fra.de.ibm.com (47.49.7a9f.ip4.static.sl-reverse.com [159.122.73.71]) by mx0b-001b2d01.pphosted.com (PPS) with ESMTPS id 3pw5xg029c-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 11 Apr 2023 10:39:45 +0000 Received: from pps.filterd (ppma02fra.de.ibm.com [127.0.0.1]) by ppma02fra.de.ibm.com (8.17.1.19/8.17.1.19) with ESMTP id 33B4Kibh008865; Tue, 11 Apr 2023 10:39:43 GMT Received: from smtprelay01.fra02v.mail.ibm.com ([9.218.2.227]) by ppma02fra.de.ibm.com (PPS) with ESMTPS id 3pu0h5sdc3-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 11 Apr 2023 10:39:43 +0000 Received: from smtpav04.fra02v.mail.ibm.com (smtpav04.fra02v.mail.ibm.com [10.20.54.103]) by smtprelay01.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 33BAdfc127853378 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 11 Apr 2023 10:39:41 GMT Received: from smtpav04.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 63D3F2004B; Tue, 11 Apr 2023 10:39:41 +0000 (GMT) Received: from smtpav04.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 1104520043; Tue, 11 Apr 2023 10:39:40 +0000 (GMT) Received: from linux.ibm.com (unknown [9.179.16.229]) by smtpav04.fra02v.mail.ibm.com (Postfix) with ESMTPS; Tue, 11 Apr 2023 10:39:39 +0000 (GMT) Date: Tue, 11 Apr 2023 13:39:38 +0300 From: Mike Rapoport To: Peter Xu Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, Mike Kravetz , Mike Rapoport , Andrea Arcangeli , Leonardo Bras Soares Passos , Andrew Morton , David Hildenbrand , Axel Rasmussen , Nadav Amit Subject: Re: [PATCH 17/29] selftests/mm: Drop global mem_fd in uffd tests Message-ID: References: <20230330155707.3106228-1-peterx@redhat.com> <20230330160806.3107497-1-peterx@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230330160806.3107497-1-peterx@redhat.com> X-TM-AS-GCONF: 00 X-Proofpoint-GUID: 5m0oJ4Sx0jFzY0Hb3aXI-u3Y5PKcEbka X-Proofpoint-ORIG-GUID: 7ssZyfZGPXLAYHbTc1YmtqJju9E9CVoF X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22 definitions=2023-04-11_06,2023-04-11_01,2023-02-09_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 suspectscore=0 mlxlogscore=999 impostorscore=0 phishscore=0 lowpriorityscore=0 spamscore=0 adultscore=0 malwarescore=0 clxscore=1015 priorityscore=1501 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2303200000 definitions=main-2304110099 X-Rspamd-Queue-Id: EB2D6140023 X-Stat-Signature: shjubffykn4fuz8zdrrsdcjgj5ktc364 X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1681209587-432871 X-HE-Meta: U2FsdGVkX18dnvNEtgvYPRlPLHGB0TalAzlD31EtsaNKoyJOO/A8EQuCbHREcJbWkSIQn6KonaCEWDyTxDyshzSgs5WkgArgSNhTwnG0yb/PBWx42Eh7EbSIY7YRv8tbCZ2CfjmO+rdJi8ax7toQ/pkCIFY1RfODTf7SyzkvJfhBvixgwa+licj1fcKficCrSSZgagJp/Rz5tuf91szKoTuawprJF/3uGqY9BZ9JxumYFHHQn5T9HTvLmZs9Yeloi2on3MslomROBLv4TjtrxvQ7VGKChiemDiPp9XT9q1QBHrjvr9C86IGintQvDd1/DYZu9XruB/K9aRXKFuvYDBF/6NKgCAUdJ+szBK+6MVKI1gWbRr0ExTZEea66p1VBz2YlSnM+fsK4NEHM2I2gU3jQTGJC/jAY8+ICxpez6KmIxElLB8TEpBvG7OwHrnlVKv/yY8dgx/OQDBLg0C8NxXBJ8UwHnugCnfyiUXJDIc2bfxe54Kqi5dKrxd2xHxoZsfqlEe8dag5xcBAjJ7jI+cKGwFdn92KtuNLVDwYaynHJyFCaouxNlTS/8z2Sbiws3+SBU1BSpmekPDlPUbkJVrQ4Q6abnm3bz9jjjXn6+nz+k1ZEIIxehmUMa6iRNOfErd+XRgE3KEHNm5lDL8XI48tQiQAZxo3zIK57pge9omfTRtOH7cw/i0qk/xCxQ8A5XgrCWYH/4YyRG2zjYw0rMJNTsgRC+BLgBt8CE0TpKW5vs3zYkr6Jz8sBYV1g4tULWrhYpYRoQhDsbU82GmErs4udjDv2PGhwmMkZ7edD1+EQ2IDszkvyEq0oZw88a90SXvIchtOQ1/4lBjUXe2Iaikoe86GBhd72jB4l0Tfi8DgtwBJXxAsB0zvnwKrSnNW0+k9bvpDqairoCDc3UjXYjwy9djw6gDShh8qY0cpmE0FPE1F0C9hGiM4H3Osj+sZZDxr8BTzJ9/GGzAGZER9 XOpaMPY4 Icv+D7alBjuVJXq7nryStmFIQIF6/hZuK6Lcp0Cm0zG39elxfEVxhIj2YJ9O2TqIaQLqAXaDOZdeEmpDEv//SpRz5Rji8w43nfkb9pb0miRgK58pCm+ls766yQL14Ngt4B8x+MWIlUd6WCKfjGtHvbEC5HZTlXTt/cwFWLE4wQpY2wz7ZPVU+wizTHZHlms61hufyTKLrqe/fflCC0QEo/Lr7AUY1nzYcPbC1jxlz0JFBoFiKRW++ror8d27BlvIERZ41p75zsRsaKTD94rbQ2TuqukxqMGRfFevDwDYdPymFTa/eyLNu8esiFFTKCpoWywiFZYIXQWNdClvbwm/4KFxpQtzFh6Uw1/qo 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 Thu, Mar 30, 2023 at 12:08:06PM -0400, Peter Xu wrote: > Drop it by creating the memfd dynamically in the tests. > > Signed-off-by: Peter Xu Reviewed-by: Mike Rapoport (IBM) > --- > tools/testing/selftests/mm/uffd-common.c | 28 +++++++++++++++++++++++- > tools/testing/selftests/mm/uffd-common.h | 2 +- > tools/testing/selftests/mm/uffd-stress.c | 15 ------------- > 3 files changed, 28 insertions(+), 17 deletions(-) > > diff --git a/tools/testing/selftests/mm/uffd-common.c b/tools/testing/selftests/mm/uffd-common.c > index 3a9b5c1aca9d..a2b6e4957d0f 100644 > --- a/tools/testing/selftests/mm/uffd-common.c > +++ b/tools/testing/selftests/mm/uffd-common.c > @@ -12,12 +12,32 @@ > volatile bool test_uffdio_copy_eexist = true; > unsigned long nr_cpus, nr_pages, nr_pages_per_cpu, page_size, hpage_size; > char *area_src, *area_src_alias, *area_dst, *area_dst_alias, *area_remap; > -int mem_fd, uffd = -1, uffd_flags, finished, *pipefd, test_type; > +int uffd = -1, uffd_flags, finished, *pipefd, test_type; > bool map_shared, test_collapse, test_dev_userfaultfd; > bool test_uffdio_wp = true, test_uffdio_minor = false; > unsigned long long *count_verify; > uffd_test_ops_t *uffd_test_ops; > > +static int uffd_mem_fd_create(off_t mem_size, bool hugetlb) > +{ > + unsigned int memfd_flags = 0; > + int mem_fd; > + > + if (hugetlb) > + memfd_flags = MFD_HUGETLB; > + mem_fd = memfd_create("uffd-test", memfd_flags); > + if (mem_fd < 0) > + err("memfd_create"); > + if (ftruncate(mem_fd, mem_size)) > + err("ftruncate"); > + if (fallocate(mem_fd, > + FALLOC_FL_PUNCH_HOLE | FALLOC_FL_KEEP_SIZE, 0, > + mem_size)) > + err("fallocate"); > + > + return mem_fd; > +} > + > static void anon_release_pages(char *rel_area) > { > if (madvise(rel_area, nr_pages * page_size, MADV_DONTNEED)) > @@ -51,6 +71,7 @@ static void hugetlb_allocate_area(void **alloc_area, bool is_src) > off_t offset = is_src ? 0 : size; > void *area_alias = NULL; > char **alloc_area_alias; > + int mem_fd = uffd_mem_fd_create(size * 2, true); > > *alloc_area = mmap(NULL, size, PROT_READ | PROT_WRITE, > (map_shared ? MAP_SHARED : MAP_PRIVATE) | > @@ -73,6 +94,8 @@ static void hugetlb_allocate_area(void **alloc_area, bool is_src) > } > if (area_alias) > *alloc_area_alias = area_alias; > + > + close(mem_fd); > } > > static void hugetlb_alias_mapping(__u64 *start, size_t len, unsigned long offset) > @@ -95,6 +118,7 @@ static void shmem_allocate_area(void **alloc_area, bool is_src) > size_t bytes = nr_pages * page_size; > unsigned long offset = is_src ? 0 : bytes; > char *p = NULL, *p_alias = NULL; > + int mem_fd = uffd_mem_fd_create(bytes * 2, false); > > if (test_collapse) { > p = BASE_PMD_ADDR; > @@ -124,6 +148,8 @@ static void shmem_allocate_area(void **alloc_area, bool is_src) > area_src_alias = area_alias; > else > area_dst_alias = area_alias; > + > + close(mem_fd); > } > > static void shmem_alias_mapping(__u64 *start, size_t len, unsigned long offset) > diff --git a/tools/testing/selftests/mm/uffd-common.h b/tools/testing/selftests/mm/uffd-common.h > index 11f770391bd9..0dfab7057295 100644 > --- a/tools/testing/selftests/mm/uffd-common.h > +++ b/tools/testing/selftests/mm/uffd-common.h > @@ -87,7 +87,7 @@ typedef struct uffd_test_ops uffd_test_ops_t; > > extern unsigned long nr_cpus, nr_pages, nr_pages_per_cpu, page_size, hpage_size; > extern char *area_src, *area_src_alias, *area_dst, *area_dst_alias, *area_remap; > -extern int mem_fd, uffd, uffd_flags, finished, *pipefd, test_type; > +extern int uffd, uffd_flags, finished, *pipefd, test_type; > extern bool map_shared, test_collapse, test_dev_userfaultfd; > extern bool test_uffdio_wp, test_uffdio_minor; > extern unsigned long long *count_verify; > diff --git a/tools/testing/selftests/mm/uffd-stress.c b/tools/testing/selftests/mm/uffd-stress.c > index e6d39a755082..4eca1a0276c2 100644 > --- a/tools/testing/selftests/mm/uffd-stress.c > +++ b/tools/testing/selftests/mm/uffd-stress.c > @@ -1090,21 +1090,6 @@ int main(int argc, char **argv) > } > nr_pages = nr_pages_per_cpu * nr_cpus; > > - if (test_type == TEST_SHMEM || test_type == TEST_HUGETLB) { > - unsigned int memfd_flags = 0; > - > - if (test_type == TEST_HUGETLB) > - memfd_flags = MFD_HUGETLB; > - mem_fd = memfd_create(argv[0], memfd_flags); > - if (mem_fd < 0) > - err("memfd_create"); > - if (ftruncate(mem_fd, nr_pages * page_size * 2)) > - err("ftruncate"); > - if (fallocate(mem_fd, > - FALLOC_FL_PUNCH_HOLE | FALLOC_FL_KEEP_SIZE, 0, > - nr_pages * page_size * 2)) > - err("fallocate"); > - } > printf("nr_pages: %lu, nr_pages_per_cpu: %lu\n", > nr_pages, nr_pages_per_cpu); > return userfaultfd_stress(); > -- > 2.39.1 > -- Sincerely yours, Mike.