Re: [PATCH libdrm v4 1/1] amdgpu: move asic id table to a separate file

2017-06-01 Thread Michel Dänzer
On 01/06/17 11:27 PM, Li, Samuel wrote: >> If amdgpu.ids living in the amdgpu directory prevents it from being used by >> libdrm_radeon (why?), let's put it in a new toplevel directory, e.g. >> "data". >>> README is also located in this directory. >> Not the same thing. It documents things about

Re: [PATCH v6 5/5] drm/amdgpu: resize VRAM BAR for CPU access v2

2017-06-01 Thread Bjorn Helgaas
[+cc ADMGPU, DRM folks] On Tue, May 09, 2017 at 06:49:07PM +0200, Christian König wrote: > From: Christian König > > Try to resize BAR0 to let CPU access all of VRAM. > > v2: rebased, style cleanups, disable mem decode before resize, > handle gmc_v9 as well, round

Re: iMac 10,1 with Ubuntu 16.04: black screen after suspend

2017-06-01 Thread Alex Deucher
On Thu, Jun 1, 2017 at 4:27 AM, Florian Echtler wrote: > On 31.05.2017 14:57, Alex Deucher wrote: >> On Wed, May 31, 2017 at 5:21 AM, Lukas Wunner wrote: >>> On Wed, May 31, 2017 at 10:48:37AM +0200, Florian Echtler wrote: >>> >>> Hm, try booting with

Re: [Intel-gfx] [PATCH i-g-t] tests/kms_setmode: increase MAX_CRTCS to 6

2017-06-01 Thread Harry Wentland
Please ignore what I wrote. Looks like I was a bit hasty in my judgement. The code just doesn't read easily but seems correct, other than the array size. Harry On 2017-05-31 04:17 PM, Harry Wentland wrote: On 2017-05-31 09:32 AM, Harry Wentland wrote: On 2017-05-31 05:37 AM, Daniel Vetter

RE: [PATCH libdrm v4 1/1] amdgpu: move asic id table to a separate file

2017-06-01 Thread Li, Samuel
>If amdgpu.ids living in the amdgpu directory prevents it from being used by >libdrm_radeon (why?), let's put it in a new toplevel directory, e.g. >"data". >> README is also located in this directory. > Not the same thing. It documents things about the header files, and doesn't > get installed

RE: [PATCH v2 00/21] S3 following up

2017-06-01 Thread Deucher, Alexander
> -Original Message- > From: Huang Rui [mailto:ray.hu...@amd.com] > Sent: Thursday, June 01, 2017 7:18 AM > To: amd-gfx@lists.freedesktop.org; Deucher, Alexander; Koenig, Christian > Cc: Wang, Ken; Huan, Alvin; Huang, Ray > Subject: [PATCH v2 00/21] S3 following up > > Hi all, > >

Re: [PATCH] drm/amdgpu: Move compute vm bug logic to amdgpu_vm.c

2017-06-01 Thread Leo Liu
When you git email-send first time, there is a message-id, when you would send follow up patches, using --in-reply-to="message-id" in your comment line, then that will be in the same thread. Cheers, Leo On 06/01/2017 09:40 AM, Xie, AlexBin wrote: Ok. I will add the v2 next time. I did

Re: [PATCH] drm/amdgpu: Move compute vm bug logic to amdgpu_vm.c

2017-06-01 Thread Xie, AlexBin
Ok. I will add the v2 next time. I did not add the v2 in case this email list does not put the two patches under same thread... -Alex Bin From: Christian König Sent: Thursday, June 1, 2017 9:26 AM To: Xie, AlexBin;

Re: [PATCH] drm/amdgpu: Move compute vm bug logic to amdgpu_vm.c

2017-06-01 Thread Christian König
Am 01.06.2017 um 01:08 schrieb Alex Xie: In review, Christian would like to keep the logic inside amdgpu_vm.c with a cost of slightly slower. The loop is still optimized out with this patch. v2: remove the if statement. Now it is not slower. Signed-off-by: Alex Xie

Re: [PATCH] drm/amdgpu: Move compute vm bug logic to amdgpu_vm.c

2017-06-01 Thread Xie, AlexBin
Hi Christian, Please review this v2 patch. It is slightly faster. Thanks, Alex Bin From: Xie, AlexBin Sent: Wednesday, May 31, 2017 7:08 PM To: amd-gfx@lists.freedesktop.org Cc: Xie, AlexBin Subject: [PATCH] drm/amdgpu: Move compute vm bug logic to

Re: [PATCH 2/3] drm/amdgpu: use existing function amdgpu_bo_create_kernel

2017-06-01 Thread Leo Liu
On 06/01/2017 06:55 AM, Christian König wrote: Am 31.05.2017 um 21:28 schrieb Leo Liu: To simplify vce bo create Signed-off-by: Leo Liu --- drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 27 +++ 1 file changed, 3 insertions(+), 24 deletions(-) diff

Re: iMac 10,1 with Ubuntu 16.04: black screen after suspend

2017-06-01 Thread Florian Echtler
On 31.05.2017 14:57, Alex Deucher wrote: > On Wed, May 31, 2017 at 5:21 AM, Lukas Wunner wrote: >> On Wed, May 31, 2017 at 10:48:37AM +0200, Florian Echtler wrote: >> >> Hm, try booting with drm.debug=0xf to see if link training for the >> eDP connector succeeds. If the link

[PATCH trivial] drm/amdkfd: Spelling s/apreture/aperture/

2017-06-01 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven --- drivers/gpu/drm/amd/amdkfd/kfd_process.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c b/drivers/gpu/drm/amd/amdkfd/kfd_process.c index

Re: [PATCH] drm/amdgpu: enable gfx 9.0 cp interrupts

2017-06-01 Thread Alex Deucher
On Wed, May 31, 2017 at 11:10 PM, Ken Wang wrote: > Change-Id: If7dee7bd1074eac7faa6af724a7272f9ce6f3a98 > Signed-off-by: Ken Wang Reviewed-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 3 --- > 1

Re: [PATCH 03/19] drm/amd/display: Limit DCN to x86 arch

2017-06-01 Thread Alex Deucher
On Thu, Jun 1, 2017 at 12:15 AM, zhoucm1 wrote: > > > On 2017年05月31日 23:52, Harry Wentland wrote: >> >> DCN bw calcs currently rely on the following gcc options: >>-mhard-float -msse -mpreferred-stack-boundary=4 >> >> We probably shouldn't really try building this on

Re: [PATCH v2 00/21] S3 following up

2017-06-01 Thread Christian König
Patch #18 is Reviewed-by: Christian König . The rest of the set is Acked-by: Christian König . Regards, Christian. Am 01.06.2017 um 13:17 schrieb Huang Rui: Hi all, Changes from V1 -> V2: - Rename pt_regs to gart_pt_regs,

[PATCH v2 01/21] drm/amdgpu: abstract gart table initialization for gfxhub/mmhub

2017-06-01 Thread Huang Rui
Signed-off-by: Huang Rui --- drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 34 drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 34 2 files changed, 42 insertions(+), 26 deletions(-) diff --git

[PATCH v2 21/21] drm/amdgpu: update to use RREG32_SOC15/WREG32_SOC15 for gmc9

2017-06-01 Thread Huang Rui
Signed-off-by: Huang Rui --- drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c index a901f73..3325104 100644 ---

[PATCH v2 20/21] drm/amdgpu: update to use RREG32_SOC15/WREG32_SOC15 for mmhub

2017-06-01 Thread Huang Rui
Signed-off-by: Huang Rui --- drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 173 +++- 1 file changed, 81 insertions(+), 92 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c b/drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c index

[PATCH v2 19/21] drm/amdgpu: update to use RREG32_SOC15/WREG32_SOC15 for gfxhub

2017-06-01 Thread Huang Rui
Signed-off-by: Huang Rui --- drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 141 ++- 1 file changed, 64 insertions(+), 77 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c b/drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c index

[PATCH v2 18/21] drm/amdgpu: fix the gart table cleared issue for S3

2017-06-01 Thread Huang Rui
Signed-off-by: Huang Rui --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 +- drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c | 2 ++ drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c | 2 ++ drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c | 2 ++

[PATCH v2 02/21] drm/amdgpu: abstract gart aperture initialization for gfxhub/mmhub

2017-06-01 Thread Huang Rui
Signed-off-by: Huang Rui --- drivers/gpu/drm/amd/amdgpu/gfxhub_v1_0.c | 37 +--- drivers/gpu/drm/amd/amdgpu/mmhub_v1_0.c | 36 +-- 2 files changed, 40 insertions(+), 33 deletions(-) diff --git

[PATCH v2 00/21] S3 following up

2017-06-01 Thread Huang Rui
Hi all, Changes from V1 -> V2: - Rename pt_regs to gart_pt_regs, dis_identity_aperture to disable_identity_aperture. - Use upper_32_bits/lower_32_bits macros. - Add comments and fix "uint64_t" for stolen size. - Merge page table regs function into page table aperture. - Add three patches to use

Re: [PATCH 3/5] drm/syncobj: add sync_file interaction. (v1.2)

2017-06-01 Thread Chris Wilson
On Thu, Jun 01, 2017 at 11:06:41AM +1000, Dave Airlie wrote: > From: Dave Airlie > > This interface allows importing the fence from a sync_file into > an existing drm sync object, or exporting the fence attached to > an existing drm sync object into a new sync file object. >

Re: [RFC] Exclusive gpu access for SteamVR usecases

2017-06-01 Thread Christian König
Am 31.05.2017 um 20:46 schrieb Andres Rodriguez: On 2017-05-31 02:53 AM, Christian König wrote: 2. How are the priorities from processes supposed to interact with the per context priority? Do you mean process niceness? There isn't any relationship between niceness and gpu priority. Let

Re: [PATCH 1/5] drm: introduce sync objects (v4)

2017-06-01 Thread Chris Wilson
On Thu, Jun 01, 2017 at 11:06:39AM +1000, Dave Airlie wrote: > From: Dave Airlie > > Sync objects are new toplevel drm object, that contain a > pointer to a fence. This fence can be updated via command > submission ioctls via drivers. > > There is also a generic wait obj API

Re: [PATCH 03/19] drm/amd/display: Limit DCN to x86 arch

2017-06-01 Thread zhoucm1
On 2017年06月01日 16:50, Christian König wrote: Am 01.06.2017 um 06:15 schrieb zhoucm1: On 2017年05月31日 23:52, Harry Wentland wrote: DCN bw calcs currently rely on the following gcc options: -mhard-float -msse -mpreferred-stack-boundary=4 We probably shouldn't really try building this on

Re: [PATCH 03/19] drm/amd/display: Limit DCN to x86 arch

2017-06-01 Thread Christian König
Am 01.06.2017 um 06:15 schrieb zhoucm1: On 2017年05月31日 23:52, Harry Wentland wrote: DCN bw calcs currently rely on the following gcc options: -mhard-float -msse -mpreferred-stack-boundary=4 We probably shouldn't really try building this on architectures other than x86. Does it mean our

Re: [PATCH] drm/amdgpu: Move compute vm bug logic to amdgpu_vm.c

2017-06-01 Thread Christian König
Am 31.05.2017 um 20:09 schrieb Alex Xie: In review, Christian would like to keep the logic inside amdgpu_vm.c with a cost of slightly slower. The loop is still optimized out with this patch. Signed-off-by: Alex Xie Reviewed-by: Christian König

Re: [PATCH libdrm v4 1/1] amdgpu: move asic id table to a separate file

2017-06-01 Thread Michel Dänzer
On 01/06/17 12:32 AM, Li, Samuel wrote: >> From: Michel Dänzer [mailto:mic...@daenzer.net] >> On 31/05/17 07:31 AM, Li, Samuel wrote: >>> From: Michel Dänzer [mailto:mic...@daenzer.net] On 30/05/17 06:16 AM, Samuel Li wrote: > diff --git a/amdgpu/amdgpu_asic_id.c