Bug 484148 - VTd: changeset 10 of 16: avoid redundant context mapping
Summary: VTd: changeset 10 of 16: avoid redundant context mapping
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel-xen
Version: 5.4
Hardware: All
OS: Linux
low
medium
Target Milestone: rc
: 5.4
Assignee: Xen Maintainance List
QA Contact: Martin Jenner
URL:
Whiteboard:
Depends On: 484147
Blocks: 331741 484150
TreeView+ depends on / blocked
 
Reported: 2009-02-04 23:51 UTC by Don Dugger
Modified: 2014-07-25 03:22 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-02-05 15:23:43 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
avoid redundant context mapping (3.25 KB, patch)
2009-02-04 23:51 UTC, Don Dugger
no flags Details | Diff

Description Don Dugger 2009-02-04 23:51:39 UTC
Created attachment 330942 [details]
avoid redundant context mapping

Description of patch:

In case DEV_TYPE_PCI in domain_context_mapping(), redundant context
mapping may occur for some devices.
                                                                                For example, the NIC is 03:00.0, and its parent bridge is: 00:1e.0.
After domain_context_mapping_one() 03:00.0 and 00:1e.0, the
'secbus' is 3 and 'bus' is 0, so will domain_context_mapping_one()
03:00.0 again -- this redundant invocation returns -EINVAL because
we have created the mapping but haven't changed pdev->domain from
Dom0 to a new domain at this time and eventually the
XEN_DOMCTL_assign_device hypercall returns a failure.
 
This patch detects this case and avoids the redundant invocation.


Upstream Status: Accepted (CS 18964)

Comment 1 Don Dugger 2009-02-05 15:23:43 UTC
Process error, don't need a separate BZ per patch


Note You need to log in before you can comment on or make changes to this bug.