diff options
author | Eric Anholt <[email protected]> | 2011-10-22 09:37:57 -0700 |
---|---|---|
committer | Eric Anholt <[email protected]> | 2011-10-29 12:16:05 -0700 |
commit | 3d851ae48865ac6a20a615e6e06d913c2265cda5 (patch) | |
tree | 61983729c6319a982cc0842b70f0c9d73cab0999 /src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | |
parent | da8f052560120010b7f37a300f03c0847cca3aa5 (diff) |
i965: Use the batch save/reset code to avoid needing the BO validate step.
We were doing the BO validate step in prepare() (brw_validate_state())
hooks of atoms so that we could check_aperture before emitting the
relocation trees during brw_upload_state() that would actually make
the batchbuffer reference too much memory to be executed. Now that
all relocations occur in the batchbuffer, we can instead
check_aperture after emitting our state into the batchbuffer, and
easily roll back, flush, and retry if we happened to go over the
limits.
This will let us remove the whole prepare() vs emit() split in our
state atoms, which is a source of tricky dependencies and duplicated
code.
Reviewed-by: Kenneth Graunke <[email protected]>
Acked-by: Paul Berry <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965/gen7_wm_surface_state.c')
0 files changed, 0 insertions, 0 deletions