From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 0/9 v3] KVM: Nested SVM fixes and tracepoint conversion Date: Mon, 12 Oct 2009 11:26:02 +0200 Message-ID: <4AD2F62A.5010704@redhat.com> References: <1255097313-23267-1-git-send-email-joerg.roedel@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , Alexander Graf , kvm@vger.kernel.org, linux-kernel@vger.kernel.org To: Joerg Roedel Return-path: In-Reply-To: <1255097313-23267-1-git-send-email-joerg.roedel@amd.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 10/09/2009 04:08 PM, Joerg Roedel wrote: > Hi Avi, Marcelo, > > this series of patches contains bugfixes for the Nested SVM code and the > conversion of Nested SVM debugging to tracepoints. The fixes are: > > 1) A patch Alex already sent (1/9) but which was not yet > applied. It fixes a lost event_inj problem when we emulate a > vmrun and a vmexit without entering the guest in the > meantime. > > 2) The patch 2/9 fixes a schedule() while atomic bug in the > Nested SVM code. The KVM interrupt injection code runs with > preemtion and interrupts disabled. But the > enable_irq_window() function from SVM may emulate a #vmexit. > This emulation might sleep which causes the schedule() while > atomic() bug. > > These fixes (patches 1 and 2) should also be considered for -stable > backporting. The patches 3 to 8 convert the old printk based debugging > for Nested SVM to tracepoints. Patch 9 removes the nsvm_printk code. > Please review and/or consider to apply these changes. > Looks good to me. -- error compiling committee.c: too many arguments to function