diff options
author | Kenneth Graunke <[email protected]> | 2018-06-04 03:09:59 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2018-06-04 09:43:09 -0700 |
commit | b3ba47c5926b3a5850403b1ab0df37815a203d9d (patch) | |
tree | 2add50bf8f7650ab1cea3b9df0834e00f1eae45b /scripts | |
parent | 06d3c65098097675a34035da3043a71061fad17b (diff) |
i965: Fix batch-last mode to properly swap BOs.
On pre-4.13 kernels, which don't support I915_EXEC_BATCH_FIRST, we move
the validation list entry to the end...but incorrectly left the exec_bo
array alone, causing a mismatch where exec_bos[0] no longer corresponded
with validation_list[0] (and similarly for the last entry).
One example of resulting breakage is that we'd update bo->gtt_offset
based on the wrong buffer. This wreaked total havoc when trying to use
softpin, and likely caused unnecessary relocations in the normal case.
Fixes: 29ba502a4e28471f67e4e904ae503157087efd20 (i965: Use I915_EXEC_BATCH_FIRST when available.)
Reviewed-by: Chris Wilson <[email protected]>
Reviewed-by: Lionel Landwerlin <[email protected]>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions