AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

i965: Merge VS/GS and WM pull constant buffer upload paths.

Browse Source 
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
Eric Anholt
2014-05-21 14:22:47 -07:00
committed by Kenneth Graunke
parent 34f4e614dd
commit e24ef5ab18
4 changed files with 42 additions and 53 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/mesa/drivers/dri/i965/brw_gs_surface_state.c
+3 -3
View File
@@ -47,11 +47,11 @@ brw_upload_gs_pull_constants(struct brw_context *brw)
return;
/* CACHE_NEW_GS_PROG */
const struct brw_vec4_prog_data *prog_data = &brw->gs.prog_data->base;
const struct brw_stage_prog_data *prog_data = &brw->gs.prog_data->base.base;
/* _NEW_PROGRAM_CONSTANTS */
brw_upload_vec4_pull_constants(brw, BRW_NEW_GS_CONSTBUF, &gp->program.Base,
stage_state, prog_data);
brw_upload_pull_constants(brw, BRW_NEW_GS_CONSTBUF, &gp->program.Base,
stage_state, prog_data, false);
}
const struct brw_tracked_state brw_gs_pull_constants = {
src/mesa/drivers/dri/i965/brw_state.h
+6 -5
View File
@@ -264,11 +264,12 @@ calculate_attr_overrides(const struct brw_context *brw,
/* brw_vs_surface_state.c */
void
brw_upload_vec4_pull_constants(struct brw_context *brw,
GLbitfield brw_new_constbuf,
const struct gl_program *prog,
struct brw_stage_state *stage_state,
const struct brw_vec4_prog_data *prog_data);
brw_upload_pull_constants(struct brw_context *brw,
GLbitfield brw_new_constbuf,
const struct gl_program *prog,
struct brw_stage_state *stage_state,
const struct brw_stage_prog_data *prog_data,
bool dword_pitch);
/* gen7_vs_state.c */
void
src/mesa/drivers/dri/i965/brw_vs_surface_state.c
+29 -16
View File
@@ -36,22 +36,35 @@
#include "brw_state.h"
#include "intel_buffer_objects.h"
/**
* Creates a temporary BO containing the pull constant data for the shader
* stage, and the SURFACE_STATE struct that points at it.
*
* Pull constants are GLSL uniforms (and other constant data) beyond what we
* could fit as push constants, or that have variable-index array access
* (which is easiest to support using pull constants, and avoids filling
* register space with mostly-unused data).
*
* Compare this path to brw_curbe.c for gen4/5 push constants, and
* gen6_vs_state.c for gen6+ push constants.
*/
void
brw_upload_vec4_pull_constants(struct brw_context *brw,
GLbitfield brw_new_constbuf,
const struct gl_program *prog,
struct brw_stage_state *stage_state,
const struct brw_vec4_prog_data *prog_data)
brw_upload_pull_constants(struct brw_context *brw,
GLbitfield brw_new_constbuf,
const struct gl_program *prog,
struct brw_stage_state *stage_state,
const struct brw_stage_prog_data *prog_data,
bool dword_pitch)
{
int i;
uint32_t surf_index = prog_data->base.binding_table.pull_constants_start;
uint32_t surf_index = prog_data->binding_table.pull_constants_start;
/* Updates the ParamaterValues[i] pointers for all parameters of the
* basic type of PROGRAM_STATE_VAR.
*/
_mesa_load_state_parameters(&brw->ctx, prog->Parameters);
if (!prog_data->base.nr_pull_params) {
if (!prog_data->nr_pull_params) {
if (stage_state->surf_offset[surf_index]) {
stage_state->surf_offset[surf_index] = 0;
brw->state.dirty.brw |= brw_new_constbuf;
@@ -59,19 +72,19 @@ brw_upload_vec4_pull_constants(struct brw_context *brw,
return;
}
/* _NEW_PROGRAM_CONSTANTS */
uint32_t size = prog_data->base.nr_pull_params * 4;
/* CACHE_NEW_*_PROG | _NEW_PROGRAM_CONSTANTS */
uint32_t size = prog_data->nr_pull_params * 4;
drm_intel_bo *const_bo = NULL;
uint32_t const_offset;
float *constants = intel_upload_space(brw, size, 64,
&const_bo, &const_offset);
for (i = 0; i < prog_data->base.nr_pull_params; i++) {
constants[i] = *prog_data->base.pull_param[i];
for (i = 0; i < prog_data->nr_pull_params; i++) {
constants[i] = *prog_data->pull_param[i];
}
if (0) {
for (i = 0; i < ALIGN(prog_data->base.nr_pull_params, 4) / 4; i++) {
for (i = 0; i < ALIGN(prog_data->nr_pull_params, 4) / 4; i++) {
float *row = &constants[i * 4];
fprintf(stderr, "const surface %3d: %4.3f %4.3f %4.3f %4.3f\n",
i, row[0], row[1], row[2], row[3]);
@@ -80,7 +93,7 @@ brw_upload_vec4_pull_constants(struct brw_context *brw,
brw_create_constant_surface(brw, const_bo, const_offset, size,
&stage_state->surf_offset[surf_index],
false);
dword_pitch);
drm_intel_bo_unreference(const_bo);
brw->state.dirty.brw |= brw_new_constbuf;
@@ -103,11 +116,11 @@ brw_upload_vs_pull_constants(struct brw_context *brw)
(struct brw_vertex_program *) brw->vertex_program;
/* CACHE_NEW_VS_PROG */
const struct brw_vec4_prog_data *prog_data = &brw->vs.prog_data->base;
const struct brw_stage_prog_data *prog_data = &brw->vs.prog_data->base.base;
/* _NEW_PROGRAM_CONSTANTS */
brw_upload_vec4_pull_constants(brw, BRW_NEW_VS_CONSTBUF, &vp->program.Base,
stage_state, prog_data);
brw_upload_pull_constants(brw, BRW_NEW_VS_CONSTBUF, &vp->program.Base,
stage_state, prog_data, false);
}
const struct brw_tracked_state brw_vs_pull_constants = {
src/mesa/drivers/dri/i965/brw_wm_surface_state.c
+4 -29
View File
@@ -480,41 +480,16 @@ static void
brw_upload_wm_pull_constants(struct brw_context *brw)
{
struct gl_context *ctx = &brw->ctx;
struct brw_stage_state *stage_state = &brw->wm.base;
/* BRW_NEW_FRAGMENT_PROGRAM */
struct brw_fragment_program *fp =
(struct brw_fragment_program *) brw->fragment_program;
struct gl_program_parameter_list *params = fp->program.Base.Parameters;
const int size = brw->wm.prog_data->base.nr_pull_params * sizeof(float);
const int surf_index =
brw->wm.prog_data->base.binding_table.pull_constants_start;
unsigned int i;
_mesa_load_state_parameters(ctx, params);
/* CACHE_NEW_WM_PROG */
if (brw->wm.prog_data->base.nr_pull_params == 0) {
if (brw->wm.base.surf_offset[surf_index]) {
brw->wm.base.surf_offset[surf_index] = 0;
brw->state.dirty.brw |= BRW_NEW_SURFACES;
}
return;
}
struct gl_stage_prog_data *prog_data = &brw->wm.prog_data->base;
/* _NEW_PROGRAM_CONSTANTS */
drm_intel_bo *const_bo = NULL;
uint32_t const_offset;
float *constants = intel_upload_space(brw, size, 64,
&const_bo, &const_offset);
for (i = 0; i < brw->wm.prog_data->base.nr_pull_params; i++) {
constants[i] = *brw->wm.prog_data->base.pull_param[i];
}
brw_create_constant_surface(brw, const_bo, const_offset, size,
&brw->wm.base.surf_offset[surf_index],
true);
drm_intel_bo_unreference(const_bo);
brw->state.dirty.brw |= BRW_NEW_SURFACES;
brw_upload_pull_constants(brw, BRW_NEW_SURFACES, &fp->program.Base,
stage_state, prog_data, true);
}
const struct brw_tracked_state brw_wm_pull_constants = {