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 1CAC0C001DF for ; Fri, 20 Oct 2023 16:33:52 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A5CDC8D010B; Fri, 20 Oct 2023 12:33:51 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A0CA58D0003; Fri, 20 Oct 2023 12:33:51 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8ACFA8D010B; Fri, 20 Oct 2023 12:33:51 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 779AC8D0003 for ; Fri, 20 Oct 2023 12:33:51 -0400 (EDT) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 3A439A0A2F for ; Fri, 20 Oct 2023 16:33:51 +0000 (UTC) X-FDA: 81366386262.24.904E134 Received: from madras.collabora.co.uk (madras.collabora.co.uk [46.235.227.172]) by imf10.hostedemail.com (Postfix) with ESMTP id 3DF9DC001C for ; Fri, 20 Oct 2023 16:33:47 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=kxIb9rhY; spf=pass (imf10.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.172 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com; dmarc=pass (policy=quarantine) header.from=collabora.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1697819628; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=49QLDqH/9+YRY+2hRFU7DcnKJ78PzFsNBGB67PhzUz8=; b=Y9ivxBag5u8QvohcGw8OfqDpmRXFx4NWzRLDI6Xl5tgejvwMN6g50vDbcBlXJ6pP8tk6AD wBdk0RR0ert/HDlpg68V1RuxU/GnppPdrMXFOnojHCPCCDRInz0IIfRGIPKeU4rpV1vMYL tkLPWz34uwjJ03NuCuPRgjhiFOTrHIo= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1697819628; a=rsa-sha256; cv=none; b=js+s6BAkyr9708szqKS73AUp2smx/2xBNL3kZcym2/d2xTPmCYp2rt+4xVDSE2jR1Am+D3 JdJ1a7JBtwm6eimaUOB6vY8Svu29KGLpj+T0gUEBiwN1PMv+2MXu8OR7Rdkt10gnE+fvLT YuwrvYbSG2MnyD2mk5GKXhExNsmNA0I= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=kxIb9rhY; spf=pass (imf10.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.172 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com; dmarc=pass (policy=quarantine) header.from=collabora.com Received: from [192.168.100.7] (unknown [39.34.188.12]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: usama.anjum) by madras.collabora.co.uk (Postfix) with ESMTPSA id 571ED660737B; Fri, 20 Oct 2023 17:33:29 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1697819626; bh=qAf3VEpznG3prdWZXqc557DgYUNT22t1DAY9WghAu+k=; h=Date:Cc:Subject:To:References:From:In-Reply-To:From; b=kxIb9rhY56Zh09c+xKKNsfn+CdSM2I7MVoJaUtB4doGcTTUT6bxvDoh8wQo7+qsD0 K8nzYVZbmnL3/wWhcydm+ipN0T9m/0nbqKWqE52KE8BkSZt6DmGOEuLw3mvHmkpDFd /an8HY4m3/GCQQA40CRMiNZBTjb/fTpG7d8iq6sFZNvuD34ia+IctP8S4A06l/ikO+ DWmmNyaaSUATq/+kUtSoUBKzpV26GHx17pgcFL8Zj+q2815xSfgV/e4NsMP1bCGyAB bI0I+4Ta5zBgOUbWQWY8J0SJRT3rkXLtZdPDWsoD6p80d/eluP6qp6BFcVOwylpd6l 3WzBwuIJdq+ZA== Message-ID: <53fcf507-c00e-4715-bc81-082282c37c86@collabora.com> Date: Fri, 20 Oct 2023 21:33:23 +0500 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: Muhammad Usama Anjum , jeffxu@chromium.org, akpm@linux-foundation.org, keescook@chromium.org, jannh@google.com, sroettger@google.com, willy@infradead.org, gregkh@linuxfoundation.org, torvalds@linux-foundation.org, jeffxu@google.com, jorgelo@chromium.org, groeck@chromium.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-mm@kvack.org, surenb@google.com, alex.sierra@amd.com, apopple@nvidia.com, aneesh.kumar@linux.ibm.com, axelrasmussen@google.com, ben@decadent.org.uk, catalin.marinas@arm.com, david@redhat.com, dwmw@amazon.co.uk, ying.huang@intel.com, hughd@google.com, joey.gouly@arm.com, corbet@lwn.net, wangkefeng.wang@huawei.com, Liam.Howlett@oracle.com, lstoakes@gmail.com, mawupeng1@huawei.com, linmiaohe@huawei.com, namit@vmware.com, peterx@redhat.com, ryan.roberts@arm.com, shr@devkernel.io, vbabka@suse.cz, xiujianfeng@huawei.com, yu.ma@intel.com, zhangpeng362@huawei.com, dave.hansen@intel.com, luto@kernel.org, linux-hardening@vger.kernel.org Subject: Re: [RFC PATCH v2 8/8] selftest mm/mseal mprotect/munmap/mremap/mmap Content-Language: en-US To: Peter Zijlstra References: <20231017090815.1067790-1-jeffxu@chromium.org> <20231017090815.1067790-9-jeffxu@chromium.org> <3cf29cd5-8346-419f-88f1-3a5c8ddbacad@collabora.com> <20231020152354.GC31411@noisy.programming.kicks-ass.net> From: Muhammad Usama Anjum In-Reply-To: <20231020152354.GC31411@noisy.programming.kicks-ass.net> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 3DF9DC001C X-Rspam-User: X-Rspamd-Server: rspam11 X-Stat-Signature: 3uhdqaer1cntzwhab67r96hk4fyrx7sb X-HE-Tag: 1697819627-440538 X-HE-Meta: U2FsdGVkX18sm/h6CPwfnWmcjuLA+BRUlBodA87Z2S1zfa7Pqnzjwg6jeILh8A7qXqJ85MjBM/bhqbJ7PpvbofzKRnUwESk//xWi4v2u+fKTHt7lYnp3fEcctWY3qffPs56siIdYB/aga2AdgniLcEr7+moyKuLK4RrKy1TTeAuUmrYhF8rLUrsvxQlXwPFn+nUhZRQ37g3zAWlIJDfXsciy/PxljesnHPvDNaFMw/ZcAD5cQkTQ3BM0lO17cx0y6qsIpB88TehmczWlcfw25KtJ2bhAjM2glaKVHBn65QKKaqTtxpSVqaznVrEEnbnGILnR61UHG4kw8E/p2u62SqbpfF+T22ow4gN/BH0xnyh7zWH4EoKbPzTUmYqSCglhNKikeQ6AQVHX2LLzFmikFLI2B3DCgMkIgmP4a7Z6givcZZyAJX71nvnXkAf2ezq5ZXM0QcdF897Xz0my+bHfbVCIM+T/OaLnpfucsSMSG2grf+4gnVlqSi8P+D2POiBz5dtXQCq+CfnNpTKStYfZ8CvgOvr7QFw8xZWmxfWZNgGzk1fn8fbglpb74S/53hTWdhhkpzMOEYKXimxr+hRVo6lTyQP9cIoURjwrNCVY69fU1sWA1Q11xce6i4Y6DEIpMQlcm9psqUZpILUDGDz4cpioSq1mjidbYiNmAudtKqpLpj2F1lEguwvE4EnNwGDclu9ioXh1HVcNL/K75ndnpumDIt75vKTMDTfmSPFUrFnO2Oc/sJeItCZzaHhMV3TtWsoYu4moqJMC6rPJbdDrncFN8W0yOS9QktYVW4nFiy05eJdK1qXq+rAkAUqLqVaGNwVPavOXmvFRay/a5r92b8QQN/rnd8sk8d8EhJZnVyOzPOvBxDYJ2Ieub7fMcXPR5OoTRj7BpEClGfGhRMzlOwj7BmG0MSHMhcNMeLVMgsadatiHX6Uq2dlVovZ3OAgyvmnFDnYCB6C2JFelJul t8ro9mck KLZT3CcaAajxpuMQ/sWshYNWk0TPDp//Yz2x7fYyIHCeSUJZZorIneIdKCidUAd1j81E1qFNQ55OVmZiyClGOVheRMS2V+J9jwPBtutBLCPJuoW0HBfpoQzO3MFwfB+si0l6uuKgMu27ta0hUhg9rr7hJnHklTFBLJVk9rQAzXvhK7RzPRZMMZfGzazJ1S74k/lOUHKyhwZV2ASJya0R9OQCSqHPnBT0zm+k9E0vvh/IYE1k0o3dGtM6sng== 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 10/20/23 8:23 PM, Peter Zijlstra wrote: > On Fri, Oct 20, 2023 at 07:24:03PM +0500, Muhammad Usama Anjum wrote: > >> Please remove these. These macros would be picked up from the kernel >> headers automatically. > > As per the previous discussions, how does that work if you have O= build > directories? Then headers should be prepared in that O= directory first. make headers O=abc && make -C tools/testing/selftests O=abc > > I find this push to force people to do 'make headers' in order to use > simple selftests quite misguided. You're making it *harder* to use, > leading to less use. I'm just following what we have been doing over selftests mailing list to fix build issues in different use cases and kselfest.rst. Let me share the history: Around 2 years ago, selftest Makefile used to prepare kernel headers from source automatically and include them to build selftests. It had several bugs. So they separated the header preparation from selftest build. After a while people started getting build failures because they weren't building headers which were previously built automatically. So someone had written a patch (already in v6.6-rc6) to show informative error if headers aren't present. So now selftests can't be built until headers are built. The understanding here is that selftests come with kernel source and they should be built using in-source kernel headers as people don't always have updated headers. I think, if someone want to build just one selftest without doing make headers, he should install kernel headers from source before doing so instead of adding duplicate defines in the test itself. It helps while development to not to keep the duplicate copy of these macros in selftest as well. -- BR, Muhammad Usama Anjum