From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu Date: Wed, 23 Jun 2010 16:26:36 +0300 Message-ID: <4C220B8C.8070306@redhat.com> References: <20100323163709.033115262@redhat.com> <20100323163812.726785030@redhat.com> <4C21FE28.1040704@redhat.com> <4C21FF20.4030706@redhat.com> <4C2209F2.6030109@siemens.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , kvm@vger.kernel.org To: Jan Kiszka Return-path: Received: from mx1.redhat.com ([209.132.183.28]:64301 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751515Ab0FWN0j (ORCPT ); Wed, 23 Jun 2010 09:26:39 -0400 In-Reply-To: <4C2209F2.6030109@siemens.com> Sender: kvm-owner@vger.kernel.org List-ID: On 06/23/2010 04:19 PM, Jan Kiszka wrote: > >>> I'm testing uq/master, and I see this breaks it. qemu freezes >>> immediately on startup. >>> >>> (might be due to a rebase?) >>> >>> >> The symptoms, btw, are a vcpu started from 0:0 instead of 0xf000:0xfff0. >> >> > Might be unrelated, still: Does [1] make any difference? No, and this is not surprising - looks like reset is completely bypassed. > (Upsream is a > bit hairy ATM.) > Well, it boots at least, which uq/master doesn't with this patch. -- error compiling committee.c: too many arguments to function