sriov would not use agp, so seperate the fb aperture setting.

Change-Id: I1372cd355326731a31361bff13d79e12121b8651
Signed-off-by: Frank.Min <frank....@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 39 ++++++++++++++++++++------------
 drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c    | 12 +++++-----
 drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c  | 27 +++++++++++++++-------
 3 files changed, 49 insertions(+), 29 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
index 6ce37ce..ec78c8b 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c
@@ -75,23 +75,32 @@ static void gfxhub_v1_0_init_system_aperture_regs(struct 
amdgpu_device *adev)
        WREG32_SOC15_RLC(GC, 0, mmMC_VM_AGP_BOT, adev->gmc.agp_start >> 24);
        WREG32_SOC15_RLC(GC, 0, mmMC_VM_AGP_TOP, adev->gmc.agp_end >> 24);
 
-       /* Program the system aperture low logical page number. */
-       WREG32_SOC15_RLC(GC, 0, mmMC_VM_SYSTEM_APERTURE_LOW_ADDR,
-                    min(adev->gmc.fb_start, adev->gmc.agp_start) >> 18);
+       if (amdgpu_sriov_vf(adev)) {
+               /* Program the system aperture low logical page number. */
+               WREG32_SOC15_RLC(GC, 0, mmMC_VM_SYSTEM_APERTURE_LOW_ADDR,
+                                adev->gmc.fb_start >> 18);
 
-       if (adev->asic_type == CHIP_RAVEN && adev->rev_id >= 0x8)
-               /*
-                * Raven2 has a HW issue that it is unable to use the vram which
-                * is out of MC_VM_SYSTEM_APERTURE_HIGH_ADDR. So here is the
-                * workaround that increase system aperture high address (add 1)
-                * to get rid of the VM fault and hardware hang.
-                */
                WREG32_SOC15_RLC(GC, 0, mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR,
-                            max((adev->gmc.fb_end >> 18) + 0x1,
-                                adev->gmc.agp_end >> 18));
-       else
-               WREG32_SOC15_RLC(GC, 0, mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR,
-                            max(adev->gmc.fb_end, adev->gmc.agp_end) >> 18);
+                                adev->gmc.fb_end >> 18);
+       } else {
+               /* Program the system aperture low logical page number. */
+               WREG32_SOC15_RLC(GC, 0, mmMC_VM_SYSTEM_APERTURE_LOW_ADDR,
+                            min(adev->gmc.fb_start, adev->gmc.agp_start) >> 
18);
+
+               if (adev->asic_type == CHIP_RAVEN && adev->rev_id >= 0x8)
+                       /*
+                        * Raven2 has a HW issue that it is unable to use the 
vram which
+                        * is out of MC_VM_SYSTEM_APERTURE_HIGH_ADDR. So here 
is the
+                        * workaround that increase system aperture high 
address (add 1)
+                        * to get rid of the VM fault and hardware hang.
+                        */
+                       WREG32_SOC15_RLC(GC, 0, 
mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR,
+                                    max((adev->gmc.fb_end >> 18) + 0x1,
+                                        adev->gmc.agp_end >> 18));
+               else
+                       WREG32_SOC15_RLC(GC, 0, 
mmMC_VM_SYSTEM_APERTURE_HIGH_ADDR,
+                                    max(adev->gmc.fb_end, adev->gmc.agp_end) 
>> 18);
+       }
 
        /* Set default page address. */
        value = adev->vram_scratch.gpu_addr - adev->gmc.vram_start
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c 
b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index 6de1726..1f8bdfa 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -920,12 +920,12 @@ static void gmc_v9_0_vram_gtt_location(struct 
amdgpu_device *adev,
                                        struct amdgpu_gmc *mc)
 {
        u64 base = 0;
-       if (!amdgpu_sriov_vf(adev)) {
-               if (adev->asic_type == CHIP_ARCTURUS)
-                       base = mmhub_v9_4_get_fb_location(adev);
-               else
-                       base = mmhub_v1_0_get_fb_location(adev);
-       }
+
+       if (adev->asic_type == CHIP_ARCTURUS)
+               base = mmhub_v9_4_get_fb_location(adev);
+       else
+               base = mmhub_v1_0_get_fb_location(adev);
+
        /* add the xgmi offset of the physical node */
        base += adev->gmc.xgmi.physical_node_id * 
adev->gmc.xgmi.node_segment_size;
        amdgpu_gmc_vram_location(adev, mc, base);
diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c 
b/drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c
index 0cf7ef4..ea3359f 100644
--- a/drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c
+++ b/drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c
@@ -118,14 +118,25 @@ static void mmhub_v9_4_init_system_aperture_regs(struct 
amdgpu_device *adev,
                            adev->gmc.agp_start >> 24);
 
        /* Program the system aperture low logical page number. */
-       WREG32_SOC15_OFFSET(MMHUB, 0,
-                           mmVMSHAREDVC0_MC_VM_SYSTEM_APERTURE_LOW_ADDR,
-                           hubid * MMHUB_INSTANCE_REGISTER_OFFSET,
-                           min(adev->gmc.fb_start, adev->gmc.agp_start) >> 18);
-       WREG32_SOC15_OFFSET(MMHUB, 0,
-                           mmVMSHAREDVC0_MC_VM_SYSTEM_APERTURE_HIGH_ADDR,
-                           hubid * MMHUB_INSTANCE_REGISTER_OFFSET,
-                           max(adev->gmc.fb_end, adev->gmc.agp_end) >> 18);
+       if (amdgpu_sriov_vf(adev)) {
+               WREG32_SOC15_OFFSET(MMHUB, 0,
+                                       
mmVMSHAREDVC0_MC_VM_SYSTEM_APERTURE_LOW_ADDR,
+                                       hubid * MMHUB_INSTANCE_REGISTER_OFFSET,
+                                       adev->gmc.fb_start >> 18);
+               WREG32_SOC15_OFFSET(MMHUB, 0,
+                                       
mmVMSHAREDVC0_MC_VM_SYSTEM_APERTURE_HIGH_ADDR,
+                                       hubid * MMHUB_INSTANCE_REGISTER_OFFSET,
+                                       adev->gmc.fb_end >> 18);
+       } else {
+               WREG32_SOC15_OFFSET(MMHUB, 0,
+                                   
mmVMSHAREDVC0_MC_VM_SYSTEM_APERTURE_LOW_ADDR,
+                                   hubid * MMHUB_INSTANCE_REGISTER_OFFSET,
+                                   min(adev->gmc.fb_start, 
adev->gmc.agp_start) >> 18);
+               WREG32_SOC15_OFFSET(MMHUB, 0,
+                                   
mmVMSHAREDVC0_MC_VM_SYSTEM_APERTURE_HIGH_ADDR,
+                                   hubid * MMHUB_INSTANCE_REGISTER_OFFSET,
+                                   max(adev->gmc.fb_end, adev->gmc.agp_end) >> 
18);
+       }
 
        /* Set default page address. */
        value = adev->vram_scratch.gpu_addr - adev->gmc.vram_start +
-- 
2.7.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to