Re: [Intel-gfx] [PATCH v3 04/18] drm/selftest: Add drm damage helper selftest

Deepak Singh Rawat
2018-10-16T16:00:17Z

> > > Obviously needs some adjusting on the igt side too, since we seem to be
> > > missing the igt scaffolding for tests-drm-helper.ko.
> > > -Daniel
> >
> > Hi Daniel,
> >
> > Thanks for the review. I am a little confused here. Should we have single
> > kernel module for drm plane helper selftest and damage helper selftest?
> > Also shall I rename the kernel selfttest to kms_*?
> >
> > For user-space igt test it should be it makes sense to rename to
> kms_selftets?
>
> Since I went back&forth on this way too many times:
> - igt should be called kms_selftest. Please work together with igt
> maintainers (Arek and Petri), since we also need to update the CI
> building infrastructure to make sure it updates the list of subtests
> implemented by the kernel.
>
> - Kernel module I'd call test-drm_modeset.ko. That kernel module can then
> include the existing test-drm-helper.c (could probably rename to
> test-drm_plane_helper.c for clarity) and your new damage helper (named
> test-drm_damage_helper.c for consistency).
>
> Does that make sense to everyone?
>

Yes it makes sense to me and in fact I had similar changes in mind. And,
since existing plane selftest are not invoked from igt it's safe to rename
kernel module.