IOMMU/x86: drop locking from quarantine_init() hooks
authorJan Beulich <jbeulich@suse.com>
Wed, 20 Apr 2022 08:52:13 +0000 (10:52 +0200)
committerJan Beulich <jbeulich@suse.com>
Wed, 20 Apr 2022 08:52:13 +0000 (10:52 +0200)
commit02b0e2d0303869a222bb8c2a888756394e99d15b
treef5a379d3b0467a1ac3f2b223479693a1373fa6b2
parent075817681c2e9cc1d0d9ac51e45f7a7dd86123f6
IOMMU/x86: drop locking from quarantine_init() hooks

Prior extension of these functions to enable per-device quarantine page
tables already didn't add more locking there, but merely left in place
what had been there before. But really locking is unnecessary here:
We're running with pcidevs_lock held (i.e. multiple invocations of the
same function [or their teardown equivalents] are impossible, and hence
there are no "local" races), while all consuming of the data being
populated here can't race anyway due to happening sequentially
afterwards, and unlike ordinary domains' page tables quarantine ones
are never modified once fully constructed. See also the comment in
struct arch_pci_dev.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
Reviewed-by: Kevin Tian <kevin.tian@intel.com>
xen/drivers/passthrough/amd/iommu_map.c
xen/drivers/passthrough/vtd/iommu.c