From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 1/2] x86: eliminate TS_XSAVE Date: Tue, 04 May 2010 10:41:19 +0300 Message-ID: <4BDFCF9F.3030600@redhat.com> References: <1272812038-32484-1-git-send-email-avi@redhat.com> <1272812038-32484-2-git-send-email-avi@redhat.com> <4BDDBA18.3080909@redhat.com> <4BDF4407.8000503@zytor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Suresh Siddha , Brian Gerst , Dexuan Cui , Sheng Yang , Ingo Molnar , linux-kernel@vger.kernel.org, kvm@vger.kernel.org To: "H. Peter Anvin" Return-path: In-Reply-To: <4BDF4407.8000503@zytor.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 05/04/2010 12:45 AM, H. Peter Anvin wrote: > >> I was trying to avoid a performance regression relative to the current >> code, as it appears that some care was taken to avoid the memory reference. >> >> I agree that it's probably negligible compared to the save/restore >> code. If the x86 maintainers agree as well, I'll replace it with >> cpu_has_xsave. >> >> > I asked Suresh to comment on this, since he wrote the original code. He > did confirm that the intent was to avoid a global memory reference. > > Ok, so you're happy with the patch as is? -- error compiling committee.c: too many arguments to function