mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2025-12-23 15:30:14 +01:00
i965: Use brw_bo_map[_gtt]() in intel_miptree_map_raw().
This moves the intel_batchbuffer_flush before the drm_intel_bo_busy call, which is a change in behavior. However, the old behavior was broken. In the future, we may want to only flush in the batchbuffer references the BO being mapped. That's certainly more typical. Signed-off-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Carl Worth <cworth@cworth.org> Reviewed-by: Eric Anholt <eric@anholt.net>
This commit is contained in:
parent
e396674d5f
commit
adaa5a6ca6
1 changed files with 2 additions and 8 deletions
|
|
@ -1706,18 +1706,12 @@ intel_miptree_map_raw(struct brw_context *brw, struct intel_mipmap_tree *mt)
|
|||
|
||||
drm_intel_bo *bo = mt->region->bo;
|
||||
|
||||
if (unlikely(INTEL_DEBUG & DEBUG_PERF)) {
|
||||
if (drm_intel_bo_busy(bo)) {
|
||||
perf_debug("Mapping a busy miptree, causing a stall on the GPU.\n");
|
||||
}
|
||||
}
|
||||
|
||||
intel_batchbuffer_flush(brw);
|
||||
|
||||
if (mt->region->tiling != I915_TILING_NONE)
|
||||
drm_intel_gem_bo_map_gtt(bo);
|
||||
brw_bo_map_gtt(brw, bo, "miptree");
|
||||
else
|
||||
drm_intel_bo_map(bo, true);
|
||||
brw_bo_map(brw, bo, true, "miptree");
|
||||
|
||||
return bo->virtual;
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue