From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v2 17/17] kvm: Drop dependencies on very old capabilities Date: Mon, 03 Jan 2011 19:01:55 +0200 Message-ID: <4D220103.5070707@redhat.com> References: <4ffa4f23bc93aa5af90d836986771bb6d9856bf9.1294043582.git.jan.kiszka@web.de> <4D21F464.1070807@redhat.com> <4D21FF48.70103@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , kvm@vger.kernel.org, qemu-devel@nongnu.org, Jan Kiszka To: Jan Kiszka Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57888 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750995Ab1ACRCC (ORCPT ); Mon, 3 Jan 2011 12:02:02 -0500 In-Reply-To: <4D21FF48.70103@web.de> Sender: kvm-owner@vger.kernel.org List-ID: On 01/03/2011 06:54 PM, Jan Kiszka wrote: > Am 03.01.2011 17:08, Avi Kivity wrote: > > On 01/03/2011 10:33 AM, Jan Kiszka wrote: > >> From: Jan Kiszka > >> > >> COALESCED_MMIO, SYNC_MMU, EXT_CPUID, CLOCKSOURCE, NOP_IO_DELAY, PV_MMU - > >> all these caps predate features on which we already depend at build > >> time. Moreover, the check for KVM_CAP_EXT_CPUID is unneeded as we > >> already test& fail is a more recent feature is missing. > > > > No. Each test documents a dependency of qemu on a kvm feature. Even > > though something like SYNC_MMU is unlikely to go away, as long as we > > depend on it, we require the feature. > > > > Then at least move all those KVM_CAPs we need at build time into > configure. Need a run time check as well (build on new kernel, run on old kernel, or run on even newer kernel that lost a feature). > I really see no value in keeping ugly conditional code > around, A) because those paths won't be tested and B) none of the CAPs > touched here are to pass away without a replacement that will require > user space adaption anyway. I'm fine with a series of checks during init time with no fallback. I'm not fine with just dropping those away. Reducing code size is great, but not at the cost of undiagnosed runtime failures. -- error compiling committee.c: too many arguments to function