Attachment 118056 Details for Bug 91790 – possible fix

[patch] possible fix

0001-drm-amdgpu-fix-buffer-placement-under-memory-pressur.patch (text/plain), 1.33 KB, created by Alex Deucher on 2015-09-02 20:09:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2015-09-02 20:09:42 UTC

Size: 1.33 KB

patch

obsolete

>From cff1986ebd057a2a5953e67151870bc45af63b1a Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Christian=20K=C3=B6nig?= <christian.koenig@amd.com>
>Date: Wed, 2 Sep 2015 20:25:48 +0200
>Subject: [PATCH] drm/amdgpu: fix buffer placement under memory pressure
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Some buffers (UVD/VM page tables) must be placed in VRAM,
>but the byte restriction for moving buffers didn't took this
>into account.
>
>Port of radeon commit 4b09556660bfe1b43d72ca858524c6baf2c6cb1d.
>
>Signed-off-by: Christian KÃ¶nig <christian.koenig@amd.com>
>---
> drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>index d3aa498..6cc2264 100644
>--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>@@ -354,7 +354,7 @@ int amdgpu_cs_list_validate(struct amdgpu_cs_parser *p)
> 			 * into account. We don't want to disallow buffer moves
> 			 * completely.
> 			 */
>-			if (current_domain != AMDGPU_GEM_DOMAIN_CPU &&
>+			if ((lobj->allowed_domains & current_domain) != 0 &&
> 			    (domain & current_domain) == 0 && /* will be moved */
> 			    bytes_moved > bytes_moved_threshold) {
> 				/* don't move it */
>-- 
>1.9.1
>

Actions: View | Splinter Review

Attachments on bug 91790: 117962 | 117963 | 117967 | 118056 | 118060

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.