As tonga/fiji/polaris10/11 use ppt table version 1.
so rename tonga_processtable to process_pptables_v1_0
Rex Zhu (3):
drm/amd/powerplay: rename tonga_processpptable* to
processpptable_v1_0*.
drm/amd/powerplay: add pp_table_version in hwmgr.
drm/amd/powerplay: add pptable point check b
Change-Id: Iee3c7ab9c1318ec7206bf5a5ae59d8c48b8e823b
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 1 +
drivers/gpu/drm/amd/powerplay/inc/hwmgr.h | 8
2 files changed, 9 insertions(+)
diff --git a/drivers/gpu/drm/amd/powerplay/hwmg
Change-Id: Ibad281cd7fd660abe612894df82be284c9b865a9
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c
b/drivers/gpu/drm/amd/powerplay
This series is,
Reviewed-by: Edward O'Callaghan
On 09/12/2016 06:22 PM, Rex Zhu wrote:
> As tonga/fiji/polaris10/11 use ppt table version 1.
> so rename tonga_processtable to process_pptables_v1_0
>
>
> Rex Zhu (3):
> drm/amd/powerplay: rename tonga_processpptable* to
> processpptable_v1
v2: 1. move out ppt renamed patches.
2. wrap ppt version related functions in smu7_hwmgr.c
implement smu7_hwmgr smu7_smumgr to manager asics with smu version 7.
add delete duplicated code.
There are subtle differences in firmware image between different asics,
add asic_name_smc.c under smumgr
Change-Id: I2ea5b1dc70883e8656fe0b1fd7a2b9a9c4ddcb27
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/inc/smumgr.h| 69 ++
drivers/gpu/drm/amd/powerplay/smumgr/smumgr.c | 101 +-
2 files changed, 169 insertions(+), 1
Change-Id: I22767dc03fcec497e5cd76e82459916abf130105
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/smumgr/Makefile |2 +-
.../gpu/drm/amd/powerplay/smumgr/polaris10_smc.c | 2284
.../gpu/drm/amd/powerplay/smumgr/polaris10_smc.h
Reviewed-by: Alex Deucher
Signed-off-by: Rex Zhu
---
drivers/gpu/drm/amd/powerplay/hwmgr/Makefile | 3 +--
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 25 +++--
2 files changed, 24 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/Makefile
b
Change-Id: I1bab73135461c289264b14972fbf15eb424734ee
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/smumgr/Makefile |2 +-
drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c| 2371
drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.h
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/hwmgr/Makefile | 4 +---
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 30 +---
2 files changed, 28 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/Makefil
Change-Id: Ia3dae5b7ef0889acd9479f83d5f3b8a91786f4bf
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/hwmgr/Makefile | 2 --
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 35 ++--
drivers/gpu/drm/amd/powerplay/inc/hwmgr.h| 1 +
3 f
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
.../gpu/drm/amd/powerplay/inc/polaris10_pwrvirus.h | 3 +-
.../gpu/drm/amd/powerplay/smumgr/polaris10_smc.c | 81 +--
.../drm/amd/powerplay/smumgr/polaris10_smumgr.c| 681 ++---
.../drm/amd/powerplay/smumgr/polaris10_s
Change-Id: If9ca14557f99f3e13e17086e8c0a90c53f232631
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c | 74 +--
.../gpu/drm/amd/powerplay/smumgr/tonga_smumgr.c| 657 +
.../gpu/drm/amd/powerplay/smumgr/tonga_smumgr.h
Change-Id: Ic5b83029c4e326ef4e02ee341ba8b1109d17328b
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
.../drm/amd/powerplay/hwmgr/iceland_powertune.h| 14 -
drivers/gpu/drm/amd/powerplay/smumgr/Makefile |5 +-
drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c | 2577 +
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/hwmgr/Makefile | 4 +---
drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 36 +---
2 files changed, 28 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/Makefi
Change-Id: Ie63ee99aa515e37943aee726103260819c9e2913
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c| 69 +--
drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c | 605 +
drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr
Change-Id: I8144e52141f7e6a49a18be3523540a6edd898ec0
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
.../gpu/drm/amd/powerplay/hwmgr/iceland_hwmgr.c| 50 +-
.../drm/amd/powerplay/hwmgr/iceland_powertune.c| 6 +-
.../gpu/drm/amd/powerplay/hwmgr/iceland_thermal.c | 2 +-
.../gpu
Signed-off-by: Rex Zhu
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/amd/powerplay/inc/smumgr.h | 1 +
drivers/gpu/drm/amd/powerplay/smumgr/Makefile | 2 +-
drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c | 589 +
drivers/gpu/drm/amd/powerplay/smumgr/smu7_
Am 12.09.2016 um 04:16 schrieb Huang Rui:
Clean up the codes to move the release handles into fail labels.
Signed-off-by: Huang Rui
Reviewed-by: Christian König .
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 21 ++---
1 file changed, 10 insertions(+), 11 deletions(-)
Am 12.09.2016 um 10:36 schrieb Edward O'Callaghan:
This series is,
Reviewed-by: Edward O'Callaghan
Acked-by: Christian König for the series.
On 09/12/2016 06:22 PM, Rex Zhu wrote:
As tonga/fiji/polaris10/11 use ppt table version 1.
so rename tonga_processtable to process_pptables_v1_0
R
A few coding style issues, but nothing major.
Taking patch #7 as an example, but most of this applies to the rest as well:
+/*sviLoadLIneEn, SviLoadLineVddC, TDC_VDDC_ThrottleReleaseLimitPerc,
TDC_MAWt, TdcWaterfallCtl, DTEAmbientTempBase, DisplayCac, BAPM_TEMP_GRADIENT */
That line looks t
From: Christian König
Use an explicit limit when moving things around to
force address space allocation.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b
From: Christian König
Doesn't make much sense to clear the GTT when it was never filled.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/drivers/gpu/drm/amd/amdgpu/amdgp
From: Christian König
This way the address space gets allocated immediately.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 20 ++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/drivers/g
From: Christian König
Only allocate address space when we really need it.
v2: fix a typo, add correct function description,
stop leaking the node in the error case.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/Makefile | 3 +-
drivers/gpu/drm/amd/amdgpu/amdgpu_c
From: Christian König
An invalid offset to be used by custom VRAM/GGT manager when we
don't have an offset for an BO.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 1 +
drivers/gpu/drm/amd/amdgpu/amdgpu_object.h | 2 ++
2 files changed, 3 insertions(+)
diff -
From: Christian König
We need to validate the offset to make sure that we don't write after the BO.
Additional to that a page should be enough and can make address space
handling much easier.
Signed-off-by: Christian König
---
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 6 ++
1 file changed,
Would it make sense to free the GTT address space in amdgpu_ttm_unbind?
No, when amdgpu_ttm_unbind() is called the address space is about to be
freed anyway.
Currently any migration of VRAM to GTT would allocate GTT address space
and never free that space until the BO is destroyed. But you cou
From: "monk.liu"
Change-Id: Idf3d3bf0f2d2396a77341f97174d0a173fdd8932
Signed-off-by: monk.liu
---
amdgpu/amdgpu.h| 3 ++-
amdgpu/amdgpu_cs.c | 12 +---
include/drm/amdgpu_drm.h | 3 ++-
tests/amdgpu/basic_tests.c | 2 +-
4 files changed, 14 insertions(+), 6 dele
From: Junwei Zhang
Signed-off-by: Junwei Zhang
Reviewed-by: Jammy Zhou
---
tests/amdgpu/basic_tests.c | 100 +
1 file changed, 100 insertions(+)
diff --git a/tests/amdgpu/basic_tests.c b/tests/amdgpu/basic_tests.c
index 11f6a63..5144e03 100644
--- a
From: Junwei Zhang
Signed-off-by: Junwei Zhang
Reviewed-by: Christian König
Reviewed-by: Jammy Zhou
---
amdgpu/amdgpu.h | 22 +++
amdgpu/amdgpu_cs.c | 72
include/drm/amdgpu_drm.h | 27 ++
3 files chan
From: Junwei Zhang
This function is used to look up the marking name
for a specific board.
v2: agd: Squash in subsequent updates to the table.
Signed-off-by: Junwei Zhang
Reviewed-by: Flora Cui
---
amdgpu/amdgpu.h | 10 +++
amdgpu/amdgpu_asic_id.h | 170 +
Attached image of scan-view's report. Basically the variable "sem" is free'ed
but then used as sem->next in the for loop. Should use safe version of macro.
Tom
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailm
From: Junwei Zhang
v2: agd: rebase and squash in all the previous optimizations and
changes so everything compiles.
Signed-off-by: Junwei Zhang
Reviewed-by: Monk Liu
Reviewed-by: Jammy Zhou
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 171 +++
From: Junwei Zhang
v2: agd: rebase and squash in all the previous optimizations and
changes so everything compiles.
v3: squash in Slava's 32bit build fix
Signed-off-by: Junwei Zhang
Reviewed-by: Monk Liu
Reviewed-by: Jammy Zhou
---
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 2 +
drivers/gpu
From: "monk.liu"
original we use ttm_dma path to allocate GTT bo, which is too much
slower than the path of ttm_pool, in most cases.
The swiotlb checks don't seem to work and we always end up in the
slow path even when an IOMMU is available.
Signed-off-by: monk.liu
Reviewed-by: Jammy Zhou
Sig
Switch to _SAFE variant of list macro to avoid use-after-free bug.
Signed-off-by: Tom St Denis
---
amdgpu/amdgpu_cs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/amdgpu/amdgpu_cs.c b/amdgpu/amdgpu_cs.c
index fb5b3a8c4bc4..cd92a0957293 100644
--- a/amdgpu/amdgpu_cs.c
+
Am 12.09.2016 um 20:18 schrieb Tom St Denis:
Switch to _SAFE variant of list macro to avoid use-after-free bug.
Signed-off-by: Tom St Denis
---
amdgpu/amdgpu_cs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/amdgpu/amdgpu_cs.c b/amdgpu/amdgpu_cs.c
index fb5b3a8c4bc
Am 12.09.2016 um 18:44 schrieb Alex Deucher:
From: "monk.liu"
original we use ttm_dma path to allocate GTT bo, which is too much
slower than the path of ttm_pool, in most cases.
The swiotlb checks don't seem to work and we always end up in the
slow path even when an IOMMU is available.
While
Seems like this is missing the update corresponding to libdrm patch
"amdgpu: cs_wait_fences now can return the first signaled fence
index". Is the kernel patch for that update also available or going
to be sent to the list?
Yours sincerely,
Bas Nieuwenhuizen
On Mon, Sep 12, 2016 at 6:40 PM, Alex
Yeah, we added that later on.
Looks like Alex (or somebody else) squashed the original and follow up
patch for libdrm together, but forgot to do so for the kernel side.
Just wait a moment, if Alex can't find that of hand I'm going to take a
look through the branches tomorrow.
Regards,
Chris
> -Original Message-
> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Christian König
> Sent: Monday, September 12, 2016 2:46 PM
> To: Bas Nieuwenhuizen; Alex Deucher
> Cc: Zhang, Jerry; amd-gfx@lists.freedesktop.org
> Subject: Re: [PATCH] drm/amdgpu: add the in
From: "monk.liu"
Return the index of the first signaled fence. This information
is useful in some APIs like Vulkan.
Signed-off-by: monk.liu
Signed-off-by: Alex Deucher
---
drivers/dma-buf/fence.c | 19 ++-
include/linux/fence.h | 2 +-
2 files changed, 15 insertions(+), 6
From: "monk.liu"
Return the index of the first signaled fence in the fences ioctl.
This information is useful in some APIs like Vulkan.
Signed-off-by: monk.liu
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 4 +++-
drivers/gpu/drm/amd/amdgpu/amdgpu_sa.c | 2 +-
inclu
On Mon, Sep 12, 2016 at 2:26 PM, Christian König
wrote:
> Am 12.09.2016 um 18:44 schrieb Alex Deucher:
>>
>> From: "monk.liu"
>>
>> original we use ttm_dma path to allocate GTT bo, which is too much
>> slower than the path of ttm_pool, in most cases.
>>
>> The swiotlb checks don't seem to work an
Useful for validating certain tests involving the CB where the
chicken bit status may have an affect on the results.
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/vi.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgp
Useful for validating certain tests involving the CB where the
chicken bit status may have an affect on the results.
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/cik.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/cik.c b/drivers/gpu/drm/amd/amd
On Mon, Sep 12, 2016 at 10:39 AM, Christian König
wrote:
> From: Christian König
>
> Use an explicit limit when moving things around to
> force address space allocation.
>
> Signed-off-by: Christian König
Typo in the patch title:
s/expolicit/explicit/
With that fixed:
Reviewed-by: Alex Deucher
On Mon, Sep 12, 2016 at 10:39 AM, Christian König
wrote:
> From: Christian König
>
> Only allocate address space when we really need it.
>
> v2: fix a typo, add correct function description,
> stop leaking the node in the error case.
>
> Signed-off-by: Christian König
Other than the comment
The comment header on top of amdgpu_gtt_mgr.c should say "Copyright
2016" (wrong year). With that fixed, this patch is Reviewed-by: Felix
Kuehling .
Regards,
Felix
On 16-09-12 10:39 AM, Christian König wrote:
> From: Christian König
>
> Only allocate address space when we really need it.
>
>
Yong is currently working on a follow-up fix to handle process shutdown
on APUs as well as S3 suspend correctly.
Regards,
Felix
On 16-09-10 12:06 PM, Oded Gabbay wrote:
> On Sat, Sep 10, 2016 at 4:31 AM, Edward O'Callaghan
> wrote:
>> pqm_uninit() will be called in kfd_process_notifier_releas
This regresses vce perf badly on tonga
https://bugs.freedesktop.org/show_bug.cgi?id=97494
Maybe I need a better test case or faster cpu or something,
but vce powerplay encode issues disappeared for me with the latest
firmware.
root wrote:
From: Rex Zhu
fix issue that encode test failed on th
On 13/09/16 01:44 AM, Alex Deucher wrote:
> From: "monk.liu"
>
> original we use ttm_dma path to allocate GTT bo, which is too much
> slower than the path of ttm_pool, in most cases.
>
> The swiotlb checks don't seem to work and we always end up in the
> slow path even when an IOMMU is available
On 13/09/16 06:05 AM, Alex Deucher wrote:
> On Mon, Sep 12, 2016 at 10:39 AM, Christian König
> wrote:
>> From: Christian König
>>
>> Use an explicit limit when moving things around to
>> force address space allocation.
>>
>> Signed-off-by: Christian König
>
> Typo in the patch title:
> s/expol
On Mon, Sep 12, 2016 at 9:17 PM, Michel Dänzer wrote:
> On 13/09/16 01:44 AM, Alex Deucher wrote:
>> From: "monk.liu"
>>
>> original we use ttm_dma path to allocate GTT bo, which is too much
>> slower than the path of ttm_pool, in most cases.
>>
>> The swiotlb checks don't seem to work and we alw
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/si_dpm.c | 287 +++---
drivers/gpu/drm/amd/amdgpu/si_smc.c | 37 ++--
drivers/gpu/drm/amd/amdgpu/sislands_smc.h | 30 ++--
3 files changed, 181 insertions(+), 173 deletions(-)
diff --git a/drivers/gp
remnants of the port from radeon.
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/amd/amdgpu/si_dpm.c | 34 --
1 file changed, 16 insertions(+), 18 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/si_dpm.c
b/drivers/gpu/drm/amd/amdgpu/si_dpm.c
index e4f7c10.
Patches 1&2 are,
Reviewed-by: Edward O'Callaghan
On 09/13/2016 02:11 PM, Alex Deucher wrote:
> remnants of the port from radeon.
>
> Signed-off-by: Alex Deucher
> ---
> drivers/gpu/drm/amd/amdgpu/si_dpm.c | 34 --
> 1 file changed, 16 insertions(+), 18 deletions
From: Maruthi Srinivas Bayyavarapu
This adds VCE VM mode support from Stoney onwards. Session tracking
is an open issue, yet to be supported.
v2: Fixed warnings from checkpatch.pl
Signed-off-by: Maruthi Bayyavarapu
Reviewed-by: Christian König
Signed-off-by: Alex Deucher
---
drivers/gpu/drm
59 matches
Mail list logo
