All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hu Tao <hutao@cn.fujitsu.com>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: kvm list <kvm@vger.kernel.org>,
	qemu-devel <qemu-devel@nongnu.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Gleb Natapov <gleb@redhat.com>, Blue Swirl <blauwirbel@gmail.com>,
	Eric Blake <eblake@redhat.com>, Andrew Jones <drjones@redhat.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	Sasha Levin <levinsasha928@gmail.com>,
	Luiz Capitulino <lcapitulino@redhat.com>,
	Anthony Liguori <aliguori@us.ibm.com>,
	Markus Armbruster <armbru@redhat.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Stefan Hajnoczi <stefanha@redhat.com>,
	Juan Quintela <quintela@redhat.com>,
	Orit Wasserman <owasserm@redhat.com>,
	Kevin Wolf <kwolf@redhat.com>,
	Wen Congyang <wency@cn.fujitsu.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Alexander Graf <agraf@suse.de>,
	Alex Williamson <alex.will
Subject: Re: [PATCH v13 2/8] start vm after resetting it
Date: Tue, 5 Mar 2013 11:05:57 +0800	[thread overview]
Message-ID: <20130305030557.GL16362@localhost.localdomain> (raw)
In-Reply-To: <512F5A5E.9020504@siemens.com>

On Thu, Feb 28, 2013 at 02:23:42PM +0100, Jan Kiszka wrote:
> On 2013-02-28 13:13, Hu Tao wrote:
> > From: Wen Congyang <wency@cn.fujitsu.com>
> > 
> > The guest should run after resetting it, but it does not run if its
> > old state is RUN_STATE_INTERNAL_ERROR or RUN_STATE_PAUSED.
> > 
> > We don't set runstate to RUN_STATE_PAUSED when resetting the guest,
> > so the runstate will be changed from RUN_STATE_INTERNAL_ERROR or
> > RUN_STATE_PAUSED to RUN_STATE_RUNNING(not RUN_STATE_PAUSED).
> 
> I just wonder what will happen if I interrupted the guest via gdb and
> then issue "monitor system_reset", also via gdb - common pattern if you
> set a breakpoint on some BUG() or fault handler and then want to restart
> the guest. Will the guest continue then while gdb thinks it is still
> stopped? Likely, we do not differentiate between gdb-initiated stops and
> the rest. Could you clarify?

Guest won't continue unless issue gdb "continue". Anyway, I'll seperate
this patch, as Paolo requested.

WARNING: multiple messages have this Message-ID (diff)
From: Hu Tao <hutao@cn.fujitsu.com>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: kvm list <kvm@vger.kernel.org>,
	qemu-devel <qemu-devel@nongnu.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Gleb Natapov <gleb@redhat.com>, Blue Swirl <blauwirbel@gmail.com>,
	Eric Blake <eblake@redhat.com>, Andrew Jones <drjones@redhat.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	Sasha Levin <levinsasha928@gmail.com>,
	Luiz Capitulino <lcapitulino@redhat.com>,
	Anthony Liguori <aliguori@us.ibm.com>,
	Markus Armbruster <armbru@redhat.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Stefan Hajnoczi <stefanha@redhat.com>,
	Juan Quintela <quintela@redhat.com>,
	Orit Wasserman <owasserm@redhat.com>,
	Kevin Wolf <kwolf@redhat.com>,
	Wen Congyang <wency@cn.fujitsu.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Alexander Graf <agraf@suse.de>,
	Alex Williamson <alex.williamson@redhat.com>,
	Peter Maydell <peter.maydell@linaro.org>
Subject: Re: [PATCH v13 2/8] start vm after resetting it
Date: Tue, 5 Mar 2013 11:05:57 +0800	[thread overview]
Message-ID: <20130305030557.GL16362@localhost.localdomain> (raw)
In-Reply-To: <512F5A5E.9020504@siemens.com>

On Thu, Feb 28, 2013 at 02:23:42PM +0100, Jan Kiszka wrote:
> On 2013-02-28 13:13, Hu Tao wrote:
> > From: Wen Congyang <wency@cn.fujitsu.com>
> > 
> > The guest should run after resetting it, but it does not run if its
> > old state is RUN_STATE_INTERNAL_ERROR or RUN_STATE_PAUSED.
> > 
> > We don't set runstate to RUN_STATE_PAUSED when resetting the guest,
> > so the runstate will be changed from RUN_STATE_INTERNAL_ERROR or
> > RUN_STATE_PAUSED to RUN_STATE_RUNNING(not RUN_STATE_PAUSED).
> 
> I just wonder what will happen if I interrupted the guest via gdb and
> then issue "monitor system_reset", also via gdb - common pattern if you
> set a breakpoint on some BUG() or fault handler and then want to restart
> the guest. Will the guest continue then while gdb thinks it is still
> stopped? Likely, we do not differentiate between gdb-initiated stops and
> the rest. Could you clarify?

Guest won't continue unless issue gdb "continue". Anyway, I'll seperate
this patch, as Paolo requested.

WARNING: multiple messages have this Message-ID (diff)
From: Hu Tao <hutao@cn.fujitsu.com>
To: Jan Kiszka <jan.kiszka@siemens.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	kvm list <kvm@vger.kernel.org>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	Markus Armbruster <armbru@redhat.com>,
	Blue Swirl <blauwirbel@gmail.com>,
	Orit Wasserman <owasserm@redhat.com>,
	Gleb Natapov <gleb@redhat.com>,
	Juan Quintela <quintela@redhat.com>,
	Alexander Graf <agraf@suse.de>, Andrew Jones <drjones@redhat.com>,
	Alex Williamson <alex.williamson@redhat.com>,
	Sasha Levin <levinsasha928@gmail.com>,
	Stefan Hajnoczi <stefanha@redhat.com>,
	Luiz Capitulino <lcapitulino@redhat.com>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Kevin Wolf <kwolf@redhat.com>,
	Anthony Liguori <aliguori@us.ibm.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v13 2/8] start vm after resetting it
Date: Tue, 5 Mar 2013 11:05:57 +0800	[thread overview]
Message-ID: <20130305030557.GL16362@localhost.localdomain> (raw)
In-Reply-To: <512F5A5E.9020504@siemens.com>

On Thu, Feb 28, 2013 at 02:23:42PM +0100, Jan Kiszka wrote:
> On 2013-02-28 13:13, Hu Tao wrote:
> > From: Wen Congyang <wency@cn.fujitsu.com>
> > 
> > The guest should run after resetting it, but it does not run if its
> > old state is RUN_STATE_INTERNAL_ERROR or RUN_STATE_PAUSED.
> > 
> > We don't set runstate to RUN_STATE_PAUSED when resetting the guest,
> > so the runstate will be changed from RUN_STATE_INTERNAL_ERROR or
> > RUN_STATE_PAUSED to RUN_STATE_RUNNING(not RUN_STATE_PAUSED).
> 
> I just wonder what will happen if I interrupted the guest via gdb and
> then issue "monitor system_reset", also via gdb - common pattern if you
> set a breakpoint on some BUG() or fault handler and then want to restart
> the guest. Will the guest continue then while gdb thinks it is still
> stopped? Likely, we do not differentiate between gdb-initiated stops and
> the rest. Could you clarify?

Guest won't continue unless issue gdb "continue". Anyway, I'll seperate
this patch, as Paolo requested.

  reply	other threads:[~2013-03-05  3:05 UTC|newest]

Thread overview: 136+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-28 12:13 [PATCH v13 0/8] pv event interface between host and guest Hu Tao
2013-02-28 12:13 ` Hu Tao
2013-02-28 12:13 ` [PATCH v13] kvm: notify host when the guest is panicked Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-02-28 12:13 ` [PATCH v13 1/8] save/load cpu runstate Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-02-28 21:12   ` Eric Blake
2013-02-28 21:12     ` Eric Blake
2013-03-01  7:36     ` Hu Tao
2013-03-01  7:36       ` [Qemu-devel] " Hu Tao
2013-03-01  7:36       ` Hu Tao
2013-03-01 16:29       ` Eric Blake
2013-03-01 16:29         ` [Qemu-devel] " Eric Blake
2013-03-01 16:29         ` Eric Blake
2013-03-04  9:30   ` Paolo Bonzini
2013-03-04  9:30     ` [Qemu-devel] " Paolo Bonzini
2013-03-04  9:30     ` Paolo Bonzini
2013-03-05  2:33     ` Hu Tao
2013-03-05  2:33       ` [Qemu-devel] " Hu Tao
2013-03-05  2:33       ` Hu Tao
2013-03-05  8:24       ` Paolo Bonzini
2013-03-05  8:24         ` [Qemu-devel] " Paolo Bonzini
2013-03-05  8:24         ` Paolo Bonzini
2013-02-28 12:13 ` [PATCH v13 2/8] start vm after resetting it Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-02-28 13:23   ` Jan Kiszka
2013-02-28 13:23     ` Jan Kiszka
2013-03-05  3:05     ` Hu Tao [this message]
2013-03-05  3:05       ` [Qemu-devel] " Hu Tao
2013-03-05  3:05       ` Hu Tao
2013-03-04  9:32   ` Paolo Bonzini
2013-03-04  9:32     ` [Qemu-devel] " Paolo Bonzini
2013-03-04  9:32     ` Paolo Bonzini
2013-03-05  3:06     ` Hu Tao
2013-03-05  3:06       ` [Qemu-devel] " Hu Tao
2013-03-05  3:06       ` Hu Tao
2013-02-28 12:13 ` [PATCH v13 3/8] update kernel headers Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-02-28 12:13 ` [PATCH v13 4/8] add a new runstate: RUN_STATE_GUEST_PANICKED Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-03-04  9:40   ` Paolo Bonzini
2013-03-04  9:40     ` [Qemu-devel] " Paolo Bonzini
2013-03-04  9:40     ` Paolo Bonzini
2013-03-05  3:17     ` Hu Tao
2013-03-05  3:17       ` [Qemu-devel] " Hu Tao
2013-03-05  3:17       ` Hu Tao
2013-03-05  8:26       ` Paolo Bonzini
2013-03-05  8:26         ` [Qemu-devel] " Paolo Bonzini
2013-03-05  8:26         ` Paolo Bonzini
2013-03-06  9:03         ` Hu Tao
2013-03-06  9:03           ` [Qemu-devel] " Hu Tao
2013-03-06  9:03           ` Hu Tao
2013-02-28 12:13 ` [PATCH v13 5/8] add a new qevent: QEVENT_GUEST_PANICKED Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-03-01 16:31   ` Eric Blake
2013-03-01 16:31     ` [Qemu-devel] " Eric Blake
2013-03-01 16:31     ` Eric Blake
2013-03-05  3:17     ` Hu Tao
2013-03-05  3:17       ` [Qemu-devel] " Hu Tao
2013-03-05  3:17       ` Hu Tao
2013-03-04  9:40   ` Paolo Bonzini
2013-03-04  9:40     ` [Qemu-devel] " Paolo Bonzini
2013-03-04  9:40     ` Paolo Bonzini
2013-02-28 12:13 ` [PATCH v13 6/8] introduce a new qom device to deal with panicked event Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-03-04  9:42   ` Paolo Bonzini
2013-03-04  9:42     ` [Qemu-devel] " Paolo Bonzini
2013-03-04  9:42     ` Paolo Bonzini
2013-03-04 10:10   ` Christian Borntraeger
2013-03-04 10:10     ` [Qemu-devel] " Christian Borntraeger
2013-03-04 10:21     ` Paolo Bonzini
2013-03-04 10:21       ` [Qemu-devel] " Paolo Bonzini
2013-02-28 12:13 ` [PATCH v13 7/8] allower the user to disable pv event support Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-03-04  9:47   ` Paolo Bonzini
2013-03-04  9:47     ` [Qemu-devel] " Paolo Bonzini
2013-03-04  9:47     ` Paolo Bonzini
2013-02-28 12:13 ` [PATCH v13 8/8] pv event: add document to describe the usage Hu Tao
2013-02-28 12:13   ` Hu Tao
2013-03-03  9:17 ` [PATCH v13 0/8] pv event interface between host and guest Gleb Natapov
2013-03-03  9:17   ` [Qemu-devel] " Gleb Natapov
2013-03-03  9:17   ` Gleb Natapov
2013-03-04 10:05   ` Paolo Bonzini
2013-03-04 10:05     ` [Qemu-devel] " Paolo Bonzini
2013-03-04 10:05     ` Paolo Bonzini
2013-03-04 10:21     ` Gleb Natapov
2013-03-04 10:21       ` [Qemu-devel] " Gleb Natapov
2013-03-04 10:21       ` Gleb Natapov
2013-03-04 10:28       ` Paolo Bonzini
2013-03-04 10:28         ` [Qemu-devel] " Paolo Bonzini
2013-03-04 10:43         ` Gleb Natapov
2013-03-04 10:43           ` [Qemu-devel] " Gleb Natapov
2013-03-04 10:43           ` Gleb Natapov
2013-03-04 10:49           ` Paolo Bonzini
2013-03-04 10:49             ` [Qemu-devel] " Paolo Bonzini
2013-03-04 10:49             ` Paolo Bonzini
2013-03-04 10:59             ` Gleb Natapov
2013-03-04 10:59               ` [Qemu-devel] " Gleb Natapov
2013-03-04 10:59               ` Gleb Natapov
2013-03-04 11:10               ` Paolo Bonzini
2013-03-04 11:10                 ` [Qemu-devel] " Paolo Bonzini
2013-03-04 11:10                 ` Paolo Bonzini
2013-03-04 11:20                 ` Gleb Natapov
2013-03-04 11:20                   ` [Qemu-devel] " Gleb Natapov
2013-03-04 11:20                   ` Gleb Natapov
2013-03-04 11:35                   ` Paolo Bonzini
2013-03-04 11:35                     ` [Qemu-devel] " Paolo Bonzini
2013-03-04 11:35                     ` Paolo Bonzini
2013-03-04 11:52                     ` Gleb Natapov
2013-03-04 11:52                       ` [Qemu-devel] " Gleb Natapov
2013-03-04 11:52                       ` Gleb Natapov
2013-03-04 12:21                       ` Paolo Bonzini
2013-03-04 12:21                         ` [Qemu-devel] " Paolo Bonzini
2013-03-04 12:21                         ` Paolo Bonzini
2013-03-06  8:56     ` Hu Tao
2013-03-06  8:56       ` [Qemu-devel] " Hu Tao
2013-03-06  8:56       ` Hu Tao
2013-03-06  9:07       ` Paolo Bonzini
2013-03-06  9:07         ` [Qemu-devel] " Paolo Bonzini
2013-03-06  9:28         ` li guang
2013-03-06  9:28           ` li guang
2013-03-06  9:38         ` Gleb Natapov
2013-03-06  9:38           ` [Qemu-devel] " Gleb Natapov
2013-03-06  9:38           ` Gleb Natapov
2013-03-06  9:48           ` Paolo Bonzini
2013-03-06  9:48             ` [Qemu-devel] " Paolo Bonzini
2013-03-06  9:48             ` Paolo Bonzini
2013-03-06  9:59             ` Gleb Natapov
2013-03-06  9:59               ` [Qemu-devel] " Gleb Natapov
2013-03-06  9:59               ` Gleb Natapov
2013-03-06  8:46   ` Hu Tao
2013-03-06  8:46     ` [Qemu-devel] " Hu Tao
2013-03-06  8:46     ` Hu Tao
2013-03-06  9:37     ` Gleb Natapov
2013-03-06  9:37       ` [Qemu-devel] " Gleb Natapov
2013-03-06  9:37       ` Gleb Natapov

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=20130305030557.GL16362@localhost.localdomain \
    --to=hutao@cn.fujitsu.com \
    --cc=agraf@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=blauwirbel@gmail.com \
    --cc=drjones@redhat.com \
    --cc=eblake@redhat.com \
    --cc=gleb@redhat.com \
    --cc=jan.kiszka@siemens.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=kvm@vger.kernel.org \
    --cc=kwolf@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=levinsasha928@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=mtosatti@redhat.com \
    --cc=owasserm@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=stefanha@redhat.com \
    --cc=wency@cn.fujitsu.com \
    /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.