This is a note to let you know that I've just added the patch titled
drm/radeon: drop ttm two ended allocation
to the 3.19-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-drop-ttm-two-ended-allocation.patch
and it can be found in the queue-3.19 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.
>From a239118a24b3bf9089751068e431dfb63dc4168b Mon Sep 17 00:00:00 2001
From: Alex Deucher <alexander.deucher@xxxxxxx>
Date: Tue, 17 Mar 2015 11:53:33 -0400
Subject: drm/radeon: drop ttm two ended allocation
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
From: Alex Deucher <alexander.deucher@xxxxxxx>
commit a239118a24b3bf9089751068e431dfb63dc4168b upstream.
radeon_bo_create() calls radeon_ttm_placement_from_domain()
before ttm_bo_init() is called. radeon_ttm_placement_from_domain()
uses the ttm bo size to determine when to select top down
allocation but since the ttm bo is not initialized yet the
check is always false. It only took effect when buffers
were validated later. It also seemed to regress suspend
and resume on some systems possibly due to it not
taking effect in radeon_bo_create().
radeon_bo_create() and radeon_ttm_placement_from_domain()
need to be reworked substantially for this to be optimally
effective. Re-enable it at that point.
Noticed-by: Oded Gabbay <oded.gabbay@xxxxxxx>
Reviewed-by: Christian König <christian.koenig@xxxxxxx>
Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
drivers/gpu/drm/radeon/radeon_object.c | 11 -----------
1 file changed, 11 deletions(-)
--- a/drivers/gpu/drm/radeon/radeon_object.c
+++ b/drivers/gpu/drm/radeon/radeon_object.c
@@ -173,17 +173,6 @@ void radeon_ttm_placement_from_domain(st
else
rbo->placements[i].lpfn = 0;
}
-
- /*
- * Use two-ended allocation depending on the buffer size to
- * improve fragmentation quality.
- * 512kb was measured as the most optimal number.
- */
- if (rbo->tbo.mem.size > 512 * 1024) {
- for (i = 0; i < c; i++) {
- rbo->placements[i].flags |= TTM_PL_FLAG_TOPDOWN;
- }
- }
}
int radeon_bo_create(struct radeon_device *rdev,
Patches currently in stable-queue which might be from alexander.deucher@xxxxxxx are
queue-3.19/drm-radeon-fix-wait-to-actually-occur-after-the-signaling-callback.patch
queue-3.19/drm-radeon-do-a-posting-read-in-cik_set_irq.patch
queue-3.19/drm-radeon-drop-ttm-two-ended-allocation.patch
queue-3.19/drm-radeon-fix-drm_ioctl_radeon_cs-oops.patch
queue-3.19/drm-radeon-fix-interlaced-modes-on-dce8.patch
queue-3.19/drm-radeon-do-a-posting-read-in-r100_set_irq.patch
queue-3.19/drm-radeon-do-a-posting-read-in-evergreen_set_irq.patch
queue-3.19/drm-radeon-do-a-posting-read-in-r600_set_irq.patch
queue-3.19/drm-radeon-do-a-posting-read-in-rs600_set_irq.patch
queue-3.19/drm-radeon-drop-setting-upll-to-sleep-mode.patch
queue-3.19/drm-radeon-do-a-posting-read-in-si_set_irq.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