From: Avi Kivity <avi@qumranet.com>
To: Rusty Russell <rusty@rustcorp.com.au>
Cc: arnd@arndb.de, virtualization@lists.linux-foundation.org
Subject: Re: [RFC 0/4] Using a generic bus_type for virtio
Date: Sun, 08 Jul 2007 12:45:34 +0300 [thread overview]
Message-ID: <4690B23E.8030009@qumranet.com> (raw)
In-Reply-To: <1183860930.6005.212.camel@localhost.localdomain>
Rusty Russell wrote:
> On Fri, 2007-07-06 at 14:42 +0200, arnd@arndb.de wrote:
>
>> This is a subject that came up in the virtio BOF session
>> at OLS. I decided to go forward and implement something
>> that I like, based on the latest virtio proposal at the
>> time, which was draft III.
>>
>> It's not a drop-in replacement, because it's missing a
>> host implementation. I first started my own, which is
>> not done yet, but wanted to do one for lguest and one
>> for emulated PCI next. It's also entirely untested.
>>
>
> Hi Arnd,
>
> 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).
>
This is liable to cause breakage when something changes. And if I
weren't such a sweet person I'd say it's also horribly ugly.
> 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.
>
> Basically, I'd like to see someone start with work from the PCI side,
> then make sure non-PCI isn't overly burdened.
>
We can certainly make non-bus specific code, like the feature bitmaps,
shared. I do agree that it makes sense to start from the PCI side as
virtbus doesn't have any constraints.
--
error compiling committee.c: too many arguments to function
next prev parent reply other threads:[~2007-07-08 9:45 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 [this message]
2007-07-08 15:55 ` Arnd Bergmann
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=4690B23E.8030009@qumranet.com \
--to=avi@qumranet.com \
--cc=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 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.