From mboxrd@z Thu Jan 1 00:00:00 1970 From: Radim =?utf-8?B?S3LEjW3DocWZ?= Subject: Re: [kvm-unit-tests PATCH] kvm-unit-tests: Gtests Framework and Example Tests Date: Thu, 1 Feb 2018 16:51:57 +0100 Message-ID: <20180201155156.GA27896@flask> References: <20180125184803.228086-1-hofsass@google.com> <20180130160924.GD22015@flask> <20180131085601.4yhk6tc3kqq2vv3t@kamzik.brq.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Cc: Andrew Jones , Paolo Bonzini , Jim Mattson , kvm@vger.kernel.org To: Ken Hofsass Return-path: Received: from mx1.redhat.com ([209.132.183.28]:60520 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751706AbeBAPwL (ORCPT ); Thu, 1 Feb 2018 10:52:11 -0500 Content-Disposition: inline In-Reply-To: <20180131085601.4yhk6tc3kqq2vv3t@kamzik.brq.redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: 2018-01-31 09:56+0100, Andrew Jones: > On Tue, Jan 30, 2018 at 10:28:34AM -0800, Ken Hofsass wrote: > > Thanks for looking at this Radim, > > > > On Tue, Jan 30, 2018 at 8:09 AM, Radim Krčmář wrote: > > > 2018-01-25 10:48-0800, Ken Hofsass: > > >> Test framework for KVM and the Linux kernel that enables standalone > > > .... > > > This email didn't make it to the kvm list. Might be because it is too > > > big, please see what the server said and potentially resend after > > > splitting into parts, thanks. > > > > OK. The gtests framework code doesn't readily break into smaller > > functionally independent patches. I will break it into patches that > > separately add the large files and then have the last patch update the > > Makefiles/etc, so the project still builds after each patch. (If > > that's not acceptable, please recommend an alternative.) Sounds good, thanks. > > > (Also, doesn't compile here due to missing #include > > > and I didn't look further.) > > > > My apologies. Please try "KERNEL= make" > > > > The gtests code needs the actual KVM/kernel headers for the API/ABI > > details. I should have added a check for $KERNEL before building > > gtests. If you have a different recommendation for referencing the > > kernel sources from the kvm-unit-test directory, let me know. > > > > We've been copying the kernel files we need (see lib/linux/), or even > just cherry-picking the defines we need from linux headers and dropping > them into their respective kvm-unit-tests' lib/$ARCH/asm/ headers. Is > there some reason you've opted not to do that? > > On a historical note, kvm-unit-tests used to look in kernel sources for > headers, but we decided it would be more useful for the test suite to > remove the dependency. That was pre 'make standalone' though. We could > start a discussion about how much we need kvm-unit-tests to _build_ > without the dependency now, as one can always build standalone tests > on a build machine and then easily copy them to test machines. Yeah, I'd just copy the file(s) for now.