All of lore.kernel.org
 help / color / mirror / Atom feed
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.


  parent reply	other threads:[~2007-11-07  5:40 UTC|newest]

Thread overview: 26+ 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  3:38   ` Gregory Haskins
2007-11-07  5:40     ` Avi Kivity
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  6:29         ` Anthony Liguori
2007-11-07 17:33         ` Anthony Liguori
2007-11-07 17:33         ` Anthony Liguori
2007-11-07  6:09       ` Rusty Russell
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-08 16:40             ` Anthony Liguori
2007-11-08  9:17           ` Gerd Hoffmann
2007-11-13 13:18           ` Gregory Haskins
2007-11-13 13:59             ` Zachary Amsden
2007-11-13 13:59             ` Zachary Amsden
2007-11-13 13:18           ` Gregory Haskins
2007-11-08  6:37         ` Avi Kivity
2007-11-07 20:38       ` Gregory Haskins
2007-11-06 18:49 ` Anthony Liguori
  -- strict thread matches above, loose matches on Subject: below --
2007-11-06 17:16 Anthony Liguori

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 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.