Attachment 41073 Details for Bug 32188 – possible fix

[patch] possible fix

0001-vbo-only-flush-for-vbos-not-cbufs.patch (text/plain), 1011 bytes, created by Alex Deucher on 2010-12-13 10:52:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2010-12-13 10:52:11 UTC

Size: 1011 bytes

patch

obsolete

>From eb6c4caffe0d546e102a43e815e6487e48874f06 Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexdeucher@gmail.com>
>Date: Mon, 13 Dec 2010 13:49:23 -0500
>Subject: [PATCH] vbo: only flush for vbos not cbufs
>
>---
> src/radeon_exa_shared.c |    9 ++++++---
> 1 files changed, 6 insertions(+), 3 deletions(-)
>
>diff --git a/src/radeon_exa_shared.c b/src/radeon_exa_shared.c
>index 2ef0751..da4a3fb 100644
>--- a/src/radeon_exa_shared.c
>+++ b/src/radeon_exa_shared.c
>@@ -180,9 +180,12 @@ void radeon_vb_no_space(ScrnInfoPtr pScrn,
> #if defined(XF86DRM_MODE)
>     if (info->cs) {
> 	if (vbo->vb_bo) {
>-	    if (vbo->vb_start_op != vbo->vb_offset) {
>-		accel_state->finish_op(pScrn, vert_size);
>-		accel_state->ib_reset_op = info->cs->cdw;
>+	    /* only flush for vbo, not cbuf */
>+	    if (vbo == &accel_state->vbo) {
>+		if (vbo->vb_start_op != vbo->vb_offset) {
>+		    accel_state->finish_op(pScrn, vert_size);
>+		    accel_state->ib_reset_op = info->cs->cdw;
>+		}
> 	    }
> 
> 	    /* release the current VBO */
>-- 
>1.7.1.1
>

Actions: View | Splinter Review

Attachments on bug 32188: 40875 | 40876 | 40877 | 41073 | 41088 | 41603 | 41634 | 41698 | 42268 | 42285 | 42350

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.