Received-SPF: pass (google.com: domain of linux-kernel+bounces-123367-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <834542e1-bb29-49ed-98ee-13a79168819f@nvidia.com>
Date: Thu, 28 Mar 2024 10:28:38 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 0/2] Fix selftests/mm build without requiring "make
 headers"
To: Muhammad Usama Anjum <usama.anjum@collabora.com>,
 Andrew Morton <akpm@linux-foundation.org>, Shuah Khan <shuah@kernel.org>
Cc: David Hildenbrand <david@redhat.com>, Peter Xu <peterx@redhat.com>,
 Axel Rasmussen <axelrasmussen@google.com>,
 Andrea Arcangeli <aarcange@redhat.com>, linux-mm@kvack.org,
 linux-kselftest@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
 Mark Brown <broonie@kernel.org>, Suren Baghdasaryan <surenb@google.com>,
 Peter Zijlstra <peterz@infradead.org>, Guillaume Tucker <gtucker@gtucker.io>
References: <20240328033418.203790-1-jhubbard@nvidia.com>
 <6d82298b-b17b-440a-beef-590177d0ff50@collabora.com>
Content-Language: en-US
From: John Hubbard <jhubbard@nvidia.com>
In-Reply-To: <6d82298b-b17b-440a-beef-590177d0ff50@collabora.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?RUtQS2tRY256QmVyaXpwNS96K2x4dWxoMHdsOVpkb2t5MFJwcGpLUllMNHJ1?=
 =?utf-8?B?ZEFaRWpJdFNoWmdFb1piQTBXdmd5enVneEEwYWc1azNSTTF5eHU1TTlheUhz?=
 =?utf-8?B?Mm5VQWV1ZC9lSG5KQnZaMmZ2UitTQjkvYW40NnhxTFFpczU3aW5NME9IVlhk?=
 =?utf-8?B?dEgzcTY3Y21iQk9JRHU1UzFHWDAvR1AyVGxkamZPWWhoUEJPcTQ5dGphRTVP?=
 =?utf-8?B?MUdYS3lBOWFpR2d2bTdYQzJjQkZVSGxXTVNvSEt0QzNOak5MWHI4TjdnRW1U?=
 =?utf-8?B?REVrRVd3L1lqbkJoR2c5c1R4cDdOUVJnVWh1Qkh5V09Qa2ltdFJpTkVmUURn?=
 =?utf-8?B?SDZLN1VPTnhZK25Xa2tVeVpua0IzVEdEbjR4N2RQK0U4bTI2Q0wzNk44aGUz?=
 =?utf-8?B?T21jOVdxOGNTcjhodkdFVFRDbWhPVEF0cDlVQ1JKMGhsZEFSc2RIQnM5b01W?=
 =?utf-8?B?RS8rL04xbGpYV1lic05SeHpkT2dXMUEvaXVJenIySHlTdEdFQjQrdnhNZWls?=
 =?utf-8?B?SGt2Uldnc0Nob1R0Z214alNWTDZnS2NNdWdPUkx6TThYWjJqdVRqM0I4b2dQ?=
 =?utf-8?B?SlF1SW9ZdE5Kc0k3c1ZQKytNZW1reUpiMkN4Z1gvNTU3Mnh0eXAwcEVSVjJE?=
 =?utf-8?B?L1hhK3JOQzNlUVpNOGF3SDRZb1dHV0k1RytKelM1NU0wOGt0SktCMEM4cDFI?=
 =?utf-8?B?MXBOcnh3R1RvK29Id3BGdWg3SEVJM01kcTA5YXIvNFJCdVJPSktaSXhlbkhX?=
 =?utf-8?B?VHpKOW9aOW1lQm1VcXVmVVg4V0JTSzBrVjFETU40Q1MzVzVSbjhWODNJUVZR?=
 =?utf-8?B?SVlaYVBWRDMrQlVGMEFDUVBpSXFsR1Z3UmdrZjFpWkhlODBudUJET29nY1VZ?=
 =?utf-8?B?RmhmRjhRQXlTVnNtWDJrbEpEaFVabmRMWHhlTXRrczMvM0R4WGlCb3VvSlM5?=
 =?utf-8?B?QnNpeUxnejF5ZUpJY0JyWTRrY2tZSVV3RUljOHcvREpFdEpDZTdKWnJISWM5?=
 =?utf-8?B?RWpqVm82UDBtN1NITWxqdFlwY0s0WGxoeU9wSU5UMUNPb3JRbVRONTJWSGxK?=
 =?utf-8?B?aFFoQWpyOXA5Y0FmN1FpTzBtREVxSWs2ZGFJOGo2Vzk1QUR4Sjk1WGZHbmdH?=
 =?utf-8?B?cEZDUUlQaGg1YmdxdjlGWGtld2R1c0ZDQXhOejVERTc5b2tnZVVOek5DOWZN?=
 =?utf-8?B?clhzNWM5SVVIWXdzdzBYcVR3RmJpMXpWeE9FVS9GMm1lQzQ0VG9uM1VmU1ZS?=
 =?utf-8?B?MjFhcVdpWWRMVy9tMGlqU3MvL1JMMlpYeUFIOXlZNGxxdUJteWJNTWR3cmxz?=
 =?utf-8?B?K1ppeHlxWlJRMlhqQnZmcW4yR2hiQ3AxaHFwdmkwSWllRHN6OVUrQS85dEsz?=
 =?utf-8?B?VDRTdjlBNE14TjRVN2t0aVpuNVFQdDBxTHhtS2l3YVVXNUJ5cW5KSEcwY3B4?=
 =?utf-8?B?ZmlaOEhKMFZJd2JpUmp6eXFHU0dSRzdBMnlFVWxPSEdvdk94TStDak9GandK?=
 =?utf-8?B?YlhxYTNVR1FIYUxxMHpjV3k3OXRxMzF4dTJBVG9LTDArWStsN09rRHlrSmR6?=
 =?utf-8?B?bENDcU80RWZSZ0NUeE5DV3k5aHlCMG5lOFhSWE5pL2xERDEyNDdSc3MvbmRk?=
 =?utf-8?B?ME1PSzFYTFRldGxFamxiYVcxY2tsSm1wWUhLTTIreWdlQWVML0R0RVlBVFNp?=
 =?utf-8?B?M3NaUUNUMDRtaVdrS2ZvNy9vUUdGclI0QkloK1BXWUhTOHAzdHl4eXZQVTdp?=
 =?utf-8?B?dTNUeE5UZi9kNU1RVS82WDRTODFwVHVBTmtjOTBsQXZ0cU5mQW9iQnhJRkVn?=
 =?utf-8?B?bCtXYVVON0Q4SXhVV0Z5Y3lEa2VORWFsWXFFNTN5dUc0eElhb2J0UGhkVVdR?=
 =?utf-8?B?VEN5UldWUCs3N3JPbUcwUC9QeC9OVUZ0a3MramVFcnJHNkNtMktuRXBJb1JE?=
 =?utf-8?B?L0ExZVJydy9SNVplcktZWjZRZXZPaWtHa2ZmN2gwNUY2ODg2VjRWeWdiK1VX?=
 =?utf-8?B?NkxvRFFpNXZtMmpaN1dxOWtoc1BhWmFacXdodm85eDVhc3l1SzhhajhzL004?=
 =?utf-8?B?Ry9rd2VkL1FLRUc5c0hwR0tEMWVKY3FEN01IQUNpdDIyZW9ySVM5bmFrTGVX?=
 =?utf-8?B?c2k3Um5LdUx6eHh0MHlNemFBdEJuakk1MzYyY0pudXdqRUdBenFNeWRWV1M3?=
 =?utf-8?B?Y3c9PQ==?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c4de883d-cfff-43c9-2199-08dc4f4caea1
X-MS-Exchange-CrossTenant-AuthSource: BY5PR12MB4130.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Mar 2024 17:29:54.6822
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: pepeeygQHwRWjBbKqKvmyLXB7r/Oy3gZsBfHnGtnuissL4oXaX95Yo4psCikZJBpqOEFOG2m1TIBDFDZgbtNFQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB8175

On 3/28/24 2:02 AM, Muhammad Usama Anjum wrote:
> 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
> 

I think we talked through this already: Peter Z. pointed out the problems
with requiring this kind of prerequisite. And it really it overkill. The
approach here is simple, easy to maintain, and avoids breaking the various
unusual build setups that people have.

I'll shut up now and let others weigh in, though. :)


thanks,
-- 
John Hubbard
NVIDIA