On Thu, Jul 13, 2023 at 12:19:13PM +0800, David Gow wrote:
While I love the idea of this, it breaks the default UML --alltests build, as all of ALSA is behind an "if !UML".
Oh, UML finally works again - that's good.
ALSA folks, how horrifying a prospect is removing the "if !UML" everywhere? If it's not trivial, how do we feel about adding "sound/soc/.kunitconfig" containing these tests?
I suspect this is due to UML not providing some key APIs like DMA, if it's building now I guess those dependencies might have been fixed. Do all the drivers build properly if you enable UML, otherwise all the randconfig people will get upset, it could've been to save all the drivers from having to deal with UML features issues?
Another option would be for the KUnit runner to just ignore kconfig options getting disabled.