RE: [PATCH v2] drm/amdgpu/gmc8: SRIOV need to program fb location

2017-07-18 Thread Deng, Emily
Hi Alex, Modified the fix, please help to review again. Best Wishes, Emily Deng > -Original Message- > From: Emily Deng [mailto:emily.d...@amd.com] > Sent: Wednesday, July 19, 2017 11:44 AM > To: amd-gfx@lists.freedesktop.org > Cc: Deng, Emily > Subject: [PATCH

Re: [PATCH] drm/amdgpu: disable firmware loading for psp v10

2017-07-18 Thread Alex Deucher
On Tue, Jul 18, 2017 at 8:26 PM, Junwei Zhang wrote: > Now asd firmware is not ready for psp v10, will enable it when it's available > > Signed-off-by: Junwei Zhang Reviewed-by: Alex Deucher > --- >

RE: [PATCH libdrm] test/amdgpu: fix test failure for SI

2017-07-18 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Flora Cui > Sent: Tuesday, July 18, 2017 10:53 PM > To: amd-gfx@lists.freedesktop.org; dri-de...@lists.freedesktop.org > Cc: Cui, Flora > Subject: [PATCH libdrm] test/amdgpu: fix test failure

[PATCH libdrm] test/amdgpu: fix test failure for SI

2017-07-18 Thread Flora Cui
Change-Id: I646f1bf844bd92962b9f71aa287f90173ae233c6 Signed-off-by: Flora Cui --- tests/amdgpu/basic_tests.c | 273 ++--- tests/amdgpu/cs_tests.c| 41 +++ tests/amdgpu/vce_tests.c | 41 +++ 3 files changed, 229

Re: [PATCH 0/6] Experimental P2P buffer sharing v2

2017-07-18 Thread Felix Kuehling
Sorry for the spam. The second (old) cover letter was sent by mistake. Please look at v3. Regards, Felix On 17-07-18 10:22 PM, Felix Kuehling wrote: > This patch series adds experimental P2P buffer sharing in amdgpu. It's > disabled by default and can be enabled with amdgpu.p2p_sharing=1. > >

[PATCH 6/6] drm/amdgpu: enable foreign DMA-buf objects v2

2017-07-18 Thread Felix Kuehling
From: Christian König We should be able to handle BOs from other instances as well. v2: * Add a module option that is off-by-default * Use new DRM helper function to check the exporting driver Signed-off-by: Christian König Signed-off-by:

[PATCH 4/6] drm/amdgpu: separate BO from GEM object

2017-07-18 Thread Felix Kuehling
From: Christian König This allows us to have multiple GEM objects for one BO. Signed-off-by: Christian König Reviewed-by: Felix Kuehling --- drivers/gpu/drm/amd/amdgpu/amdgpu.h| 12 +++--

[PATCH 0/6] Experimental P2P buffer sharing v2

2017-07-18 Thread Felix Kuehling
This patch series adds experimental P2P buffer sharing in amdgpu. It's disabled by default and can be enabled with amdgpu.p2p_sharing=1. v2: * Changed drm helper function to cast to GEM object * Added foreign BO checks to DC code paths * Updated commit message for amdgpu_cs change Amber Lin (1):

[PATCH 3/6] drm/amdgpu: disallow foreign BOs in the display path v2

2017-07-18 Thread Felix Kuehling
From: Christian König Pinning them in other devices VRAM would obviously not work. v2: Add checks to DC code paths Signed-off-by: Christian König Signed-off-by: Felix Kuehling ---

[PATCH 0/6] Experimental P2P buffer sharing v3

2017-07-18 Thread Felix Kuehling
This patch series adds experimental P2P buffer sharing in amdgpu. It's disabled by default and can be enabled with amdgpu.p2p_sharing=1. v2: * Changed drm helper function to cast to GEM object * Added foreign BO checks to DC code paths * Updated commit message for amdgpu_cs change v3: * Use

[PATCH 2/6] drm/amdgpu: disallow foreign BOs for CS w/o GPUVM mapping

2017-07-18 Thread Felix Kuehling
From: Christian König Signed-off-by: Christian König Reviewed-by: Felix Kuehling --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 5/6] drm/amdgpu: handle foreign BOs in the VM mapping

2017-07-18 Thread Felix Kuehling
From: Amber Lin Set the system bit for foreign BO mappings and use the remote VRAM BAR address as the VRAM base offset. Signed-off-by: Amber Lin Reviewed-by: Felix Kuehling --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 17

Re: [PATCH libdrm] libdrm_amdgpu: add kernel semaphore support

2017-07-18 Thread zhoucm1
On 2017年07月18日 21:57, Christian König wrote: Am 18.07.2017 um 04:29 schrieb zhoucm1: On 2017年07月18日 01:35, Christian König wrote: Am 17.07.2017 um 19:22 schrieb Marek Olšák: On Sun, Jul 16, 2017 at 11:36 PM, Dave Airlie wrote: I can take a look at it, I just won't have

[PATCH] drm/amdgpu: disable firmware loading for psp v10

2017-07-18 Thread Junwei Zhang
Now asd firmware is not ready for psp v10, will enable it when it's available Signed-off-by: Junwei Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c

Re: [PATCH 3/3] drm/amdgpu: Clear active for HIQ in RLC_CP_SCHEDULERS

2017-07-18 Thread Felix Kuehling
Hi Shaoyun, You'd need to squash these 3 patches into 1 because otherwise you break the build. But I think there should be a way to do this without requiring an interface change. In kgd_hqd_load you could remember the pipe_id and queue_id of the HIQ somewhere in adev. Then you can use that to

[PATCH 2/3] drm/amdkfd: hqd_destroy interface change from KFD side

2017-07-18 Thread Shaoyun Liu
Change-Id: I5bd514b4357d1082f4e8be3df2a1b37051c9bd9f Signed-off-by: Shaoyun Liu --- drivers/gpu/drm/amd/amdkfd/kfd_kernel_queue.c| 2 +- drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_cik.c | 2 +- drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_v9.c | 2 +-

[PATCH 2/3] drm/amdkfd: hqd_destroy interface change from KFD side

2017-07-18 Thread Shaoyun Liu
Change-Id: I5bd514b4357d1082f4e8be3df2a1b37051c9bd9f Signed-off-by: Shaoyun Liu --- drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_cik.c | 2 +- drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_v9.c | 2 +- drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_vi.c | 2 +- 3 files changed, 3

[PATCH 3/3] drm/amdgpu: Clear active for HIQ in RLC_CP_SCHEDULERS

2017-07-18 Thread Shaoyun Liu
Change-Id: Ibc3ae5ac852405b77908bc26f899fe97bde88d86 Signed-off-by: Shaoyun Liu --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c | 4 ++-- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v8.c | 15 +-- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v9.c | 16

Re: [RFC PATCH] radeonsi: set a per-buffer flag that disables inter-process sharing (v2)

2017-07-18 Thread Marek Olšák
On Tue, Jul 18, 2017 at 5:11 PM, Michel Dänzer wrote: > On 18/07/17 04:08 PM, Marek Olšák wrote: >> >> From: Marek Olšák >> >> For lower overhead in the CS ioctl. >> Winsys allocators are not used with interprocess-sharable resources. >> >> v2: It

Re: [RFC PATCH] radeonsi: set a per-buffer flag that disables inter-process sharing (v2)

2017-07-18 Thread Michel Dänzer
On 18/07/17 04:08 PM, Marek Olšák wrote: From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with interprocess-sharable resources. v2: It shouldn't crash anymore, but the kernel will reject the new flag. ---

Re: [PATCH] drm/radeon: Set depth on low mem to 16 bpp instead of 8 bpp

2017-07-18 Thread Michel Dänzer
On 18/07/17 11:20 AM, Takashi Iwai wrote: From: Egbert Eich The radeon driver reduces the framebuffer resolution to 8bpp if a device with less than 32MB VRAM is found. This causes the framebuffer to run in 8 bit paletted mode. For a text console this is not an issue as 256

[RFC PATCH] radeonsi: set a per-buffer flag that disables inter-process sharing (v2)

2017-07-18 Thread Marek Olšák
From: Marek Olšák For lower overhead in the CS ioctl. Winsys allocators are not used with interprocess-sharable resources. v2: It shouldn't crash anymore, but the kernel will reject the new flag. --- src/gallium/drivers/radeon/r600_buffer_common.c | 7 +

Re: [PATCH 2/2] drm/amdgpu: enable huge page handling in the VM v4

2017-07-18 Thread Felix Kuehling
If you submit the code, it allows more people to experiment with it. Regards, Felix On 17-07-18 09:54 AM, Christian König wrote: > Yeah, I mean it looks good from the software side but we still don't > see the hardware react as it should. > > It doesn't seem to hurt anything, so I'm torn

Re: [PATCH 1/2] drm/ttm: Implement vm_operations_struct.access v2

2017-07-18 Thread Michel Dänzer
On 17/07/17 11:57 PM, Felix Kuehling wrote: Allows gdb to access contents of user mode mapped BOs. System memory is handled by TTM using kmap. Other memory pools require a new driver callback in ttm_bo_driver. v2: * kmap only one page at a time * swap in BO if needed * make driver callback more

Re: [RFC PATCH] radeonsi: set a per-buffer flag that disables inter-process sharing

2017-07-18 Thread Marek Olšák
For comments only. There are some assertion failures. Marek On Tue, Jul 18, 2017 at 1:47 PM, Marek Olšák wrote: > From: Marek Olšák > > for lower overhead in the CS ioctl > --- > src/gallium/drivers/radeon/r600_buffer_common.c | 7 +++ >

[RFC PATCH] radeonsi: set a per-buffer flag that disables inter-process sharing

2017-07-18 Thread Marek Olšák
From: Marek Olšák for lower overhead in the CS ioctl --- src/gallium/drivers/radeon/r600_buffer_common.c | 7 +++ src/gallium/drivers/radeon/radeon_winsys.h | 1 + src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 6 ++ 3 files changed, 14 insertions(+) diff

[PATCH] drm/radeon: Set depth on low mem to 16 bpp instead of 8 bpp

2017-07-18 Thread Takashi Iwai
From: Egbert Eich The radeon driver reduces the framebuffer resolution to 8bpp if a device with less than 32MB VRAM is found. This causes the framebuffer to run in 8 bit paletted mode. For a text console this is not an issue as 256 different colors is more than one gets on a VGA

Re: [PATCH libdrm] libdrm_amdgpu: add kernel semaphore support

2017-07-18 Thread Marek Olšák
Hi Dave, If you just add "get" functions for what you need from amdgpu objects, that should be fine. Marek On Mon, Jul 17, 2017 at 11:00 PM, Dave Airlie wrote: > On 18 July 2017 at 03:02, Christian König wrote: >> Am 17.07.2017 um 05:36 schrieb Dave

Re: [PATCH 1/2] drm/ttm: Implement vm_operations_struct.access v2

2017-07-18 Thread Christian König
Am 18.07.2017 um 05:57 schrieb Felix Kuehling: Allows gdb to access contents of user mode mapped BOs. System memory is handled by TTM using kmap. Other memory pools require a new driver callback in ttm_bo_driver. v2: * kmap only one page at a time * swap in BO if needed * make driver callback

Re: [PATCH] drm/amdgpu:fix gfx fence allocate size

2017-07-18 Thread Christian König
Am 18.07.2017 um 14:50 schrieb Deucher, Alexander: -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Monk Liu Sent: Tuesday, July 18, 2017 12:40 AM To: amd-gfx@lists.freedesktop.org Cc: Yu, Xiangliang; Liu, Monk Subject: [PATCH] drm/amdgpu:fix

Re: [PATCH libdrm] drm/amdgpu: add new low overhead command submission API. (v2)

2017-07-18 Thread Christian König
Am 18.07.2017 um 05:52 schrieb Dave Airlie: From: Dave Airlie This just sends chunks to the kernel API for a single command stream. This should provide a more future proof and extensible API for command submission. v2: use amdgpu_bo_list_handle, add two helper functions

Re: [PATCH libdrm] libdrm_amdgpu: add kernel semaphore support

2017-07-18 Thread Christian König
Am 18.07.2017 um 04:29 schrieb zhoucm1: On 2017年07月18日 01:35, Christian König wrote: Am 17.07.2017 um 19:22 schrieb Marek Olšák: On Sun, Jul 16, 2017 at 11:36 PM, Dave Airlie wrote: I can take a look at it, I just won't have time until next week most likely. I've taken

Re: [PATCH 2/2] drm/amdgpu: enable huge page handling in the VM v4

2017-07-18 Thread Christian König
Yeah, I mean it looks good from the software side but we still don't see the hardware react as it should. It doesn't seem to hurt anything, so I'm torn apart between pushing it and completely fixing it later on or wait till we have figured everything out. Felix what is your opinion on that?

Re: [PATCH libdrm 2/2] drm/amdgpu: add new low overhead command submission API.

2017-07-18 Thread Christian König
Am 18.07.2017 um 02:48 schrieb Dave Airlie: From: Dave Airlie This just sends chunks to the kernel API for a single command stream. This should provide a more future proof and extensible API for command submission. Signed-off-by: Dave Airlie ---

Re: [PATCH libdrm 1/2] drm/amdgpu: add syncobj create/destroy/import/export apis

2017-07-18 Thread Christian König
Am 18.07.2017 um 02:48 schrieb Dave Airlie: From: Dave Airlie These are just wrappers using the amdgpu device handle. Signed-off-by: Dave Airlie Reviewed-by: Christian König for this one. --- amdgpu/amdgpu.h| 55

RE: [PATCH] drm/amdgpu:fix gfx fence allocate size

2017-07-18 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Monk Liu > Sent: Tuesday, July 18, 2017 12:40 AM > To: amd-gfx@lists.freedesktop.org > Cc: Yu, Xiangliang; Liu, Monk > Subject: [PATCH] drm/amdgpu:fix gfx fence allocate size > > 1, for

RE: [PATCH v3 1/4] drm/amdgpu: Fix KFD oversubscription by tracking queues correctly

2017-07-18 Thread Deucher, Alexander
> -Original Message- > From: Kuehling, Felix > Sent: Tuesday, July 18, 2017 12:41 AM > To: amd-gfx@lists.freedesktop.org; Deucher, Alexander > Subject: Re: [PATCH v3 1/4] drm/amdgpu: Fix KFD oversubscription by > tracking queues correctly > > Hi Alex, > > This patch series went into

Fw: [PATCH] drm/amdgpu:fix gfx fence allocate size

2017-07-18 Thread Liu, Monk
From: Monk Liu Sent: Tuesday, July 18, 2017 1:56 PM To: amd-gfx-boun...@lists.freedesktop.org Cc: Liu, Monk; Yu, Xiangliang Subject: [PATCH] drm/amdgpu:fix gfx fence allocate size 1, for sriov, we need 8dw for the gfx fence due to CP