diff options
| author | Jan Beulich <jbeulich@suse.com> | 2021-09-17 12:50:38 +0200 |
|---|---|---|
| committer | Juergen Gross <jgross@suse.com> | 2021-09-20 17:01:21 +0200 |
| commit | 794d5b8a497ff053f56856472e2fae038fa761aa (patch) | |
| tree | d3f8197f07e19b666414abba45ec055898fbe042 /arch | |
| parent | 8e1034a526652f265ed993fab7f659eb8ae4b6f0 (diff) | |
| download | linux-stable-794d5b8a497ff053f56856472e2fae038fa761aa.tar.gz linux-stable-794d5b8a497ff053f56856472e2fae038fa761aa.tar.bz2 linux-stable-794d5b8a497ff053f56856472e2fae038fa761aa.zip | |
swiotlb-xen: this is PV-only on x86
The code is unreachable for HVM or PVH, and it also makes little sense
in auto-translated environments. On Arm, with
xen_{create,destroy}_contiguous_region() both being stubs, I have a hard
time seeing what good the Xen specific variant does - the generic one
ought to be fine for all purposes there. Still Arm code explicitly
references symbols here, so the code will continue to be included there.
Instead of making PCI_XEN's "select" conditional, simply drop it -
SWIOTLB_XEN will be available unconditionally in the PV case anyway, and
is - as explained above - dead code in non-PV environments.
This in turn allows dropping the stubs for
xen_{create,destroy}_contiguous_region(), the former of which was broken
anyway - it failed to set the DMA handle output.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
Link: https://lore.kernel.org/r/5947b8ae-fdc7-225c-4838-84712265fc1e@suse.com
Signed-off-by: Juergen Gross <jgross@suse.com>
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/x86/Kconfig | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index 88fb922c23a0..a71ced4c711f 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig @@ -2605,7 +2605,6 @@ config PCI_OLPC config PCI_XEN def_bool y depends on PCI && XEN - select SWIOTLB_XEN config MMCONF_FAM10H def_bool y |