From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [patch 0/2] do not run halted vcpu's Date: Sun, 17 Aug 2008 09:31:08 +0300 Message-ID: <48A7C5AC.1020402@qumranet.com> References: <20080801230912.013317832@localhost.localdomain> <48A2BB11.2020909@qumranet.com> <20080815001955.GB14844@dmt.cnet> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Marcelo Tosatti Return-path: Received: from il.qumranet.com ([212.179.150.194]:40339 "EHLO il.qumranet.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751331AbYHQGbI (ORCPT ); Sun, 17 Aug 2008 02:31:08 -0400 In-Reply-To: <20080815001955.GB14844@dmt.cnet> Sender: kvm-owner@vger.kernel.org List-ID: Marcelo Tosatti wrote: > KVM: x86: do not execute halted vcpus > > Alright, need_resched was actually checked in vcpu_enter_guest, so this > checks signal_pending too: > > Offline or uninitialized vcpu's can be executed if requested to perform > userspace work. > > Follow Avi's suggestion to handle halted vcpu's in the main loop, > simplifying kvm_emulate_halt(). Introduce a new vcpu->requests bit to > indicate events that promote state from halted to running. > > Also standardize vcpu wake sites. > Applied, thanks. -- I have a truly marvellous patch that fixes the bug which this signature is too narrow to contain.