diff options
| author | Alex Williamson <alex.williamson@redhat.com> | 2012-04-17 21:46:44 -0600 |
|---|---|---|
| committer | Marcelo Tosatti <mtosatti@redhat.com> | 2012-04-19 00:04:18 -0300 |
| commit | 21a1416a1c945c5aeaeaf791b63c64926018eb77 (patch) | |
| tree | 4a6ac6aae66648bba9de57902ec98dfa5ae9b98d /lib | |
| parent | 2225fd56049643c1a7d645c0ce9d499d43c7974e (diff) | |
| download | linux-21a1416a1c945c5aeaeaf791b63c64926018eb77.tar.gz linux-21a1416a1c945c5aeaeaf791b63c64926018eb77.tar.bz2 linux-21a1416a1c945c5aeaeaf791b63c64926018eb77.zip | |
KVM: lock slots_lock around device assignment
As pointed out by Jason Baron, when assigning a device to a guest
we first set the iommu domain pointer, which enables mapping
and unmapping of memory slots to the iommu. This leaves a window
where this path is enabled, but we haven't synchronized the iommu
mappings to the existing memory slots. Thus a slot being removed
at that point could send us down unexpected code paths removing
non-existent pinnings and iommu mappings. Take the slots_lock
around creating the iommu domain and initial mappings as well as
around iommu teardown to avoid this race.
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions