Re: [PATCH] drm/amdgpu/vg20:support new UVD FW version naming convention

2018-06-07 Thread James Zhu
On 2018-06-07 03:10 AM, Christian König wrote: Am 06.06.2018 um 20:59 schrieb James Zhu: Vega20 UVD Firmware has a new version naming convention:    [31, 30] for encode interface major    [29, 24] for encode interface minor    [15, 8] for firmware revision    [7, 0] for hardware family id

Re: [PATCH xf86-video-amdgpu 0/7] Enabling Color Management - Round 3

2018-06-07 Thread Michel Dänzer
On 2018-06-07 03:26 PM, Mike Lothian wrote: > It seems messy having to create a whole xorg.conf just for one parameter Not sure I can agree, anyway it's the same whether the file is called /etc/X11/xorf.conf or /etc/X11/xorg.conf.d/foobar . :) -- Earthling Michel Dänzer |

Re: [PATCH] drm/amd/display: avoid sleeping in atomic context while creating new context or state

2018-06-07 Thread Michel Dänzer
On 2018-05-31 08:05 PM, Christian König wrote: > Am 30.05.2018 um 18:03 schrieb Harry Wentland: >> On 2018-05-30 06:17 AM, Shirish S wrote: >>> This patch fixes the warning messages that are caused due to calling >>> sleep in atomic context as below: >>> >>> BUG: sleeping function called from

Re: [PATCH] drm/amdgpu/vg20:support new UVD FW version naming convention

2018-06-07 Thread James Zhu
On 2018-06-07 08:38 AM, Leo Liu wrote: On 06/07/2018 03:10 AM, Christian König wrote: Am 06.06.2018 um 20:59 schrieb James Zhu: Vega20 UVD Firmware has a new version naming convention:    [31, 30] for encode interface major    [29, 24] for encode interface minor    [15, 8] for firmware

Re: [PATCH] drm/amdgpu/vg20:support new UVD FW version naming convention

2018-06-07 Thread Leo Liu
On 06/07/2018 03:10 AM, Christian König wrote: Am 06.06.2018 um 20:59 schrieb James Zhu: Vega20 UVD Firmware has a new version naming convention:    [31, 30] for encode interface major    [29, 24] for encode interface minor    [15, 8] for firmware revision    [7, 0] for hardware family id

Re: [PATCH xf86-video-amdgpu 0/7] Enabling Color Management - Round 3

2018-06-07 Thread Mike Lothian
It seems messy having to create a whole xorg.conf just for one parameter On Thu, 7 Jun 2018 at 08:31 Michel Dänzer wrote: > On 2018-06-07 09:22 AM, Mike Lothian wrote: > > Hi > > > > Is there a way to set depth using a xorg.conf.d snippet rather than > > creating an xorg.conf? > > Sure (the

RE: [PATCH 2/3] drm/amd/amdgpu: Fix crash in amdgpu_bo_reserve

2018-06-07 Thread Vishwakarma, Pratik
Ok. Abandoning the series. I am working on identifying the root cause and will post the new patch soon. Regards Pratik -Original Message- From: Koenig, Christian Sent: Wednesday, June 6, 2018 3:17 PM To: Vishwakarma, Pratik ; amd-gfx@lists.freedesktop.org; alexdeuc...@gmail.com;

Re: [PATCH] drm/amd/display: avoid sleeping in atomic context while creating new context or state

2018-06-07 Thread Harry Wentland
On 2018-06-07 10:44 AM, Michel Dänzer wrote: > On 2018-05-31 08:05 PM, Christian König wrote: >> Am 30.05.2018 um 18:03 schrieb Harry Wentland: >>> On 2018-05-30 06:17 AM, Shirish S wrote: This patch fixes the warning messages that are caused due to calling sleep in atomic context as

[PATCH] drm/amdgpu: update documentation for amdgpu_irq.c

2018-06-07 Thread Slava Abramov
Add/update function level documentation and add reference to amdgpu_irq.c in amdgpu.rst Signed-off-by: Slava Abramov --- Documentation/gpu/amdgpu.rst| 9 +++ drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c | 102 ++-- 2 files changed, 80 insertions(+), 31

[PATCH v2 2/3] drm/amdgpu: Add interrupt SQ source struct to amdgpu_gfx struct v2.

2018-06-07 Thread Andrey Grodzovsky
From: David Panariti SQ can generate interrupts on EDC/ECC errors and this struct controls how the interrupt is handled. The guts are filled in in the gf_v_.c files. v2: Rebase. Signed-off-by: David Panariti --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 1 + 1 file changed, 1 insertion(+) diff

[PATCH v2 3/3] drm/amdgpu: Add plumbing for handling SQ EDC/ECC interrupts v2.

2018-06-07 Thread Andrey Grodzovsky
From: David Panariti SQ can generate interrupts and installs the ISR to handle the SQ interrupts. Add parsing SQ data in interrupt handler. v2: Remove CZ only limitation. Rebase. Signed-off-by: David Panariti Signed-off-by: Andrey Grodzovsky --- drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 109

Re: [PATCH xf86-video-amdgpu 0/7] Enabling Color Management - Round 3

2018-06-07 Thread Leo Li
On 2018-06-06 01:03 PM, Michel Dänzer wrote: On 2018-06-06 06:01 PM, Michel Dänzer wrote: On 2018-06-01 06:03 PM, sunpeng...@amd.com wrote: From: "Leo (Sunpeng) Li" This ended up being different enough from v2 to warrant a new patchset. Per Michel's suggestions, there have been various

[PATCH] drm/amdgpu: change gfx8 ib test to use WB

2018-06-07 Thread Shirish S
This patch is extends the usage of WB in gfx8's ib test which was originally implemented in the below upstream patch: "ed9324a drm/amdgpu: change gfx9 ib test to use WB" Signed-off-by: Shirish S --- drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 35 +-- 1 file changed,

Re: [PATCH] drm/amdgpu: change gfx8 ib test to use WB

2018-06-07 Thread zhoucm1
On 2018年06月08日 12:54, Shirish S wrote: This patch is extends the usage of WB in gfx8's ib test which was originally implemented in the below upstream patch: "ed9324a drm/amdgpu: change gfx9 ib test to use WB" Signed-off-by: Shirish S Reviewed-by: Chunming Zhou ---

[PATCH] drm/amdgpu: Correct the ndw of bo update mapping.

2018-06-07 Thread Emily Deng
For buffer object that has shadow buffer, need twice commands. Signed-off-by: Emily Deng --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c index

Re: [PATCH][next] drm/amd/pp: initialized result to zero before or'ing in data

2018-06-07 Thread Colin Ian King
On 06/06/18 13:18, Colin King wrote: > From: Colin Ian King > > The current use of result is or'ing in values and checking for > a non-zero result, however, result is not initialized to zero > so it potentially contains garbage to start with. Fix this by > initializing it to the first return

Re: radeon 0000:1d:00.0: GPU lockup (current fence id 0x00000000010da43f last fence id 0x00000000010da52d on ring 0)

2018-06-07 Thread Borislav Petkov
On Wed, Jun 06, 2018 at 10:26:15AM +0200, Christian König wrote: > Well what did you do to trigger the lockup? Looks like an application send > something to the hardware to crash the GFX block. So what I observed was (in that order): machine was building a kernel so was busy, X didn't respond for

[PATCH][next] drm/amd/pp: initialized result to zero before or'ing in data

2018-06-07 Thread Colin King
From: Colin Ian King The current use of result is or'ing in values and checking for a non-zero result, however, result is not initialized to zero so it potentially contains garbage to start with. Fix this by initializing it to the first return from the call to

[PATCH] drm/amdgpu/vg20:support new UVD FW version naming convention

2018-06-07 Thread James Zhu
Vega20 UVD Firmware has a new version naming convention: [31, 30] for encode interface major [29, 24] for encode interface minor [15, 8] for firmware revision [7, 0] for hardware family id Inside kernel log UVD firmware Version: 1.1.2 (denote major.minor.revision) Signed-off-by: James Zhu

Re: [PATCH] drm/amdgpu/vg20:support new UVD FW version naming convention

2018-06-07 Thread Christian König
Am 06.06.2018 um 20:59 schrieb James Zhu: Vega20 UVD Firmware has a new version naming convention: [31, 30] for encode interface major [29, 24] for encode interface minor [15, 8] for firmware revision [7, 0] for hardware family id Inside kernel log UVD firmware Version: 1.1.2 (denote

Re: [PATCH xf86-video-amdgpu 0/7] Enabling Color Management - Round 3

2018-06-07 Thread Mike Lothian
Hi Is there a way to set depth using a xorg.conf.d snippet rather than creating an xorg.conf? Cheers Mike On Wed, 6 Jun 2018 at 17:01 Michel Dänzer wrote: > > Hi Leo, > > > On 2018-06-01 06:03 PM, sunpeng...@amd.com wrote: > > From: "Leo (Sunpeng) Li" > > > > This ended up being different

Re: [PATCH xf86-video-amdgpu 0/7] Enabling Color Management - Round 3

2018-06-07 Thread Michel Dänzer
On 2018-06-07 09:22 AM, Mike Lothian wrote: > Hi > > Is there a way to set depth using a xorg.conf.d snippet rather than > creating an xorg.conf? Sure (the xorg.conf.d snippets are simply concatenated together with xorg.conf, so anything can go in either of them), but it's the wrong way to do

Re: [PATCH xf86-video-amdgpu 0/7] Enabling Color Management - Round 3

2018-06-07 Thread Michel Dänzer
On 2018-06-06 07:03 PM, Michel Dänzer wrote: > On 2018-06-06 06:01 PM, Michel Dänzer wrote: >> On 2018-06-01 06:03 PM, sunpeng...@amd.com wrote: >>> From: "Leo (Sunpeng) Li" >>> >>> This ended up being different enough from v2 to warrant a new patchset. Per >>> Michel's suggestions, there have

Re: [PATCH 1/3 v2] drm/v3d: Take a lock across GPU scheduler job creation and queuing.

2018-06-07 Thread Lucas Stach
Am Mittwoch, den 06.06.2018, 10:48 -0700 schrieb Eric Anholt: > Between creation and queueing of a job, you need to prevent any other > job from being created and queued.  Otherwise the scheduler's fences > may be signaled out of seqno order. > > v2: move mutex unlock to the error label. > > >

Re: [PATCH][next] drm/amd/pp: initialized result to zero before or'ing in data

2018-06-07 Thread Huang Rui
On Wed, Jun 06, 2018 at 01:18:31PM +0100, Colin King wrote: > From: Colin Ian King > > The current use of result is or'ing in values and checking for > a non-zero result, however, result is not initialized to zero > so it potentially contains garbage to start with. Fix this by > initializing it