diff options
author | Jordan Justen <[email protected]> | 2016-11-07 14:08:22 -0800 |
---|---|---|
committer | Jordan Justen <[email protected]> | 2016-12-07 09:00:49 -0800 |
commit | edf3113aeddcf66cb24906e53a2d4f41616f8985 (patch) | |
tree | 2b95a5e10b6c333bcb6939abcbb7b03efecce5eb /src | |
parent | 12e0a6e25967e097f9d18e9ee25b30248f617b28 (diff) |
intel/blorp_blit: Adjust blorp surface parameters for split blits
If try_blorp_blit() previously returned that a blit was too large,
shrink_surface_params() will be used to update the surface parameters
for the smaller blit so the blit operation can proceed.
v2:
* Use double instead of float. (Jason)
Signed-off-by: Jordan Justen <[email protected]>
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src')
-rw-r--r-- | src/intel/blorp/blorp_blit.c | 97 |
1 files changed, 94 insertions, 3 deletions
diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c index 05977f06b6c..b976eefef8e 100644 --- a/src/intel/blorp/blorp_blit.c +++ b/src/intel/blorp/blorp_blit.c @@ -1484,6 +1484,12 @@ surf_retile_w_to_y(const struct isl_device *isl_dev, info->tile_y_sa /= 2; } +static bool +can_shrink_surfaces(const struct blorp_params *params) +{ + return false; +} + struct blt_axis { double src0, src1, dst0, dst1; bool mirror; @@ -1738,12 +1744,88 @@ adjust_split_source_coords(const struct blt_axis *orig, split_coords->src1 = orig->src1 + (scale >= 0.0 ? delta1 : delta0); } +static const struct isl_extent2d +get_px_size_sa(const struct isl_surf *surf) +{ + static const struct isl_extent2d one_to_one = { .w = 1, .h = 1 }; + + if (surf->msaa_layout != ISL_MSAA_LAYOUT_INTERLEAVED) + return one_to_one; + else + return isl_get_interleaved_msaa_px_size_sa(surf->samples); +} + +static void +shrink_surface_params(const struct isl_device *dev, + struct brw_blorp_surface_info *info, + double *x0, double *x1, double *y0, double *y1) +{ + uint32_t byte_offset, x_offset_sa, y_offset_sa, size; + struct isl_extent2d px_size_sa; + int adjust; + + surf_convert_to_single_slice(dev, info); + + px_size_sa = get_px_size_sa(&info->surf); + + /* Because this gets called after we lower compressed images, the tile + * offsets may be non-zero and we need to incorporate them in our + * calculations. + */ + x_offset_sa = (uint32_t)*x0 * px_size_sa.w + info->tile_x_sa; + y_offset_sa = (uint32_t)*y0 * px_size_sa.h + info->tile_y_sa; + isl_tiling_get_intratile_offset_sa(dev, info->surf.tiling, + info->surf.format, info->surf.row_pitch, + x_offset_sa, y_offset_sa, + &byte_offset, + &info->tile_x_sa, &info->tile_y_sa); + + info->addr.offset += byte_offset; + + adjust = (int)info->tile_x_sa / px_size_sa.w - (int)*x0; + *x0 += adjust; + *x1 += adjust; + info->tile_x_sa = 0; + + adjust = (int)info->tile_y_sa / px_size_sa.h - (int)*y0; + *y0 += adjust; + *y1 += adjust; + info->tile_y_sa = 0; + + size = MIN2((uint32_t)ceil(*x1), info->surf.logical_level0_px.width); + info->surf.logical_level0_px.width = size; + info->surf.phys_level0_sa.width = size * px_size_sa.w; + + size = MIN2((uint32_t)ceil(*y1), info->surf.logical_level0_px.height); + info->surf.logical_level0_px.height = size; + info->surf.phys_level0_sa.height = size * px_size_sa.h; +} + +static void +shrink_surfaces(const struct isl_device *dev, + struct blorp_params *params, + struct brw_blorp_blit_prog_key *wm_prog_key, + struct blt_coords *coords) +{ + /* Shrink source surface */ + shrink_surface_params(dev, ¶ms->src, &coords->x.src0, &coords->x.src1, + &coords->y.src0, &coords->y.src1); + wm_prog_key->need_src_offset = false; + + /* Shrink destination surface */ + shrink_surface_params(dev, ¶ms->dst, &coords->x.dst0, &coords->x.dst1, + &coords->y.dst0, &coords->y.dst1); + wm_prog_key->need_dst_offset = false; +} + static void do_blorp_blit(struct blorp_batch *batch, - struct blorp_params *params, + const struct blorp_params *orig_params, struct brw_blorp_blit_prog_key *wm_prog_key, const struct blt_coords *orig) { + struct blorp_params params; + struct blt_coords blit_coords; struct blt_coords split_coords = *orig; double w = orig->x.dst1 - orig->x.dst0; double h = orig->y.dst1 - orig->y.dst0; @@ -1755,9 +1837,15 @@ do_blorp_blit(struct blorp_batch *batch, y_scale = -y_scale; bool x_done, y_done; + bool shrink = false; do { + params = *orig_params; + blit_coords = split_coords; + if (shrink) + shrink_surfaces(batch->blorp->isl_dev, ¶ms, wm_prog_key, + &blit_coords); enum blit_shrink_status result = - try_blorp_blit(batch, params, wm_prog_key, &split_coords); + try_blorp_blit(batch, ¶ms, wm_prog_key, &blit_coords); if (result & BLIT_WIDTH_SHRINK) { w /= 2.0; @@ -1772,8 +1860,11 @@ do_blorp_blit(struct blorp_batch *batch, adjust_split_source_coords(&orig->y, &split_coords.y, y_scale); } - if (result != 0) + if (result != 0) { + assert(can_shrink_surfaces(orig_params)); + shrink = true; continue; + } y_done = (orig->y.dst1 - split_coords.y.dst1 < 0.5); x_done = y_done && (orig->x.dst1 - split_coords.x.dst1 < 0.5); |