Reviewed-by: Guchun Chen
Regards,
Guchun
-Original Message-
From: Zhou1, Tao
Sent: Tuesday, September 17, 2019 2:22 PM
To: amd-gfx@lists.freedesktop.org; Chen, Guchun ; Zhang,
Hawking ; Deucher, Alexander
Cc: Zhou1, Tao
Subject: [PATCH] drm/amdgpu: use GPU PAGE SHIFT for umc retired
-Original Message-
From: Zhou1, Tao
Sent: Tuesday, September 17, 2019 2:25 PM
To: amd-gfx@lists.freedesktop.org; Chen, Guchun ; Zhang,
Hawking
Cc: Zhou1, Tao
Subject: [PATCH] drm/amdgpu: replace DRM_ERROR with DRM_WARN in
ras_reserve_bad_pages
There are two cases of reserve error s
Series is:
Reviewed-by: Guchun Chen
Regards,
Guchun
-Original Message-
From: Zhou1, Tao
Sent: Tuesday, September 17, 2019 2:17 PM
To: amd-gfx@lists.freedesktop.org; Chen, Guchun ; Koenig,
Christian ; Zhang, Hawking
Cc: Zhou1, Tao
Subject: [PATCH 2/2] drm/amdgpu: fix size calc error
From: zhexzhan
Issue: DROOP coef read by HDT appear to be mismatch with requirement of BKM0.83
Root cause: These values are supposed to be overwritten by PPLIB.
However, driver missed code of this part.
Solution: Add overwriting process when reading pptable from vBIOS
Hardcode specific coef wit
Issue:
quark didn't trigger TDR correctly on compute ring
Root cause:
Default timeout value for compute ring is infinite
Solution:
In SR-IOV and passthrough mode, if compute ring timeout
is set, then use user set value; if not set, then use
same value as gfx ring timeout.
Signed-off-by: Jesse Zh
There are two cases of reserve error should be ignored:
1) a ras bad page has been allocated (used by someone);
2) a ras bad page has been reserved (duplicate error injection for one page);
DRM_ERROR is unnecessary for the failure of bad page reserve
Signed-off-by: Tao Zhou
---
drivers/gpu/drm/
umc retired page belongs to vram and it should be aligned to gpu page
size
Signed-off-by: Tao Zhou
---
drivers/gpu/drm/amd/amdgpu/umc_v6_1.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/umc_v6_1.c
b/drivers/gpu/drm/amd/amdgpu/umc_v6_1.c
index 1c
the bo pointer is reused for bad pages, initialize it in each loop
Signed-off-by: Tao Zhou
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
index 5f623daf5078..090daf
replace offset with size
Signed-off-by: Tao Zhou
---
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
index ecad84e1b4e2..2fcd2d14cbf0 100644
---
On Mon, 16 Sep 2019, Marek Olšák wrote:
> The purpose is to get rid of all PCI ID tables for all drivers in
> userspace. (or at least stop updating them)
>
> Mesa common code and modesetting will use this.
I'd think this would warrant a high level description of what you want
to achieve in the co
On Wed, Sep 4, 2019 at 1:58 PM Randy Dunlap wrote:
>
> On 9/4/19 6:34 AM, Stephen Rothwell wrote:
> > Hi all,
> >
> > News: this will be the last linux-next I will release until Sept 30.
> >
> > Changes since 20190903:
> >
>
> on x86_64:
>
> In file included from
> ../drivers/gpu/drm/amd/amdgpu/.
In dcn*_clock_source_create when dcn20_clk_src_construct fails allocated
clk_src needs release.
Signed-off-by: Navid Emamdoost
---
drivers/gpu/drm/amd/display/dc/dce100/dce100_resource.c | 3 ++-
drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c | 1 +
drivers/gpu/drm/amd/display/dc/dce112
Reviewed-by: Evan Quan
-Original Message-
From: Liang, Prike
Sent: Tuesday, September 17, 2019 10:54 AM
To: amd-gfx@lists.freedesktop.org
Cc: Quan, Evan ; Feng, Kenneth ;
Huang, Ray ; Liu, Aaron ; Liang, Prike
Subject: [PATCH] drm/amd/powerplay: implement VCN power gating control int
Thanks Hawking!
From: Zhang, Hawking
Sent: Tuesday, September 17, 2019 11:02
To: Yin, Tianci (Rico)
Cc: amd-gfx@lists.freedesktop.org ; Deucher,
Alexander ; Xu, Feifei ; Yuan,
Xiaojie ; Xiao, Jack ; Long, Gang
Subject: Re: [Patch] drm/amdgpu: add navi12 pci id
Reviewed-by: Hawking Zhang mailto:hawking.zh...@amd.com>>
Regards,
Hawking
Sent from my iPhone
On Sep 16, 2019, at 22:57, Yin, Tianci (Rico)
mailto:tianci@amd.com>> wrote:
<0001-drm-amdgpu-add-navi12-pci-id.patch>
___
amd-gfx mailing list
amd-gf
Reviewed-by: Tao Zhou
> -Original Message-
> From: amd-gfx On Behalf Of Alex
> Deucher
> Sent: 2019年9月17日 3:51
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander
> Subject: [PATCH] drm/amdgpu/ras: use GPU PAGE_SIZE/SHIFT for reserving
> pages
>
> We are reserving vram pages s
From ec0d4e0069877a5a763972da4393b3c11404108d Mon Sep 17 00:00:00 2001
From: "Tianci.Yin"
Date: Tue, 17 Sep 2019 10:35:34 +0800
Subject: [PATCH] drm/amdgpu: add navi12 pci id
Add Navi12 PCI id support.
Change-Id: Ic4e63f46f6ddf856910018dc3ddc2093d87873d0
Signed-off-by: Tianci.Yin
---
drivers/
VCN Gat/Ungat by processing the SMU power up/down message, otherwise
S3 will resume failed as JPEG always power off during start VCN stage.
Signed-off-by: Prike Liang
---
drivers/gpu/drm/amd/powerplay/renoir_ppt.c | 28 +++-
1 file changed, 27 insertions(+), 1 deletion(-)
The purpose is to get rid of all PCI ID tables for all drivers in
userspace. (or at least stop updating them)
Mesa common code and modesetting will use this.
Marek
On Sat, Sep 7, 2019 at 3:48 PM Daniel Vetter wrote:
> On Sat, Sep 7, 2019 at 3:18 AM Rob Clark wrote:
> >
> > On Fri, Sep 6, 2019
On Mon, Sep 16, 2019 at 11:12:03PM +0100, Mark Brown wrote:
> On Mon, Sep 16, 2019 at 01:51:15PM -0700, Nick Desaulniers wrote:
>
> > + Mark
> > I think this was a result of the resolved merge conflict. See the
> > -next only commit titled:
> > Merge remote-tracking branch 'drm/drm-next'
> Y
On Tue, Sep 17, 2019 at 02:46:48AM +0900, Masahiro Yamada wrote:
> On Mon, Sep 16, 2019 at 1:46 PM Austin Kim wrote:
> > gcc throws compile error with below message:
> GNU Make throws ...
Xinpeng Liu via Nick Desaulniers sent a description of the problem and a
patch so I think I'll be able to f
On Mon, Sep 16, 2019 at 01:51:15PM -0700, Nick Desaulniers wrote:
> + Mark
> I think this was a result of the resolved merge conflict. See the
> -next only commit titled:
> Merge remote-tracking branch 'drm/drm-next'
Yes, the DRM and the Kbuild people really need to coordinate with each
othe
On 2019-09-13 8:08 a.m., Christian König wrote:
> The placement is something TTM/BO internal and the RAS code should
> avoid touching that directly.
>
> Add a helper to create a BO at a fixed location and use that instead.
>
> Signed-off-by: Christian König
> ---
> drivers/gpu/drm/amd/amdgpu/
On Mon, Sep 16, 2019 at 3:28 AM Adam Zerella wrote:
>
> Some of the documentation formatting could be improved
> which will resolve some Sphinx amdgpu build warnings e.g
>
> WARNING: Unexpected indentation.
> WARNING: Block quote ends without a blank line; unexpected unindent.
> WARNING: Inline em
We are reserving vram pages so they should be aligned to the
GPU page size.
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_r
Am 16.09.19 um 21:35 schrieb Andrey Grodzovsky:
Regression on Vega20 caused by 0d9a80213829e092c1a85166b660ebf2c7af573e.
Not setting BO ptr to null will confuse amdgpu_bo_create_reserved.
Signed-off-by: Andrey Grodzovsky
Reviewed-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_
Regression on Vega20 caused by 0d9a80213829e092c1a85166b660ebf2c7af573e.
Not setting BO ptr to null will confuse amdgpu_bo_create_reserved.
Signed-off-by: Andrey Grodzovsky
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu
Am 16.09.19 um 20:43 schrieb Alex Deucher:
Looks like a copy and paste typo.
Fixes: 0d9a80213829 ("drm/amdgpu: cleanup creating BOs at fixed location")
Signed-off-by: Alex Deucher
Reviewed-by: Christian König for the series.
Sorry for the noise, I don't have hardware to actually test that :
Am 16.09.19 um 20:22 schrieb Alex Deucher:
Add new parameters.
Signed-off-by: Alex Deucher
Reviewed-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
b/drivers/gpu/drm/amd/amdgpu/a
Series is Reviewed-by: Andrey Grodzovsky
Andrey
On 9/16/19 2:43 PM, Alex Deucher wrote:
> Looks like a copy and paste typo.
>
> Fixes: 0d9a80213829 ("drm/amdgpu: cleanup creating BOs at fixed location")
> Signed-off-by: Alex Deucher
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 +-
> 1
Looks like a copy and paste typo.
Fixes: 0d9a80213829 ("drm/amdgpu: cleanup creating BOs at fixed location")
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
b/driv
Typo in the function name.
Fixes: 0d9a80213829 ("drm/amdgpu: cleanup creating BOs at fixed location")
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
b/drive
Reviewed-by: Andrey Grodzovsky
Andrey
On 9/16/19 2:22 PM, Alex Deucher wrote:
> Add new parameters.
>
> Signed-off-by: Alex Deucher
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> b/drivers/gp
Add new parameters.
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
index f1175f9f4b80..54549a26ee01 100644
--- a/drivers/gpu/drm/amd/amdg
On Tue, Sep 17, 2019 at 02:46:48AM +0900, Masahiro Yamada wrote:
> (+CC Stephen Rothwell, Mark Brown)
>
> On Mon, Sep 16, 2019 at 1:46 PM Austin Kim wrote:
> >
> > gcc throws compile error with below message:
>
> GNU Make throws ...
>
>
I don't have the original patch so I don't know what the
kfd needs drm_device to call into drm_cgroup functions
Signed-off-by: Harish Kasiviswanathan
Reviewed-by: Felix Kuehling
---
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c | 2 +-
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h | 1 +
drivers/gpu/drm/amd/amdkfd/kfd_device.c| 2 ++
drivers/gpu/drm/am
For AMD compute (amdkfd) driver.
All AMD compute devices are exported via single device node /dev/kfd. As
a result devices cannot be controlled individually using device cgroup.
AMD compute devices will rely on its graphics counterpart that exposes
/dev/dri/renderN node for each device. For each
Participate in device cgroup. All kfd devices are exposed via /dev/kfd.
So use /dev/dri/renderN node.
Before exposing the device to a task check if it has permission to
access it. If the task (based on its cgroup) can access /dev/dri/renderN
then expose the device via kfd node.
If the task cannot
This is required to check against cgroup permissions.
Signed-off-by: Harish Kasiviswanathan
Reviewed-by: Felix Kuehling
---
drivers/gpu/drm/amd/amdkfd/kfd_topology.c | 10 ++
drivers/gpu/drm/amd/amdkfd/kfd_topology.h | 3 +++
2 files changed, 13 insertions(+)
diff --git a/drivers/gpu/
Hi Tejun,
Sorry. Resending these patches as I got distracted with other stuff and
couldn't follow through. As per our previous discussion the plan was to
upstream these changes (mainly patch 3 "Export devcgroup_check_permission")
through Alex Deucher's amd-staging-drm-next and Dave Airlie's drm-ne
On 2019-09-16 12:33 p.m., Zhao, Yong wrote:
> These were deleted before, but somehow showed up again. Delete them again.
>
> Change-Id: I19b3063932380cb74a01d505e8e92f897a2c2cb7
> Signed-off-by: Yong Zhao
Reviewed-by: Felix Kuehling
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 4
>
(+CC Stephen Rothwell, Mark Brown)
On Mon, Sep 16, 2019 at 1:46 PM Austin Kim wrote:
>
> gcc throws compile error with below message:
GNU Make throws ...
This is probably a merge mistake in linux-next.
If so, this should be directly fixed in the linux-next.
If it is not fixed in time,
please
On 2019-09-16 12:26 p.m., wrote:
> This was deleted before, but somehow showed up again. Delete it again.
>
> Change-Id: I19b3063932380cb74a01d505e8e92f897a2c2cb7
> Signed-off-by: Yong Zhao
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/
Acked-by: Alex Deucher
From: amd-gfx on behalf of Zhao, Yong
Sent: Monday, September 16, 2019 12:33 PM
To: amd-gfx@lists.freedesktop.org
Cc: Zhao, Yong
Subject: [PATCH] drm/amdkfd: Delete unused KFD_IS_* macro
These were deleted before, but somehow showed up
These were deleted before, but somehow showed up again. Delete them again.
Change-Id: I19b3063932380cb74a01d505e8e92f897a2c2cb7
Signed-off-by: Yong Zhao
---
drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
b/drive
This was deleted before, but somehow showed up again. Delete it again.
Change-Id: I19b3063932380cb74a01d505e8e92f897a2c2cb7
Signed-off-by: Yong Zhao
---
drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h
b/drivers/gp
Hi Linus,
Continuing on the cleanup theme from last merge window, here is
another set of hmm and closely related patches that are largely
cleanups and bug fixes without any particularly notable functional
change.
The diffstat below covers a wide section of the kernel, but a lot of
this is caused
drivers/gpu/drm/amd/amdgpu/../display/dc/dml/Makefile:70: *** missing
`endif'. Stop.
make[4]: *** [drivers/gpu/drm/amd/amdgpu] Error 2
Signed-off-by: Xinpeng Liu
---
drivers/gpu/drm/amd/display/dc/dml/Makefile | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/amd/display/dc/dm
Reviewed-by: Guchun Chen
-Original Message-
From: Christian König
Sent: Friday, September 13, 2019 8:08 PM
To: Grodzovsky, Andrey ; Zhou1, Tao
; Chen, Guchun ;
amd-gfx@lists.freedesktop.org
Subject: [PATCH] drm/amdgpu: cleanup creating BOs at fixed location
The placement is something
Am 16.09.19 um 07:47 schrieb Chen, Guchun:
Use debugfs_remove_recursive to remove the whole debugfs
directory instead of removing the node one by one.
Change-Id: I27e9edab20d568b16d0163f8c7c3d81fb6832acd
Signed-off-by: Guchun Chen
Reviewed-by: Christian König
---
drivers/gpu/drm/amd/amdg
Some of the documentation formatting could be improved
which will resolve some Sphinx amdgpu build warnings e.g
WARNING: Unexpected indentation.
WARNING: Block quote ends without a blank line; unexpected unindent.
WARNING: Inline emphasis start-string without end-string.
Signed-off-by: Adam Zerel
Reviewed-by: Emily Deng
>-Original Message-
>From: Zhao, Jiange
>Sent: Monday, September 16, 2019 3:02 PM
>To: amd-gfx@lists.freedesktop.org
>Cc: Nieto, David M ; Deng, Emily
>; Koenig, Christian ;
>Zhao, Jiange
>Subject: [PATCH] drm/amdgpu/SRIOV: Navi12 SRIOV VF gets GTT base
>
>From:
From: Jiange Zhao
With changes in PSP and HV, SRIOV VF will handle
vram gtt location just like bare metal. There is
no need to differentiate it anymore.
Signed-off-by: Jiange Zhao
---
drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a
53 matches
Mail list logo
