All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wen Congyang <wency@cn.fujitsu.com>
To: kvm list <kvm@vger.kernel.org>,
	qemu-devel <qemu-devel@nongnu.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Avi Kivity <avi@redhat.com>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	Gleb Natapov <gleb@redhat.com>
Subject: [PATCH 2/2 v3] kvm: set exit_reason to KVM_EXIT_GUEST_PANICKED when guest panicked
Date: Thu, 08 Mar 2012 16:04:28 +0800	[thread overview]
Message-ID: <4F58680C.2070007@cn.fujitsu.com> (raw)
In-Reply-To: <4F58664D.1070800@cn.fujitsu.com>

This patch introduces new request bit KVM_REQ_GUEST_PANICEKD.
If this bit is set, set vcpu's exit_reason to KVM_EXIT_GUEST_PANICKED.
And then the user space can know the guest panicked.

Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
---
 arch/x86/kvm/x86.c       |   11 +++++++++++
 include/linux/kvm.h      |    1 +
 include/linux/kvm_host.h |    1 +
 3 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index c9d99e5..978e398 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -5013,6 +5013,10 @@ int kvm_emulate_hypercall(struct kvm_vcpu *vcpu)
 	case KVM_HC_VAPIC_POLL_IRQ:
 		ret = 0;
 		break;
+	case KVM_HC_GUEST_PANICKED:
+		ret = 0;
+		kvm_make_request(KVM_REQ_GUEST_PANICKED, vcpu);
+		break;
 	default:
 		ret = -KVM_ENOSYS;
 		break;
@@ -5228,6 +5232,13 @@ static int vcpu_enter_guest(struct kvm_vcpu *vcpu)
 			r = 0;
 			goto out;
 		}
+
+		if (kvm_check_request(KVM_REQ_GUEST_PANICKED, vcpu)) {
+			vcpu->run->exit_reason = KVM_EXIT_GUEST_PANICKED;
+			r = 0;
+			goto out;
+		}
+
 		if (kvm_check_request(KVM_REQ_DEACTIVATE_FPU, vcpu)) {
 			vcpu->fpu_active = 0;
 			kvm_x86_ops->fpu_deactivate(vcpu);
diff --git a/include/linux/kvm.h b/include/linux/kvm.h
index acbe429..a46052e 100644
--- a/include/linux/kvm.h
+++ b/include/linux/kvm.h
@@ -163,6 +163,7 @@ struct kvm_pit_config {
 #define KVM_EXIT_OSI              18
 #define KVM_EXIT_PAPR_HCALL	  19
 #define KVM_EXIT_S390_UCONTROL	  20
+#define KVM_EXIT_GUEST_PANICKED	  21
 
 /* For KVM_EXIT_INTERNAL_ERROR */
 #define KVM_INTERNAL_ERROR_EMULATION 1
diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h
index 355e445..026bbe5 100644
--- a/include/linux/kvm_host.h
+++ b/include/linux/kvm_host.h
@@ -54,6 +54,7 @@
 #define KVM_REQ_IMMEDIATE_EXIT    15
 #define KVM_REQ_PMU               16
 #define KVM_REQ_PMI               17
+#define KVM_REQ_GUEST_PANICKED     18
 
 #define KVM_USERSPACE_IRQ_SOURCE_ID	0
 
-- 
1.7.1


WARNING: multiple messages have this Message-ID (diff)
From: Wen Congyang <wency@cn.fujitsu.com>
To: kvm list <kvm@vger.kernel.org>,
	qemu-devel <qemu-devel@nongnu.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Avi Kivity <avi@redhat.com>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	Gleb Natapov <gleb@redhat.com>
Subject: [Qemu-devel] [PATCH 2/2 v3] kvm: set exit_reason to KVM_EXIT_GUEST_PANICKED when guest panicked
Date: Thu, 08 Mar 2012 16:04:28 +0800	[thread overview]
Message-ID: <4F58680C.2070007@cn.fujitsu.com> (raw)
In-Reply-To: <4F58664D.1070800@cn.fujitsu.com>

This patch introduces new request bit KVM_REQ_GUEST_PANICEKD.
If this bit is set, set vcpu's exit_reason to KVM_EXIT_GUEST_PANICKED.
And then the user space can know the guest panicked.

Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
---
 arch/x86/kvm/x86.c       |   11 +++++++++++
 include/linux/kvm.h      |    1 +
 include/linux/kvm_host.h |    1 +
 3 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
index c9d99e5..978e398 100644
--- a/arch/x86/kvm/x86.c
+++ b/arch/x86/kvm/x86.c
@@ -5013,6 +5013,10 @@ int kvm_emulate_hypercall(struct kvm_vcpu *vcpu)
 	case KVM_HC_VAPIC_POLL_IRQ:
 		ret = 0;
 		break;
+	case KVM_HC_GUEST_PANICKED:
+		ret = 0;
+		kvm_make_request(KVM_REQ_GUEST_PANICKED, vcpu);
+		break;
 	default:
 		ret = -KVM_ENOSYS;
 		break;
@@ -5228,6 +5232,13 @@ static int vcpu_enter_guest(struct kvm_vcpu *vcpu)
 			r = 0;
 			goto out;
 		}
+
+		if (kvm_check_request(KVM_REQ_GUEST_PANICKED, vcpu)) {
+			vcpu->run->exit_reason = KVM_EXIT_GUEST_PANICKED;
+			r = 0;
+			goto out;
+		}
+
 		if (kvm_check_request(KVM_REQ_DEACTIVATE_FPU, vcpu)) {
 			vcpu->fpu_active = 0;
 			kvm_x86_ops->fpu_deactivate(vcpu);
diff --git a/include/linux/kvm.h b/include/linux/kvm.h
index acbe429..a46052e 100644
--- a/include/linux/kvm.h
+++ b/include/linux/kvm.h
@@ -163,6 +163,7 @@ struct kvm_pit_config {
 #define KVM_EXIT_OSI              18
 #define KVM_EXIT_PAPR_HCALL	  19
 #define KVM_EXIT_S390_UCONTROL	  20
+#define KVM_EXIT_GUEST_PANICKED	  21
 
 /* For KVM_EXIT_INTERNAL_ERROR */
 #define KVM_INTERNAL_ERROR_EMULATION 1
diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h
index 355e445..026bbe5 100644
--- a/include/linux/kvm_host.h
+++ b/include/linux/kvm_host.h
@@ -54,6 +54,7 @@
 #define KVM_REQ_IMMEDIATE_EXIT    15
 #define KVM_REQ_PMU               16
 #define KVM_REQ_PMI               17
+#define KVM_REQ_GUEST_PANICKED     18
 
 #define KVM_USERSPACE_IRQ_SOURCE_ID	0
 
-- 
1.7.1

  parent reply	other threads:[~2012-03-08  8:02 UTC|newest]

Thread overview: 206+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-08  7:57 [PATCH 0/2 v3] kvm: notify host when guest panicked Wen Congyang
2012-03-08  7:57 ` [Qemu-devel] " Wen Congyang
2012-03-08  8:02 ` [PATCH 1/2 " Wen Congyang
2012-03-08  8:02   ` [Qemu-devel] " Wen Congyang
2012-03-08  8:04 ` Wen Congyang [this message]
2012-03-08  8:04   ` [Qemu-devel] [PATCH 2/2 v3] kvm: set exit_reason to KVM_EXIT_GUEST_PANICKED " Wen Congyang
2012-03-08  8:06 ` [PATCH 1/2 v3] update linux-headers Wen Congyang
2012-03-08  8:06   ` [Qemu-devel] " Wen Congyang
2012-03-08  8:07 ` [PATCH 2/2 v3] deal with guest panicked event Wen Congyang
2012-03-08  8:07   ` [Qemu-devel] " Wen Congyang
2012-03-08 10:08   ` Jan Kiszka
2012-03-08 10:08     ` [Qemu-devel] " Jan Kiszka
2012-03-08 10:11     ` Wen Congyang
2012-03-08 10:11       ` [Qemu-devel] " Wen Congyang
2012-03-08 10:15   ` [RESEND][PATCH " Wen Congyang
2012-03-08 10:15     ` [Qemu-devel] " Wen Congyang
2012-03-08 11:28     ` Avi Kivity
2012-03-08 11:28       ` [Qemu-devel] " Avi Kivity
2012-03-08 11:36       ` Daniel P. Berrange
2012-03-08 11:36         ` [Qemu-devel] " Daniel P. Berrange
2012-03-08 11:52         ` Avi Kivity
2012-03-08 11:52           ` [Qemu-devel] " Avi Kivity
2012-03-08 11:56           ` Daniel P. Berrange
2012-03-08 11:56             ` [Qemu-devel] " Daniel P. Berrange
2012-03-08 11:56             ` Daniel P. Berrange
2012-03-09 22:22             ` Marcelo Tosatti
2012-03-09 22:22               ` [Qemu-devel] " Marcelo Tosatti
2012-03-21 19:01               ` Anthony Liguori
2012-03-21 19:01                 ` Anthony Liguori
2012-03-12  1:46             ` Wen Congyang
2012-03-12  1:46               ` [Qemu-devel] " Wen Congyang
2012-03-12  1:46               ` Wen Congyang
2012-03-08 11:13 ` [PATCH 0/2 v3] kvm: notify host when guest panicked Avi Kivity
2012-03-08 11:13   ` [Qemu-devel] " Avi Kivity
2012-03-08 11:13   ` Avi Kivity
2012-03-09  1:21   ` Wen Congyang
2012-03-09  1:21     ` [Qemu-devel] " Wen Congyang
2012-03-09  1:21     ` Wen Congyang
2012-03-12  9:04     ` Wen Congyang
2012-03-12  9:04       ` [Qemu-devel] " Wen Congyang
2012-03-12 10:33       ` Avi Kivity
2012-03-12 10:33         ` [Qemu-devel] " Avi Kivity
2012-03-13  6:44         ` Wen Congyang
2012-03-13  6:44           ` [Qemu-devel] " Wen Congyang
2012-03-13  8:54           ` Avi Kivity
2012-03-13  8:54             ` [Qemu-devel] " Avi Kivity
2012-03-13  9:18         ` Daniel P. Berrange
2012-03-13  9:18           ` [Qemu-devel] " Daniel P. Berrange
2012-03-13 10:47           ` Avi Kivity
2012-03-13 10:47             ` [Qemu-devel] " Avi Kivity
2012-03-13 10:47             ` Avi Kivity
2012-03-14  8:29             ` Wen Congyang
2012-03-14  8:29               ` [Qemu-devel] " Wen Congyang
2012-03-14  8:29               ` Wen Congyang
2012-03-14  9:24               ` Avi Kivity
2012-03-14  9:24                 ` [Qemu-devel] " Avi Kivity
2012-03-14  9:53                 ` Wen Congyang
2012-03-14  9:53                   ` [Qemu-devel] " Wen Congyang
2012-03-14 10:07                   ` Avi Kivity
2012-03-14 10:07                     ` [Qemu-devel] " Avi Kivity
2012-03-14 10:26                     ` Wen Congyang
2012-03-14 10:26                       ` [Qemu-devel] " Wen Congyang
2012-03-14 10:29                       ` Avi Kivity
2012-03-14 10:29                         ` [Qemu-devel] " Avi Kivity
2012-03-14 10:46                         ` Gleb Natapov
2012-03-14 10:46                           ` [Qemu-devel] " Gleb Natapov
2012-03-14 10:48                           ` Avi Kivity
2012-03-14 10:48                             ` [Qemu-devel] " Avi Kivity
2012-03-14 11:11                             ` Wen Congyang
2012-03-14 11:11                               ` [Qemu-devel] " Wen Congyang
2012-03-14 11:11                               ` Wen Congyang
2012-03-14 13:07                               ` Avi Kivity
2012-03-14 13:07                                 ` [Qemu-devel] " Avi Kivity
2012-03-14 13:13                                 ` Avi Kivity
2012-03-14 13:13                                   ` [Qemu-devel] " Avi Kivity
2012-03-14 13:14                                 ` Gleb Natapov
2012-03-14 13:14                                   ` [Qemu-devel] " Gleb Natapov
2012-03-14 13:16                                   ` Avi Kivity
2012-03-14 13:16                                     ` [Qemu-devel] " Avi Kivity
2012-03-14 13:25                                     ` Gleb Natapov
2012-03-14 13:25                                       ` [Qemu-devel] " Gleb Natapov
2012-03-14 18:46                                       ` Eric Northup
2012-03-14 18:46                                         ` [Qemu-devel] " Eric Northup
2012-03-15  7:01                                         ` Wen Congyang
2012-03-15  7:01                                           ` [Qemu-devel] " Wen Congyang
2012-03-15  7:01                                           ` Wen Congyang
2012-03-15 10:39                                         ` Gleb Natapov
2012-03-15 10:39                                           ` [Qemu-devel] " Gleb Natapov
2012-03-15 11:25                                           ` Jan Kiszka
2012-03-15 11:25                                             ` [Qemu-devel] " Jan Kiszka
2012-03-15 11:46                                             ` Avi Kivity
2012-03-15 11:46                                               ` [Qemu-devel] " Avi Kivity
2012-03-16  8:05                                               ` Wen Congyang
2012-03-16  8:05                                                 ` [Qemu-devel] " Wen Congyang
2012-03-21 19:12                                               ` Anthony Liguori
2012-03-21 19:12                                                 ` Anthony Liguori
2012-03-22  8:34                                                 ` Wen Congyang
2012-03-22  8:34                                                   ` Wen Congyang
2012-03-14 18:47                                       ` Eric Northup
2012-03-14 18:47                                         ` [Qemu-devel] " Eric Northup
2012-03-14 18:47                                         ` Eric Northup
2012-03-14 10:37                   ` Amit Shah
2012-03-14 10:37                     ` [Qemu-devel] " Amit Shah
2012-03-14 10:37                     ` Amit Shah
2012-03-14 10:52                     ` Wen Congyang
2012-03-14 10:52                       ` [Qemu-devel] " Wen Congyang
2012-03-14 10:52                       ` Gleb Natapov
2012-03-14 10:52                         ` [Qemu-devel] " Gleb Natapov
2012-03-14 10:57                         ` Wen Congyang
2012-03-14 10:57                           ` [Qemu-devel] " Wen Congyang
2012-03-14 10:58                           ` Gleb Natapov
2012-03-14 10:58                             ` [Qemu-devel] " Gleb Natapov
2012-03-14 11:13                             ` Wen Congyang
2012-03-14 11:13                               ` [Qemu-devel] " Wen Congyang
2012-03-14 11:13                               ` Wen Congyang
2012-03-14 10:52                       ` Avi Kivity
2012-03-14 10:52                         ` [Qemu-devel] " Avi Kivity
2012-03-14 10:58                         ` Wen Congyang
2012-03-14 10:58                           ` [Qemu-devel] " Wen Congyang
2012-03-14 10:59                           ` Daniel P. Berrange
2012-03-14 10:59                             ` [Qemu-devel] " Daniel P. Berrange
2012-03-14 11:06                             ` Wen Congyang
2012-03-14 11:06                               ` [Qemu-devel] " Wen Congyang
2012-03-14 11:11                               ` Gleb Natapov
2012-03-14 11:11                                 ` [Qemu-devel] " Gleb Natapov
2012-03-14 11:17                               ` Daniel P. Berrange
2012-03-14 11:17                                 ` [Qemu-devel] " Daniel P. Berrange
2012-03-14 10:59                           ` Gleb Natapov
2012-03-14 10:59                             ` [Qemu-devel] " Gleb Natapov
2012-03-14 10:57                       ` Amit Shah
2012-03-14 10:57                         ` [Qemu-devel] " Amit Shah
2012-03-14  9:51               ` Amit Shah
2012-03-14  9:51                 ` [Qemu-devel] " Amit Shah
2012-03-14 10:04                 ` Wen Congyang
2012-03-14 10:04                   ` [Qemu-devel] " Wen Congyang
2012-03-14 10:08                   ` Avi Kivity
2012-03-14 10:08                     ` [Qemu-devel] " Avi Kivity
2012-03-14 10:08                     ` Avi Kivity
2012-03-14 10:40                   ` Amit Shah
2012-03-14 10:40                     ` [Qemu-devel] " Amit Shah
2012-03-14 10:42                     ` Gleb Natapov
2012-03-14 10:42                       ` [Qemu-devel] " Gleb Natapov
2012-03-14 10:57                 ` Daniel P. Berrange
2012-03-14 10:57                   ` [Qemu-devel] " Daniel P. Berrange
2012-03-14 10:57                   ` Daniel P. Berrange
2012-03-14 11:01                   ` Wen Congyang
2012-03-14 11:01                     ` [Qemu-devel] " Wen Congyang
2012-03-14 11:01                     ` Wen Congyang
2012-03-21 19:04             ` [Qemu-devel] " Anthony Liguori
2012-03-21 19:04               ` Anthony Liguori
2012-03-22  7:33               ` Gleb Natapov
2012-03-22  7:33                 ` Gleb Natapov
2012-03-12 10:31     ` Avi Kivity
2012-03-12 10:31       ` [Qemu-devel] " Avi Kivity
2012-03-19  7:33 ` Wen Congyang
2012-03-19  7:33   ` [Qemu-devel] " Wen Congyang
2012-03-20  9:59   ` Wen Congyang
2012-03-20  9:59     ` [Qemu-devel] " Wen Congyang
2012-03-20 15:45     ` Gleb Natapov
2012-03-20 15:45       ` [Qemu-devel] " Gleb Natapov
2012-03-21  0:56       ` Wen Congyang
2012-03-21  0:56         ` [Qemu-devel] " Wen Congyang
2012-03-21  9:11         ` Gleb Natapov
2012-03-21  9:11           ` [Qemu-devel] " Gleb Natapov
2012-03-21  9:35           ` Wen Congyang
2012-03-21  9:35             ` [Qemu-devel] " Wen Congyang
2012-03-21  9:35             ` Wen Congyang
2012-03-21  9:42             ` Gleb Natapov
2012-03-21  9:42               ` [Qemu-devel] " Gleb Natapov
2012-03-21 16:18           ` Corey Minyard
2012-03-21 16:18             ` [Qemu-devel] " Corey Minyard
2012-03-21 16:24             ` Gleb Natapov
2012-03-21 16:24               ` [Qemu-devel] " Gleb Natapov
2012-03-21 16:24               ` Gleb Natapov
2012-03-21 16:25             ` Avi Kivity
2012-03-21 16:25               ` [Qemu-devel] " Avi Kivity
2012-03-21 17:04               ` Daniel P. Berrange
2012-03-21 17:04                 ` [Qemu-devel] " Daniel P. Berrange
2012-03-21 17:34                 ` Avi Kivity
2012-03-21 17:34                   ` [Qemu-devel] " Avi Kivity
2012-03-21 18:17                   ` Jan Kiszka
2012-03-21 18:17                     ` [Qemu-devel] " Jan Kiszka
2012-03-21 18:17                     ` Jan Kiszka
2012-03-21 19:19               ` [Qemu-devel] " Anthony Liguori
2012-03-21 19:19                 ` Anthony Liguori
2012-03-22  1:05                 ` Wen Congyang
2012-03-22  1:05                   ` [Qemu-devel] " Wen Congyang
2012-03-22  1:05                   ` Wen Congyang
2012-03-22  7:31                   ` Gleb Natapov
2012-03-22  7:31                     ` Gleb Natapov
2012-03-22  7:44                     ` Wen Congyang
2012-03-22  7:44                       ` Wen Congyang
2012-03-22  8:36                       ` Gleb Natapov
2012-03-22  8:36                         ` [Qemu-devel] " Gleb Natapov
2012-03-22  8:36                         ` Gleb Natapov
2012-03-22  7:28                 ` Gleb Natapov
2012-03-22  7:28                   ` [Qemu-devel] " Gleb Natapov
2012-03-22  7:28                   ` Gleb Natapov
2012-03-22  7:40                   ` Wen Congyang
2012-03-22  7:40                     ` Wen Congyang
2012-04-17  3:14       ` Wen Congyang
2012-04-02 10:05   ` Wen Congyang
2012-04-02 10:05     ` [Qemu-devel] " Wen Congyang
2012-04-02 10:54     ` Amit Shah
2012-04-02 10:54       ` [Qemu-devel] " Amit Shah
2012-04-02 10:54       ` Amit Shah

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F58680C.2070007@cn.fujitsu.com \
    --to=wency@cn.fujitsu.com \
    --cc=avi@redhat.com \
    --cc=berrange@redhat.com \
    --cc=gleb@redhat.com \
    --cc=jan.kiszka@siemens.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.