[PATCH xf86-video-amdgpu 1/2] Skip some initialization steps for GPU screens

From: Michel Dänzer Xorg doesn't use the following functionality of GPU screens, so don't bother initializing it: * DRI page flipping * DRI3 / Present / SYNC fences * XVideo / XvMC * Root window with background None (Ported from radeon commit

答复: [PATCH 4/4] drm/amdgpu:enable MCBP for SR-IOV

Anyone review the 4 patches and another libdrm patch ? -邮件原件- 发件人: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] 代表 Monk Liu 发送时间: Wednesday, March 08, 2017 3:54 PM 收件人: amd-gfx@lists.freedesktop.org 抄送: Liu, Monk 主题: [PATCH 4/4] drm/amdgpu:enable MCBP for

Re: [PATCH xf86-video-ati 2/2] Pass TRUE to drmmode_set_desired_modes the first time for GPU screens

On Wed, Mar 8, 2017 at 4:03 AM, Michel Dänzer wrote: > From: Michel Dänzer > > This is the only place we call drmmode_set_desired_modes for GPU screens > during server startup. Without this change, the display outputs of > secondary GPUs may stay on

Re: [PATCH xf86-video-amdgpu] glamor: Use glamor_finish when available

On Wed, Mar 8, 2017 at 3:58 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher > --- > configure.ac| 9 +++-- >

Re: Shared semaphores for amdgpu

On 28 February 2017 at 11:46, zhoucm1 wrote: > Hi Dave, > > The attached is our semaphore implementation, amdgpu_cs.c is drm file, the > others are kernel file. > Any suggestion? Thanks, I've built a tree with all these in it, and started looking into the interface. I do

[PATCH 23/23] drm/amdgpu: get cs support of AMDGPU_HW_IP_UVD_ENC

From: Leo Liu Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 9 + 1 file changed, 9 insertions(+) diff --git

[PATCH 17/23] drm/amdgpu: move amdgpu_vce structure to vce header

From: Leo Liu Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 29 +

[PATCH 18/23] drm/amdgpu: add uvd enc rings

From: Leo Liu And initialize them Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h | 3 ++-

[PATCH 16/23] drm/amdgpu: move amdgpu_uvd structure to uvd header

From: Leo Liu Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 30 +-

[PATCH 20/23] drm/amdgpu: add uvd enc ring type and functions

From: Leo Liu Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 19/23] drm/amdgpu: add uvd enc run queue

From: Leo Liu Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 14/23] drm/amdgpu: add DF MGCG flag

From: Huang Rui Signed-off-by: Huang Rui Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/include/amd_shared.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 13/23] drm/amdgpu: add DRM MGCG header

From: Huang Rui Signed-off-by: Huang Rui Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/include/amd_shared.h | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH 15/23] drm/amdgpu: add a ring func for end command

From: Leo Liu Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 3 +++ drivers/gpu/drm/amd/amdgpu/amdgpu_ring.h | 1 + 2 files

[PATCH 12/23] drm/amdgpu: add asic callback to get memsize register

Newer asics use different registers so abstract it. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h| 3 +++ drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- drivers/gpu/drm/amd/amdgpu/cik.c | 6 ++

[PATCH 08/23] drm/amdgpu: add a ucode size member into firmware info

From: Huang Rui This will be used for newer asics. Signed-off-by: Huang Rui Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_ucode.c | 7 +--

[PATCH 00/23] Prep patches for new asics

This set of patches updates various infrastructure in the driver to support new asics. Alex Deucher (3): drm/amdgpu: switch ih handling to two levels drm/amdgpu/ih: store the full context id drm/amdgpu: add asic callback to get memsize register Alex Xie (2): drm/amdgpu: set GART PTE asic

[PATCH 05/23] drm/amdgpu: switch ih handling to two levels

Newer asics have a two levels of irq ids now: client id - the IP src id - the interrupt src within the IP Signed-off-by: Alex Deucher Signed-off-by: Ken Wang Reviewed-by: Ken Wang Signed-off-by: Alex Deucher

[PATCH 01/23] drm/amdgpu: init aperture definitions

From: Junwei Zhang Signed-off-by: Flora Cui Reviewed-by: Jammy Zhou Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.h | 6 ++

[PATCH 04/23] drm/amdgpu: add 64bit doorbell functions

From: Ken Wang Newer asics need 64 bit doorbells. Signed-off-by: Ken Wang Reviewed-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h| 4

[PATCH 06/23] drm/amdgpu/ih: store the full context id

The contextID field (formerly known as src_data) of the IH vector stores client specific information about an interrupt. It was expanded from 32 bits to 128 on newer asics. Expand the src_id field to handle this. Reviewed-by: Harry Wentland Reviewed-by: Christian König

[PATCH 03/23] drm/amdgpu: add 64bit wb functions

From: Ken Wang Newer asics need 64 bit writeback slots. Signed-off-by: Ken Wang Reviewe-by: Alex Deucher Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h| 2 ++

Re: [PATCH umr] PM4 packets list number of dwords not bytes

Reviewed-by: Edward O'Callaghan On 03/09/2017 07:03 AM, Tom St Denis wrote: > When counting size of IBs count in dwords... > > Signed-off-by: Tom St Denis > --- > src/lib/ring_decode.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [PATCH 2/2] drm/amdgpu: bump driver version for some new features

On 03/09/2017 10:18 AM, Alex Deucher wrote: > We added new gem ioctl flags and the new fences ioctl, but forgot > to bump the version. Reviewed-by: Edward O'Callaghan Form which commit Alex? perhaps CC stable? > > Signed-off-by: Alex Deucher

[PATCH 1/2] drm/amdgpu: validate paramaters in the gem ioctl

Reject it if there are any invalid flags or domains. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c

[PATCH 2/2] drm/amdgpu: bump driver version for some new features

We added new gem ioctl flags and the new fences ioctl, but forgot to bump the version. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c

Re: [PATCH umr] Fix ring offset to dword instead of bytes

Acked-by: Edward O'Callaghan On 03/09/2017 03:31 AM, Tom St Denis wrote: > This allows you to specify (say) > > -R gfx[16:32] to read words 16 through 32 which is > consistent with how the data is presented. > > This patch also correctly enables the PM4 decoder >

[PATCH 21/27] drm/amd/display: Switch to DRM helpers in s3.

From: Andrey Grodzovsky Change-Id: I50db672b5b1f2eed7933863a2e901466dcb636d0 Signed-off-by: Andrey Grodzovsky Acked-by: Harry Wentland Reviewed-by: Tony Cheng Reviewed-by: Jordan Lazare

[PATCH 22/27] drm/amd/display: Refactor on dc_sink structure.

From: Zeyu Fan Change-Id: I47456746d26307356b62379b1cf3c221e5935eb5 Signed-off-by: Zeyu Fan Acked-by: Harry Wentland Reviewed-by: Tony Cheng --- drivers/gpu/drm/amd/display/dc/core/dc_link.c | 9 -

[PATCH 08/27] drm/amd/display: Refactor headless to use atomic commit.

From: Andrey Grodzovsky Headless mode set needs to be synchronized against outstanding nonblocking commits. This achieved by building atomic state and commiting it. Change-Id: Ie5e778afb33dd5c303a169216a7bb8c2d857037e Signed-off-by: Andrey Grodzovsky

[PATCH 04/27] drm/amd/display: freesync pipe split :VTotal_Min_Mask for Hflip/lock.

From: Charlene Liu Change-Id: Icb5d285ed455e7081f88cd09df3845c08f7ceb20 Signed-off-by: Charlene Liu Acked-by: Harry Wentland Reviewed-by: Jordan Lazare --- drivers/gpu/drm/amd/display/dc/core/dc.c

[PATCH 18/27] drm/amd/display: TPS4 logic typo fix

From: Charlene Liu Change-Id: I334d4dfa6f5db2366457b1db9062f8ae6ec04e19 Signed-off-by: Charlene Liu Acked-by: Harry Wentland Reviewed-by: Wenjing Liu Reviewed-by: Tony Cheng ---

[PATCH 25/27] drm/amd/display: fix viewport adjustment on rotated surface

From: Dmytro Laktyushkin Change-Id: I1d7db6e097c5a0d307e4d08d70f52542afba0f93 Signed-off-by: Dmytro Laktyushkin Acked-by: Harry Wentland Reviewed-by: Tony Cheng ---

[PATCH 17/27] drm/amd/display: move refclk from dc to resource_pool

From: Charlene Liu Change-Id: Ib84c5ff3887c4b096c8716279a7741b46350e433 Signed-off-by: Charlene Liu Acked-by: Harry Wentland Reviewed-by: Tony Cheng --- drivers/gpu/drm/amd/display/dc/core/dc.c |

[PATCH 05/27] drm/amd/display: We don't support interlace and doublescan

Don't tell DRM otherwise. Change-Id: I6c01701b3be35cb5badf236459ee5d9a1ab8c309 Signed-off-by: Harry Wentland Acked-by: Harry Wentland Reviewed-by: Tony Cheng --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c | 5

[PATCH 07/27] drm/amd/display: Refactor atomic commit implementation. (v2)

From: Andrey Grodzovsky Modify amdgpu_dm_atomic_comit to implement atomic_comit_tail hook. Unify Buffer objects allocation and dealocation for surface updates and page flips. Simplify wait for fences and target_vbank logic for non blockiing commit. Remove hacky update

[PATCH 19/27] drm/amd/display: After program backend, also program front end regs.

From: Yongqiang Sun Issue: In case of two 4K@60 + one non-4k monitor, when unplug non-4k monitor, the remain two 4k monitor don't work properly. Reason: In that case, two 4k use two pipes and no split, when unplug happens, those two monitor will use 4 pipes and split, but

[PATCH 24/27] drm/amd/display: fix hsplit viewport calculation for rotated/mirrored usecases

From: Dmytro Laktyushkin Change-Id: I603f1f04c9bbac3236dfd56b3587b6943e25acd1 Signed-off-by: Dmytro Laktyushkin Acked-by: Harry Wentland Reviewed-by: Tony Cheng ---

[PATCH 20/27] drm/amd/display: Do not copy bottom pipe when map resource.

From: Yongqiang Sun Change-Id: I30658257832be3865f3f3af0b471fd490fab4426 Signed-off-by: Yongqiang Sun Acked-by: Harry Wentland Reviewed-by: Tony Cheng ---

[PATCH 10/27] drm/amd/display: clean up and simply locking logic

From: Tony Cheng always take update lock instead of using HW built in update lock trigger with write to primary_addr_lo. we will be a little more inefficient with the extra registers write to lock, but this simplify code and make it always correct. Will revisit locking

[PATCH 16/27] drm/amd/display: Memory leak fix during disable

From: Leon Elazar 1.current_context memory wasn't released at dc_post_update_surfaces_to_stream during context swap. Change-Id: Iff814cf675856708daadfa540ef2b57d633139df Signed-off-by: Leon Elazar Acked-by: Harry Wentland

[PATCH 27/27] drm/amd/display: fix incorrect vp adjustment

From: Dmytro Laktyushkin Viewport would be incorrectly adjusted when surface was used for multiple displays Change-Id: Iffd7e03973edc72f63d21aa83b0482f6013d2ecd Signed-off-by: Dmytro Laktyushkin Acked-by: Harry Wentland

[PATCH 14/27] drm/amd/display: Simplify some DMCU waits

From: Amy Zhang Change-Id: I3fbfab5a617bed56bb5013e820b0e37c507e0bfe Signed-off-by: Amy Zhang Acked-by: Harry Wentland Reviewed-by: Tony Cheng --- drivers/gpu/drm/amd/display/dc/dce/dce_clocks.c | 1 +

[PATCH 12/27] drm/amd/display: sometime VtotalMin less than VTotal (rounding issue)

From: Charlene Liu Change-Id: I1f8de5415aac149d85b73a28fb630c50696a9250 Signed-off-by: Charlene Liu Acked-by: Harry Wentland Reviewed-by: Jordan Lazare ---

[PATCH 13/27] drm/amd/display: Adding FastUpdate functionality

From: Leon Elazar Exposing DC Api dc_check_update_surfaces_for_stream validation will return the answer which type of update is required, so upper layers can is it safe to call the update API fro high IRQ yes/no. Change-Id: I094592c5df4227ed2fea2ceb5de5b2604173fa20

[PATCH 03/27] drm/amd/display: Remove unused define from amdgpu_dm_types

Change-Id: Idfd0d0d6e537eddbc75378ba394b0f36bd89dd50 Signed-off-by: Harry Wentland Acked-by: Harry Wentland Reviewed-by: Tony Cheng --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.h | 5 - 1 file changed, 5

[PATCH 09/27] drm/amd/display: Remove page_fleep_needed function.

From: Andrey Grodzovsky This function makes a lot of assumptions to try and deduce tha an atomic_commit came from FLIP ioctl. Those assumptions are not explicit contract with DRM API and might lead to wrong result as the code changes. Instead use

[PATCH 01/27] drm/amd/display: Use amdgpu mode funcs statically

No need to assign them dynamically. This is much more readable. Change-Id: I2d8a356e8d916800c4553c4e9b19ce42f7f1c391 Signed-off-by: Harry Wentland Acked-by: Harry Wentland Reviewed-by: Tony Cheng ---

[PATCH 06/27] drm/amd/display: extended the programming sequence to VFlip as well

From: Charlene Liu Change-Id: Iccf6c436dc0807f4d5e0ce96ae1e64ed31c20a38 Signed-off-by: Charlene Liu Acked-by: Harry Wentland Reviewed-by: Charlene Liu --- drivers/gpu/drm/amd/display/dc/core/dc.c | 19

[PATCH 00/27] DC Patches Mar 8, 2017

* Use atomic helpers for commit, suspend/resume, and gamma * Whole bunch of fixes * Updated scaler calculations Amy Zhang (1): drm/amd/display: Simplify some DMCU waits Andrey Grodzovsky (4): drm/amd/display: Refactor atomic commit implementation. (v2) drm/amd/display: Refactor headless to

Re: [PATCH 2/2] drm/amdgpu: trace fence details in amdgpu_sched_run_job

I'm working on a follow-up that has Christian's extra requests to remove the job/fence pointers. But that is coming as a separate commit. Regards, Andres ​ ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org

Re: [PATCH 2/2] drm/amdgpu: trace fence details in amdgpu_sched_run_job

Is this patch ready to land or is there more work required? Thanks, Alex On Sun, Feb 26, 2017 at 5:54 AM, Christian König wrote: > Am 25.02.2017 um 18:28 schrieb Andres Rodriguez: > > > > On Feb 25, 2017 4:40 AM, "Christian König" wrote: > >

Re: [PATCH 1/2] drm/amdgpu: make trace format uniform csv name=value

On Sat, Feb 25, 2017 at 4:33 AM, Christian König wrote: > Am 24.02.2017 um 19:20 schrieb Andres Rodriguez: >> >> Most of the traces have uniform format except for two of them. Having >> all the traces match makes it simple to run awk on the ftrace output. >> >>

Re: [PATCH 4/5] drm/radeon: Don't register Thunderbolt eGPU with vga_switcheroo

On Wed, Mar 8, 2017 at 12:01 AM, Lukas Wunner wrote: > On Tue, Mar 07, 2017 at 03:30:30PM -0500, Alex Deucher wrote: >> On Fri, Feb 24, 2017 at 2:19 PM, Lukas Wunner wrote: >> > An external Thunderbolt GPU can neither drive the laptop's panel nor be >> > powered

[PATCH umr] PM4 packets list number of dwords not bytes

When counting size of IBs count in dwords... Signed-off-by: Tom St Denis --- src/lib/ring_decode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/ring_decode.c b/src/lib/ring_decode.c index 401d6caaa088..a991f5ffa237 100644 ---

Re: [PATCH 15/24] drm/radeon: Merge pre/postclose hooks

Reviewed-by: Christian König for this one and #19. Christian. Am 08.03.2017 um 15:12 schrieb Daniel Vetter: Again no apparent explanation for the split except hysterical raisins. Merging them also makes it a bit more obviuos what's going on wrt the runtime pm

[PATCH 15/24] drm/radeon: Merge pre/postclose hooks

Again no apparent explanation for the split except hysterical raisins. Merging them also makes it a bit more obviuos what's going on wrt the runtime pm refdancing. Cc: Alex Deucher Cc: Christian König Cc: amd-gfx@lists.freedesktop.org

Re: [PATCH 4/5] drm/radeon: Don't register Thunderbolt eGPU with vga_switcheroo

On Wed, Mar 08, 2017 at 11:46:33AM +0100, Peter Wu wrote: > On Wed, Mar 08, 2017 at 06:01:54AM +0100, Lukas Wunner wrote: > > On Tue, Mar 07, 2017 at 03:30:30PM -0500, Alex Deucher wrote: > > > On Fri, Feb 24, 2017 at 2:19 PM, Lukas Wunner wrote: > > > > An external Thunderbolt

Re: [PATCH 4/5] drm/radeon: Don't register Thunderbolt eGPU with vga_switcheroo

On Wed, Mar 08, 2017 at 06:01:54AM +0100, Lukas Wunner wrote: > On Tue, Mar 07, 2017 at 03:30:30PM -0500, Alex Deucher wrote: > > On Fri, Feb 24, 2017 at 2:19 PM, Lukas Wunner wrote: > > > An external Thunderbolt GPU can neither drive the laptop's panel nor be > > > powered off

Re: [PATCH] drm:IB flag cleanups

Am 08.03.2017 um 11:22 schrieb Liu, Monk: Rebase done: will send out another patch for review Additional to that I would provide the implementation (e.g. forwarding it to the IB command) for the PREEMPT flag in the same patch. No, we don't have such imple, because this flag will be tagged by

[PATCH] amdgpu:update IB flags with PREEMPT

PREEMPT flag is used for MCBP Change-Id: I2ddf0f3912c219663e80680f76d671c7856e Signed-off-by: Monk Liu --- include/drm/amdgpu_drm.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h index

RE: [PATCH] drm:IB flag cleanups

Rebase done: will send out another patch for review >Additional to that I would provide the implementation (e.g. forwarding it to >the IB command) for the PREEMPT flag in the same patch. No, we don't have such imple, because this flag will be tagged by UMD, instead of by libdrm BR Monk

Re: [PATCH] drm:IB flag cleanups

Please rebase that on the amd-staging-4.9 branch, the GDS flag never made it upstream. Additional to that I would provide the implementation (e.g. forwarding it to the IB command) for the PREEMPT flag in the same patch. Christian. Am 08.03.2017 um 09:13 schrieb Monk Liu: we don't use

[PATCH xf86-video-ati 2/2] Pass TRUE to drmmode_set_desired_modes the first time for GPU screens

From: Michel Dänzer This is the only place we call drmmode_set_desired_modes for GPU screens during server startup. Without this change, the display outputs of secondary GPUs may stay on even while Xorg isn't using them. Signed-off-by: Michel Dänzer

[PATCH xf86-video-ati 1/2] Skip some initialization steps for GPU screens

From: Michel Dänzer Xorg doesn't use the following functionality of GPU screens, so don't bother initializing it: * DRI page flipping * DRI3 / Present / SYNC fences * XVideo / XvMC * Root window with background None Signed-off-by: Michel Dänzer

[PATCH xf86-video-amdgpu] glamor: Use glamor_finish when available

From: Michel Dänzer Signed-off-by: Michel Dänzer --- configure.ac| 9 +++-- src/amdgpu_glamor.c | 7 +++ 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 00249a599..591c2b9f6 100644

[PATCH] drm:IB flag cleanups

we don't use FLAG_GDS actually, and we need PREEMPT flag Change-Id: I87f704e3fa97d7a7ffa956f0c94b77c333de380d Signed-off-by: Monk Liu --- include/drm/amdgpu_drm.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/include/drm/amdgpu_drm.h