From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Jones Subject: Re: [kvm-unit-tests PATCH 00/18] bunch of mostly trivial patches Date: Tue, 10 Nov 2015 11:54:22 -0500 Message-ID: <20151110165422.GD3452@hawk.localdomain> References: <1446769483-21586-1-git-send-email-drjones@redhat.com> <56421D8E.4060904@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: kvm@vger.kernel.org, alex.bennee@linaro.org, cov@codeaurora.org To: Paolo Bonzini Return-path: Received: from mx1.redhat.com ([209.132.183.28]:54727 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753240AbbKJQy2 (ORCPT ); Tue, 10 Nov 2015 11:54:28 -0500 Content-Disposition: inline In-Reply-To: <56421D8E.4060904@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Tue, Nov 10, 2015 at 05:38:38PM +0100, Paolo Bonzini wrote: >=20 >=20 > On 06/11/2015 01:24, Andrew Jones wrote: > > Many of these patches were posted once. Some weren't, but anyway > > almost everything is pretty trivial. I'd like to get these in, or > > at least get definitive nacks on them (and then drop them) in order > > to clean my queue before more patches (coming from Alex Benn=E9e an= d > > Chistopher are reposted). > >=20 > > All patches also available here > > https://github.com/rhdrjones/kvm-unit-tests/commits/queue >=20 > I applied all of these=20 Thanks! > except 1 (question asked) and 14/15/16/17 (not sure I like the idea). At one point I recall that you liked the uapi patches, although I'm not 100% married to it myself, as it does add a new dependency. I'm open to suggestions. I'm not sure what you're opposed to wrt to map files (patch 15). They aren't 100% necessary, but don't really hurt either to generate either. I won't fight for them though. The TEST=3D patch is quite useful. I find it annoying to always have to modify a makefile whenever I throw together a few line test. It may not be for everyone, but then it doesn't do anything when it's not used, so it shouldn't hurt that it exists. I would agree that maybe the patch should also document it though, if you argued that. Or, that fact that it's undocumented, and does nothing when not used, could be an argument to just commit it :-) Thanks, drew >=20 > Paolo >=20 > > Thanks, > > drew > >=20 > >=20 > > Alex Benn=E9e (4): > > README: add some CONTRIBUTING notes > > configure: emit HOST=3D$host to config.mak > > lib/printf: support the %u unsigned fmt field > > lib/arm: add flush_tlb_page mmu function > >=20 > > Andrew Jones (13): > > makefiles: use bash > > trivial: lib: fail hard on failed mallocs > > trivial: alloc: don't use 'top' outside spinlock > > trivial: lib: missing extern in string.h > > README: add pointer to new wiki page > > run_tests: pass test name to run script > > arm/run: use ACCEL to choose between kvm and tcg > > run_tests: probe for max-smp > > arm/arm64: allow building a single test > > arm/arm64: generate map files > > lib: link in linux kernel headers (uapi) > > Revert "arm/arm64: import include/uapi/linux/psci.h" > > arm/arm64: uart0_init: check /chosen/stdout-path > >=20 > > Christopher Covington (1): > > arm: Fail on unknown subtest > >=20 > > .gitignore | 2 ++ > > Makefile | 6 ++-- > > README | 32 +++++++++++++++++++ > > arm/run | 43 ++++++++++++++++++++++---- > > arm/selftest.c | 3 ++ > > arm/unittests.cfg | 7 +++-- > > config/config-arm-common.mak | 9 +++++- > > configure | 11 +++++++ > > lib/alloc.c | 8 +++-- > > lib/arm/asm/mmu.h | 11 +++++++ > > lib/arm/asm/page.h | 2 +- > > lib/arm/asm/psci.h | 2 +- > > lib/arm/asm/uapi-psci.h | 73 --------------------------------= ------------ > > lib/arm/io.c | 36 ++++++++++++++++------ > > lib/arm64/asm/mmu.h | 8 +++++ > > lib/arm64/asm/page.h | 2 +- > > lib/arm64/asm/psci.h | 2 +- > > lib/arm64/asm/uapi-psci.h | 1 - > > lib/asm-generic/page.h | 2 +- > > lib/const.h | 11 ------- > > lib/printf.c | 13 ++++++++ > > lib/string.h | 2 +- > > lib/virtio-mmio.c | 7 ++--- > > run_tests.sh | 12 +++++++- > > scripts/functions.bash | 8 +++-- > > scripts/mkstandalone.sh | 22 ++++++++++--- > > x86/unittests.cfg | 1 + > > 27 files changed, 210 insertions(+), 126 deletions(-) > > delete mode 100644 lib/arm/asm/uapi-psci.h > > delete mode 100644 lib/arm64/asm/uapi-psci.h > > delete mode 100644 lib/const.h > >=20 > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html