virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Rusty Russell <rusty@rustcorp.com.au>
Cc: virtualization@lists.linux-foundation.org
Subject: Re: [RFC 0/4] Using a generic bus_type for virtio
Date: Sun, 8 Jul 2007 17:55:53 +0200	[thread overview]
Message-ID: <200707081755.53395.arnd@arndb.de> (raw)
In-Reply-To: <1183860930.6005.212.camel@localhost.localdomain>

On Sunday 08 July 2007, Rusty Russell wrote:
>         I think it will come down to how neat PCI<->virtio is.  Can we push
> further towards PCI without screwing non-PCI?  eg. can we use
> pci_device_id?  struct pci_driver?  (Might be pushing it, but should
> probably be considered: it'd be neat if some platforms could #define
> virtio_driver_register pci_driver_register).

My idea was to have the virtio device as a child of the PCI device
instead. That way the PCI virtio host can do its own probing without
burdening the virtio bus layer with PCI specific implementation details.

>         Standardizing how to pack the info for each device into the config
> space would be especially useful.  Our drivers are going to get more
> featureful, and we're going to need a versioning/compatibility scheme
> too.

So do you like the splitting of the config space into host and driver part
into 128 bytes each?
That would split the need for versioning between host and driver.

Do you think the config space needs to be writable by the driver?
For the host part, we need to have it writable in case of a PCI host,
which finds stuff like BAR sizes using write access, but requiring that
every host has writable configuration data could make it harder for
some hosts.

	Arnd <><

  parent reply	other threads:[~2007-07-08 15:55 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-06 12:42 [RFC 0/4] Using a generic bus_type for virtio arnd
2007-07-06 12:42 ` [RFC 1/4] New virtio bus driver arnd
2007-07-08  9:59   ` Avi Kivity
2007-07-08 15:29     ` Arnd Bergmann
2007-07-08 15:48       ` Avi Kivity
2007-07-08 20:29         ` Arnd Bergmann
2007-07-08 23:42           ` Rusty Russell
2007-07-09  6:49           ` Avi Kivity
2007-07-09 11:18             ` Arnd Bergmann
2007-07-09 11:41               ` Avi Kivity
2007-07-09 11:38                 ` Arnd Bergmann
2007-07-09 12:09                   ` Avi Kivity
2007-07-09 14:24                     ` Arnd Bergmann
2007-07-09 14:56                       ` Avi Kivity
2007-07-09 16:33                         ` Arnd Bergmann
2007-07-10  1:53                     ` Rusty Russell
2007-07-10  7:56                       ` Avi Kivity
2007-07-10  1:17             ` Rusty Russell
2007-07-10  6:06               ` Avi Kivity
2007-07-06 12:42 ` [RFC 2/4] Convert virtio_net to new virtio bus arnd
2007-07-06 12:42 ` [RFC 3/4] Convert virtio_blk " arnd
2007-07-06 12:42 ` [RFC 4/4] Example virtio host implementation, using chardev arnd
2007-07-08  2:15 ` [RFC 0/4] Using a generic bus_type for virtio Rusty Russell
2007-07-08  9:45   ` Avi Kivity
2007-07-08 15:55   ` Arnd Bergmann [this message]
2007-07-08  9:42 ` Avi Kivity

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=200707081755.53395.arnd@arndb.de \
    --to=arnd@arndb.de \
    --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;
as well as URLs for NNTP newsgroup(s).