Re: [PATCH] drm/amdgpu: re-validate per VM BOs if required

On 2018年03月21日 20:00, Christian König wrote: Am 21.03.2018 um 11:31 schrieb zhoucm1: On 2018年03月20日 17:13, zhoucm1 wrote: On 2018年03月20日 15:49, zhoucm1 wrote: On 2018年03月19日 18:50, Christian König wrote: If a per VM BO ends up in a allowed domain it never moves back into the

RE: [PATCH] drm/amdgpu: give more chance for tlb flush if failed

Hi Christian, I agree with that the patch will hide the real problem, it is just a workaround, I will change the patch as you suggest. as the sriov has lots of issues on the staging, maybe we could first submit the two workarounds, later, I will spend some time to find out the root cause.

RE: [PATCH] drm/amdgpu: Fix NULL ptr on driver unload due to init failure.

kfree(adev->irq.client[i].sources); +adev->irq.client[i].sources = NULL; Set adev->irq.client[i].sources to NULL in amdgpu_irq_fini also can fix NULL ptr in amdgpu_irq_disable_all.. But I didn't check why amdgpu_device_fini was called twice. This patch looks good. Best Regards Rex

Re: [trivial PATCH V2] treewide: Align function definition open/close braces

On Wed, Mar 21, 2018 at 03:09:32PM -0700, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work > properly for

[trivial PATCH V2] treewide: Align function definition open/close braces

Some functions definitions have either the initial open brace and/or the closing brace outside of column 1. Move those braces to column 1. This allows various function analyzers like gnu complexity to work properly for these modified functions. Signed-off-by: Joe Perches

Re: [trivial PATCH V2] treewide: Align function definition open/close braces

Joe, > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. drivers/scsi and drivers/message/fusion parts look fine. Acked-by: Martin K. Petersen -- Martin K. Petersen

[pull] radeon and amdgpu drm-fixes-4.16

Hi Dave, A few more fixes for 4.16. Mostly for displays: - A fix for DP handling on radeon - Fix banding on eDP panels - Fix HBR audio - Fix for disabling VGA mode on Raven that leads to a corrupt or blank display on some platforms The following changes since commit

Re: [trivial PATCH V2] treewide: Align function definition open/close braces

On Wed, Mar 21, 2018 at 11:09 PM, Joe Perches wrote: > Some functions definitions have either the initial open brace and/or > the closing brace outside of column 1. > > Move those braces to column 1. > > This allows various function analyzers like gnu complexity to work >

Re: [PATCH] drm/amd/pp: use mlck_table.count for array loop index limit

On Wed, 2018-03-21 at 18:26 +, Colin King wrote: > From: Colin Ian King > > The for-loops process data in the mclk_table but use slck_table.count > as the loop index limit. I believe these are cut-n-paste errors from > the previous almost identical loops as

[PATCH] drm/amd/pp: use mlck_table.count for array loop index limit

From: Colin Ian King The for-loops process data in the mclk_table but use slck_table.count as the loop index limit. I believe these are cut-n-paste errors from the previous almost identical loops as indicated by static analysis. Fix these. Detected by CoverityScan,

Re: [PATCH] drm/amdgpu: Fix NULL ptr on driver unload due to init failure.

On Wed, Mar 21, 2018 at 2:22 PM, Andrey Grodzovsky wrote: > Problem: > When unloading due to failure amdgpu_device_fini was called twice > which was leading to NULL ptr in amdgpu_irq_disable_all. > > Fix: > Call amdgpu_device_fini only once from

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

Am 21.03.2018 um 19:23 schrieb Marek Olšák: On Wed, Mar 21, 2018 at 2:15 PM, Christian König > wrote: Am 21.03.2018 um 19:04 schrieb Marek Olšák: On Wed, Mar 21, 2018 at 10:07 AM, Christian König

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

On Wed, Mar 21, 2018 at 2:15 PM, Christian König wrote: > Am 21.03.2018 um 19:04 schrieb Marek Olšák: > > On Wed, Mar 21, 2018 at 10:07 AM, Christian König < > christian.koe...@amd.com> wrote: > >> Am 21.03.2018 um 14:57 schrieb Marek Olšák: >> >> On Wed, Mar 21, 2018

[PATCH] drm/amdgpu: Fix NULL ptr on driver unload due to init failure.

Problem: When unloading due to failure amdgpu_device_fini was called twice which was leading to NULL ptr in amdgpu_irq_disable_all. Fix: Call amdgpu_device_fini only once from amdgpu_driver_unload_kms. Signed-off-by: Andrey Grodzovsky ---

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

Am 21.03.2018 um 19:04 schrieb Marek Olšák: On Wed, Mar 21, 2018 at 10:07 AM, Christian König > wrote: Am 21.03.2018 um 14:57 schrieb Marek Olšák: On Wed, Mar 21, 2018 at 4:13 AM, Christian König

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

On Wed, Mar 21, 2018 at 10:07 AM, Christian König wrote: > Am 21.03.2018 um 14:57 schrieb Marek Olšák: > > On Wed, Mar 21, 2018 at 4:13 AM, Christian König < > ckoenig.leichtzumer...@gmail.com> wrote: > >> Am 21.03.2018 um 06:08 schrieb Marek Olšák: >> >> On Tue, Mar

Re: [PATCH 00/20] Add KFD GPUVM support for dGPUs v4

On 2018-03-21 03:52 AM, Oded Gabbay wrote: > > Hi Felix, > I did a quick pass on the patch-set and didn't see anything scary. > Patches 1-14 are already applied to my -next tree. If I will send it > now to Dave I believe we would be ok from schedule POV. > I suggest we delay userptr support for

RE: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

Ø But we still have to support cases where we have 1GB stolen VRAM, and wasting those 1GB would suck a bit. Not really, since we only move display buffer here. Regards, Samuel Li From: Koenig, Christian Sent: Wednesday, March 21, 2018 10:07 AM To: Marek Olšák Cc: Deucher,

Re: [PATCH] drm/amdgpu: fix "mitigate workaround for i915"

On 21 March 2018 at 13:08, Christian König wrote: > Mixed up exporter and importer here. E.g. while mapping the BO we need > to check the importer not the exporter. > > Signed-off-by: Christian König > --- >

Re: [PATCH 16/42] drm/amdgpu/gmc9: fix vega12's athub golden setting.

On Wed, Mar 21, 2018 at 10:19 AM, Christian König wrote: > Am 21.03.2018 um 14:46 schrieb Alex Deucher: >> >> From: Feifei Xu >> >> The athub's golden setting is for vega10 only now. >> Remove this from vega12, which is introduced by branch

Re: [PATCH 00/42] Add vega12 support

Apart from patch #16 Acked-by: Christian König for the series. Christian. Am 21.03.2018 um 14:45 schrieb Alex Deucher: Vega12 is a new GPU from AMD. This adds support for it. Patch 1 just adds new register headers and is pretty big, so I haven't sent it to the

Re: [PATCH 16/42] drm/amdgpu/gmc9: fix vega12's athub golden setting.

Am 21.03.2018 um 14:46 schrieb Alex Deucher: From: Feifei Xu The athub's golden setting is for vega10 only now. Remove this from vega12, which is introduced by branch merge. Signed-off-by: Feifei Xu Reviewed-by: Ken Wang Signed-off-by:

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

Am 21.03.2018 um 14:57 schrieb Marek Olšák: On Wed, Mar 21, 2018 at 4:13 AM, Christian König > wrote: Am 21.03.2018 um 06:08 schrieb Marek Olšák: On Tue, Mar 20, 2018 at 4:16 PM, Christian König

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

On Wed, Mar 21, 2018 at 4:13 AM, Christian König < ckoenig.leichtzumer...@gmail.com> wrote: > Am 21.03.2018 um 06:08 schrieb Marek Olšák: > > On Tue, Mar 20, 2018 at 4:16 PM, Christian König > wrote: > >> That's what I meant with use up the otherwise unused VRAM. I

Re: [PATCH] drm/amdgpu: fix "mitigate workaround for i915"

Acked-by: Alex Deucher From: amd-gfx on behalf of Christian König Sent: Wednesday, March 21, 2018 9:08:18 AM To: amd-gfx@lists.freedesktop.org Subject: [PATCH]

[PATCH 42/42] drm/amdgpu: add vega12 pci ids (v2)

v2: add additional pci ids Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c index

[PATCH 40/42] drm/amd/powerplay: add the smu manager for vega12 (v4)

From: Evan Quan handles the driver interaction with the smu firmware v2: squash in: - s3 fix for firmware loading - smu loading through the psp - unecessary calls to is_smc_ram_running() - smu table cleanups v3: rebase v4: rebase, smu bo allocation fixes, add dpm running

[PATCH 33/42] drm/amd/powerplay: add vega12_inc.h

From: Evan Quan Signed-off-by: Evan Quan Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/powerplay/hwmgr/vega12_inc.h | 39 1 file changed, 39

[PATCH 30/42] drm/amdgpu/soc15: update vega12 cg_flags

From: Evan Quan Add the appropriate clockgating flags for vega12 Signed-off-by: Evan Quan Acked-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/soc15.c | 19

[PATCH 34/42] drm/amd/powerplay: update atomfirmware.h (v2)

From: Evan Quan Add new smu_info table. v2: update table format. Signed-off-by: Evan Quan Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/include/atomfirmware.h | 82

[PATCH 32/42] drm/amdgpu/soc15: initialize reg base for vega12

From: Hawking Zhang Signed-off-by: Hawking Zhang Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/soc15.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 39/42] drm/amd/powerplay: add new pp_psm infrastructure for vega12 (v2)

From: Evan Quan New psm infrastructure for vega12. v2: rebase (Alex) Signed-off-by: Evan Quan Acked-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/powerplay/hwmgr/Makefile

[PATCH 36/42] drm/amd/powerplay: add vega12_ppsmc.h

From: Evan Quan Signed-off-by: Evan Quan Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/powerplay/inc/vega12_ppsmc.h | 123 +++ 1 file changed, 123

[PATCH 37/42] drm/amd/powerplay: add vega12_pptable.h

From: Evan Quan Signed-off-by: Evan Quan Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- .../gpu/drm/amd/powerplay/hwmgr/vega12_pptable.h | 109 + 1 file changed, 109

[PATCH 28/42] drm/amdgpu/gfx9: add golden setting for vega12 (v3)

From: Hawking Zhang Add gfx9_2_1 golden setting. v2: switch to soc15_program_register_sequence for golden setting programming v3: squash in additional golden updates Signed-off-by: Feifei Xu Reviewed-by: Ken Wang Signed-off-by:

[PATCH 26/42] drm/amdgpu/gfx9: add support for vega12

Same as vega10 and raven. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c

[PATCH 29/42] drm/amdgpu/soc15: add support for vega12

Add the IP blocks, clock and powergating flags, and common clockgating support. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/soc15.c | 7 +++ 1 file changed, 7 insertions(+) diff --git

[PATCH 31/42] drm/amd/soc15: Add external_rev_id for vega12.

From: Feifei Xu Add external_rev_id for vega12. Signed-off-by: Feifei Xu Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/soc15.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 25/42] drm/amdgpu/gfx9: add gfx config for vega12

Just a place holder for now. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c

[PATCH 27/42] drm/amdgpu/gfx9: add clockgating support for vega12

Same as vega10 and raven. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c

[PATCH 22/42] drm/amdgpu/sdma4: Update vega12 sdma golden setting.

From: Feifei Xu Update vega12 sdma golden setting. Signed-off-by: Feifei Xu Reviewed-by: Ken Wang Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 8 1 file changed, 4

[PATCH 20/42] drm/amdgpu/sdma4: add clockgating support for vega12

Same as vega10 for now. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c

[PATCH 12/42] drm/amd/display/dm: add vega12 support

Add support for vega12 to the display manager. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 24/42] drm/amdgpu/gfx9: Add placeholder for vega12 golden settings

Fill these in when we get them. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c

[PATCH 19/42] drm/amdgpu/sdma4: Add placeholder for vega12 golden settings

Fill these in when we get them. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c

[PATCH 23/42] drm/amdgpu/gfx9: add support for vega12 firmware

Declare and fetch the appriopriate files. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c

[PATCH 13/42] drm/amd/display: Add bios firmware info version for VG12

From: "Jerry (Fangzhi) Zuo" VG12 shows minor revision version of 2 which is not handled in bios_parser_get_firmware_info() routine. Signed-off-by: Jerry (Fangzhi) Zuo Signed-off-by: Alex Deucher ---

[PATCH 21/42] drm/amdgpu/sdma4: add sdma4_0_1 support for vega12 (v3)

From: Hawking Zhang Add sdma golden setting for vega12. v2: switch to soc15_program_register_sequence for golden register programming v3: squash in unused declaration fix Signed-off-by: Feifei Xu Reviewed-by: Alex Deucher

[PATCH 18/42] drm/amdgpu/sdma4: specify vega12 firmware

Declare the firmware and fetch the proper file. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c

[PATCH 16/42] drm/amdgpu/gmc9: fix vega12's athub golden setting.

From: Feifei Xu The athub's golden setting is for vega10 only now. Remove this from vega12, which is introduced by branch merge. Signed-off-by: Feifei Xu Reviewed-by: Ken Wang Signed-off-by: Alex Deucher ---

[PATCH 15/42] drm/amdgpu/gmc9: add vega12 support

Same as vega10. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c index

[PATCH 17/42] drm/amdgpu/mmhub: add clockgating support for vega12

Treat it the same as vega10 for now. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c

[PATCH 10/42] drm/amdgpu: specify vega12 vce firmware

Declare firmware and add support for the file. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git

[PATCH 14/42] drm/amdgpu: add vega12 to dc support check

DC is used for modesetting on vega12. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c

[PATCH 09/42] drm/amdgpu: specify vega12 uvd firmware

Declare firmware and add support for the file. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git

[PATCH 06/42] drm/amdgpu/psp: initial vega12 support

Same as vega10 for now. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 1 + drivers/gpu/drm/amd/amdgpu/psp_v3_1.c | 2 ++ 2 files changed, 3 insertions(+) diff --git

[PATCH 03/42] drm/amdgpu: add gpu_info firmware for vega12

Stores gpu configuration details. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c

[PATCH 08/42] drm/amdgpu: add vega12 ucode loading method

From: Feifei Xu Same as vega10. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu Signed-off-by: Hawking Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 1 + 1 file changed, 1 insertion(+) diff

[PATCH 04/42] drm/amdgpu: set asic family and ip blocks for vega12

soc15 just like vega10 and raven. Signed-off-by: Alex Deucher Reviewed-by: Feifei Xu --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c

[PATCH 02/42] drm/amdgpu: add vega12 to asic_type enum

From: Feifei Xu Add vega12 to amd_asic_type enum and amdgpu_asic_name[]. Signed-off-by: Alex Deucher Signed-off-by: Hawking Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 1 + include/drm/amd_asic_type.h

[PATCH 00/42] Add vega12 support

Vega12 is a new GPU from AMD. This adds support for it. Patch 1 just adds new register headers and is pretty big, so I haven't sent it to the mailing list. The entire series can be viewed here: https://cgit.freedesktop.org/~agd5f/linux/log/?h=amd-staging-drm-next-vega12 Alex Deucher (20):

[PATCH 05/42] drm/amdgpu: vega12 to smu firmware

From: Hawking Zhang Add the cgs interface to query the smu firmware for vega12 and declare the firmware. Signed-off-by: Alex Deucher Signed-off-by: Hawking Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c | 3 +++

Re: [PATCH] drm/amd/pp: Add new asic(vega12) support in pp_psm.c

Acked-by: Alex Deucher From: amd-gfx on behalf of Rex Zhu Sent: Wednesday, March 21, 2018 7:34:34 AM To: amd-gfx@lists.freedesktop.org Cc: Zhu, Rex Subject: [PATCH] drm/amd/pp:

Re: [PATCH] drm/amd/pp: Clean up powerplay code on Vega12

Reviewed-by: Alex Deucher From: amd-gfx on behalf of Rex Zhu Sent: Wednesday, March 21, 2018 5:46:34 AM To: amd-gfx@lists.freedesktop.org Cc: Zhu, Rex Subject: [PATCH]

Re: [PATCH] drm/amd/pp: Fix set wrong temperature range on smu7

Reviewed-by: Alex Deucher From: amd-gfx on behalf of Rex Zhu Sent: Wednesday, March 21, 2018 3:56:19 AM To: amd-gfx@lists.freedesktop.org Cc: Zhu, Rex Subject: [PATCH]

[PATCH] drm/amdgpu: fix "mitigate workaround for i915"

Mixed up exporter and importer here. E.g. while mapping the BO we need to check the importer not the exporter. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git

Re: [Linaro-mm-sig] [PATCH 1/5] dma-buf: add optional invalidate_mappings callback v2

Am 21.03.2018 um 09:28 schrieb Daniel Vetter: On Tue, Mar 20, 2018 at 06:47:57PM +0100, Christian König wrote: Am 20.03.2018 um 15:08 schrieb Daniel Vetter: [SNIP] For the in-driver reservation path (CS) having a slow-path that grabs a temporary reference, drops the vram lock and then locks

[PATCH] drm/amd/pp: Add new asic(vega12) support in pp_psm.c

In new asics, no power state management in driver, no need to implement related callback functions. add some ps checks in pp_psm.c Revert "drm/amd/powerplay: add new pp_psm infrastructure for vega12 (v2)" This reverts commit 7d1a63f3aa331b853e41f92d0e7890ed31de8c13. Change-Id:

Re: [PATCH] drm/amdgpu: Don't change preferred domian when fallback GTT v5

Am 21.03.2018 um 11:27 schrieb Chunming Zhou: v2: add sanity checking v3: make code open v4: also handle visible to invisible fallback v5: Since two fallback cases, re-use goto retry Change-Id: I2cf672ad36b8b4cc1a6b2e704f786bf6a155d9ce Signed-off-by: Chunming Zhou

Re: [PATCH] drm/amdgpu: re-validate per VM BOs if required

On 2018年03月20日 17:13, zhoucm1 wrote: On 2018年03月20日 15:49, zhoucm1 wrote: On 2018年03月19日 18:50, Christian König wrote: If a per VM BO ends up in a allowed domain it never moves back into the prefered domain. Signed-off-by: Christian König Yeah, it's better

[PATCH] drm/amdgpu: Don't change preferred domian when fallback GTT v5

v2: add sanity checking v3: make code open v4: also handle visible to invisible fallback v5: Since two fallback cases, re-use goto retry Change-Id: I2cf672ad36b8b4cc1a6b2e704f786bf6a155d9ce Signed-off-by: Chunming Zhou --- drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c| 16

Re: [PATCH] drm/amdgpu: Don't change preferred domian when fallback GTT v4

Am 20.03.2018 um 08:55 schrieb Chunming Zhou: v2: add sanity checking v3: make code open v4: also handle visible to invisible fallback Change-Id: I2cf672ad36b8b4cc1a6b2e704f786bf6a155d9ce Signed-off-by: Chunming Zhou --- drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c| 16

[PATCH] drm/amd/pp: Clean up powerplay code on Vega12

Change-Id: I792a0c6170115867b99d7112d8eba9ff2faf39d7 --- drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.c | 482 + drivers/gpu/drm/amd/powerplay/hwmgr/vega12_hwmgr.h | 32 -- 2 files changed, 1 insertion(+), 513 deletions(-) diff --git

Re: [Linaro-mm-sig] [PATCH 1/5] dma-buf: add optional invalidate_mappings callback v2

Am 21.03.2018 um 09:18 schrieb Daniel Vetter: [SNIP] They're both in i915_gem_userptr.c, somewhat interleaved. Would be interesting if you could show what you think is going wrong in there compared to amdgpu_mn.c. i915 implements only one callback: static const struct mmu_notifier_ops

Re: [Linaro-mm-sig] [PATCH 1/5] dma-buf: add optional invalidate_mappings callback v2

On Tue, Mar 20, 2018 at 06:47:57PM +0100, Christian König wrote: > Am 20.03.2018 um 15:08 schrieb Daniel Vetter: > > [SNIP] > > For the in-driver reservation path (CS) having a slow-path that grabs a > > temporary reference, drops the vram lock and then locks the reservation > > normally (using

Re: [Linaro-mm-sig] [PATCH 1/5] dma-buf: add optional invalidate_mappings callback v2

On Tue, Mar 20, 2018 at 06:47:57PM +0100, Christian König wrote: > Am 20.03.2018 um 15:08 schrieb Daniel Vetter: > > [SNIP] > > For the in-driver reservation path (CS) having a slow-path that grabs a > > temporary reference, drops the vram lock and then locks the reservation > > normally (using

[PATCH] drm/amd/pp: Fix set wrong temperature range on smu7

Fix the issue thermal irq was always triggered for GPU under temperature range detected The low temp in default thermal policy was set to -273. so need to use int type for the low temp. Change-Id: I1141b2698233ecd1e984b80eaf371966ab1aeef0 Signed-off-by: Rex Zhu ---

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

Am 21.03.2018 um 06:08 schrieb Marek Olšák: On Tue, Mar 20, 2018 at 4:16 PM, Christian König > wrote: That's what I meant with use up the otherwise unused VRAM. I don't see any disadvantage of always setting GTT as second

Re: [PATCH 1/2] drm/amdgpu: Enable scatter gather display support

But for CZ/ST, due to hardware limitation as discussed before, we either use VRAM or GTT, not both. That is actually not correct, as far as I read up on that the issue for switching between VRAM and GTT placement is minimal. We should just make sure that we don't do this on every page flip,

Re: [PATCH 00/20] Add KFD GPUVM support for dGPUs v4

On Mon, Mar 19, 2018 at 9:05 PM, Felix Kuehling wrote: > On 2018-03-19 12:39 PM, Christian König wrote: >> So coming back to this series once more. >> >> Patch #1, #3 are Reviewed-by: Christian König . >> >> Patch #2, #4 - #13 and #18-#19 are

Re: [PATCH 2/2] drm/amd/pp: Register smu irq for legacy asics

Series is: Reviewed-by: Alex Deucher From: amd-gfx on behalf of Rex Zhu Sent: Wednesday, March 21, 2018 1:51:28 AM To: amd-gfx@lists.freedesktop.org Cc: Zhu, Rex Subject:

[PATCH 2/2] drm/amd/pp: Register smu irq for legacy asics

Change-Id: I1927175adfecbcfe99908f06959f8c0a507d3278 Signed-off-by: Rex Zhu --- drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 33 1 file changed, 33 insertions(+) diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c

[PATCH 1/2] drm/amd/pp: Initialize default thermal control type for each asic

Signed-off-by: Rex Zhu Change-Id: I4e1b3f4bc66f28cc6a015182452d426ddd611224 --- drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c| 9 + drivers/gpu/drm/amd/powerplay/hwmgr/vega10_hwmgr.c | 2 +- drivers/gpu/drm/amd/powerplay/inc/hwmgr.h | 1 + 3 files