Received-SPF: pass (google.com: domain of linux-kernel+bounces-80475-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Sun, 25 Feb 2024 21:42:06 -0600
From: Lucas De Marchi <lucas.demarchi@intel.com>
To: Arnd Bergmann <arnd@kernel.org>
CC: Oded Gabbay <ogabbay@kernel.org>, Thomas =?utf-8?Q?Hellstr=C3=B6m?=
	<thomas.hellstrom@linux.intel.com>, Arnd Bergmann <arnd@arndb.de>, "Maarten
 Lankhorst" <maarten.lankhorst@linux.intel.com>, Maxime Ripard
	<mripard@kernel.org>, Thomas Zimmermann <tzimmermann@suse.de>, David Airlie
	<airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>, Rodrigo Vivi
	<rodrigo.vivi@intel.com>, Matthew Brost <matthew.brost@intel.com>, "Francois
 Dugast" <francois.dugast@intel.com>, Jani Nikula <jani.nikula@intel.com>,
	Tejas Upadhyay <tejas.upadhyay@intel.com>, Matthew Auld
	<matthew.auld@intel.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, "Michal
 Wajdeczko" <michal.wajdeczko@intel.com>, Matt Roper
	<matthew.d.roper@intel.com>, Daniele Ceraolo Spurio
	<daniele.ceraolospurio@intel.com>, <intel-xe@lists.freedesktop.org>,
	<dri-devel@lists.freedesktop.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/3] drm/xe/kunit: fix link failure with built-in xe
Message-ID: <7hfobmmgs6ntnusobalqxslcyv3kqdlnhxjoxbaful4ameavj2@niodl55l7b7g>
References: <20240224121528.1972719-1-arnd@kernel.org>
Content-Type: text/plain; charset="us-ascii"; format=flowed
Content-Disposition: inline
In-Reply-To: <20240224121528.1972719-1-arnd@kernel.org>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?jJuKr1Lt54mBWjUvhWWyYgzpuiYceFD1Z9wULUEJvirRYMz5LRnLWoI7Jh+h?=
 =?us-ascii?Q?g+tDZWmnFHwHpyiN1VMW/SB4zzPqEPeb+HbrPA0x1/QUNG+t7yo9JGIEM7+0?=
 =?us-ascii?Q?KsDnVqjvR4w4rLbTNtrrzy4AD2cC53xS1tHvQa/sSxNy99lITTW2zkGZwsg8?=
 =?us-ascii?Q?6kX4u8Wshf9q/RffHkyBITcmq80OVn2AHp6uaXy8MwmDjc57LhHIP+PBZdPP?=
 =?us-ascii?Q?Z9Bs06vKg43T1OutX4lnzVepiDyw6w0v9k5k1fDoA9qyEfLBmRyID94L7KbU?=
 =?us-ascii?Q?xuCv0ovTPT18mAgQBSoCZzmoowziM8tljQjDsckQXQlXQx9VlCMa0MXasqVb?=
 =?us-ascii?Q?qkRuanzg4NgcbhmBsYifaCCzssVvA/9mnkMIVFY9foDqaKJN0Ltv091ngm9i?=
 =?us-ascii?Q?2Ixe9yf4QKYXBVOPp/2Dqqbo6faSoUemXHX+OHfDZCstHEZ53cz7z2p4u/I4?=
 =?us-ascii?Q?vAoiz4si9U0Hc5GIc9kia7pt8oXEZah62/VzcebbqDcxPcbuVhb1h1JMNZci?=
 =?us-ascii?Q?0/FDCgHAvp5SmpTEciqukgc9Y8ut58ncImROf1UrxAH85KP23Ektjg40FpRh?=
 =?us-ascii?Q?mOZD8B8uMrxcbAYectBqwMImxjmwgSnXoSrojNqfeHKp8u3rb85Cw5TxAHjy?=
 =?us-ascii?Q?KwCTqPtZNlQrAOT4NgHMMybs8ymI2XRnRMSByXJpLyYf5UywWpqYghQAM+Nn?=
 =?us-ascii?Q?6SXhw+k0e/rzG7u1SXaTh4/kCEPtJVMYzRXuDbYUev099cWx/2uAyecqPa4D?=
 =?us-ascii?Q?65bqzaj5BqVKWoZ5rA4P/6QF6RqYUDkH8G5k72k6HBnvPamWrsYVAwQsJ7Uy?=
 =?us-ascii?Q?cpvRbG6KnPoB1H62cMTTW7oLDKVu44NDqeljsJ329eja8YAGJOTpIfSc+kow?=
 =?us-ascii?Q?XHZ5N53wn5isfAU7lvczEsn9jOSm6yktoHMe3gxru0UzfXLlct+McNVx3mCp?=
 =?us-ascii?Q?B0yx8YRjiIMCQlikUKyPvIgSgI2n+kqrCjoK/SVMldxZR9roNr/rOwevclfn?=
 =?us-ascii?Q?IlUfh70iz021F08gKOw0QIDP9CZU5AugrRbmge6q6k/8Fd5SLE9vNHYp87B6?=
 =?us-ascii?Q?kOguT9F8+yWMb4meF4O6jG+e9BnS5/GYcdlqLqljIHnvWqhEfT+oqlzPf3cs?=
 =?us-ascii?Q?BwXryduaoy6GhMxNq28NLX7D0iv6DwoJuak4ThQIbXsr2MW+qqSTVPGAR7UA?=
 =?us-ascii?Q?qH7nlJ6Ze7gD7ChRuN7rgYyoX7PZ4vzpXU0neE/NrtjWxkKYG0AKjNg8Upoa?=
 =?us-ascii?Q?y3vimNMjrtMdl9Te1D4cmxWAIX29Z+B8eEe6E1qJVL2cuasOxY8as4VKxAqJ?=
 =?us-ascii?Q?W+nU4p2T+f7BPV2RFYxRc4o0hPU6TI2uTZYlQcUxUMsfq8KKa3UBixNZ6pwQ?=
 =?us-ascii?Q?wVkynUpUN5ScR4atbPlzOFQgpYSMp2VQ8a9NyDlNxHUG7KbxdJrMhFAwmAfS?=
 =?us-ascii?Q?6BsnDGA9IokRVZIdeZds3+LF8vomKH3QO+gEKG40SU7RArjldeQ1BQAymZKH?=
 =?us-ascii?Q?Q9GGIFcraoFJv2bKc5kDwfeEbJUB9HqJ/pqMFNLpaGWHyv3vVxPn14Y3RAd7?=
 =?us-ascii?Q?a54dsFZGzCTvE/vUBGnq+GJ6E6EMFAi1dcqeTHtgZfzgHoIBSK7nSovezcKe?=
 =?us-ascii?Q?NA=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 1380964c-d9f4-489d-2457-08dc367ce946
X-MS-Exchange-CrossTenant-AuthSource: CY5PR11MB6139.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Feb 2024 03:42:09.8803
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: QBnUd9IIbnT1moqcgZa9v7RCLObtoGifeznk2p7TQOpINGLOXprjoU3H04Dg4BwDY4vtYho3CkAjb99tSpYsJULz4BabkT7WAgENH3ybtqo=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7333
X-OriginatorOrg: intel.com

On Sat, Feb 24, 2024 at 01:14:59PM +0100, Arnd Bergmann wrote:
>From: Arnd Bergmann <arnd@arndb.de>
>
>When the driver is built-in but the tests are in loadable modules,
>the helpers don't actually get put into the driver:
>
>ERROR: modpost: "xe_kunit_helper_alloc_xe_device" [drivers/gpu/drm/xe/tests/xe_test.ko] undefined!
>
>Change the Makefile to ensure they are always part of the driver
>even when the rest of the kunit tests are in loadable modules.
>
>The tests/xe_kunit_helpers.c file depends on DRM_KUNIT_TEST_HELPERS,
>so this has to always be selected by the main XE module now, rather
>than the actual tests. In turn, the "depends on (m || (y && KUNIT=y))"
>doesn't really do what it tried and can just be removed.

it actually did, which was to workaround issues prior to the commit you
are pointing out.  What it did  was to make sure xe.ko is m, or if it's
built-in, kunit is also built-in. Apparently the problem here is that
the xe_test.ko is missing the symbols.

See commit 08987a8b6820 ("drm/xe: Fix build with KUNIT=m").

I'm happy to remove it though if it's indeed not needed anymore.

Lucas De Marchi

>
>Fixes: 5095d13d758b ("drm/xe/kunit: Define helper functions to allocate fake xe device")
>Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>---
> drivers/gpu/drm/xe/Kconfig       | 3 ++-
> drivers/gpu/drm/xe/Kconfig.debug | 1 -
> drivers/gpu/drm/xe/Makefile      | 6 ++++--
> 3 files changed, 6 insertions(+), 4 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/Kconfig b/drivers/gpu/drm/xe/Kconfig
>index 6d4428b19a4c..2948650680e1 100644
>--- a/drivers/gpu/drm/xe/Kconfig
>+++ b/drivers/gpu/drm/xe/Kconfig
>@@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0-only
> config DRM_XE
> 	tristate "Intel Xe Graphics"
>-	depends on DRM && PCI && MMU && (m || (y && KUNIT=y))
>+	depends on DRM && PCI && MMU
> 	depends on ACPI_VIDEO || !ACPI
> 	select INTERVAL_TREE
> 	# we need shmfs for the swappable backing store, and in particular
>@@ -11,6 +11,7 @@ config DRM_XE
> 	select DRM_BUDDY
> 	select DRM_EXEC
> 	select DRM_KMS_HELPER
>+	select DRM_KUNIT_TEST_HELPERS if DRM_XE_KUNIT_TEST != n
> 	select DRM_PANEL
> 	select DRM_SUBALLOC_HELPER
> 	select DRM_DISPLAY_DP_HELPER
>diff --git a/drivers/gpu/drm/xe/Kconfig.debug b/drivers/gpu/drm/xe/Kconfig.debug
>index 549065f57a78..df02e5d17d26 100644
>--- a/drivers/gpu/drm/xe/Kconfig.debug
>+++ b/drivers/gpu/drm/xe/Kconfig.debug
>@@ -76,7 +76,6 @@ config DRM_XE_KUNIT_TEST
> 	depends on DRM_XE && KUNIT && DEBUG_FS
> 	default KUNIT_ALL_TESTS
> 	select DRM_EXPORT_FOR_TESTS if m
>-	select DRM_KUNIT_TEST_HELPERS
> 	help
> 	  Choose this option to allow the driver to perform selftests under
> 	  the kunit framework
>diff --git a/drivers/gpu/drm/xe/Makefile b/drivers/gpu/drm/xe/Makefile
>index 4c6ffe4b2172..b596e4482a9b 100644
>--- a/drivers/gpu/drm/xe/Makefile
>+++ b/drivers/gpu/drm/xe/Makefile
>@@ -158,8 +158,10 @@ xe-$(CONFIG_PCI_IOV) += \
> 	xe_lmtt_2l.o \
> 	xe_lmtt_ml.o
>
>-xe-$(CONFIG_DRM_XE_KUNIT_TEST) += \
>-	tests/xe_kunit_helpers.o
>+# include helpers for tests even when XE is built-in
>+ifdef CONFIG_DRM_XE_KUNIT_TEST
>+xe-y += tests/xe_kunit_helpers.o
>+endif
>
> # i915 Display compat #defines and #includes
> subdir-ccflags-$(CONFIG_DRM_XE_DISPLAY) += \
>-- 
>2.39.2
>