On 03/31/2017 12:05 PM, Deucher, Alexander wrote:
-Original Message-
From: Junwei Zhang [mailto:jerry.zh...@amd.com]
Sent: Thursday, March 30, 2017 10:52 PM
To: Deucher, Alexander; Koenig, Christian
Cc: amd-gfx@lists.freedesktop.org; Zhang, Jerry
Subject: [PATCH 1/2 v2] drm/amdgpu: fix vm
From: "Zhang, Jerry"
v2: set both of them in gmc
v3: move vm size and block size in vm manager
Signed-off-by: Junwei Zhang
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 8
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 22 --
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.
On KB, KV, CZ we should read the vram width from integrated system
table, if we can. The NOOFCHAN in MC_SHARED_CHMAP is not accurate.
With this change we can enable two 4k displays on CZ again. This use
case was broken sometime in January when we started looking at
vram_width for bandwidth calcula
Hi Andy,
Thanks for bisecting the issue. I've attached a patch which should fix the
issue, if you'd like to give it a try.
Thanks,
Jordan
-Original Message-
From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of
Deucher, Alexander
Sent: Wednesday, March 29, 2017 11:
On 17-03-31 03:15 AM, Christian König wrote:
> Am 30.03.2017 um 16:55 schrieb Felix Kuehling:
>> This only makes a difference for 32-bit systems. The idea is to have a
>> fixed virtual address space size with 4-level page tables and to
>> minimize differences between 32 and 64-bit systems.
> Thanks
No need to reset the wptr and clear the rings. The UNMAP_QUEUES
packet writes the current MQD state back the MQD on suspend,
so there is no need to reset it as well.
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 26 +-
1 file changed, 13 inserti
Reviewed-by: Harry Wentland
On 2017-03-31 11:01 AM, Christian König wrote:
Am 31.03.2017 um 16:54 schrieb Alex Deucher:
Fixes: 2a5071056e6a601e ("drm/amd/powerplay: add global PowerPlay
mutex.")
Reported-by: Julia Lawall
Signed-off-by: Alex Deucher
Reviewed-by: Christian König
---
dri
Hello Flora Cui,
The patch 375d6f7057a9: "drm/amdgpu/gfx6: clean up cu configuration"
from Feb 7, 2017, leads to the following static checker warning:
drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c:3737 gfx_v6_0_get_cu_info()
warn: potential off by one cu_info->bitmap[4]
drivers/gpu/drm/a
Am 31.03.2017 um 16:54 schrieb Alex Deucher:
Fixes: 2a5071056e6a601e ("drm/amd/powerplay: add global PowerPlay mutex.")
Reported-by: Julia Lawall
Signed-off-by: Alex Deucher
Reviewed-by: Christian König
---
drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 2 +-
1 file changed, 1 insertio
Fixes: 2a5071056e6a601e ("drm/amd/powerplay: add global PowerPlay mutex.")
Reported-by: Julia Lawall
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c
b/
From: Christian König
This avoids merging them together on page fault.
Signed-off-by: Christian König
Acked-by: Michel Dänzer
---
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 4 +---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c| 16
2 files changed, 13 insertions(+), 7 deleti
From: Christian König
Implement AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS using TTM_PL_FLAG_CONTIGUOUS
instead of a placement limit. That allows us to better handle CPU
accessible placements.
Signed-off-by: Christian König
Acked-by: Michel Dänzer
---
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 11
From: Christian König
We will probably never see this combination.
Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 25 -
1 file changed, 25 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/dri
From: Christian König
This allows drivers to specify if they need a contiguous allocation or not.
v2: use space instead of tab
Signed-off-by: Christian König
---
drivers/gpu/drm/ttm/ttm_bo.c| 4 +++-
include/drm/ttm/ttm_placement.h | 1 +
2 files changed, 4 insertions(+), 1 deletion(-)
d
From: Christian König
This allows the driver to handle io_mem mappings on their own.
Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 1 +
drivers/gpu/drm/ast/ast_ttm.c | 1 +
drivers/gpu/drm/bochs/bochs_mm.c| 1 +
dr
From: Christian König
No need to implement the same logic twice. Also check if the busy placements
are identical to the already scanned placements before checking them.
v2: improve check even more as suggested by Michel.
Signed-off-by: Christian König
---
drivers/gpu/drm/ttm/ttm_bo.c | 45 +++
On 31/03/17 04:09 PM, Christian König wrote:
> Am 31.03.2017 um 03:07 schrieb Michel Dänzer:
>> On 30/03/17 10:09 PM, Alex Deucher wrote:
>>> On Thu, Mar 30, 2017 at 2:39 AM, Michel Dänzer
>>> wrote:
On 29/03/17 09:55 PM, Christian König wrote:
> From: Christian König
>
> We will
Am 30.03.2017 um 16:55 schrieb Felix Kuehling:
This only makes a difference for 32-bit systems. The idea is to have a
fixed virtual address space size with 4-level page tables and to
minimize differences between 32 and 64-bit systems.
Thanks going to use that as commit message.
Reviewed-by: Fe
Am 31.03.2017 um 03:07 schrieb Michel Dänzer:
On 30/03/17 10:09 PM, Alex Deucher wrote:
On Thu, Mar 30, 2017 at 2:39 AM, Michel Dänzer wrote:
On 29/03/17 09:55 PM, Christian König wrote:
From: Christian König
We will probably never see this combination.
Signed-off-by: Christian König
---
Am 30.03.2017 um 20:34 schrieb Alex Deucher:
Set the ready flag to reflect this.
Signed-off-by: Alex Deucher
Patches #1 and #3-#5 are Reviewed-by: Christian König
Patch #2 is Acked-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 1 +
1 file changed, 1 insertion(+)
d
20 matches
Mail list logo
