* [PATCH] drm/amdgpu: correct GART location info
@ 2018-06-19 4:41 Junwei Zhang
[not found] ` <1529383295-10431-1-git-send-email-Jerry.Zhang-5C7GfCeVMHo@public.gmane.org>
0 siblings, 1 reply; 3+ messages in thread
From: Junwei Zhang @ 2018-06-19 4:41 UTC (permalink / raw)
To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Junwei Zhang
Signed-off-by: Junwei Zhang <Jerry.Zhang@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index f77b07b..f9fe8d3 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -676,17 +676,17 @@ void amdgpu_device_vram_location(struct amdgpu_device *adev,
}
/**
- * amdgpu_device_gart_location - try to find GTT location
+ * amdgpu_device_gart_location - try to find GART location
*
* @adev: amdgpu device structure holding all necessary informations
* @mc: memory controller structure holding memory informations
*
- * Function will place try to place GTT before or after VRAM.
+ * Function will place try to place GART before or after VRAM.
*
- * If GTT size is bigger than space left then we ajust GTT size.
+ * If GART size is bigger than space left then we ajust GART size.
* Thus function will never fails.
*
- * FIXME: when reducing GTT size align new size on power of 2.
+ * FIXME: when reducing GART size align new size on power of 2.
*/
void amdgpu_device_gart_location(struct amdgpu_device *adev,
struct amdgpu_gmc *mc)
@@ -699,13 +699,13 @@ void amdgpu_device_gart_location(struct amdgpu_device *adev,
size_bf = mc->vram_start;
if (size_bf > size_af) {
if (mc->gart_size > size_bf) {
- dev_warn(adev->dev, "limiting GTT\n");
+ dev_warn(adev->dev, "limiting GART\n");
mc->gart_size = size_bf;
}
mc->gart_start = 0;
} else {
if (mc->gart_size > size_af) {
- dev_warn(adev->dev, "limiting GTT\n");
+ dev_warn(adev->dev, "limiting GART\n");
mc->gart_size = size_af;
}
/* VCE doesn't like it when BOs cross a 4GB segment, so align
@@ -714,7 +714,7 @@ void amdgpu_device_gart_location(struct amdgpu_device *adev,
mc->gart_start = ALIGN(mc->vram_end + 1, 0x100000000ULL);
}
mc->gart_end = mc->gart_start + mc->gart_size - 1;
- dev_info(adev->dev, "GTT: %lluM 0x%016llX - 0x%016llX\n",
+ dev_info(adev->dev, "GART: %lluM 0x%016llX - 0x%016llX\n",
mc->gart_size >> 20, mc->gart_start, mc->gart_end);
}
--
1.9.1
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 3+ messages in thread[parent not found: <1529383295-10431-1-git-send-email-Jerry.Zhang-5C7GfCeVMHo@public.gmane.org>]
* Re: [PATCH] drm/amdgpu: correct GART location info [not found] ` <1529383295-10431-1-git-send-email-Jerry.Zhang-5C7GfCeVMHo@public.gmane.org> @ 2018-06-19 7:04 ` Christian König [not found] ` <476e59ac-1cba-d3ca-2670-6be0d8af4b82-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 0 siblings, 1 reply; 3+ messages in thread From: Christian König @ 2018-06-19 7:04 UTC (permalink / raw) To: Junwei Zhang, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW We need a commit message, something like "Avoid confusing the GART with the GTT domain.". Am 19.06.2018 um 06:41 schrieb Junwei Zhang: > Signed-off-by: Junwei Zhang <Jerry.Zhang@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index f77b07b..f9fe8d3 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -676,17 +676,17 @@ void amdgpu_device_vram_location(struct amdgpu_device *adev, > } > > /** > - * amdgpu_device_gart_location - try to find GTT location > + * amdgpu_device_gart_location - try to find GART location > * > * @adev: amdgpu device structure holding all necessary informations > * @mc: memory controller structure holding memory informations > * > - * Function will place try to place GTT before or after VRAM. > + * Function will place try to place GART before or after VRAM. > * > - * If GTT size is bigger than space left then we ajust GTT size. > + * If GART size is bigger than space left then we ajust GART size. > * Thus function will never fails. > * > - * FIXME: when reducing GTT size align new size on power of 2. > + * FIXME: when reducing GART size align new size on power of 2. Please just drop this line. IIRC we actually don't align the gartsize parameter to power of two any more either. With that fixed the patch is Reviewed-by: Christian König <christian.koenig@amd.com>. Thanks, Christian. > */ > void amdgpu_device_gart_location(struct amdgpu_device *adev, > struct amdgpu_gmc *mc) > @@ -699,13 +699,13 @@ void amdgpu_device_gart_location(struct amdgpu_device *adev, > size_bf = mc->vram_start; > if (size_bf > size_af) { > if (mc->gart_size > size_bf) { > - dev_warn(adev->dev, "limiting GTT\n"); > + dev_warn(adev->dev, "limiting GART\n"); > mc->gart_size = size_bf; > } > mc->gart_start = 0; > } else { > if (mc->gart_size > size_af) { > - dev_warn(adev->dev, "limiting GTT\n"); > + dev_warn(adev->dev, "limiting GART\n"); > mc->gart_size = size_af; > } > /* VCE doesn't like it when BOs cross a 4GB segment, so align > @@ -714,7 +714,7 @@ void amdgpu_device_gart_location(struct amdgpu_device *adev, > mc->gart_start = ALIGN(mc->vram_end + 1, 0x100000000ULL); > } > mc->gart_end = mc->gart_start + mc->gart_size - 1; > - dev_info(adev->dev, "GTT: %lluM 0x%016llX - 0x%016llX\n", > + dev_info(adev->dev, "GART: %lluM 0x%016llX - 0x%016llX\n", > mc->gart_size >> 20, mc->gart_start, mc->gart_end); > } > _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <476e59ac-1cba-d3ca-2670-6be0d8af4b82-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]
* Re: [PATCH] drm/amdgpu: correct GART location info [not found] ` <476e59ac-1cba-d3ca-2670-6be0d8af4b82-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> @ 2018-06-19 7:08 ` Zhang, Jerry (Junwei) 0 siblings, 0 replies; 3+ messages in thread From: Zhang, Jerry (Junwei) @ 2018-06-19 7:08 UTC (permalink / raw) To: christian.koenig-5C7GfCeVMHo, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW On 06/19/2018 03:04 PM, Christian König wrote: > We need a commit message, something like "Avoid confusing the GART with the GTT > domain.". Yeah, will add such kind of info. > > Am 19.06.2018 um 06:41 schrieb Junwei Zhang: >> Signed-off-by: Junwei Zhang <Jerry.Zhang@amd.com> >> --- >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 14 +++++++------- >> 1 file changed, 7 insertions(+), 7 deletions(-) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> index f77b07b..f9fe8d3 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> @@ -676,17 +676,17 @@ void amdgpu_device_vram_location(struct amdgpu_device >> *adev, >> } >> /** >> - * amdgpu_device_gart_location - try to find GTT location >> + * amdgpu_device_gart_location - try to find GART location >> * >> * @adev: amdgpu device structure holding all necessary informations >> * @mc: memory controller structure holding memory informations >> * >> - * Function will place try to place GTT before or after VRAM. >> + * Function will place try to place GART before or after VRAM. >> * >> - * If GTT size is bigger than space left then we ajust GTT size. >> + * If GART size is bigger than space left then we ajust GART size. >> * Thus function will never fails. >> * >> - * FIXME: when reducing GTT size align new size on power of 2. >> + * FIXME: when reducing GART size align new size on power of 2. > > Please just drop this line. IIRC we actually don't align the gartsize parameter > to power of two any more either. Got it. Thanks. Jerry > > With that fixed the patch is Reviewed-by: Christian König > <christian.koenig@amd.com>. > > Thanks, > Christian. > >> */ >> void amdgpu_device_gart_location(struct amdgpu_device *adev, >> struct amdgpu_gmc *mc) >> @@ -699,13 +699,13 @@ void amdgpu_device_gart_location(struct amdgpu_device >> *adev, >> size_bf = mc->vram_start; >> if (size_bf > size_af) { >> if (mc->gart_size > size_bf) { >> - dev_warn(adev->dev, "limiting GTT\n"); >> + dev_warn(adev->dev, "limiting GART\n"); >> mc->gart_size = size_bf; >> } >> mc->gart_start = 0; >> } else { >> if (mc->gart_size > size_af) { >> - dev_warn(adev->dev, "limiting GTT\n"); >> + dev_warn(adev->dev, "limiting GART\n"); >> mc->gart_size = size_af; >> } >> /* VCE doesn't like it when BOs cross a 4GB segment, so align >> @@ -714,7 +714,7 @@ void amdgpu_device_gart_location(struct amdgpu_device *adev, >> mc->gart_start = ALIGN(mc->vram_end + 1, 0x100000000ULL); >> } >> mc->gart_end = mc->gart_start + mc->gart_size - 1; >> - dev_info(adev->dev, "GTT: %lluM 0x%016llX - 0x%016llX\n", >> + dev_info(adev->dev, "GART: %lluM 0x%016llX - 0x%016llX\n", >> mc->gart_size >> 20, mc->gart_start, mc->gart_end); >> } > _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-06-19 7:08 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-19 4:41 [PATCH] drm/amdgpu: correct GART location info Junwei Zhang
[not found] ` <1529383295-10431-1-git-send-email-Jerry.Zhang-5C7GfCeVMHo@public.gmane.org>
2018-06-19 7:04 ` Christian König
[not found] ` <476e59ac-1cba-d3ca-2670-6be0d8af4b82-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2018-06-19 7:08 ` Zhang, Jerry (Junwei)
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.