On Fri, Feb 14, 2020 at 11:14:49AM +0100, David Hildenbrand wrote: > On 13.02.20 15:33, Andrew Jones wrote: > > Users may need to temporarily provide additional VMM parameters. > > The VMM_PARAMS environment variable provides a way to do that. > > We take care to make sure when executing ./run_tests.sh that > > the VMM_PARAMS parameters come last, allowing unittests.cfg > > parameters to be overridden. However, when running a command > > line like `$ARCH/run $TEST $PARAMS` we want $PARAMS to override > > $VMM_PARAMS, so we ensure that too. > > While reading this, I was wondering why not simply "$QEMU_PARAMS"? I'd like to slowly move away from assuming QEMU is the VMM. We already have support for kvmtool (to some degree) and I'd like to see that support increase. Also, maybe we'll eventually add support for a rust-vmm reference VMM to drive kvm-unit-tests. IOW, rather than add yet another QEMU named variable I'd like to start a trend of using VMM named variables. Actually, we could add VMM named alternatives for the QEMU named ones we have now and start using them in the scripts. We'd just need to remap the old names for backward compatibility early in the run. Thanks, drew
