* [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
[parent not found: <1473844418-22724-1-git-send-email-Flora.Cui-5C7GfCeVMHo@public.gmane.org>]
* 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
[parent not found: <4a840c0a-30bd-836d-7d50-8bc8c7565f4e-ANTagKRnAhcb1SvskN2V4Q@public.gmane.org>]
* 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.