[PATCH 2/4] drm/amdgpu: remove VRAM size reduction

Alex Deucher alexdeucher at gmail.com
Fri Nov 17 15:53:50 UTC 2017


On Fri, Nov 17, 2017 at 5:12 AM, Christian König
<ckoenig.leichtzumerken at gmail.com> wrote:
> Remove some outdated comments and all code which tries to reduce the VRAM size
> mapped into the MC.
>
> This is superfluous and misleading since we never actually program the size.
>
> Signed-off-by: Christian König <christian.koenig at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 30 +-----------------------------
>  drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c      |  6 ------
>  drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c      |  6 ------

Please also fix up gmc_v6_0.c.  With that fixed:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

>  3 files changed, 1 insertion(+), 41 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 65fba5fb537e..f9bee76e7071 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -578,41 +578,13 @@ void amdgpu_wb_free(struct amdgpu_device *adev, u32 wb)
>   * @base: base address at which to put VRAM
>   *
>   * Function will try to place VRAM at base address provided
> - * as parameter (which is so far either PCI aperture address or
> - * for IGP TOM base address).
> - *
> - * If there is not enough space to fit the unvisible VRAM in the 32bits
> - * address space then we limit the VRAM size to the aperture.
> - *
> - * Note: We don't explicitly enforce VRAM start to be aligned on VRAM size,
> - * this shouldn't be a problem as we are using the PCI aperture as a reference.
> - * Otherwise this would be needed for rv280, all r3xx, and all r4xx, but
> - * not IGP.
> - *
> - * Note: we use mc_vram_size as on some board we need to program the mc to
> - * cover the whole aperture even if VRAM size is inferior to aperture size
> - * Novell bug 204882 + along with lots of ubuntu ones
> - *
> - * Note: when limiting vram it's safe to overwritte real_vram_size because
> - * we are not in case where real_vram_size is inferior to mc_vram_size (ie
> - * note afected by bogus hw of Novell bug 204882 + along with lots of ubuntu
> - * ones)
> - *
> - * Note: IGP TOM addr should be the same as the aperture addr, we don't
> - * explicitly check for that though.
> - *
> - * FIXME: when reducing VRAM size align new size on power of 2.
> + * as parameter.
>   */
>  void amdgpu_vram_location(struct amdgpu_device *adev, struct amdgpu_mc *mc, u64 base)
>  {
>         uint64_t limit = (uint64_t)amdgpu_vram_limit << 20;
>
>         mc->vram_start = base;
> -       if (mc->mc_vram_size > (adev->mc.mc_mask - base + 1)) {
> -               dev_warn(adev->dev, "limiting VRAM to PCI aperture size\n");
> -               mc->real_vram_size = mc->aper_size;
> -               mc->mc_vram_size = mc->aper_size;
> -       }
>         mc->vram_end = mc->vram_start + mc->mc_vram_size - 1;
>         if (limit && limit < mc->real_vram_size)
>                 mc->real_vram_size = limit;
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> index de7a249f0e24..d521862804ea 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> @@ -240,12 +240,6 @@ static void gmc_v7_0_vram_gtt_location(struct amdgpu_device *adev,
>         u64 base = RREG32(mmMC_VM_FB_LOCATION) & 0xFFFF;
>         base <<= 24;
>
> -       if (mc->mc_vram_size > 0xFFC0000000ULL) {
> -               /* leave room for at least 1024M GTT */
> -               dev_warn(adev->dev, "limiting VRAM\n");
> -               mc->real_vram_size = 0xFFC0000000ULL;
> -               mc->mc_vram_size = 0xFFC0000000ULL;
> -       }
>         amdgpu_vram_location(adev, &adev->mc, base);
>         amdgpu_gart_location(adev, mc);
>  }
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> index 67778744da5a..bd3f842cca00 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> @@ -405,12 +405,6 @@ static void gmc_v8_0_vram_gtt_location(struct amdgpu_device *adev,
>                 base = RREG32(mmMC_VM_FB_LOCATION) & 0xFFFF;
>         base <<= 24;
>
> -       if (mc->mc_vram_size > 0xFFC0000000ULL) {
> -               /* leave room for at least 1024M GTT */
> -               dev_warn(adev->dev, "limiting VRAM\n");
> -               mc->real_vram_size = 0xFFC0000000ULL;
> -               mc->mc_vram_size = 0xFFC0000000ULL;
> -       }
>         amdgpu_vram_location(adev, &adev->mc, base);
>         amdgpu_gart_location(adev, mc);
>  }
> --
> 2.11.0
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list