From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joerg Roedel Subject: Re: [PATCH] kvm-vmx: add module parameter to avoid trapping HLT instructions (v3) Date: Sat, 4 Dec 2010 00:32:31 +0100 Message-ID: <20101203233231.GA29502@8bytes.org> References: <1291415962-13762-1-git-send-email-aliguori@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kvm@vger.kernel.org, Avi Kivity , Marcelo Tosatti , Chris Wright , Srivatsa Vaddagiri To: Anthony Liguori Return-path: Received: from 8bytes.org ([88.198.83.132]:54060 "EHLO 8bytes.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753448Ab0LCXcd (ORCPT ); Fri, 3 Dec 2010 18:32:33 -0500 Content-Disposition: inline In-Reply-To: <1291415962-13762-1-git-send-email-aliguori@us.ibm.com> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, Dec 03, 2010 at 04:39:22PM -0600, Anthony Liguori wrote: > + if (yield_on_hlt) > + min |= CPU_BASED_HLT_EXITING; This approach won't work out on AMD because in HLT the CPU may enter C1e. In C1e the local apic timer interupt is not delivered anymore and when this is the current timer in use the cpu may miss timer ticks or never comes out of HLT again. The guest has no chance to work around this as the Linux idle routine does. If you really wan't active idling of a guest, it should idle in the hypervisor where it can work around such problems. Joerg