[PATCH v2 2/2] drm/todo: Add entry to clean up former seltests suites

[Maxime Ripard]

Hi Maira,

On Wed, Oct 25, 2023 at 02:26:44PM -0300, Maira Canal wrote:
> Hi Maxime,
> 
> Wouldn't be nice to add to the TODO list an item regarding the deleted
> drm_mm tests? Something just to remember us to develop new tests for it
> in the future.

I guess we could, but it's really not clear to me what these were
testing in the first place.

So the scope of the work would effectively be "increase our test
coverage" which I believe is already covered by the todo task just
above.

Maxime

> On 10/25/23 10:24, Maxime Ripard wrote:
> > Most of those suites are undocumented and aren't really clear about what
> > they are testing. Let's add a TODO entry as a future task to get started
> > into KUnit and DRM.
> > 
> > Signed-off-by: Maxime Ripard <mripard at kernel.org>
> > ---
> >   Documentation/gpu/todo.rst | 17 +++++++++++++++++
> >   1 file changed, 17 insertions(+)
> > 
> > diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
> > index 03fe5d1247be..b62c7fa0c2bc 100644
> > --- a/Documentation/gpu/todo.rst
> > +++ b/Documentation/gpu/todo.rst
> > @@ -621,6 +621,23 @@ Contact: Javier Martinez Canillas <javierm at redhat.com>
> >   Level: Intermediate
> > +Clean up and document former selftests suites
> > +---------------------------------------------
> > +
> > +Some KUnit test suites (drm_buddy, drm_cmdline_parser, drm_damage_helper,
> > +drm_format, drm_framebuffer, drm_dp_mst_helper, drm_mm, drm_plane_helper and
> > +drm_rect) are former selftests suites that have been converted over when KUnit
> > +was first introduced.
> > +
> > +These suites were fairly undocumented, and with different goals than what unit
> > +tests can be. Trying to identify what each test in these suites actually test
> > +for, whether that makes sense for a unit test, and either remove it if it
> > +doesn't or document it if it does would be of great help.
> > +
> > +Contact: Maxime Ripard <mripard at kernel.org>
> > +
> > +Level: Intermediate
> > +
> >   Enable trinity for DRM
> >   ----------------------
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20231026/070e024b/attachment.sig>