On Tue, Sep 27, 2016 at 09:57:14AM +0100, Will Deacon wrote:> Hi Michael, > > In commit 1a937693993f ("virtio: new feature to detect IOMMU device quirk"), > you added a new feature bit (VIRTIO_F_IOMMU_PLATFORM) to describe whether > or not a given virtio device requires physical address or bus addresses. > > Is there a plan to get this incorporated into the virtio spec [1]? At the > moment, I can't see a working draft that includes the new bit. Having it > in the spec would help convince implementors that it's not just a Linux > thing. > > Thanks, > > Will > > [1] http://docs.oasis-open.org/virtio/virtio/v1.0/cs04/virtio-v1.0-cs04.htmlAbsolutely. Want to contribute a patch? -- MST
On Fri, Oct 07, 2016 at 07:24:34AM +0300, Michael S. Tsirkin wrote:> On Tue, Sep 27, 2016 at 09:57:14AM +0100, Will Deacon wrote: > > Hi Michael, > > > > In commit 1a937693993f ("virtio: new feature to detect IOMMU device quirk"), > > you added a new feature bit (VIRTIO_F_IOMMU_PLATFORM) to describe whether > > or not a given virtio device requires physical address or bus addresses. > > > > Is there a plan to get this incorporated into the virtio spec [1]? At the > > moment, I can't see a working draft that includes the new bit. Having it > > in the spec would help convince implementors that it's not just a Linux > > thing. > > > > Thanks, > > > > Will > > > > [1] http://docs.oasis-open.org/virtio/virtio/v1.0/cs04/virtio-v1.0-cs04.html > > Absolutely. Want to contribute a patch?I could have a look. Is there a git repo for the sources to the working draft? Will
On Mon, Oct 10, 2016 at 05:59:03PM +0100, Will Deacon wrote:> On Fri, Oct 07, 2016 at 07:24:34AM +0300, Michael S. Tsirkin wrote: > > On Tue, Sep 27, 2016 at 09:57:14AM +0100, Will Deacon wrote: > > > Hi Michael, > > > > > > In commit 1a937693993f ("virtio: new feature to detect IOMMU device quirk"), > > > you added a new feature bit (VIRTIO_F_IOMMU_PLATFORM) to describe whether > > > or not a given virtio device requires physical address or bus addresses. > > > > > > Is there a plan to get this incorporated into the virtio spec [1]? At the > > > moment, I can't see a working draft that includes the new bit. Having it > > > in the spec would help convince implementors that it's not just a Linux > > > thing. > > > > > > Thanks, > > > > > > Will > > > > > > [1] http://docs.oasis-open.org/virtio/virtio/v1.0/cs04/virtio-v1.0-cs04.html > > > > Absolutely. Want to contribute a patch? > > I could have a look. Is there a git repo for the sources to the working > draft? > > WillYes, though unfortunately it's svn :( git svn clone --stdlayout https://tools.oasis-open.org/version-control/svn/virtio Check out the main branch. Patches must be sent to virtio-dev or virtio-comments list (subscription required). Thanks! -- MST