RE: [PATCH 07/83] drm/radeon: Add kfd-->kgd interface of locking srbm_gfx_cntl register

2014-07-11 Thread Bridgman, John
>Koenig, Christian >Subject: RE: [PATCH 07/83] drm/radeon: Add kfd-->kgd interface of locking >srbm_gfx_cntl register > >Checking... we shouldn't need to call the lock from kfd any more.We should >be able to do any required locking in radeon kgd code. Confirmed. The lock

RE: [PATCH 07/83] drm/radeon: Add kfd-->kgd interface of locking srbm_gfx_cntl register

2014-07-11 Thread Bridgman, John
, 2014 1:48 PM >To: 'Jerome Glisse'; Oded Gabbay >Cc: David Airlie; Deucher, Alexander; linux-kernel@vger.kernel.org; dri- >de...@lists.freedesktop.org; Lewycky, Andrew; Joerg Roedel; Gabbay, Oded; >Koenig, Christian >Subject: RE: [PATCH 07/83] drm/radeon: Add kfd-->kgd in

RE: [PATCH 07/83] drm/radeon: Add kfd-->kgd interface of locking srbm_gfx_cntl register

2014-07-11 Thread Bridgman, John
avid Airlie; Deucher, Alexander; linux-kernel@vger.kernel.org; dri- >de...@lists.freedesktop.org; Bridgman, John; Lewycky, Andrew; Joerg >Roedel; Gabbay, Oded; Koenig, Christian >Subject: Re: [PATCH 07/83] drm/radeon: Add kfd-->kgd interface of locking >srbm_gfx_cntl register > >On Fr

Re: [PATCH 07/83] drm/radeon: Add kfd-->kgd interface of locking srbm_gfx_cntl register

2014-07-11 Thread Jerome Glisse
On Fri, Jul 11, 2014 at 12:50:07AM +0300, Oded Gabbay wrote: > This patch adds a new interface to kfd2kgd_calls structure, which > allows the kfd to lock and unlock the srbm_gfx_cntl register Why does kfd needs to lock this register if kfd can not access any of those register ? This sounds broken