Attachment 47812 Details for Bug 37168 – proposed patch

[patch] proposed patch

0001-r600g-fixed-bo-memory-leak-issue.patch (text/plain), 1022 bytes, created by Pierre-Eric Pelloux-Prayer on 2011-06-10 07:34:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pierre-Eric Pelloux-Prayer

Created: 2011-06-10 07:34:43 UTC

Size: 1022 bytes

patch

obsolete

>From 8c9cb6c70785cbbbf1e1250ce3b9b54f23afb744 Mon Sep 17 00:00:00 2001
>From: Pierre-Eric Pelloux-Prayer <pelloux@gmail.com>
>Date: Fri, 10 Jun 2011 13:57:21 +0200
>Subject: [PATCH 1/2] r600g: fixed bo memory leak issue
>
>pipe_resource_reference call was miossing, thus creating massive memory under certain conditions.
>Fix : https://bugs.freedesktop.org/show_bug.cgi?id=37168
>---
> src/gallium/drivers/r600/r600_translate.c |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
>diff --git a/src/gallium/drivers/r600/r600_translate.c b/src/gallium/drivers/r600/r600_translate.c
>index 7482d15..307fd57 100644
>--- a/src/gallium/drivers/r600/r600_translate.c
>+++ b/src/gallium/drivers/r600/r600_translate.c
>@@ -48,6 +48,7 @@ void r600_translate_index_buffer(struct r600_pipe_context *r600,
> 				&r600->context, *index_buffer, 0, *start, count, ptr);
> 
> 		pipe_resource_reference(index_buffer, out_buffer);
>+		pipe_resource_reference(&out_buffer, NULL);
> 		*index_size = 2;
> 		*start = out_offset / 2;
> 		break;
>-- 
>1.7.5.1
>

Actions: View | Splinter Review

Attachments on bug 37168: 47812

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.