* [PATCH] drm/todo: Add entry for using kunit in the subsystem
@ 2022-05-04 8:02 Javier Martinez Canillas
2022-05-04 8:10 ` Maxime Ripard
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Javier Martinez Canillas @ 2022-05-04 8:02 UTC (permalink / raw)
To: linux-kernel
Cc: Thomas Zimmermann, Maxime Ripard, Javier Martinez Canillas,
Daniel Vetter, David Airlie, Jonathan Corbet, Maarten Lankhorst,
Maxime Ripard, dri-devel, linux-doc
The Kernel Unit Testing (KUnit) framework provides a common framework for
unit tests within the Linux kernel. Having a test suite would allow to
identify regressions earlier.
Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
---
Documentation/gpu/todo.rst | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
index 127e76ee0b2d..10bfb50908d1 100644
--- a/Documentation/gpu/todo.rst
+++ b/Documentation/gpu/todo.rst
@@ -603,6 +603,20 @@ Level: Advanced
Better Testing
==============
+Add unit tests using the Kernel Unit Testing (KUnit) framework
+--------------------------------------------------------------
+
+The `KUnit <https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html>`_
+provides a common framework for unit tests within the Linux kernel. Having a
+test suite would allow to identify regressions earlier.
+
+A good candidate for the first unit tests are the format-conversion helpers in
+``drm_format_helper.c``.
+
+Contact: Javier Martinez Canillas <javierm@redhat.com>
+
+Level: Intermediate
+
Enable trinity for DRM
----------------------
--
2.35.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/todo: Add entry for using kunit in the subsystem
2022-05-04 8:02 [PATCH] drm/todo: Add entry for using kunit in the subsystem Javier Martinez Canillas
@ 2022-05-04 8:10 ` Maxime Ripard
2022-05-04 8:34 ` Javier Martinez Canillas
2022-05-04 8:53 ` Daniel Vetter
2022-05-04 14:49 ` Thomas Zimmermann
2022-05-05 8:32 ` Javier Martinez Canillas
2 siblings, 2 replies; 7+ messages in thread
From: Maxime Ripard @ 2022-05-04 8:10 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: linux-kernel, Thomas Zimmermann, Daniel Vetter, David Airlie,
Jonathan Corbet, Maarten Lankhorst, dri-devel, linux-doc
[-- Attachment #1: Type: text/plain, Size: 1590 bytes --]
Hi,
On Wed, May 04, 2022 at 10:02:12AM +0200, Javier Martinez Canillas wrote:
> The Kernel Unit Testing (KUnit) framework provides a common framework for
> unit tests within the Linux kernel. Having a test suite would allow to
> identify regressions earlier.
>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
> ---
>
> Documentation/gpu/todo.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> index 127e76ee0b2d..10bfb50908d1 100644
> --- a/Documentation/gpu/todo.rst
> +++ b/Documentation/gpu/todo.rst
> @@ -603,6 +603,20 @@ Level: Advanced
> Better Testing
> ==============
>
> +Add unit tests using the Kernel Unit Testing (KUnit) framework
> +--------------------------------------------------------------
> +
> +The `KUnit <https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html>`_
> +provides a common framework for unit tests within the Linux kernel. Having a
> +test suite would allow to identify regressions earlier.
> +
> +A good candidate for the first unit tests are the format-conversion helpers in
> +``drm_format_helper.c``.
> +
> +Contact: Javier Martinez Canillas <javierm@redhat.com>
> +
> +Level: Intermediate
Kunit is fairly easy to grasp if you have some knowledge of other unit
testing frameworks already (pytest, cmocka, etc.)
Another good candidate would be to convert (some ?) selftests to kunit.
I'm not sure the others, but at least test-drm_cmdline_parser should be
fairly easy to convert.
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/todo: Add entry for using kunit in the subsystem
2022-05-04 8:10 ` Maxime Ripard
@ 2022-05-04 8:34 ` Javier Martinez Canillas
2022-05-04 14:16 ` Maxime Ripard
2022-05-04 8:53 ` Daniel Vetter
1 sibling, 1 reply; 7+ messages in thread
From: Javier Martinez Canillas @ 2022-05-04 8:34 UTC (permalink / raw)
To: Maxime Ripard
Cc: Thomas Zimmermann, Jonathan Corbet, David Airlie, linux-doc,
linux-kernel, dri-devel
Hello Maxime,
Thanks for your feedback.
On 5/4/22 10:10, Maxime Ripard wrote:
> Hi,
>
> On Wed, May 04, 2022 at 10:02:12AM +0200, Javier Martinez Canillas wrote:
>> The Kernel Unit Testing (KUnit) framework provides a common framework for
>> unit tests within the Linux kernel. Having a test suite would allow to
>> identify regressions earlier.
>>
>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
>> ---
>>
>> Documentation/gpu/todo.rst | 14 ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
>> index 127e76ee0b2d..10bfb50908d1 100644
>> --- a/Documentation/gpu/todo.rst
>> +++ b/Documentation/gpu/todo.rst
>> @@ -603,6 +603,20 @@ Level: Advanced
>> Better Testing
>> ==============
>>
>> +Add unit tests using the Kernel Unit Testing (KUnit) framework
>> +--------------------------------------------------------------
>> +
>> +The `KUnit <https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html>`_
>> +provides a common framework for unit tests within the Linux kernel. Having a
>> +test suite would allow to identify regressions earlier.
>> +
>> +A good candidate for the first unit tests are the format-conversion helpers in
>> +``drm_format_helper.c``.
>> +
>> +Contact: Javier Martinez Canillas <javierm@redhat.com>
>> +
>> +Level: Intermediate
>
> Kunit is fairly easy to grasp if you have some knowledge of other unit
> testing frameworks already (pytest, cmocka, etc.)
>
Yes, I didn't set to intermediate due kunit but rather due the format
conversions, since the functions are not easy to read and understand.
And the person writing the unit tests will have to get familiar with
the different formats to verify that conversions are done correctly.
> Another good candidate would be to convert (some ?) selftests to kunit.
> I'm not sure the others, but at least test-drm_cmdline_parser should be
> fairly easy to convert.
>
Indeed. Maybe I would add it as a separate entr though, as a follow-up.
> Maxime
--
Best regards,
Javier Martinez Canillas
Linux Engineering
Red Hat
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/todo: Add entry for using kunit in the subsystem
2022-05-04 8:10 ` Maxime Ripard
2022-05-04 8:34 ` Javier Martinez Canillas
@ 2022-05-04 8:53 ` Daniel Vetter
1 sibling, 0 replies; 7+ messages in thread
From: Daniel Vetter @ 2022-05-04 8:53 UTC (permalink / raw)
To: Maxime Ripard
Cc: Javier Martinez Canillas, linux-kernel, Thomas Zimmermann,
Daniel Vetter, David Airlie, Jonathan Corbet, Maarten Lankhorst,
dri-devel, linux-doc
On Wed, May 04, 2022 at 10:10:30AM +0200, Maxime Ripard wrote:
> Hi,
>
> On Wed, May 04, 2022 at 10:02:12AM +0200, Javier Martinez Canillas wrote:
> > The Kernel Unit Testing (KUnit) framework provides a common framework for
> > unit tests within the Linux kernel. Having a test suite would allow to
> > identify regressions earlier.
> >
> > Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
> > ---
> >
> > Documentation/gpu/todo.rst | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> > index 127e76ee0b2d..10bfb50908d1 100644
> > --- a/Documentation/gpu/todo.rst
> > +++ b/Documentation/gpu/todo.rst
> > @@ -603,6 +603,20 @@ Level: Advanced
> > Better Testing
> > ==============
> >
> > +Add unit tests using the Kernel Unit Testing (KUnit) framework
> > +--------------------------------------------------------------
> > +
> > +The `KUnit <https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html>`_
> > +provides a common framework for unit tests within the Linux kernel. Having a
> > +test suite would allow to identify regressions earlier.
> > +
> > +A good candidate for the first unit tests are the format-conversion helpers in
> > +``drm_format_helper.c``.
> > +
> > +Contact: Javier Martinez Canillas <javierm@redhat.com>
> > +
> > +Level: Intermediate
>
> Kunit is fairly easy to grasp if you have some knowledge of other unit
> testing frameworks already (pytest, cmocka, etc.)
>
> Another good candidate would be to convert (some ?) selftests to kunit.
> I'm not sure the others, but at least test-drm_cmdline_parser should be
> fairly easy to convert.
All the drm selftest should be good candidates for conversion to kunit.
Either way patch is Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Cheers, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/todo: Add entry for using kunit in the subsystem
2022-05-04 8:34 ` Javier Martinez Canillas
@ 2022-05-04 14:16 ` Maxime Ripard
0 siblings, 0 replies; 7+ messages in thread
From: Maxime Ripard @ 2022-05-04 14:16 UTC (permalink / raw)
To: Javier Martinez Canillas
Cc: Thomas Zimmermann, Jonathan Corbet, David Airlie, linux-doc,
linux-kernel, dri-devel
[-- Attachment #1: Type: text/plain, Size: 2350 bytes --]
On Wed, May 04, 2022 at 10:34:39AM +0200, Javier Martinez Canillas wrote:
> Hello Maxime,
>
> Thanks for your feedback.
>
> On 5/4/22 10:10, Maxime Ripard wrote:
> > Hi,
> >
> > On Wed, May 04, 2022 at 10:02:12AM +0200, Javier Martinez Canillas wrote:
> >> The Kernel Unit Testing (KUnit) framework provides a common framework for
> >> unit tests within the Linux kernel. Having a test suite would allow to
> >> identify regressions earlier.
> >>
> >> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
> >> ---
> >>
> >> Documentation/gpu/todo.rst | 14 ++++++++++++++
> >> 1 file changed, 14 insertions(+)
> >>
> >> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> >> index 127e76ee0b2d..10bfb50908d1 100644
> >> --- a/Documentation/gpu/todo.rst
> >> +++ b/Documentation/gpu/todo.rst
> >> @@ -603,6 +603,20 @@ Level: Advanced
> >> Better Testing
> >> ==============
> >>
> >> +Add unit tests using the Kernel Unit Testing (KUnit) framework
> >> +--------------------------------------------------------------
> >> +
> >> +The `KUnit <https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html>`_
> >> +provides a common framework for unit tests within the Linux kernel. Having a
> >> +test suite would allow to identify regressions earlier.
> >> +
> >> +A good candidate for the first unit tests are the format-conversion helpers in
> >> +``drm_format_helper.c``.
> >> +
> >> +Contact: Javier Martinez Canillas <javierm@redhat.com>
> >> +
> >> +Level: Intermediate
> >
> > Kunit is fairly easy to grasp if you have some knowledge of other unit
> > testing frameworks already (pytest, cmocka, etc.)
> >
>
> Yes, I didn't set to intermediate due kunit but rather due the format
> conversions, since the functions are not easy to read and understand.
>
> And the person writing the unit tests will have to get familiar with
> the different formats to verify that conversions are done correctly.
>
> > Another good candidate would be to convert (some ?) selftests to kunit.
> > I'm not sure the others, but at least test-drm_cmdline_parser should be
> > fairly easy to convert.
> >
>
> Indeed. Maybe I would add it as a separate entr though, as a follow-up.
Yeah, that works for me
Acked-by: Maxime Ripard <maxime@cerno.tech>
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/todo: Add entry for using kunit in the subsystem
2022-05-04 8:02 [PATCH] drm/todo: Add entry for using kunit in the subsystem Javier Martinez Canillas
2022-05-04 8:10 ` Maxime Ripard
@ 2022-05-04 14:49 ` Thomas Zimmermann
2022-05-05 8:32 ` Javier Martinez Canillas
2 siblings, 0 replies; 7+ messages in thread
From: Thomas Zimmermann @ 2022-05-04 14:49 UTC (permalink / raw)
To: Javier Martinez Canillas, linux-kernel
Cc: Jonathan Corbet, David Airlie, dri-devel, linux-doc, Maxime Ripard
[-- Attachment #1.1: Type: text/plain, Size: 1578 bytes --]
Hi
Am 04.05.22 um 10:02 schrieb Javier Martinez Canillas:
> The Kernel Unit Testing (KUnit) framework provides a common framework for
> unit tests within the Linux kernel. Having a test suite would allow to
> identify regressions earlier.
>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
> ---
>
> Documentation/gpu/todo.rst | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> index 127e76ee0b2d..10bfb50908d1 100644
> --- a/Documentation/gpu/todo.rst
> +++ b/Documentation/gpu/todo.rst
> @@ -603,6 +603,20 @@ Level: Advanced
> Better Testing
> ==============
>
> +Add unit tests using the Kernel Unit Testing (KUnit) framework
> +--------------------------------------------------------------
> +
> +The `KUnit <https://www.kernel.org/doc/html/latest/dev-tools/kunit/index.html>`_
> +provides a common framework for unit tests within the Linux kernel. Having a
> +test suite would allow to identify regressions earlier.
> +
> +A good candidate for the first unit tests are the format-conversion helpers in
> +``drm_format_helper.c``.
> +
> +Contact: Javier Martinez Canillas <javierm@redhat.com>
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
> +
> +Level: Intermediate
> +
> Enable trinity for DRM
> ----------------------
>
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/todo: Add entry for using kunit in the subsystem
2022-05-04 8:02 [PATCH] drm/todo: Add entry for using kunit in the subsystem Javier Martinez Canillas
2022-05-04 8:10 ` Maxime Ripard
2022-05-04 14:49 ` Thomas Zimmermann
@ 2022-05-05 8:32 ` Javier Martinez Canillas
2 siblings, 0 replies; 7+ messages in thread
From: Javier Martinez Canillas @ 2022-05-05 8:32 UTC (permalink / raw)
To: linux-kernel
Cc: Thomas Zimmermann, Maxime Ripard, Daniel Vetter, David Airlie,
Jonathan Corbet, Maarten Lankhorst, Maxime Ripard, dri-devel,
linux-doc
On 5/4/22 10:02, Javier Martinez Canillas wrote:
> The Kernel Unit Testing (KUnit) framework provides a common framework for
> unit tests within the Linux kernel. Having a test suite would allow to
> identify regressions earlier.
>
> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
> ---
Pushed to drm-misc (drm-misc-next). Thanks all!
--
Best regards,
Javier Martinez Canillas
Linux Engineering
Red Hat
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-05-05 8:32 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-04 8:02 [PATCH] drm/todo: Add entry for using kunit in the subsystem Javier Martinez Canillas
2022-05-04 8:10 ` Maxime Ripard
2022-05-04 8:34 ` Javier Martinez Canillas
2022-05-04 14:16 ` Maxime Ripard
2022-05-04 8:53 ` Daniel Vetter
2022-05-04 14:49 ` Thomas Zimmermann
2022-05-05 8:32 ` Javier Martinez Canillas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).