From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: Re: [PATCH 00/18] [uq/master] MCE & IO exit fixes, prepare for VCPU loop reuse Date: Mon, 10 Jan 2011 11:10:43 +0100 Message-ID: <4D2ADB23.1090505@siemens.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, qemu-devel@nongnu.org To: Avi Kivity , Marcelo Tosatti Return-path: Received: from thoth.sbs.de ([192.35.17.2]:22929 "EHLO thoth.sbs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752895Ab1AJKLB (ORCPT ); Mon, 10 Jan 2011 05:11:01 -0500 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: Am 10.01.2011 09:31, Jan Kiszka wrote: > This series has three major topics: > - add required kernel reentry after IO exits > - provide MCE forwarding under !CONFIG_IOTHREAD > - prepare kvm_cpu_exec for qemu-kvm reuse > > Along these lines, several cleanups and simplifcations are applied to > cpus.c and the KVM VCPU execution bits. The first patch should of course > be dropped if uq/master is applied without "kvm: Drop return value of > kvm_cpu_exec". > > Note that I did not yet to test the MCE support. Is there an easy way to > trigger valid MCE events at kernel level? Or do I need to fake SIGBUS at > qemu level? > > Jan Kiszka (18): > Revert "kvm: Drop return value of kvm_cpu_exec" > kvm: Drop redundant kvm_enabled from kvm_cpu_thread_fn > kvm: Provide sigbus services arch-independently > Refactor signal setup functions in cpus.c > kvm: Set up signal mask also for !CONFIG_IOTHREAD > kvm: Refactor qemu_kvm_eat_signals > kvm: Add MCE signal support for !CONFIG_IOTHREAD > kvm: Handle kvm_init_vcpu errors Ugh, I did it again: Forgot to test this "trivial" patch. Promptly, it turned out to be buggy, and it also revealed some potential issues in the kvm init code. Need to look into this later. The rest of the series is unaffected, though. Jan > Refactor kvm&tcg function names in cpus.c > Fix a few coding style violations in cpus.c > Introduce VCPU self-signaling service > kvm: Move irqchip event processing out of inner loop > kvm: Unconditionally reenter kernel after IO exits > kvm: Remove static return code of kvm_handle_io > kvm: Leave kvm_cpu_exec directly after KVM_EXIT_SHUTDOWN > kvm: Separate TCG from KVM cpu execution > kvm: x86: Prepare VCPU loop for in-kernel irqchip > kvm: Drop return values from kvm_arch_pre/post_run > > cpu-exec.c | 19 +-- > cpus.c | 521 +++++++++++++++++++++++++++++++--------------------- > kvm-all.c | 76 +++++---- > kvm-stub.c | 9 +- > kvm.h | 14 +- > qemu-common.h | 1 + > target-i386/kvm.c | 90 +++++----- > target-ppc/kvm.c | 16 ++- > target-s390x/kvm.c | 16 ++- > 9 files changed, 444 insertions(+), 318 deletions(-) > -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=36113 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PcEi8-0004KO-TU for qemu-devel@nongnu.org; Mon, 10 Jan 2011 05:10:50 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PcEi7-0008FV-LR for qemu-devel@nongnu.org; Mon, 10 Jan 2011 05:10:48 -0500 Received: from thoth.sbs.de ([192.35.17.2]:22833) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PcEi7-0008EN-AZ for qemu-devel@nongnu.org; Mon, 10 Jan 2011 05:10:47 -0500 Message-ID: <4D2ADB23.1090505@siemens.com> Date: Mon, 10 Jan 2011 11:10:43 +0100 From: Jan Kiszka MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 00/18] [uq/master] MCE & IO exit fixes, prepare for VCPU loop reuse List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity , Marcelo Tosatti Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org Am 10.01.2011 09:31, Jan Kiszka wrote: > This series has three major topics: > - add required kernel reentry after IO exits > - provide MCE forwarding under !CONFIG_IOTHREAD > - prepare kvm_cpu_exec for qemu-kvm reuse > > Along these lines, several cleanups and simplifcations are applied to > cpus.c and the KVM VCPU execution bits. The first patch should of course > be dropped if uq/master is applied without "kvm: Drop return value of > kvm_cpu_exec". > > Note that I did not yet to test the MCE support. Is there an easy way to > trigger valid MCE events at kernel level? Or do I need to fake SIGBUS at > qemu level? > > Jan Kiszka (18): > Revert "kvm: Drop return value of kvm_cpu_exec" > kvm: Drop redundant kvm_enabled from kvm_cpu_thread_fn > kvm: Provide sigbus services arch-independently > Refactor signal setup functions in cpus.c > kvm: Set up signal mask also for !CONFIG_IOTHREAD > kvm: Refactor qemu_kvm_eat_signals > kvm: Add MCE signal support for !CONFIG_IOTHREAD > kvm: Handle kvm_init_vcpu errors Ugh, I did it again: Forgot to test this "trivial" patch. Promptly, it turned out to be buggy, and it also revealed some potential issues in the kvm init code. Need to look into this later. The rest of the series is unaffected, though. Jan > Refactor kvm&tcg function names in cpus.c > Fix a few coding style violations in cpus.c > Introduce VCPU self-signaling service > kvm: Move irqchip event processing out of inner loop > kvm: Unconditionally reenter kernel after IO exits > kvm: Remove static return code of kvm_handle_io > kvm: Leave kvm_cpu_exec directly after KVM_EXIT_SHUTDOWN > kvm: Separate TCG from KVM cpu execution > kvm: x86: Prepare VCPU loop for in-kernel irqchip > kvm: Drop return values from kvm_arch_pre/post_run > > cpu-exec.c | 19 +-- > cpus.c | 521 +++++++++++++++++++++++++++++++--------------------- > kvm-all.c | 76 +++++---- > kvm-stub.c | 9 +- > kvm.h | 14 +- > qemu-common.h | 1 + > target-i386/kvm.c | 90 +++++----- > target-ppc/kvm.c | 16 ++- > target-s390x/kvm.c | 16 ++- > 9 files changed, 444 insertions(+), 318 deletions(-) > -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux