All of lore.kernel.org
 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 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.