All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tetsuya Mukawa <mukawa@igel.co.jp>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Marcel Apfelbaum <marcel.a@redhat.com>,
	jasowang@redhat.com, qemu-devel@nongnu.org,
	n.nikolaev@virtualopensystems.com, stefanha@redhat.com
Subject: Re: [Qemu-devel] [PATCH v2 0/5] Add feature to start QEMU without vhost-user backend
Date: Wed, 22 Jul 2015 11:44:28 +0900	[thread overview]
Message-ID: <55AF038C.2050300@igel.co.jp> (raw)
In-Reply-To: <20150719154650-mutt-send-email-mst@redhat.com>

On 2015/07/19 21:48, Michael S. Tsirkin wrote:
> On Wed, Jun 24, 2015 at 07:57:45AM +0200, Michael S. Tsirkin wrote:
>> On Wed, Jun 24, 2015 at 02:46:30PM +0900, Tetsuya Mukawa wrote:
>>> On 2015/06/23 18:41, Michael S. Tsirkin wrote:
>>>> On Tue, Jun 23, 2015 at 05:31:06PM +0900, Tetsuya Mukawa wrote:
>>>>> On 2015/06/22 17:14, Michael S. Tsirkin wrote:
>>>>>> On Mon, Jun 22, 2015 at 12:50:43PM +0900, Tetsuya Mukawa wrote:
>>>>>>> Hi guys,
>>>>>>>
>>>>>>> Here are patches to add feature to start QEMU without vhost-user backend.
>>>>>>> Currently, if we want to use vhost-user backend, the backend must start before
>>>>>>> QEMU.
>>>>>> Aren't you looking for something like xinetd?
>>>>> It will help for restricting starting order, but not help for
>>>>> reconnection of vhost-user backend.
>>>> At this point, I suggest that we always connect at vm start.
>>>> With that, you can add an option to reset the VM
>>>> on backend disconnect.
>>>> So
>>>> 	- detect backend disconnect
>>>> 	- reset and stop (notifying management)
>>>> 	- reconnect or detect backend reconnect
>>>> 	- proceed with boot
>>>>
>>>> As I tried to explain below, getting the full functionality
>>>> will require guest driver changes. They aren't hard to get
>>>> done, patches welcome.
>>>>
>>> Could you please let me know your thinking about using
>>> DEVICE_NEEDS_RESET for vhost-user reconnection?
>>> If it's works, I will try to submit it.
>> DEVICE_NEEDS_RESET is hard to handle correctly in guest:
>> you need to reconfigure a bunch of state,
>> so far no one wrote the necessary support.
> But yes, if you write the guest code, I think
> it can work.
>
> We probably want a feature bit so host can know guest can
> handle reconnections, but that's a minor detail, it
> would be easy to add afterwards.
>

Hi Michael,

I appreciate for your suggestion.
I will check the virtio-net driver of linux and the virtio-net device of
QEMU to know how difficult to implement DEVICE_NEEDS_RESET.

Regards,
Tetsuya

      reply	other threads:[~2015-07-22  2:44 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-22  3:50 [Qemu-devel] [PATCH v2 0/5] Add feature to start QEMU without vhost-user backend Tetsuya Mukawa
2015-06-22  3:50 ` [Qemu-devel] [PATCH v2 1/5] vhost-user: Add ability to know vhost-user backend disconnection Tetsuya Mukawa
2015-06-22  3:50 ` [Qemu-devel] [PATCH v2 2/5] qemu-char: Add qemu_chr_disconnect to close a fd accepted by listen fd Tetsuya Mukawa
2015-06-22  3:50 ` [Qemu-devel] [PATCH v2 3/5] vhost-user: Shutdown vhost-user connection when wrong messages are passed Tetsuya Mukawa
2015-06-22  7:54   ` Michael S. Tsirkin
2015-06-23  8:33     ` Tetsuya Mukawa
2015-08-06 17:29   ` Marc-André Lureau
2015-06-22  3:50 ` [Qemu-devel] [PATCH v2 4/5] vhost-user: Enable 'nowait' and 'reconnect' option Tetsuya Mukawa
2015-08-06 17:41   ` Marc-André Lureau
2015-06-22  3:50 ` [Qemu-devel] [PATCH v2 5/5] vhost-user: Add new option to specify vhost-user backend supports Tetsuya Mukawa
2015-08-06 19:11   ` Marc-André Lureau
2015-06-22  8:14 ` [Qemu-devel] [PATCH v2 0/5] Add feature to start QEMU without vhost-user backend Michael S. Tsirkin
2015-06-23  8:31   ` Tetsuya Mukawa
2015-06-23  9:41     ` Michael S. Tsirkin
2015-06-24  5:46       ` Tetsuya Mukawa
2015-06-24  5:57         ` Michael S. Tsirkin
2015-06-24  7:13           ` Tetsuya Mukawa
2015-07-19 12:48           ` Michael S. Tsirkin
2015-07-22  2:44             ` Tetsuya Mukawa [this message]

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=55AF038C.2050300@igel.co.jp \
    --to=mukawa@igel.co.jp \
    --cc=jasowang@redhat.com \
    --cc=marcel.a@redhat.com \
    --cc=mst@redhat.com \
    --cc=n.nikolaev@virtualopensystems.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.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.