Re: AMD and free and open source software

2016-08-31 Thread Michel Dänzer
On 01/09/16 11:25 AM, Bridgman, John wrote: > One suggestion that was made recently was for AMD to supply packages > containing only HW microcode images for AMD GPUs... The microcode images for AMD GPUs have already been split out into a separate firmware-amd-graphics package in Debian sid (and

RE: AMD and free and open source software

2016-08-31 Thread Bridgman, John
Right... the microcode is part of the HW design; some vendors build the microcode images into the chip, while others have the BIOS or driver load them at start-up. The industry is generally moving to driver-loaded microcode, but I don't believe any vendor is planning to start opening up their

RE: AMD and free and open source software

2016-08-31 Thread Bridgman, John
One suggestion that was made recently was for AMD to supply packages containing only HW microcode images for AMD GPUs... you would still be using a non-free package but you would not have to enable the entire non-free Debian package that contains both real "firmware" and HW microcode for a

Re: AMD and free and open source software

2016-08-31 Thread Huang Rui
We don't have the plan to open up firmware source. Thanks, Rui On Thu, Sep 01, 2016 at 04:16:59AM +0800, Frederique wrote: > Dear Huang Rui, > > I recently swapped my NVIDIA Geforce 980 Ti for an AMD R9 Fury because > of the devoted efforts that are being made towards a free and open > source

Re: [PATCH 04/10] drm/amdgpu: skip suspend/resume on DRM_SWITCH_POWER_DYNAMIC_OFF

2016-08-31 Thread Michel Dänzer
On 01/09/16 07:08 AM, Alex Deucher wrote: > Missed this case previously. No need to do anything if the > device is already off. runtime pm will handle it. > > Signed-off-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 -- > 1 file changed,

Re: [PATCH 03/10] drm/amdgpu: handle runtime pm in fbcon

2016-08-31 Thread Michel Dänzer
On 01/09/16 07:08 AM, Alex Deucher wrote: > Ported from nouveau. > > Signed-off-by: Alex Deucher [...] > +static int > +amdgpufb_open(struct fb_info *info, int user) > +{ > + struct amdgpu_fbdev *rfbdev = info->par; > + struct amdgpu_device *adev =

Re: [PATCH 01/10] drm/amdgpu: handle runtime pm in drm pre/post close

2016-08-31 Thread Michel Dänzer
On 01/09/16 07:08 AM, Alex Deucher wrote: > Close was not handled correctly. > > Signed-off-by: Alex Deucher > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c >

[PATCH 07/10] drm/radeon: handle runtime pm correctly in amdgpu_driver_open_kms

2016-08-31 Thread Alex Deucher
Need to fix the error paths. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_kms.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_kms.c b/drivers/gpu/drm/radeon/radeon_kms.c index

[PATCH 04/10] drm/amdgpu: skip suspend/resume on DRM_SWITCH_POWER_DYNAMIC_OFF

2016-08-31 Thread Alex Deucher
Missed this case previously. No need to do anything if the device is already off. runtime pm will handle it. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

[PATCH 08/10] drm/radeon: handle runtime pm in fbcon

2016-08-31 Thread Alex Deucher
Ported from nouveau. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_fb.c | 25 + 1 file changed, 25 insertions(+) diff --git a/drivers/gpu/drm/radeon/radeon_fb.c b/drivers/gpu/drm/radeon/radeon_fb.c index 0e3143a..1ae32c9

[PATCH 10/10] drm/radeon: set runtime pm state to active on resume

2016-08-31 Thread Alex Deucher
The sbios always powers up the dGPU on resume. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_drv.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/radeon/radeon_drv.c b/drivers/gpu/drm/radeon/radeon_drv.c index

[PATCH 09/10] drm/radeon: skip suspend/resume on DRM_SWITCH_POWER_DYNAMIC_OFF

2016-08-31 Thread Alex Deucher
Missed this case previously. No need to do anything if the device is already off. runtime pm will handle it. Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_device.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

[PATCH 02/10] drm/amdgpu: handle runtime pm correctly in amdgpu_driver_open_kms

2016-08-31 Thread Alex Deucher
Need to fix the error pathes. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c

RE: GPU lockup with amdgpu driver

2016-08-31 Thread Deucher, Alexander
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Luya Tshimbalanga > Sent: Wednesday, August 31, 2016 5:46 PM > To: amd-gfx@lists.freedesktop.org > Subject: Re: GPU lockup with amdgpu driver > > -BEGIN PGP SIGNED MESSAGE- > Hash:

Re: GPU lockup with amdgpu driver

2016-08-31 Thread Luya Tshimbalanga
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 To reproduce the bug, grab the latest snapshot of linux-firmware 20160816 (git7c3dfc0b). Install latest kernel 4.8.0+ and also staging kernels. GPU lockup happens on boot on both amdgpu and radeon drivers. - -- Luya Tshimbalanga Graphic & Web

[PATCH 3/4] drm/amd/powerplay: refine struct name for coding style reason.

2016-08-31 Thread Rex Zhu
delete useless member. Change-Id: I03585fc3476f2954735f487c17dffa668287ad45 Signed-off-by: Rex Zhu --- drivers/gpu/drm/amd/powerplay/inc/power_state.h | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git

[PATCH 4/4] drm/amd/powerplay: read power state table from vbios and can't be changed. so allocate memory for request ps and current ps.

2016-08-31 Thread Rex Zhu
Signed-off-by: Rex Zhu --- drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 2 +- drivers/gpu/drm/amd/powerplay/eventmgr/psm.c | 5 +++-- drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 15 +-- 3 files changed, 17 insertions(+), 5 deletions(-) diff --git

[PATCH 1/4] drm/amd/powerplay: refine struct name for code style reason.

2016-08-31 Thread Rex Zhu
Change-Id: I491a1de273f350a14576c45b2ec986365797dcc0 Signed-off-by: Rex Zhu --- drivers/gpu/drm/amd/powerplay/hwmgr/processpptables.c | 2 +- drivers/gpu/drm/amd/powerplay/inc/hwmgr.h | 4 ++-- drivers/gpu/drm/amd/powerplay/inc/power_state.h | 2 +- 3 files

[PATCH 2/4] drm/amd/powerplay: add vce state tables initialize for ppt v1.

2016-08-31 Thread Rex Zhu
Change-Id: Ib44bf1c7c1cb71ce619cc4589608e4c049f9db7e Signed-off-by: Rex Zhu --- .../amd/powerplay/hwmgr/tonga_processpptables.c| 58 ++ drivers/gpu/drm/amd/powerplay/inc/hwmgr.h | 22 ++-- 2 files changed, 75 insertions(+), 5 deletions(-)

[pull] amdgpu drm-fixes-4.8

2016-08-31 Thread Alex Deucher
Hi Dave, Fixes for 4.8: - 2 CI S4 fixes - error handling fix I have additional features for 4.9, but I'll be out of the office next week, so I probably won't get to send them out until the following week. The following changes since commit 279cf3f23870f7eb8ca071115e06d3d5ca0a2b9e:

GPU lockup with amdgpu driver

2016-08-31 Thread Luya Tshimbalanga
Upon recommendation from Phoronix comment, here is the bug report.[1] Updating linux-firmware snapshot from 20160609 (gita4bbc811) to 20160816 (git7c3dfc0b) caused a GPU lockup on enabled amdgpu driver for CIK (Sea Island) card affecting kernel 4.8+ and the staging kernel branches. An example