Messages in this thread | | | Date | Tue, 14 Jun 2022 14:58:29 +0200 | Subject | Re: [PATCH v3 3/3] drm/doc: Add KUnit documentation | From | Javier Martinez Canillas <> |
| |
Hello José,
On 6/13/22 19:17, José Expósito wrote:
[snip]
> +KUnit (Kernel unit testing framework) provides a common framework for unit tests > +within the Linux kernel. > +
I think that it will be useful to have a reference to the KUnit kernel doc here, something like the following:
`KUnit <https://docs.kernel.org/dev-tools/kunit/index.html>`_ (Kernel Unit...
> +This section covers the specifics for the DRM subsystem. For general information > +about KUnit, please refer to Documentation/dev-tools/kunit/start.rst. > + > +How to run the tests? > +~~~~~~~~~~~~~~~~~~~~~ > + > +In order to facilitate running the test suite, a configuration file is present > +in ``drivers/gpu/drm/kunit/.kunitconfig``. It can be used by ``kunit.py`` as > +follows: > + > +.. code-block:: bash > + > + $ ./tools/testing/kunit/kunit.py run --kunitconfig=drivers/gpu/drm/kunit \ > + --kconfig_add CONFIG_VIRTIO_UML=y \ > + --kconfig_add CONFIG_UML_PCI_OVER_VIRTIO=y > + > +.. note:: > + The configuration included in ``.kunitconfig`` should be as generic as > + possible. > + ``CONFIG_VIRTIO_UML`` and ``CONFIG_UML_PCI_OVER_VIRTIO`` are not > + included in it because they are only required for User Mode Linux. > + > +
Maybe also add something like this ?
For example, the following command can be used to run the test for x86_64:
$ ./tools/testing/kunit/kunit.py run --kunitconfig=drivers/gpu/drm/kunit \ --arch=x86_64
Regardless, the patch looks good to me:
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
-- Best regards,
Javier Martinez Canillas Linux Engineering Red Hat
| |