All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/amdgpu: gtt offset should be 64 bit
@ 2016-09-14  9:13 Flora Cui
       [not found] ` <1473844418-22724-1-git-send-email-Flora.Cui-5C7GfCeVMHo@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Flora Cui @ 2016-09-14  9:13 UTC (permalink / raw)
  To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Flora Cui

Change-Id: I05618a472b1ebf381575d02e226264609bf4db7f
Signed-off-by: Flora Cui <Flora.Cui@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu.h      |  4 ++--
 drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c | 10 ++++------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index 282e00d..f07f1c9 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
@@ -619,9 +619,9 @@ int amdgpu_gart_table_vram_pin(struct amdgpu_device *adev);
 void amdgpu_gart_table_vram_unpin(struct amdgpu_device *adev);
 int amdgpu_gart_init(struct amdgpu_device *adev);
 void amdgpu_gart_fini(struct amdgpu_device *adev);
-void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
+void amdgpu_gart_unbind(struct amdgpu_device *adev, u64 offset,
 			int pages);
-int amdgpu_gart_bind(struct amdgpu_device *adev, unsigned offset,
+int amdgpu_gart_bind(struct amdgpu_device *adev, u64 offset,
 		     int pages, struct page **pagelist,
 		     dma_addr_t *dma_addr, uint32_t flags);
 int amdgpu_ttm_recover_gart(struct amdgpu_device *adev);
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
index 921bce2..7968f77 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
@@ -221,11 +221,10 @@ void amdgpu_gart_table_vram_free(struct amdgpu_device *adev)
  * Unbinds the requested pages from the gart page table and
  * replaces them with the dummy page (all asics).
  */
-void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
+void amdgpu_gart_unbind(struct amdgpu_device *adev, u64 offset,
 			int pages)
 {
-	unsigned t;
-	unsigned p;
+	u64 t, p;
 	int i, j;
 	u64 page_base;
 	uint32_t flags = AMDGPU_PTE_SYSTEM;
@@ -268,12 +267,11 @@ void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
  * (all asics).
  * Returns 0 for success, -EINVAL for failure.
  */
-int amdgpu_gart_bind(struct amdgpu_device *adev, unsigned offset,
+int amdgpu_gart_bind(struct amdgpu_device *adev, u64 offset,
 		     int pages, struct page **pagelist, dma_addr_t *dma_addr,
 		     uint32_t flags)
 {
-	unsigned t;
-	unsigned p;
+	uint64_t t, p;
 	uint64_t page_base;
 	int i, j;
 
-- 
2.7.4

_______________________________________________
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
* Re: [PATCH] drm/amdgpu: gtt offset should be 64 bit
       [not found] ` <1473844418-22724-1-git-send-email-Flora.Cui-5C7GfCeVMHo@public.gmane.org>
@ 2016-09-14  9:32   ` Christian König
       [not found]     ` <4a840c0a-30bd-836d-7d50-8bc8c7565f4e-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Christian König @ 2016-09-14  9:32 UTC (permalink / raw)
  To: Flora Cui, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

Am 14.09.2016 um 11:13 schrieb Flora Cui:
> Change-Id: I05618a472b1ebf381575d02e226264609bf4db7f
> Signed-off-by: Flora Cui <Flora.Cui@amd.com>

Didn't Felix came up with the same patch a few weeks ago?

Anyway the fix is clearly not committed. Patch is Reviewed-by: Christian 
König <christian.koenig@amd.com>.

Regards,
Christian.

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu.h      |  4 ++--
>   drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c | 10 ++++------
>   2 files changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index 282e00d..f07f1c9 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -619,9 +619,9 @@ int amdgpu_gart_table_vram_pin(struct amdgpu_device *adev);
>   void amdgpu_gart_table_vram_unpin(struct amdgpu_device *adev);
>   int amdgpu_gart_init(struct amdgpu_device *adev);
>   void amdgpu_gart_fini(struct amdgpu_device *adev);
> -void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
> +void amdgpu_gart_unbind(struct amdgpu_device *adev, u64 offset,
>   			int pages);
> -int amdgpu_gart_bind(struct amdgpu_device *adev, unsigned offset,
> +int amdgpu_gart_bind(struct amdgpu_device *adev, u64 offset,
>   		     int pages, struct page **pagelist,
>   		     dma_addr_t *dma_addr, uint32_t flags);
>   int amdgpu_ttm_recover_gart(struct amdgpu_device *adev);
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> index 921bce2..7968f77 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> @@ -221,11 +221,10 @@ void amdgpu_gart_table_vram_free(struct amdgpu_device *adev)
>    * Unbinds the requested pages from the gart page table and
>    * replaces them with the dummy page (all asics).
>    */
> -void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
> +void amdgpu_gart_unbind(struct amdgpu_device *adev, u64 offset,
>   			int pages)
>   {
> -	unsigned t;
> -	unsigned p;
> +	u64 t, p;
>   	int i, j;
>   	u64 page_base;
>   	uint32_t flags = AMDGPU_PTE_SYSTEM;
> @@ -268,12 +267,11 @@ void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
>    * (all asics).
>    * Returns 0 for success, -EINVAL for failure.
>    */
> -int amdgpu_gart_bind(struct amdgpu_device *adev, unsigned offset,
> +int amdgpu_gart_bind(struct amdgpu_device *adev, u64 offset,
>   		     int pages, struct page **pagelist, dma_addr_t *dma_addr,
>   		     uint32_t flags)
>   {
> -	unsigned t;
> -	unsigned p;
> +	uint64_t t, p;
>   	uint64_t page_base;
>   	int i, j;
>   


_______________________________________________
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
* Re: [PATCH] drm/amdgpu: gtt offset should be 64 bit
       [not found]     ` <4a840c0a-30bd-836d-7d50-8bc8c7565f4e-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org>
@ 2016-09-14  9:43       ` Flora Cui
  0 siblings, 0 replies; 3+ messages in thread
From: Flora Cui @ 2016-09-14  9:43 UTC (permalink / raw)
  To: Christian König; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

yes. i just find it out. it's commit 9040cf0a in 4.6 branch yet not in
4.7. please ignore this commit.

On Wed, Sep 14, 2016 at 11:32:32AM +0200, Christian König wrote:
> Am 14.09.2016 um 11:13 schrieb Flora Cui:
> >Change-Id: I05618a472b1ebf381575d02e226264609bf4db7f
> >Signed-off-by: Flora Cui <Flora.Cui@amd.com>
> 
> Didn't Felix came up with the same patch a few weeks ago?
> 
> Anyway the fix is clearly not committed. Patch is Reviewed-by: Christian
> König <christian.koenig@amd.com>.
> 
> Regards,
> Christian.
> 
> >---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu.h      |  4 ++--
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c | 10 ++++------
> >  2 files changed, 6 insertions(+), 8 deletions(-)
> >
> >diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> >index 282e00d..f07f1c9 100644
> >--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> >+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> >@@ -619,9 +619,9 @@ int amdgpu_gart_table_vram_pin(struct amdgpu_device *adev);
> >  void amdgpu_gart_table_vram_unpin(struct amdgpu_device *adev);
> >  int amdgpu_gart_init(struct amdgpu_device *adev);
> >  void amdgpu_gart_fini(struct amdgpu_device *adev);
> >-void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
> >+void amdgpu_gart_unbind(struct amdgpu_device *adev, u64 offset,
> >  			int pages);
> >-int amdgpu_gart_bind(struct amdgpu_device *adev, unsigned offset,
> >+int amdgpu_gart_bind(struct amdgpu_device *adev, u64 offset,
> >  		     int pages, struct page **pagelist,
> >  		     dma_addr_t *dma_addr, uint32_t flags);
> >  int amdgpu_ttm_recover_gart(struct amdgpu_device *adev);
> >diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >index 921bce2..7968f77 100644
> >--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >@@ -221,11 +221,10 @@ void amdgpu_gart_table_vram_free(struct amdgpu_device *adev)
> >   * Unbinds the requested pages from the gart page table and
> >   * replaces them with the dummy page (all asics).
> >   */
> >-void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
> >+void amdgpu_gart_unbind(struct amdgpu_device *adev, u64 offset,
> >  			int pages)
> >  {
> >-	unsigned t;
> >-	unsigned p;
> >+	u64 t, p;
> >  	int i, j;
> >  	u64 page_base;
> >  	uint32_t flags = AMDGPU_PTE_SYSTEM;
> >@@ -268,12 +267,11 @@ void amdgpu_gart_unbind(struct amdgpu_device *adev, unsigned offset,
> >   * (all asics).
> >   * Returns 0 for success, -EINVAL for failure.
> >   */
> >-int amdgpu_gart_bind(struct amdgpu_device *adev, unsigned offset,
> >+int amdgpu_gart_bind(struct amdgpu_device *adev, u64 offset,
> >  		     int pages, struct page **pagelist, dma_addr_t *dma_addr,
> >  		     uint32_t flags)
> >  {
> >-	unsigned t;
> >-	unsigned p;
> >+	uint64_t t, p;
> >  	uint64_t page_base;
> >  	int i, j;
> 
> 
_______________________________________________
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:[~2016-09-14  9:43 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-14  9:13 [PATCH] drm/amdgpu: gtt offset should be 64 bit Flora Cui
     [not found] ` <1473844418-22724-1-git-send-email-Flora.Cui-5C7GfCeVMHo@public.gmane.org>
2016-09-14  9:32   ` Christian König
     [not found]     ` <4a840c0a-30bd-836d-7d50-8bc8c7565f4e-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org>
2016-09-14  9:43       ` Flora Cui

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.