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 E7090C10DC1 for ; Fri, 8 Dec 2023 12:44:13 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 130A76B0071; Fri, 8 Dec 2023 07:44:13 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 0E1D76B0072; Fri, 8 Dec 2023 07:44:13 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EEA516B0074; Fri, 8 Dec 2023 07:44:12 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id DC4606B0071 for ; Fri, 8 Dec 2023 07:44:12 -0500 (EST) Received: from smtpin11.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id B402980135 for ; Fri, 8 Dec 2023 12:44:12 +0000 (UTC) X-FDA: 81543618744.11.B44E5EA Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.223.130]) by imf08.hostedemail.com (Postfix) with ESMTP id 77C7D160009 for ; Fri, 8 Dec 2023 12:44:10 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=La9eDmjE; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=9hZ8iyq8; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=La9eDmjE; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=9hZ8iyq8; dmarc=none; spf=pass (imf08.hostedemail.com: domain of mbenes@suse.cz designates 195.135.223.130 as permitted sender) smtp.mailfrom=mbenes@suse.cz ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1702039450; 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=0n4maJTA6v/hJg+qYTmGFfPnnHUi3IlrWUXV9a2Jkcw=; b=JrM/XCCOMn7B0lFeaXvjw8hJXANwI3/9bLrkgPhArLzmtI8TxVb43gqavLpmkK/vfEBict 7pzAZoiq2/jhZ0kc96+fyioD6tyBVLi8kpEDvQNhYYouY3hDSRVc5dxugJjq8QvyTLOyLl r14uPWXLpAZGm5nWjh31/oTGK/0HwFo= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=La9eDmjE; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=9hZ8iyq8; dkim=pass header.d=suse.cz header.s=susede2_rsa header.b=La9eDmjE; dkim=pass header.d=suse.cz header.s=susede2_ed25519 header.b=9hZ8iyq8; dmarc=none; spf=pass (imf08.hostedemail.com: domain of mbenes@suse.cz designates 195.135.223.130 as permitted sender) smtp.mailfrom=mbenes@suse.cz ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1702039450; a=rsa-sha256; cv=none; b=E3I96z1TG73d1ajHAT4lUt+Pwbr0ObeDdWUfVDh2R4jhRL3d4IeZ2UZTlfZGoBPwG2KXOp 7tLOEI1rBz1ZkS4DetiDj/1gCp7PrEQiOxQAC2SrZL1MJzPEcKNWCpCataoGlXu93WPZzG ymouk6oBajsx3LLt12rkctIGz+BPb1g= Received: from pobox.suse.cz (unknown [10.100.2.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 5A52821C42; Fri, 8 Dec 2023 12:44:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_rsa; t=1702039448; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=0n4maJTA6v/hJg+qYTmGFfPnnHUi3IlrWUXV9a2Jkcw=; b=La9eDmjEtgE1tfkDvO3LLp4mDRDaXF3xKrwIFJFiMwHFHb6xlr+pMiycxsNkusq58GMart zjFdME5/0AFg9O/qizY6ctZoB2A/swvLAc5NRKIjICjPPnPDaGDIPmvx9HaOiosY59X2lu BwmBQP5W4xXRVQZkwGn2RykgTOiSKJg= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_ed25519; t=1702039448; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=0n4maJTA6v/hJg+qYTmGFfPnnHUi3IlrWUXV9a2Jkcw=; b=9hZ8iyq8cl3swcssNyUm+DJlQP4sGp3Bm2m2sorlyfHdav5kl+tzMKuwFr1duEX0Ch98Ps Cw79OFkvi7si5oCA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_rsa; t=1702039448; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=0n4maJTA6v/hJg+qYTmGFfPnnHUi3IlrWUXV9a2Jkcw=; b=La9eDmjEtgE1tfkDvO3LLp4mDRDaXF3xKrwIFJFiMwHFHb6xlr+pMiycxsNkusq58GMart zjFdME5/0AFg9O/qizY6ctZoB2A/swvLAc5NRKIjICjPPnPDaGDIPmvx9HaOiosY59X2lu BwmBQP5W4xXRVQZkwGn2RykgTOiSKJg= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_ed25519; t=1702039448; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=0n4maJTA6v/hJg+qYTmGFfPnnHUi3IlrWUXV9a2Jkcw=; b=9hZ8iyq8cl3swcssNyUm+DJlQP4sGp3Bm2m2sorlyfHdav5kl+tzMKuwFr1duEX0Ch98Ps Cw79OFkvi7si5oCA== Date: Fri, 8 Dec 2023 13:44:09 +0100 (CET) From: Miroslav Benes To: John Hubbard cc: Andrew Morton , David Hildenbrand , Peter Xu , Shuah Khan , Nathan Chancellor , linux-mm@kvack.org, linux-kselftest@vger.kernel.org, LKML , Muhammad Usama Anjum , Jonathan Corbet , linux-doc@vger.kernel.org, live-patching@vger.kernel.org Subject: Re: [PATCH v3 11/11] selftests: error out if kernel header files are not yet built In-Reply-To: <20230606071637.267103-12-jhubbard@nvidia.com> Message-ID: References: <20230606071637.267103-1-jhubbard@nvidia.com> <20230606071637.267103-12-jhubbard@nvidia.com> User-Agent: Alpine 2.21 (LSU 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 77C7D160009 X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: p37mn8fbtwydozz4gm4w4ifwp6qomeqa X-HE-Tag: 1702039450-67896 X-HE-Meta: U2FsdGVkX18I2ppJkFuyO4h0RfxISICxeWtAbwy27lB6XIFq2E/qOaT4GRUDBIcmzzD/8j0kUAXln7VTxqOR/P/xMAdcs4svrk4s9NN0n0o7FcQq0RUpUqM+C4YgT3mmJV0ymLsvPMnOZaIG7C+cOkfJTwiUqUzncqpEHzgkWaY3ExwitOCD07W9/AjgKxvpG6Ckt0zlQ2x/fcsoU55xbmH6RjqMy9PvviQPspXLmtdNnhCFT2nADqTz19hnjynlPQUUf0VylkDB4Q/bKZPmECPs/QMjOQH3TEQEZaV3Z/yiV62l5NL3566x4PwtXa0JdSecTyoP1H3zmnbFGhhPyIyi67+yNBuwfk0ibgJ1Q3mRirCy7HsDT8sLp/aMqnxBghnyb35KUFduDRf7TJZhFv99akjDAFk/xOUMdUy0Bb6DP2SS0F5ctkf31qCZjC7mJ7MGJpaHgpR0CNa6oGzMxCIDykW0rXvkgyoixaB35EMvllONf3OmtLsQY+cHtQT03VnmFoV4xadbyOmLZOQBk6+b8mJvVQFO0g9UKJliaHommo0Bh8kpoVwkV8FODxHpJnK0nAQ7Ip4g43AZeitSFnou9ywABbiDF1JVc5pGaa2a3fJyfxKD9gD0dHz/5/Vv0I6SMLlPwjYw/sxyP2/sCBjhgyap8Z11hD1rFbIuI6Chvqegaa/EPwF8cH73VaOEnlERJgN4VxCWeog010cm7E97GzPOJIgtiG/513YQKa7LYnt5bIEk1iqu3cUIRoMH+NtWTVaeL+cQ3YptTLrT93CsrnitBsCSx/l7yJ6oEk8Lk0MY32hxduT2kq1lyALp9z6Wk79si2oJihYQzhNCd86hnDREfGZR26ro+fab+jjmGECgWQ1dCW0M+q8wszBM/k9xwdtrRv5EWDwIK0/BfYJcgfdtztTOYUlNbfWwqXpsxtXE4pZe4H03awyq5M71olzI7KCUakpq/FYBozG umhgp7za YguZ0KDSc37DzYzfQo7halqMaY45wCT0pf7JmsjDK/iyEjFQTY4s6jlJHAhin2B22VWYnk7UUboadNAOVWypJtwDLItlRDbOJCFpOLMmSR3d9RXedjsSt+USrgnYMxMfSzjz/D4v/QEJi6JgDjm8RgWKcQDrjiz5nseAlY1N/Uck0tH9GvnoA3Ccd90ZgHfmxtSQ7GM6k4cKQZXoH28a0x68DtlJta0MOn+HiHmwZQGECLdqehfQff5qnWaepfE90WDxhHjulHEB1zJs2mmqj1oLTYGkft6mEtzCML+hDc8AO6xt9ml7WnsKfDfsFegsVkk7SORG0W+pMHU63LlUipzXWdwJYGlwVdaYXaNnKb1Ety+M= 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: Hi John, Muhammad, On Tue, 6 Jun 2023, John Hubbard wrote: > As per a discussion with Muhammad Usama Anjum [1], the following is how > one is supposed to build selftests: > > make headers && make -C tools/testing/selftests/mm > > Change the selftest build system's lib.mk to fail out with a helpful > message if that prerequisite "make headers" has not been done yet. > > [1] https://lore.kernel.org/all/bf910fa5-0c96-3707-cce4-5bcc656b6274@collabora.com/ could you, please, elaborate more on that one is supposed to build selftests with 'make headers'? Yes, Documentation/dev-tools/kselftest.rst mentions that because you might need headers but... The common way how we test the kernel is to build the kernel, install it somewhere and run selftests on top. The sequence basically being "make rpm-pkg; rpm -ivh; cd tools/testing/selftest/livepatch/ in source tree; sudo make run_tests" (or a similar variation of the procedure). The point is that we want to test the running kernel with its respective environment installed in /lib/modules/`uname -r`/ (if needed). This way we can run newer selftests from the current mainline tree on older kernels among others. The commit breaks the use case which worked for a long long time. It also breaks what Marcos proposed for livepatch selftests in https://lore.kernel.org/all/20231031-send-lp-kselftests-v3-0-2b1655c2605f@suse.com/ I guess we can always work around it by letting subsystem selftests to override KHDR_DIR but I am not comfortable with the behaviour that your commit introduced in the first place to be honest. Thank you, Miroslav