On 19.08.2016 08:40, Andrew Jones wrote: > On Fri, Aug 19, 2016 at 11:10:10AM +1000, Suraj Jitindar Singh wrote: >> Invoking run_tests.sh without the -g parameter will by default run all of >> the tests for a given architecture. This patch series will add a test which >> has the ability to bring down the host and thus it might be nice if we >> double check that the user actually wants to run that test instead of >> them unknowingly bringing down a machine they might not want to. >> >> In order to do this add the option for a tests' group parameter in >> unittests.cfg to include "nodefault" on order to indicate that it shouldn't >> be run be default. >> >> When tests are invoked via run_tests.sh those with the nodefault group >> parameter will be skipped unless explicitly specified by the "-g" command >> line option. When tests with the nodefault group parameter are built and >> run standalone the user will be prompted on invocation to confirm that >> they actually want to run the test. >> >> This allows a developer to mark a test as having potentially adverse >> effects and thus requires an extra level of confirmation from the user >> before they are invoked. Existing functionality will be preserved and new >> tests can choose any group other than "nodefault" if they want to be run >> by default. >> >> Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@xxxxxxxxx> >> Reviewed-by: Radim Krčmář <rkrcmar@xxxxxxxxxx> >> Reviewed-by: Andrew Jones <drjones@xxxxxxxxxx> > > Thanks for the cosmetic refresh. My r-b is already on all patches, so the > series is good to go for me. Right, series also looks fine to me, too. Radim, Paolo, could you pick up the patches directly, or do you want me to queue them and send a pull request? Thomas -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
