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.
next prev parent 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.