From: Avi Kivity <avi@qumranet.com>
To: Gregory Haskins <gregory.haskins.ml@gmail.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
Rusty Russell <rusty@rustcorp.com.au>,
Dor Laor <dor.laor@qumranet.com>,
virtualization@lists.linux-foundation.org,
linux-kernel@vger.kernel.org
Subject: Re: Use of virtio device IDs
Date: Wed, 07 Nov 2007 07:40:13 +0200 [thread overview]
Message-ID: <47314FBD.1070505@qumranet.com> (raw)
In-Reply-To: <4731334A.6090405@gmail.com>
Gregory Haskins wrote:
> Anthony Liguori wrote:
>
>
>> Right now, we would have to have every PCI vendor/device ID pair in the
>> virtio PCI driver ID table for every virtio device.
>>
>
> I realize you guys are probably far down this road in the design
> process,
That doesn't mean we can't change it if it's wrong.
> but FWIW: This is a major motivation for the reason that the
> IOQ stuff I posted a while back used strings for device identification
> instead of a fixed length, centrally managed namespace like PCI
> vendor/dev-id. Then you can just name your device something reasonably
> unique (e.g. "qumranet::veth", or "ibm-pvirt-clock").
>
I dislike strings. They make it look as if you have a nice extensible
interface, where in reality you have a poorly documented interface which
leads to poor interoperability.
I prefer nice structure where you can see all the limitations immediately.
> (I realize that if you are going to do PCI, you need to make it
> PCI-like. But I think using PCI in the first place is probably the
> wrong direction. IMHO, there's really not a lot of reason to be
> constrained by a hardware specification once you decide to go PV. This
> is even more true if you want to support as many platforms as possible
> (i.e. platforms that don't have PCI natively).
>
>
PCI means that you can reuse all of the platform's infrastructure for
irq allocation, discovery, device hotplug, and management. You can
write it for new guests but backporting it to older guests will be a
huge task.
We will support non-pci for s390, but in order to support Windows and
older Linux PCI is necessary.
--
Do not meddle in the internals of kernels, for they are subtle and quick to panic.
next prev parent reply other threads:[~2007-11-07 5:40 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-06 17:16 Use of virtio device IDs Anthony Liguori
2007-11-06 18:49 ` Anthony Liguori
2007-11-07 3:38 ` Gregory Haskins
2007-11-07 5:40 ` Avi Kivity [this message]
2007-11-07 6:09 ` Rusty Russell
2007-11-07 6:29 ` Anthony Liguori
2007-11-07 17:33 ` Anthony Liguori
2007-11-07 20:38 ` Gregory Haskins
2007-11-08 6:37 ` Avi Kivity
2007-11-08 9:17 ` Gerd Hoffmann
2007-11-08 16:40 ` Anthony Liguori
2007-11-13 13:18 ` Gregory Haskins
2007-11-13 13:59 ` Zachary Amsden
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=47314FBD.1070505@qumranet.com \
--to=avi@qumranet.com \
--cc=aliguori@us.ibm.com \
--cc=dor.laor@qumranet.com \
--cc=gregory.haskins.ml@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
--cc=virtualization@lists.linux-foundation.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox