From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: Kernel GPF in vmx_save_host_state() Date: Sun, 22 Mar 2009 11:08:33 +0200 Message-ID: <49C60011.8030405@redhat.com> References: <49C0313C.5010906@cs.cmu.edu> <49C0B702.1060807@redhat.com> <49C12ED2.4080209@cs.cmu.edu> <49C20ED4.2040700@redhat.com> <49C2B289.7030707@cs.cmu.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Benjamin Gilbert Return-path: Received: from mx2.redhat.com ([66.187.237.31]:46685 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752355AbZCVJIq (ORCPT ); Sun, 22 Mar 2009 05:08:46 -0400 In-Reply-To: <49C2B289.7030707@cs.cmu.edu> Sender: kvm-owner@vger.kernel.org List-ID: Benjamin Gilbert wrote: > Avi Kivity wrote: >> Well, that's certainly the wrmsr instruction. But I don't see how >> this can happen. >> >> Can you patch set_efer() in x86.c to print the value of the efer >> argument and of efer_reserved_bits? > > Yes, but apparently set_efer() is never called. To verify, I patched > kvm_set_msr_common() to print the msr parameter: > On 32-bit, we might actually reach the default: label of the switch in vmx_set_msr(). Can you add a printk() there? print both msr_index, and, if msr is not NULL, msr->index and msr->data. -- error compiling committee.c: too many arguments to function