From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH] Fixup for patch 5 Date: Fri, 31 Jan 2014 15:58:38 +0100 Message-ID: <52EBBA1E.6010000@redhat.com> References: <52EBA166.5060805@redhat.com> <1391175166-47032-1-git-send-email-borntraeger@de.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Cc: KVM , linux-s390 , Cornelia Huck , Alexander Graf To: Christian Borntraeger , Gleb Natapov Return-path: Received: from mx1.redhat.com ([209.132.183.28]:31750 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754335AbaAaO6u (ORCPT ); Fri, 31 Jan 2014 09:58:50 -0500 In-Reply-To: <1391175166-47032-1-git-send-email-borntraeger@de.ibm.com> Sender: kvm-owner@vger.kernel.org List-ID: Il 31/01/2014 14:32, Christian Borntraeger ha scritto: > This fixup fixes patch 5 and makes it equivalent to the code that > went through testing. Looks like the change from patch 5 does > not cause real problems. x86 will simply inject the completion > via kvm_arch_async_page_present in kvm_check_async_pf_completion. > s390 will inject twice (sync in execute but also async), but the > guest OS can handle that. (thats why I did not catch this in > my regression test after rebasing). So the only visible effect is > in the counters and in performance. We can handle this as an addon > patch Good! I will pull and apply this patch as soon as -rc1 is out. Paolo