From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [Qemu-devel] [PATCH v3 1/3] target-i386: add pkeys support for cpuid handling Date: Thu, 19 Nov 2015 12:10:49 +0100 Message-ID: <564DAE39.3050203@redhat.com> References: <1447813217-10532-1-git-send-email-huaitong.han@intel.com> <1447813217-10532-2-git-send-email-huaitong.han@intel.com> <20151118155855.GI20436@thinpad.lan.raisama.net> <1447915017.3961.22.camel@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: "kvm@vger.kernel.org" , "qemu-devel@nongnu.org" , "rth@twiddle.net" , "afaerber@suse.de" To: "Han, Huaitong" , "ehabkost@redhat.com" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:47048 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756140AbbKSLK6 (ORCPT ); Thu, 19 Nov 2015 06:10:58 -0500 In-Reply-To: <1447915017.3961.22.camel@intel.com> Sender: kvm-owner@vger.kernel.org List-ID: On 19/11/2015 07:36, Han, Huaitong wrote: > I understand it has always been that QEMU considers the feature of > cpuid_7_0_ecx_feature_name as migratable. If the feature is > unmigratable, it will been added to unmigratable_flags. > > A series of patches do complete a full function, moving > cpuid_7_0_ecx_feature_name to 2/3 patch may make 2/3 patch look > better, but make 1/3 patch look somewhat incomplete. > > Maybe it is a solution that adding the feature to unmigratable_flags in > 1/3 patch, and deleting unmigratable_flags in 2/3 patch, but I think it > is pointless. Or just squash everything together. After all we're talking of 4 files changed, 55 insertions(+), 1 deletion(-) It's not a large patch. Paolo