All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rusty Russell <rusty@rustcorp.com.au>
To: Sasha Levin <levinsasha928@gmail.com>, dlaor@redhat.com
Cc: Peter Maydell <peter.maydell@linaro.org>,
	wency@cn.fujitsu.com, kvm@vger.kernel.org, mst@redhat.com,
	linux-kernel@vger.kernel.org,
	virtualization@lists.linux-foundation.org, penberg@kernel.org,
	avi@redhat.com, anthony@codemonkey.ws,
	Amit Shah <amit.shah@redhat.com>
Subject: Re: [RFC 0/2] virtio: provide a way for host to monitor critical events in the device
Date: Wed, 25 Jul 2012 10:06:37 +0930	[thread overview]
Message-ID: <87haswptgq.fsf@rustcorp.com.au> (raw)
In-Reply-To: <500E9CC7.9020201@gmail.com>

On Tue, 24 Jul 2012 15:01:59 +0200, Sasha Levin <levinsasha928@gmail.com> wrote:
> virtio on it's own was introduced to help solve the fragmentation
> around virtualized devices, so I don't think that the main purpose of
> doing virtio drivers is due to any performance benefits virtio may
> provide.

There's one argument in your favor (with my Linaro hat on): ARM wants a
virtio reboot button, which would look remarkably similar.  There's no
standard ARM hardware for this.

So a more generalized virtio-event device might make sense.  But there
are almost an infinite number of guest events we might want: panics,
oom, low memory, stuck devices, deadlock, etc, etc.  I'm concerned about
trying to standardize them.  If we include a unspecified free-form
string, people will end up relying on the contents.  If we add a feature
bit for every new event, we'll end up running out of feature bits :)

CC'ing Amit for opinion over how much of this should be done via
virtio-serial.

Cheers,
Rusty.

WARNING: multiple messages have this Message-ID (diff)
From: Rusty Russell <rusty@rustcorp.com.au>
To: Sasha Levin <levinsasha928@gmail.com>, dlaor@redhat.com
Cc: mst@redhat.com, penberg@kernel.org, asias.hejun@gmail.com,
	virtualization@lists.linux-foundation.org,
	linux-kernel@vger.kernel.org, kvm@vger.kernel.org,
	avi@redhat.com, anthony@codemonkey.ws, wency@cn.fujitsu.com,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Amit Shah" <amit.shah@redhat.com>
Subject: Re: [RFC 0/2] virtio: provide a way for host to monitor critical events in the device
Date: Wed, 25 Jul 2012 10:06:37 +0930	[thread overview]
Message-ID: <87haswptgq.fsf@rustcorp.com.au> (raw)
In-Reply-To: <500E9CC7.9020201@gmail.com>

On Tue, 24 Jul 2012 15:01:59 +0200, Sasha Levin <levinsasha928@gmail.com> wrote:
> virtio on it's own was introduced to help solve the fragmentation
> around virtualized devices, so I don't think that the main purpose of
> doing virtio drivers is due to any performance benefits virtio may
> provide.

There's one argument in your favor (with my Linaro hat on): ARM wants a
virtio reboot button, which would look remarkably similar.  There's no
standard ARM hardware for this.

So a more generalized virtio-event device might make sense.  But there
are almost an infinite number of guest events we might want: panics,
oom, low memory, stuck devices, deadlock, etc, etc.  I'm concerned about
trying to standardize them.  If we include a unspecified free-form
string, people will end up relying on the contents.  If we add a feature
bit for every new event, we'll end up running out of feature bits :)

CC'ing Amit for opinion over how much of this should be done via
virtio-serial.

Cheers,
Rusty.

  reply	other threads:[~2012-07-25  0:36 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-23 20:32 [RFC 0/2] virtio: provide a way for host to monitor critical events in the device Sasha Levin
2012-07-23 20:32 ` Sasha Levin
2012-07-23 20:32 ` [RFC 1/2] virtio: Introduce virtio-notifier Sasha Levin
2012-07-23 20:32   ` Sasha Levin
2012-07-23 20:32 ` [RFC 2/2] kvm tools: support virtio-notifier Sasha Levin
2012-07-23 20:32   ` Sasha Levin
2012-07-24  4:55 ` [RFC 0/2] virtio: provide a way for host to monitor critical events in the device Rusty Russell
2012-07-24  4:55   ` Rusty Russell
2012-07-24  4:55   ` Rusty Russell
2012-07-24  8:26   ` Dor Laor
2012-07-24  8:26     ` Dor Laor
2012-07-24 12:30     ` Sasha Levin
2012-07-24 12:30       ` Sasha Levin
2012-07-24 12:46       ` Dor Laor
2012-07-24 12:46         ` Dor Laor
2012-07-24 13:01         ` Sasha Levin
2012-07-24 13:01           ` Sasha Levin
2012-07-25  0:36           ` Rusty Russell [this message]
2012-07-25  0:36             ` Rusty Russell
2012-07-25  8:46             ` Amit Shah
2012-07-25  8:46               ` Amit Shah
2012-07-24 12:23   ` Sasha Levin
2012-07-24 12:23     ` Sasha Levin
2012-07-24  7:44 ` Gleb Natapov
2012-07-24  7:44   ` Gleb Natapov
2012-07-24 12:26   ` Sasha Levin
2012-07-24 12:28     ` Gleb Natapov
2012-07-24 12:28       ` Gleb Natapov
2012-07-24 12:31       ` Sasha Levin
2012-07-24 12:31         ` Sasha Levin
2012-07-24 12:33         ` Gleb Natapov
2012-07-24 12:33           ` Gleb Natapov
2012-07-24 12:26   ` Sasha Levin

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=87haswptgq.fsf@rustcorp.com.au \
    --to=rusty@rustcorp.com.au \
    --cc=amit.shah@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=avi@redhat.com \
    --cc=dlaor@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=levinsasha928@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=penberg@kernel.org \
    --cc=peter.maydell@linaro.org \
    --cc=virtualization@lists.linux-foundation.org \
    --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.