r600g: Cleaned up index buffer reference handling in the draw module.
This fixes a buffer leak. Signed-off-by: Tilman Sauerbeck <tilman@code-monkey.de>
This commit is contained in:
@@ -280,7 +280,10 @@ int r600_upload_index_buffer(struct r600_context *rctx,
|
||||
goto done;
|
||||
}
|
||||
draw->index_buffer_offset = index_offset;
|
||||
draw->index_buffer = upload_buffer;
|
||||
|
||||
/* Transfer ownership. */
|
||||
pipe_resource_reference(&draw->index_buffer, upload_buffer);
|
||||
pipe_resource_reference(&upload_buffer, NULL);
|
||||
}
|
||||
|
||||
done:
|
||||
|
||||
@@ -168,7 +168,7 @@ void r600_draw_vbo(struct pipe_context *ctx, const struct pipe_draw_info *info)
|
||||
info->count);
|
||||
|
||||
draw.index_size = rctx->index_buffer.index_size;
|
||||
draw.index_buffer = rctx->index_buffer.buffer;
|
||||
pipe_resource_reference(&draw.index_buffer, rctx->index_buffer.buffer);
|
||||
draw.index_buffer_offset = draw.start * draw.index_size;
|
||||
draw.start = 0;
|
||||
r600_upload_index_buffer(rctx, &draw);
|
||||
@@ -185,4 +185,6 @@ void r600_draw_vbo(struct pipe_context *ctx, const struct pipe_draw_info *info)
|
||||
r = r600_draw_common(&draw);
|
||||
if (r)
|
||||
fprintf(stderr,"draw common failed %d\n", r);
|
||||
|
||||
pipe_resource_reference(&draw.index_buffer, NULL);
|
||||
}
|
||||
|
||||
@@ -672,7 +672,7 @@ static void r600_draw_vbo2(struct pipe_context *ctx, const struct pipe_draw_info
|
||||
info->count);
|
||||
|
||||
draw.index_size = rctx->index_buffer.index_size;
|
||||
draw.index_buffer = rctx->index_buffer.buffer;
|
||||
pipe_resource_reference(&draw.index_buffer, rctx->index_buffer.buffer);
|
||||
draw.index_buffer_offset = draw.start * draw.index_size;
|
||||
draw.start = 0;
|
||||
r600_upload_index_buffer2(rctx, &draw);
|
||||
@@ -684,6 +684,8 @@ static void r600_draw_vbo2(struct pipe_context *ctx, const struct pipe_draw_info
|
||||
draw.index_bias = info->start;
|
||||
}
|
||||
r600_draw_common(&draw);
|
||||
|
||||
pipe_resource_reference(&draw.index_buffer, NULL);
|
||||
}
|
||||
|
||||
static void r600_flush2(struct pipe_context *ctx, unsigned flags,
|
||||
@@ -2463,7 +2465,10 @@ int r600_upload_index_buffer2(struct r600_pipe_context *rctx, struct r600_drawl
|
||||
goto done;
|
||||
}
|
||||
draw->index_buffer_offset = index_offset;
|
||||
draw->index_buffer = upload_buffer;
|
||||
|
||||
/* Transfer ownership. */
|
||||
pipe_resource_reference(&draw->index_buffer, upload_buffer);
|
||||
pipe_resource_reference(&upload_buffer, NULL);
|
||||
}
|
||||
|
||||
done:
|
||||
|
||||
Reference in New Issue
Block a user