This is a note to let you know that I've just added the patch titled drm/radeon/cik: stop the sdma engines in the enable() function to the 3.13-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: drm-radeon-cik-stop-the-sdma-engines-in-the-enable-function.patch and it can be found in the queue-3.13 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From 07ae78c9798b79bad3d3adf983c94ba23fde54d4 Mon Sep 17 00:00:00 2001 From: Alex Deucher <alexander.deucher@xxxxxxx> Date: Wed, 12 Mar 2014 15:26:34 -0400 Subject: drm/radeon/cik: stop the sdma engines in the enable() function From: Alex Deucher <alexander.deucher@xxxxxxx> commit 07ae78c9798b79bad3d3adf983c94ba23fde54d4 upstream. We always stop the rings when disabling the engines so just call the stop functions directly from the sdma enable function. This way the rings' status is set correctly on suspend so there are no problems on resume. Fixes resume failures that result in acceleration getting disabled. Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- drivers/gpu/drm/radeon/cik_sdma.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) --- a/drivers/gpu/drm/radeon/cik_sdma.c +++ b/drivers/gpu/drm/radeon/cik_sdma.c @@ -222,6 +222,11 @@ void cik_sdma_enable(struct radeon_devic u32 me_cntl, reg_offset; int i; + if (enable == false) { + cik_sdma_gfx_stop(rdev); + cik_sdma_rlc_stop(rdev); + } + for (i = 0; i < 2; i++) { if (i == 0) reg_offset = SDMA0_REGISTER_OFFSET; @@ -349,10 +354,6 @@ static int cik_sdma_load_microcode(struc if (!rdev->sdma_fw) return -EINVAL; - /* stop the gfx rings and rlc compute queues */ - cik_sdma_gfx_stop(rdev); - cik_sdma_rlc_stop(rdev); - /* halt the MEs */ cik_sdma_enable(rdev, false); @@ -421,9 +422,6 @@ int cik_sdma_resume(struct radeon_device */ void cik_sdma_fini(struct radeon_device *rdev) { - /* stop the gfx rings and rlc compute queues */ - cik_sdma_gfx_stop(rdev); - cik_sdma_rlc_stop(rdev); /* halt the MEs */ cik_sdma_enable(rdev, false); radeon_ring_fini(rdev, &rdev->ring[R600_RING_TYPE_DMA_INDEX]); Patches currently in stable-queue which might be from alexander.deucher@xxxxxxx are queue-3.13/drm-radeon-fix-runpm-disabling-on-non-px-harder.patch queue-3.13/drm-radeon-dpm-fix-typo-in-evergreen_smc_firmware_header_softregisters.patch queue-3.13/drm-radeon-cik-properly-set-sdma-ring-status-on-disable.patch queue-3.13/drm-radeon-cik-stop-the-sdma-engines-in-the-enable-function.patch queue-3.13/drm-radeon-re-order-firmware-loading-in-preparation-for-dpm-rework.patch queue-3.13/drm-radeon-atom-select-the-proper-number-of-lanes-in.patch queue-3.13/drm-radeon-ttm-must-be-init-with-cpu-visible-vram-v2.patch queue-3.13/drm-radeon-cik-properly-set-compute-ring-status-on-disable.patch queue-3.13/drm-radeon-fix-minor-typos-in-si_dpm.c.patch queue-3.13/drm-radeon-si-fix-typo-in-dpm-sq-ramping-setup.patch -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html