On 5/10/19 9:17 AM, Logan Gunthorpe wrote:
On 2019-05-09 11:18 p.m., Frank Rowand wrote:
> YES, kselftest has in-kernel tests. (Excuse the shouting...)
Cool. From my cursory look, in my opinion, these would be greatly
improved by converting them to the framework Brendan is proposing for Kunit.
>> If they do exists, it seems like it would make sense to
>> convert those to kunit and have Kunit tests run-able in a VM or
>> baremetal instance.
> They already run in a VM.
> They already run on bare metal.
> They already run in UML.
Simply being able to run in UML is not the only thing here. Kunit
provides the infrastructure to quickly build, run and report results for
all the tests from userspace without needing to worry about the details
of building and running a UML kernel, then parsing dmesg to figure out
what tests were run or not.
Yes. But that is not the only environment that KUnit must support to be
of use to me for devicetree unittests (this is not new, Brendan is quite
aware of my needs and is not ignoring them).
> This is not to say that KUnit does not make sense. But I'm
> to get a better description of the KUnit features (and there are
So read the patches, or the documentation or the LWN article. It's
pretty well described in a lot of places -- that's one of the big
advantages of it. In contrast, few people seems to have any concept of
what kselftests are or where they are or how to run them (I was
surprised to find the in-kernel tests in the lib tree).
I have been following the RFC versions. I have installed the RFC patches
and run them to the extent that they worked (devicetree unittests were
a guinea pig for test conversion in the RFC series, but the converted
tests did not work). I read portions of the code while trying to
understand the unittests conversion. I made review comments based on
the portion of the code that I did read. I have read the documentation
(very nice btw, as I have said before, but should be expanded).
My comment is that the description to submit the patch series should
be fuller -- KUnit potentially has a lot of nice attributes, and I
still think I have only scratched the surface. The average reviewer
may have even less in-depth knowledge than I do. And as I have
commented before, I keep diving into areas that I had no previous
experience with (such as kselftest) to be able to properly judge this