From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53342) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vbt9l-0003BM-Ov for qemu-devel@nongnu.org; Thu, 31 Oct 2013 10:23:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vbt9e-0002aw-H2 for qemu-devel@nongnu.org; Thu, 31 Oct 2013 10:23:29 -0400 Received: from mx1.redhat.com ([209.132.183.28]:56860) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vbt9e-0002aS-8w for qemu-devel@nongnu.org; Thu, 31 Oct 2013 10:23:22 -0400 Date: Thu, 31 Oct 2013 16:26:09 +0200 From: "Michael S. Tsirkin" Message-ID: <20131031142609.GA9929@redhat.com> References: <1381949935-8692-1-git-send-email-mst@redhat.com> <20131030230618.GA5961@hawk.usersys.redhat.com> <20131031074844.GO4651@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH] import kvm-unittest in QEMU source tree List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Paolo Bonzini , Andrew Jones , qemu-devel , Gleb Natapov On Thu, Oct 31, 2013 at 03:17:57PM +0100, Anthony Liguori wrote: > On Thu, Oct 31, 2013 at 8:48 AM, Gleb Natapov wrote: > > On Wed, Oct 30, 2013 at 04:06:19PM -0700, Andrew Jones wrote: > >> On Wed, Oct 16, 2013 at 10:03:37PM +0300, Michael S. Tsirkin wrote: > >> > This simply imports kvm-unittest git into qemu source tree. > >> > > >> > We can next work on making make check run it > >> > automatically. > >> > > >> > Squashed 'kvm-unittest/' content from commit 2bc0e29 > >> > > >> > git-subtree-dir: kvm-unittest > >> > git-subtree-split: 2bc0e29ee4447bebcd3b90053881f59265306adc > >> > > >> > Signed-off-by: Michael S. Tsirkin > >> > > >> > --- > >> > > >> > Gleb, Paolo, any objections to this? I really want a small guest for > >> > running ACPI tests during make check, and kvm-unittest seems to fit the > >> > bill. > >> > > >> > Ability to test e.g. PCI with this in-tree would be very benefitial. > >> > > >> > >> Sorry for slow reply, I just noticed this mail now. > >> > >> So far qemu is a dependency for kvm-unit-tests (both x86 and arm use it), > >> but at some point that could change (if another arch doesn't want to use > >> qemu). So if that happens, then it won't make much sense for the tests to > >> live in qemu. Thus I'm not 100% in favor of moving them. However, if the > >> consensus is to move them, then I have two comments on this patch. > >> > > Do not worry, we are far from such consensus :) > > I don't think kvm-unit-tests belong in QEMU either. They are tied to > the kernel side of things, not QEMU and many of the tests don't make > sense for TCG. > > If you want to execute guest code within QEMU's make check, we can > copy the basic infrastructure in kvm-unit-tests. It's so simple that > a "fork" isn't really much of a concern to me. > > Regards, > > Anthony Liguori For now I'm only testing the bios so it's even simpler, I just put 16 bit code in the boot sector. > >> 1) There are a couple pendings patches on the kvm list that tidy up the > >> kvm-unit-tests repo - removing lots of the files. That should be > >> committed first to avoid importing a bunch of files right before deleting > >> them. > >> > >> 2) The name shouldn't change from kvm-unit-tests to kvm-unittest. > >> > >> drew > > > > -- > > Gleb.