virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
* VIRTIO_F_IOMMU_PLATFORM
@ 2016-09-27  8:57 Will Deacon
  2016-10-07  4:24 ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
  0 siblings, 1 reply; 5+ messages in thread
From: Will Deacon @ 2016-09-27  8:57 UTC (permalink / raw)
  To: mst; +Cc: virtualization

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: VIRTIO_F_IOMMU_PLATFORM
  2016-09-27  8:57 VIRTIO_F_IOMMU_PLATFORM Will Deacon
@ 2016-10-07  4:24 ` Michael S. Tsirkin
  2016-10-10 16:59   ` VIRTIO_F_IOMMU_PLATFORM Will Deacon
  0 siblings, 1 reply; 5+ messages in thread
From: Michael S. Tsirkin @ 2016-10-07  4:24 UTC (permalink / raw)
  To: Will Deacon; +Cc: virtualization

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?

-- 
MST

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: VIRTIO_F_IOMMU_PLATFORM
  2016-10-07  4:24 ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
@ 2016-10-10 16:59   ` Will Deacon
  2016-10-10 18:07     ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
  0 siblings, 1 reply; 5+ messages in thread
From: Will Deacon @ 2016-10-10 16:59 UTC (permalink / raw)
  To: Michael S. Tsirkin; +Cc: virtualization

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: VIRTIO_F_IOMMU_PLATFORM
  2016-10-10 16:59   ` VIRTIO_F_IOMMU_PLATFORM Will Deacon
@ 2016-10-10 18:07     ` Michael S. Tsirkin
  2016-10-11  1:51       ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
  0 siblings, 1 reply; 5+ messages in thread
From: Michael S. Tsirkin @ 2016-10-10 18:07 UTC (permalink / raw)
  To: Will Deacon; +Cc: virtualization

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?
> 
> Will

Yes, 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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: VIRTIO_F_IOMMU_PLATFORM
  2016-10-10 18:07     ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
@ 2016-10-11  1:51       ` Michael S. Tsirkin
  0 siblings, 0 replies; 5+ messages in thread
From: Michael S. Tsirkin @ 2016-10-11  1:51 UTC (permalink / raw)
  To: Will Deacon; +Cc: virtualization

On Mon, Oct 10, 2016 at 09:07:35PM +0300, Michael S. Tsirkin wrote:
> 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?
> > 
> > Will
> 
> Yes, 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!

A faster way is to go through a git mirror. See

https://tools.oasis-open.org/version-control/svn/virtio/trunk/git-svn.txt
section named "Faster initial clone from git mirror".



> -- 
> MST

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-10-11  1:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-27  8:57 VIRTIO_F_IOMMU_PLATFORM Will Deacon
2016-10-07  4:24 ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
2016-10-10 16:59   ` VIRTIO_F_IOMMU_PLATFORM Will Deacon
2016-10-10 18:07     ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin
2016-10-11  1:51       ` VIRTIO_F_IOMMU_PLATFORM Michael S. Tsirkin

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