From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] use upstream cpuid code Date: Mon, 03 Aug 2009 14:57:47 +0300 Message-ID: <4A76D0BB.3080000@redhat.com> References: <1248811515-6877-1-git-send-email-glommer@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Glauber Costa Return-path: Received: from mx2.redhat.com ([66.187.237.31]:39155 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752344AbZHCLwc (ORCPT ); Mon, 3 Aug 2009 07:52:32 -0400 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n73BqWbX018328 for ; Mon, 3 Aug 2009 07:52:32 -0400 In-Reply-To: <1248811515-6877-1-git-send-email-glommer@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 07/28/2009 11:05 PM, Glauber Costa wrote: > use cpuid code from upstream. By doing that, we lose the following snippet > in kvm_get_supported_cpuid(): > > ret |= 1<< 12; /* MTRR */ > ret |= 1<< 16; /* PAT */ > ret |= 1<< 7; /* MCE */ > ret |= 1<< 14; /* MCA */ > > A quick search in mailing lists says this code is not really necessary, and we're > keeping it just for backwards compatibility. This is not that important, because > we'd lose it anyway in the golden day in which we totally merge with qemu. > Anyway, if it do _is_ important, we can send a patch to qemu with it. > It is important. Please don't introduce regressions (if you do, introduce them in separate patches). The procedure to drop such workarounds for kernel bugs is to verify that major distros have the kernel fixes in their supported kernels. -- error compiling committee.c: too many arguments to function