From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: guest reboot error Date: Fri, 28 Sep 2007 16:38:00 +0200 Message-ID: <46FD11C8.8040701@qumranet.com> References: <10EA09EFD8728347A513008B6B0DA77A022AE441@pdsmsx411.ccr.corp.intel.com><10EA09EFD8728347A513008B6B0DA77A022AE445@pdsmsx411.ccr.corp.intel.com><46FB8A65.6000504@qumranet.com> <10EA09EFD8728347A513008B6B0DA77A022AE67A@pdsmsx411.ccr.corp.intel.com> <10EA09EFD8728347A513008B6B0DA77A022AE9B4@pdsmsx411.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel To: "Dong, Eddie" Return-path: In-Reply-To: <10EA09EFD8728347A513008B6B0DA77A022AE9B4-wq7ZOvIWXbNpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org Dong, Eddie wrote: >> BTW, I downloaded KVM-36 with REHL5U 32e guest, reboot doesn't work >> either. But if I use no-kvm option, it can. >> So I would like to know which release is the latest known working >> tree for "reboot". >> >> > More update: > Using today's master, without any patch, reboot works!!! > It could be explained that at reboot time, the eflag.IF is > cleared, > so no matter how PIC/APIC beahivors at that stage, it doesn't matter. > Later on, guest will reset those chips and then keep forwarding... > Some other patches fixed the issue from KVM-36, or anybody has more > finding? > > It is good that I don't need to push the kernel device reset > patch out that soon, > but we still need to do so for a graceful reset especially kernel > devices need to be reseted since > Yes, that patch is needed. I thought of another way to do it: have a single vm ioctl reset, which can set a reset bit in all vcpu->requests and then kick the vcpus. When the vcpus execute, they'll check the bit and reset the cpu and lapic then; no locking needed. -- Any sufficiently difficult bug is indistinguishable from a feature. ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/