All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Wang2 <wei.wang2@amd.com>
To: Ian Pratt <Ian.Pratt@eu.citrix.com>
Cc: xen-devel <xen-devel@lists.xensource.com>,
	Keir Fraser <Keir.Fraser@eu.citrix.com>
Subject: RE: [PATCH 0/2] IOMMU: Handle sibling device assignmentcorrectly (re-send)
Date: Thu, 29 May 2008 11:12:47 +0200	[thread overview]
Message-ID: <1212052367.18132.43.camel@gran.amd.com> (raw)
In-Reply-To: <DD74FBB8EE28D441903D56487861CD9D2F46CFB4@lonpexch01.citrite.net>


On Wed, 2008-05-28 at 17:32 +0100, Ian Pratt wrote:
> > This patch set is revised according to comments from community. Domctl
> > interface is extended to allow libxc retrieve device group information
> > from hypervisor. Vendor-specific iommu_ops is also extended by adding a
> > new operation "get_device_group_id()", which is currently a null
> > pointer
> > but could be implemented later for vt-d.
> > 
> > Error will be raised from tools side when user trying to assign PCI
> > device with a sibling device being driven by dom0. User will keep being
> > prompted until he has hidden the entire device group (at least, the
> > sibling devices driven by dom0) in dom0 kernel parameter. Hopefully
> > this
> > framework could be flexible enough to support both amd iommu and vt-d.
> > 
> > The following 2 cases are not covered by this patch, but should be easy
> > to handle.
> > * Checking for hot-plug devices (maybe we can delay calling
> > ImageHandler.signalDeviceModel() until all checks are done?)
> > * Checking for splitted device group between different passthru domains
> 
> With this patch, what happens if you assign a device that is behind a bridge. Does the guest get all the devices behind the bridge?
> 
> This would be reasonable behaviour, particularly if we prevent other VMs from claiming the sibling devices.
> 
> Ian
This patch aims to prevent a single device group from being splitted
between dom0 and passthru domain if some of sibling devices in this
group are driven by dom0.  A successful device assignment can only
happen if all the sibling devices are hidden from dom0 or non of them
are used by dom0. However, this patch does not extend the default
behavior of device assignment, only the single device is assigned event
it is behind  a bridge. 

IMO, it is more reasonable to just a assigned a single device behind a
bridge than to give target guest the whole group? Other sibling devices
in this group still get chance to be assigned to the same target domain
dynamically. If the target domain is not the same, assignment will be
rejected. To do this , we might have a global structure to indicate the
owner of each device group. 

-Wei







> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

  reply	other threads:[~2008-05-29  9:12 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-28 13:23 [PATCH 0/2] IOMMU: Handle sibling device assignment correctly (re-send) Wei Wang2
2008-05-28 16:32 ` [PATCH 0/2] IOMMU: Handle sibling device assignmentcorrectly (re-send) Ian Pratt
2008-05-29  9:12   ` Wei Wang2 [this message]
2008-05-29  9:19     ` [PATCH 0/2] IOMMU: Handle sibling deviceassignmentcorrectly (re-send) Ian Pratt

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=1212052367.18132.43.camel@gran.amd.com \
    --to=wei.wang2@amd.com \
    --cc=Ian.Pratt@eu.citrix.com \
    --cc=Keir.Fraser@eu.citrix.com \
    --cc=xen-devel@lists.xensource.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 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.