[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <a43fea8c-4c62-422e-a3f0-7192ab51992a@redhat.com>
Date: Mon, 1 Sep 2025 14:59:08 +0200
From: Jocelyn Falempe <jfalempe@...hat.com>
To: Maxime Ripard <mripard@...nel.org>
Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Thomas Zimmermann <tzimmermann@...e.de>, David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>,
Javier Martinez Canillas <javierm@...hat.com>, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH 2/3] drm/panic: Add kunit tests for drm_panic
On 27/08/2025 11:44, Maxime Ripard wrote:
> Hi,
>
> On Thu, Aug 21, 2025 at 11:49:06AM +0200, Jocelyn Falempe wrote:
>> Add kunit tests for drm_panic.
>> They check that drawing the panic screen doesn't crash, but they
>> don't check the correctness of the resulting image.
>>
>> Signed-off-by: Jocelyn Falempe <jfalempe@...hat.com>
>> ---
>> MAINTAINERS | 1 +
>> drivers/gpu/drm/drm_panic.c | 4 +
>> drivers/gpu/drm/tests/drm_panic_test.c | 164 +++++++++++++++++++++++++
>> 3 files changed, 169 insertions(+)
>> create mode 100644 drivers/gpu/drm/tests/drm_panic_test.c
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index cfa28b3470ab..285d1e27734f 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -8465,6 +8465,7 @@ T: git https://gitlab.freedesktop.org/drm/misc/kernel.git
>> F: drivers/gpu/drm/drm_draw.c
>> F: drivers/gpu/drm/drm_draw_internal.h
>> F: drivers/gpu/drm/drm_panic*.c
>> +F: drivers/gpu/drm/tests/drm_panic_test.c
>> F: include/drm/drm_panic*
>>
>> DRM PANIC QR CODE
>> diff --git a/drivers/gpu/drm/drm_panic.c b/drivers/gpu/drm/drm_panic.c
>> index 1e06e3a18d09..d89812ff1935 100644
>> --- a/drivers/gpu/drm/drm_panic.c
>> +++ b/drivers/gpu/drm/drm_panic.c
>> @@ -986,3 +986,7 @@ void drm_panic_exit(void)
>> {
>> drm_panic_qr_exit();
>> }
>> +
>> +#ifdef CONFIG_DRM_KUNIT_TEST
>> +#include "tests/drm_panic_test.c"
>> +#endif
>> diff --git a/drivers/gpu/drm/tests/drm_panic_test.c b/drivers/gpu/drm/tests/drm_panic_test.c
>> new file mode 100644
>> index 000000000000..46ff3e5e0e5d
>> --- /dev/null
>> +++ b/drivers/gpu/drm/tests/drm_panic_test.c
>> @@ -0,0 +1,164 @@
>> +// SPDX-License-Identifier: GPL-2.0 or MIT
>> +/*
>> + * Copyright (c) 2025 Red Hat.
>> + * Author: Jocelyn Falempe <jfalempe@...hat.com>
>> + *
>> + * KUNIT tests for drm panic
>> + */
>> +
>> +#include <drm/drm_fourcc.h>
>> +#include <drm/drm_panic.h>
>> +
>> +#include <kunit/test.h>
>> +
>> +#include <linux/units.h>
>> +#include <linux/vmalloc.h>
>> +
>> +struct drm_test_mode {
>> + const int width;
>> + const int height;
>> + const u32 format;
>> + void (*draw_screen)(struct drm_scanout_buffer *sb);
>> + const char *fname;
>> +};
>> +
>> +/*
>> + * Run all tests for the 3 panic screens: user, kmsg and qr_code
>> + */
>> +#define DRM_TEST_MODE_LIST(func) \
>> + DRM_PANIC_TEST_MODE(1024, 768, DRM_FORMAT_XRGB8888, func) \
>> + DRM_PANIC_TEST_MODE(300, 200, DRM_FORMAT_XRGB8888, func) \
>> + DRM_PANIC_TEST_MODE(1920, 1080, DRM_FORMAT_XRGB8888, func) \
>> + DRM_PANIC_TEST_MODE(1024, 768, DRM_FORMAT_RGB565, func) \
>> + DRM_PANIC_TEST_MODE(1024, 768, DRM_FORMAT_RGB888, func) \
>> +
>> +#define DRM_PANIC_TEST_MODE(w, h, f, name) { \
>> + .width = w, \
>> + .height = h, \
>> + .format = f, \
>> + .draw_screen = draw_panic_screen_##name, \
>> + .fname = #name, \
>> + }, \
>> +
>> +static const struct drm_test_mode drm_test_modes_cases[] = {
>> + DRM_TEST_MODE_LIST(user)
>> + DRM_TEST_MODE_LIST(kmsg)
>> + DRM_TEST_MODE_LIST(qr_code)
>> +};
>> +#undef DRM_PANIC_TEST_MODE
>> +
>> +static int drm_test_panic_init(struct kunit *test)
>> +{
>> + struct drm_scanout_buffer *priv;
>> +
>> + priv = kunit_kzalloc(test, sizeof(*priv), GFP_KERNEL);
>> + KUNIT_ASSERT_NOT_NULL(test, priv);
>> +
>> + test->priv = priv;
>> +
>> + drm_panic_set_description("Kunit testing");
>> +
>> + return 0;
>> +}
>> +
>> +static void drm_test_panic_screen_user_map(struct kunit *test)
>> +{
>> + struct drm_scanout_buffer *sb = test->priv;
>> + const struct drm_test_mode *params = test->param_value;
>> + void *fb;
>> + int fb_size;
>> +
>> + sb->format = drm_format_info(params->format);
>> + fb_size = params->width * params->height * sb->format->cpp[0];
>> +
>> + fb = vmalloc(fb_size);
>> + KUNIT_ASSERT_NOT_NULL(test, fb);
>> +
>> + iosys_map_set_vaddr(&sb->map[0], fb);
>> + sb->width = params->width;
>> + sb->height = params->height;
>> + sb->pitch[0] = params->width * sb->format->cpp[0];
>> +
>> + params->draw_screen(sb);
>> + vfree(fb);
>> +}
>
> You need to document what you're testing, and what the expected result
> is.
Sure, I will add some comments.
I think I can also add a few checks, as this doesn't check the result at
all.
>
> Maxime
Powered by blists - more mailing lists