[Xen-changelog] [xen staging] AMD/IOMMU: Fix passthrough following c/s d7cfeb7c13e

[patchbot@xxxxxxx]

commit aaef3d904bbbde1fcf9c07943878bd2aa64cc2bc
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Mon Nov 11 20:46:08 2019 +0000
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Tue Nov 12 13:08:10 2019 +0000

    AMD/IOMMU: Fix passthrough following c/s d7cfeb7c13e
    
    "AMD/IOMMU: don't blindly allocate interrupt remapping tables" introduces a
    call at runtime from amd_iommu_add_device() to 
amd_iommu_set_intremap_table()
    which is still marked as __init.
    
    On one AMD Rome machine we have, this results in a crash the moment we try 
to
    use an SR-IOV VF in a VM.
    
    Reported-by: Jennifer Herbert <jennifer.herbert@xxxxxxxxxx>
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
    Release-acked-by: Juergen Gross <jgross@xxxxxxxx>
---
 xen/drivers/passthrough/amd/iommu_map.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/drivers/passthrough/amd/iommu_map.c 
b/xen/drivers/passthrough/amd/iommu_map.c
index 03c1ad4dc5..cd5c7de7c5 100644
--- a/xen/drivers/passthrough/amd/iommu_map.c
+++ b/xen/drivers/passthrough/amd/iommu_map.c
@@ -112,7 +112,7 @@ void amd_iommu_set_root_page_table(struct amd_iommu_dte 
*dte,
     dte->v = valid;
 }
 
-void __init amd_iommu_set_intremap_table(
+void amd_iommu_set_intremap_table(
     struct amd_iommu_dte *dte, const void *ptr,
     const struct amd_iommu *iommu, bool valid)
 {
--
generated by git-patchbot for /home/xen/git/xen.git#staging

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog