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 632A8CD1283 for ; Thu, 28 Mar 2024 09:02:18 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 8C6046B0085; Thu, 28 Mar 2024 05:02:17 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 876136B0088; Thu, 28 Mar 2024 05:02:17 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 765D76B0089; Thu, 28 Mar 2024 05:02:17 -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 588936B0085 for ; Thu, 28 Mar 2024 05:02:17 -0400 (EDT) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 1A5FD1C12FC for ; Thu, 28 Mar 2024 09:02:17 +0000 (UTC) X-FDA: 81945856314.07.7F596C0 Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) by imf28.hostedemail.com (Postfix) with ESMTP id 2D685C0011 for ; Thu, 28 Mar 2024 09:02:14 +0000 (UTC) Authentication-Results: imf28.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=b2FZwegd; dmarc=pass (policy=quarantine) header.from=collabora.com; spf=pass (imf28.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.194 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1711616535; a=rsa-sha256; cv=none; b=qhCvK88cFcCajcDwLpayikvijNDfXntWPuNJksaYjbfqZEx56n5tq5rd1pzzldVguLB2s2 mKAAEz8pQjZgY+m1PZueAYjR72oeB8mBI6Mjp3Rloxwh0hOjO9BPADV4ssEjqLe1sVyQAW zsLZGxWBNPP49h3DMFKrICLXkb68KT0= ARC-Authentication-Results: i=1; imf28.hostedemail.com; dkim=pass header.d=collabora.com header.s=mail header.b=b2FZwegd; dmarc=pass (policy=quarantine) header.from=collabora.com; spf=pass (imf28.hostedemail.com: domain of usama.anjum@collabora.com designates 46.235.227.194 as permitted sender) smtp.mailfrom=usama.anjum@collabora.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1711616535; 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=GNckrkR4h9BeKhWgP0efMkJkv8Ilu1ASHJhQf/MUldw=; b=rQpLUZlNsBBwR0YPTZTN21LXVizE485SPcY7C4pCPaz9ODDwyMLRbI0br/TBUogiss4K3c EIHEA7Aq9on61LOi+YXSQFbiiEmUwj0+1EKFBm3mj3GhMiAlOHhjXFYTcnL1V9M+ZFZ7Fw 16eZJlgyrZpw3AbI9IN3K+JDVx+yKYU= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1711616533; bh=Ufh8ofbcGIpZqebBgeU63gQtywNxj6fTHPQwgJxbxWg=; h=Date:Cc:Subject:To:References:From:In-Reply-To:From; b=b2FZwegdsqBcB5ZhCCFwc/bldEv6XVivywv06kdoiGWU0RcYjHt5zMCWulL3/NR1r 23xglYTKKi+7RS/HgMV4gOr2rj8wVGGG+UYU9fo9W+2F2WBT9t4AQbrRB1KzkiHTUK F6KqIPM+KaKmq+vvh9Itvt7mlQz4i049g8DVeNZradHZ6S78T4o5/Ps+rokYDOsAP8 CWKuBOfTG6JfAAtFdigDMwPjBm7GVSeH5LWf+MSbxKDir8s1iEds7Ln0sAAvNLuQlG qk/0S0dub42+CpvkX5ezpk4YMpKk8R+TEqF+YlyBbvl6ryPwMGUQn3ZOVkOm5pibg6 2HKQnaGsx/law== Received: from [100.113.15.66] (ec2-34-240-57-77.eu-west-1.compute.amazonaws.com [34.240.57.77]) (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 madrid.collaboradmins.com (Postfix) with ESMTPSA id 6D43B3782115; Thu, 28 Mar 2024 09:02:09 +0000 (UTC) Message-ID: <6d82298b-b17b-440a-beef-590177d0ff50@collabora.com> Date: Thu, 28 Mar 2024 14:02:37 +0500 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Cc: Muhammad Usama Anjum , David Hildenbrand , Peter Xu , Axel Rasmussen , Andrea Arcangeli , linux-mm@kvack.org, linux-kselftest@vger.kernel.org, LKML , Mark Brown , Suren Baghdasaryan , Peter Zijlstra , Guillaume Tucker Subject: Re: [PATCH 0/2] Fix selftests/mm build without requiring "make headers" Content-Language: en-US To: John Hubbard , Andrew Morton , Shuah Khan References: <20240328033418.203790-1-jhubbard@nvidia.com> From: Muhammad Usama Anjum In-Reply-To: <20240328033418.203790-1-jhubbard@nvidia.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspam-User: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: 2D685C0011 X-Stat-Signature: aseqf818au6k8n5jk9t1gmx1w9ymqc7g X-HE-Tag: 1711616534-951256 X-HE-Meta: U2FsdGVkX1+0FKtfDjXjvkueocFc5xDwU8ynN16v07/vfGUb9kU4oqbQMPrCAYhpTAIKNZzVuruqvWa7y8thZaPxV0X+5+87wpzC9dpu+Rx/zngHV0r6eAT34KW7CpDebr5a5VN4T6thMn9+rgk8PeRqhjV2SCRf9Zdj99jvcM0UybLL79nT/pUSTcA0tIoNeiYFTW5j2zhc9qorZsmx76bJy8SYTm+neQAz7+sF3paNnVU8UwUwn+eqwFSlvMyoLHXi/DjbYLh971VVIuZgLSQrLIL1OREz+OwDM3SsPXUCaYinXBCZespLOy+nnn02dokpHhIFC0VWMjYSsqX4rI3yL0QfJaL5VaaFak5TuJelSquVdQKyWY71dwA0Et4WsATjuOgtQS2cB/D/PxgJTW5NN/JABkLyTSftd93AmY0ZcLFPpz9eEf4pb2EDT47+VzMfqjnZWZSsHXSnTbtgUij5F6Vn0I3r8+uYO+vAhI5Nb+gNHS2FXVkISjoayfaKux3RGmETMCzOQ/STO1U7CV6jtgjOSNGVUcNhlsT0m/kfFDClWAqbhx1FR1Z36xeLqPXheYNNWAXV2rCe+wpWRmlW5qXmMxtdFonnAsDEEZ4SWhF17+WuBvmtG9x4DLKtvM7HXM7BJl1UtgMmA4Io5YHGKOFJOwUPq68oWC+wVCr7I825VTEBFC2riGzFo4m1Ar/vkNXumAq2lXr2lfFaSGUdGmw6UT3PQwpTGaDRujgoQTb5Xaa0UH3oGHdYbJsyUXKQaPaTPB7SbM2XwEirv86h4TY7mx7PlTHcoa/tXo4uT0uGfb8NX+zu/QT5A5Rj7Ym1KTfYDqX3CIEiM+7WK7gQsKP/hphqr3k3mTBkrVl86SBA8UMIp32RjcUrkSH5FhiOE9PDrKCjx0LUdApQVKsKpbTmyNhbmQsjA2tqM5n6DGyQEWc2DKb+V5c/iZ5Br6wDsGta+KFM32Tj0UQ MFcs4n1s GApfcwZ3R7FUTNIpbDhFVZbfCeHG0EYTo1wPAYmscysAhn2YK86RphQXohwFuBY5HShFyakwDKvWFLGun5EPeG0QDBvxLpcf5uglEVltYln9MB5SulwmFxv+H/THDuVjbhm9JYidpeGYSCo3OI7U4Qjc4Vnud7xpvpP7/Y4qufbgiBfmsclRvfLLkpf2wejqp9scVQdEU+8v5EZjEMaMlQGxKSk4mzuaKEwYyLSnKjOJ1tFOjMdU+7cfLZ2fZ1BBsMNwk1gYyZ2AwnqiP9qkKcn6k3A2gIb0pW+e7PI7xdnd8lYd7MCM6OoFRhLrNp+YuX7XVXU6p8/8etxtW+ybb7wqT1jbCxw+X41emhY4zzKvRKgnatxqeGPtoPkJbNT1vBE3pxvZ6bdgzw2gVl1lOCd820Lp1TOvKcgPEqWIZXyKSmSzY/FCyrb6fBlyFJTG4aMhE1Ya2fN2oeff6jdXa2CG2hJ664jENHMeZk9Oj3bWyJYGhseIW9SMI9q2CxlEUCVBQAjZeEo53mRtRpcdN6rngG+8BlXH/1I5hWNEZ48UWifU2wer08eY3B61mpsmqsz3JR/OwsNCdUyb2eLJo3W9LHPDjOkJtZtgLdiA1gliE4dU= 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: On 3/28/24 8:34 AM, John Hubbard wrote: > Hi, > > As mentioned in each patch, this implements the solution that we discussed in > December 2023, in [1]. This turned out to be very clean and easy. It should also > be quite easy to maintain. There is another way. The headers should be built automatically by make dependency. The topmost make file always builds headers before building kselftest i.e., make kselftest The tools/testing/selftests/Makefile and others Makefiles in test suites should be updated to build the headers as well. This used to be the behavior until there were un-resolvable bugs in the Makefiles and it was reverted: https://lore.kernel.org/all/cover.1657614127.git.guillaume.tucker@collabora.com/ We should come up with the revert of this series such that all the different scenarios are covered instead of yet again a new solution; resorting to the duplication of header files or any sort of duplication. > > This should also make Peter Zijlstra happy, because it directly addresses the > root cause of his "NAK NAK NAK" reply [2]. :) > > I haven't done much build testing, because selftests are not so easy to build > with a cross-compiler. So it's just tested on x86 64-bit so far. > > [1] https://lore.kernel.org/all/783a4178-1dec-4e30-989a-5174b8176b09@redhat.com/ > [2] https://lore.kernel.org/lkml/20231103121652.GA6217@noisy.programming.kicks-ass.net/ > > Cc: David Hildenbrand > Cc: Mark Brown > Cc: Muhammad Usama Anjum > Cc: Suren Baghdasaryan > Cc: Peter Zijlstra > > John Hubbard (2): > selftests: break the dependency upon local header files > selftests/mm: fix additional build errors for selftests > > tools/include/uapi/linux/memfd.h | 39 +++ > tools/include/uapi/linux/userfaultfd.h | 386 +++++++++++++++++++++++++ > tools/testing/selftests/lib.mk | 9 + > tools/testing/selftests/mm/Makefile | 2 +- > 4 files changed, 435 insertions(+), 1 deletion(-) > create mode 100644 tools/include/uapi/linux/memfd.h > create mode 100644 tools/include/uapi/linux/userfaultfd.h > > > base-commit: 98560e9019851bf55b8a4073978a623a3bcf98c0 -- BR, Muhammad Usama Anjum