kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: "Han, Weidong" <weidong.han@intel.com>
Cc: Joerg Roedel <joerg.roedel@amd.com>,
	kvm@vger.kernel.org, Amit Shah <amit.shah@redhat.com>,
	"Kay, Allen M" <allen.m.kay@intel.com>,
	"Yang, Sheng" <sheng.yang@intel.com>,
	benami@il.ibm.com, muli@il.ibm.com
Subject: Re: [PATCH] [RESEND] VT-d: Support multiple device assignment to one guest
Date: Wed, 08 Oct 2008 12:32:12 +0200	[thread overview]
Message-ID: <48EC8C2C.5070801@redhat.com> (raw)
In-Reply-To: <0122C7C995D32147B66BF4F440D3016301D2955D@pdsmsx415.ccr.corp.intel.com>

Han, Weidong wrote:
>> Oh, I see it now.  Different devices may need to go under different
>> iommus. 
>>
>> This really feels like it should be handled by the iommu API.  Users
>> shouldn't need to bother with it.
>>     
>
> Why do you say it should be handled by iommu API? 

Because the logic of which iommu controls which device is only 
understood by iommu developers.  Also, because this logic would be 
duplicated by anyone attempting to do the same thing.

So it seems reasonable it should be implemented by the iommu API, not 
its users.

> The direct way to
> support multiple device assignment is keep a dmar_domain list for each
> guest, each device corresponds to one dmar_domain. But this will cost
> more memory because each dmar_domain has its own VT-d page table. Our
> method lets the devices which are under smae iommu and assigned to the
> same guest share the same VT-d page table. 
>   

If we devolve this to the iommu API, the same io page table can be 
shared by all iommus, so long as they all use the same page table format.


-- 
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.


  reply	other threads:[~2008-10-08 10:35 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-06  6:38 [PATCH] [RESEND] VT-d: Support multiple device assignment to one guest Han, Weidong
2008-10-07 10:04 ` Zhang, Xiantao
2008-10-07 13:59   ` Avi Kivity
2008-10-08  1:58     ` Zhang, Xiantao
2008-10-07 13:29 ` Avi Kivity
2008-10-08  5:40   ` Han, Weidong
2008-10-08 10:32     ` Avi Kivity [this message]
2008-10-08 15:06       ` Han, Weidong
2008-10-08 19:49         ` Avi Kivity
2008-10-09  6:11           ` Han, Weidong
2008-10-09  8:31             ` Avi Kivity
2008-10-09  9:25               ` Han, Weidong
2008-10-09 12:50                 ` Avi Kivity
2008-10-09 14:31                   ` Han, Weidong
     [not found]                   ` <0122C7C995D32147B66BF4F440D3016301CB08EF@pdsmsx415.ccr.corp.intel.com>
2008-10-10  5:50                     ` Han, Weidong
2008-10-10  6:40                       ` Avi Kivity
2008-10-10  7:22                         ` Han, Weidong
2008-10-10  7:32                           ` Avi Kivity
2008-10-10  7:50                             ` Han, Weidong
2008-10-29 10:25   ` Joerg Roedel

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=48EC8C2C.5070801@redhat.com \
    --to=avi@redhat.com \
    --cc=allen.m.kay@intel.com \
    --cc=amit.shah@redhat.com \
    --cc=benami@il.ibm.com \
    --cc=joerg.roedel@amd.com \
    --cc=kvm@vger.kernel.org \
    --cc=muli@il.ibm.com \
    --cc=sheng.yang@intel.com \
    --cc=weidong.han@intel.com \
    /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).