From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH/RFC] KVM: track pid for VCPU only on KVM_RUN ioctl Date: Tue, 19 Aug 2014 12:31:41 +0200 Message-ID: <53F3278D.9070107@redhat.com> References: <1407249854-2953-1-git-send-email-borntraeger@de.ibm.com> <53E38193.8060206@redhat.com> <53F30CF3.5030606@de.ibm.com> <53F3187E.6080706@redhat.com> <53F31D35.4010901@de.ibm.com> <53F31E9F.9020402@redhat.com> <53F32005.2060902@de.ibm.com> <53F320E8.3040502@redhat.com> <53F32267.6090106@de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: KVM , Gleb Natapov , Rik van Riel , Raghavendra K T , Michael Mueller To: Christian Borntraeger Return-path: Received: from mx1.redhat.com ([209.132.183.28]:17489 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750721AbaHSKbu (ORCPT ); Tue, 19 Aug 2014 06:31:50 -0400 In-Reply-To: <53F32267.6090106@de.ibm.com> Sender: kvm-owner@vger.kernel.org List-ID: Il 19/08/2014 12:09, Christian Borntraeger ha scritto: >> I'm not sure, this does seem like a workaround for another >> limitation after all... Gleb? > > Yes. We want to get rid of KVM_S390_INITIAL_RESET in QEMU. This comes > from a time, when we had another userspace prototype for KVM on s390 > (kuli). Its really a wart that has to go. Its just that we are not > there yet to remove the call to KVM_S390_INITIAL_RESET. Doing so can > result in hard to debug errors after reboot, if an interrupt was made > pending just before reboot that gets delivered in the new instance. > > The new way for local interrupt read/write will probably be some > onereg or syncreg interface with a bitmask register and payload > registers. We have to solve some concurrency and implemenation issues > here. Yes, I understand; the plan is fine and it's good that it was already on your todo list. But since you acknowledge that KVM_S390_INITIAL_RESET will go, I'm not sure we want to apply this patch (except for the pid == 0 part, of course---that one is good). Paolo