On Tue, Jan 09, 2018 at 01:13:08PM -0500, Alex Deucher wrote:
> On Tue, Jan 9, 2018 at 10:56 AM, Deucher, Alexander
> wrote:
> > I'll can push this and a few other misc patches today.
> >
>
> Pushed to drm-misc-next.
One thing I just noticed: Some kerneldoc for the newly exported functions
and m
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index ae976e3..5f2ae77 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.
Am 10.01.2018 um 09:18 schrieb Junwei Zhang:
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index ae976e3..5f2ae77
On 01/10/2018 04:57 PM, Christian König wrote:
Am 10.01.2018 um 09:18 schrieb Junwei Zhang:
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
b/drivers/gpu/dr
On Tue, Jan 2, 2018 at 8:10 PM, Yong Zhao wrote:
> Signed-off-by: Yong Zhao
> Reviewed-by: Oak Zeng
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manag
Signed-off-by: Oded Gabbay
---
drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
index 6a48d29ada47..0bedcf9cc08c 100644
--- a/drivers/gpu/drm/amd/amdkfd/kf
Change-Id: I0a554cb6a7a56db63a8fc5af60d5c63f65e021d1
Signed-off-by: Rex Zhu
---
drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 39 +++
drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 78 ++
drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.h | 1 +
drivers/g
when cat pp_power_profile_mode on Vega10
NUMMODE_NAME BUSY_SET_POINT FPS USE_RLC_BUSY MIN_ACTIVE_LEVEL
0 3D_FULL_SCREEN : 70 60 1 3
1 POWER_SAVING : 90 60 0 0
2 VIDEO*: 70 60 0
On Wed, Jan 10, 2018 at 1:01 PM, Rex Zhu wrote:
> Change-Id: I0a554cb6a7a56db63a8fc5af60d5c63f65e021d1
> Signed-off-by: Rex Zhu
> ---
> drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 39 +++
> drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 78
> ++
> driv
Trace all allocated PASIDs.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 12 ++--
drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 22 ++
2 files changed, 32 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms
Start to always allocate a pasid for each VM.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 43 ++---
1 file changed, 29 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
b/drivers/gpu/drm/amd/amdgpu/a
Makes more sense than tracing the kernel pointer.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h
index
Change reservation_object_get_fences_rcu to make the exclusive fence
pointer optional.
If not specified the exclusive fence is put into the fence array as
well.
This is helpful for a couple of cases where we need all fences in a
single array.
Signed-off-by: Christian König
---
drivers/dma-buf/
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/vce_v4_0.c | 14 +++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/vce_v4_0.c
b/drivers/gpu/drm/amd/amdgpu/vce_v4_0.c
index 4e93dfe945
Print that extra information on GMC v7.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
index b73599912b42..6b
Free up a pasid after all fences signaled.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c | 73 +
drivers/gpu/drm/amd/amdgpu/amdgpu_ids.h | 2 +
2 files changed, 75 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c
b
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index 5a9cf6a8
rd the pasid from the VM code to the emit_vm_flush function and update
all implementations with the new parameter.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 5 +++--
drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c | 2 ++
drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h | 2
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
index a17db3ff7e5c..f55
As soon as the lock is dropped the VM pointer can be invalid.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c | 26 +++---
1 file changed, 19 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c
b/drivers/gpu/drm/amd/amdgpu/uvd_v7_0.c
inde
sed -i "s/pas_id/pasid/g" drivers/gpu/drm/amd/amdgpu/*.c
sed -i "s/pas_id/pasid/g" drivers/gpu/drm/amd/amdgpu/*.h
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ih.h| 2 +-
drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 8
drivers/gpu/drm/amd/amdgpu/cik_ih.c
Print that extra information on GMC v8.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
index 287228315b76..5b
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
b/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
index b843f5bc52c9..824
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
index ff9111656ad7..
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/cik_sdma.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
b/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
index 985a6868b0a8..1d4787b
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c | 25 +++--
1 file changed, 19 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
b/drivers/gpu/drm/amd/amdgpu/vcn_v1_0.c
index
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 14 +++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
index 4d1f9404d1
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
index 64926191
This way we can see the PASID in VM faults.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
index c6e22eff89e7..
Reviewed-by: Ken Wang
From: Junwei Zhang
Sent: Wednesday, January 10, 2018 4:18:30 PM
To: amd-gfx@lists.freedesktop.org
Cc: Wang, Ken; Zhang, Jerry
Subject: [PATCH] drm/amdgpu/gfx9: fix ngg enablement to clear gds reserved
memory
Signed-off-by: Junwei Zhang
---
On Thu, Dec 21, 2017 at 1:45 PM, Daniel Drake wrote:
> Hi,
Hi again,
in Endless we are still actively working with DC on several laptops,
so I just wanted to give you a quick update regarding the regressions
indicated in Daniel's email.
> 1. System hangs on S3 resume.
> Bug can't be reproduced w
OK. I'll do it.
Samuel Li
> -Original Message-
> From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel
> Vetter
> Sent: Wednesday, January 10, 2018 3:17 AM
> To: Alex Deucher
> Cc: Deucher, Alexander ; Li, Samuel
> ; Daniel Vetter ; Koenig, Christian
> ; dri-de...@list
Ping? Alex, David, anybody?
Am 08.01.2018 um 14:51 schrieb Christian König:
Windows added by the BIOS are not marked as 64bit because they are
usually not changeable anyway.
This fixes large BAR support on my new Ryzen build system.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amd
Reviewed-by: Felix Kuehling
On 2018-01-10 06:01 AM, Oded Gabbay wrote:
> Signed-off-by: Oded Gabbay
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
> b/drivers/gpu/drm/amd/amdkfd
On Mon, Jan 8, 2018 at 8:51 AM, Christian König
wrote:
> Windows added by the BIOS are not marked as 64bit because they are
> usually not changeable anyway.
>
> This fixes large BAR support on my new Ryzen build system.
>
> Signed-off-by: Christian König
Reviewed-by: Alex Deucher
> ---
> driv
In case kfd_get_process_device_data returns null, there are some
null pointer dereferences in functions kfd_bind_processes_to_device
and kfd_unbind_processes_from_device.
Fix this by null checking pdd before dereferencing it.
Addresses-Coverity-ID: 1463794 ("Dereference null return value")
Addres
Hi Gustavo,
Thanks for catching that. When returning a fault, I think you also need
to srcu_read_unlock(&kfd_processes_srcu, idx).
However, instead of returning an error, I think I'd prefer to skip PDDs
that can't be found with continue statements. That way others would
still suspend and resume s
On Wed, Jan 10, 2018 at 6:01 AM, Rex Zhu wrote:
> when cat pp_power_profile_mode on Vega10
> NUMMODE_NAME BUSY_SET_POINT FPS USE_RLC_BUSY MIN_ACTIVE_LEVEL
> 0 3D_FULL_SCREEN : 70 60 1 3
> 1 POWER_SAVING : 90 60 0 0
Hi Dave,
A few fixes for 4.16:
- Cleanup the the remains of ttm io_mem_pfn
- A couple dpm quirks for SI
- Add Chunming as another amdgpu maintainer
- A few more huge page fixes
- A few other misc fixes
The following changes since commit fb8baefc7b2d7b93ad96abacbe63fa99e3d213d6:
drm/amdgpu: use
FWIW, Christian's "simplify huge page handling" broke CPU page table
updates for KFD on large-BAR Vega10. It doesn't matter for Dave because
KFD support on dGPUs isn't upstream yet.
Regards,
Felix
On 2018-01-10 04:18 PM, Alex Deucher wrote:
> Hi Dave,
>
> A few fixes for 4.16:
> - Cleanup the
On 01/09/2018 09:44 AM, Johannes Hirte wrote:
On 2018 Jan 03, Johannes Hirte wrote:
On 2018 Jan 03, Johannes Hirte wrote:
This should be fixed already with
https://lists.freedesktop.org/archives/amd-gfx/2017-October/014932.html
but's still missing upstream.
With this patch, the use-after-fr
The new register settings are needed to fix a tlb invalidation issue
when MMHUB power gating is turned on for Raven.
Change-Id: I846befbb2fcbddf40ca4ecbdc06da1cd442e3554
Signed-off-by: Yong Zhao
---
drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 61 ++---
1 file changed, 3
On Wed, Jan 10, 2018 at 3:10 PM, Yong Zhao wrote:
> The new register settings are needed to fix a tlb invalidation issue
> when MMHUB power gating is turned on for Raven.
>
> Change-Id: I846befbb2fcbddf40ca4ecbdc06da1cd442e3554
> Signed-off-by: Yong Zhao
Acked-by: Alex Deucher
> ---
> drivers
Yes, this new interface is more common, I just save the input and send to hw
backend. Smu7 and Vega can share this interface.
Best Regards
Rex
From: Alex Deucher
Sent: Thursday, January 11, 2018 4:18:11 AM
To: Zhu, Rex
Cc: amd-gfx list
Subject: Re: [PATCH 1/2] d
Yeah, this looks good to me.
Regards,
Felix
On 2018-01-10 04:58 PM, Gustavo A. R. Silva wrote:
> Hi Felix,
>
> Quoting Felix Kuehling :
>
>> Hi Gustavo,
>>
>> Thanks for catching that. When returning a fault, I think you also need
>> to srcu_read_unlock(&kfd_processes_srcu, idx).
>>
>> However
v2: fix register access
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index ae976e3..c73a476 100644
--- a/drivers/
In case kfd_get_process_device_data returns null, there are some
null pointer dereferences in functions kfd_bind_processes_to_device
and kfd_unbind_processes_from_device.
Fix this by printing a WARN_ON for PDDs that aren't found and skip
them with continue statements.
Addresses-Coverity-ID: 14637
Hi Felix,
Quoting Felix Kuehling :
Hi Gustavo,
Thanks for catching that. When returning a fault, I think you also need
to srcu_read_unlock(&kfd_processes_srcu, idx).
However, instead of returning an error, I think I'd prefer to skip PDDs
that can't be found with continue statements. That way
v2: fix register access
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index ae976e3..6db3645 100644
--- a/drivers/gp
Please ignore this one, a minor update is coming.
Sorry for annoyance.
Jerry
On 01/11/2018 09:44 AM, Junwei Zhang wrote:
v2: fix register access
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --gi
Could I how to verify this is valid fix?
Regards,
David Zhou
On 2018年01月11日 10:30, Junwei Zhang wrote:
v2: fix register access
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/dr
On 01/11/2018 10:50 AM, Chunming Zhou wrote:
Could I how to verify this is valid fix?
For now, check if there is no vm fault or any other side effect when loading
amdgpu with ngg=1.
Later development will be implemented with OGL team together.
Jerry
Regards,
David Zhou
On 2018年01月11日
On 2018年01月10日 20:54, Christian König wrote:
Free up a pasid after all fences signaled.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ids.c | 73 +
drivers/gpu/drm/amd/amdgpu/amdgpu_ids.h | 2 +
2 files changed, 75 insertions(+)
dif
On 2018年01月10日 20:54, Christian König wrote:
Trace all allocated PASIDs.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 12 ++--
drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 22 ++
2 files changed, 32 insertions(+), 2 deletions(-)
On 2018年01月10日 20:54, Christian König wrote:
Start to always allocate a pasid for each VM.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 43 ++---
1 file changed, 29 insertions(+), 14 deletions(-)
diff --git a/drivers/gpu/drm/amd/
Except some small nitpicks in patch #3, #4, #5, the series looks ok to
me, Reviewed-by: Chunming Zhou
Regards,
David Zhou
On 2018年01月10日 20:54, Christian König wrote:
Print that extra information on GMC v8.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 13
57 matches
Mail list logo