broadcom/vc5: Don't skip job submit just because everything is scissored.

The coordinate shaders may now have side effects in the form of transform
feedback.

Part of fixing
GTF-GLES3.gtf.GL3Tests.transform_feedback.transform_feedback_misc
This commit is contained in:
Eric Anholt 2018-03-20 12:52:19 -07:00
parent 024e814dee
commit 9f0c9c6d18
2 changed files with 7 additions and 10 deletions

View File

@ -377,14 +377,6 @@ vc5_job_submit(struct vc5_context *vc5, struct vc5_job *job)
if (!job->needs_flush)
goto done;
/* The RCL setup would choke if the draw bounds cause no drawing, so
* just drop the drawing if that's the case.
*/
if (job->draw_max_x <= job->draw_min_x ||
job->draw_max_y <= job->draw_min_y) {
goto done;
}
if (vc5->screen->devinfo.ver >= 41)
v3d41_emit_rcl(job);
else

View File

@ -696,8 +696,13 @@ v3dX(emit_rcl)(struct vc5_job *job)
uint32_t supertile_h_in_pixels = job->tile_height * supertile_h;
uint32_t min_x_supertile = job->draw_min_x / supertile_w_in_pixels;
uint32_t min_y_supertile = job->draw_min_y / supertile_h_in_pixels;
uint32_t max_x_supertile = (job->draw_max_x - 1) / supertile_w_in_pixels;
uint32_t max_y_supertile = (job->draw_max_y - 1) / supertile_h_in_pixels;
uint32_t max_x_supertile = 0;
uint32_t max_y_supertile = 0;
if (job->draw_max_x != 0 && job->draw_max_y != 0) {
max_x_supertile = (job->draw_max_x - 1) / supertile_w_in_pixels;
max_y_supertile = (job->draw_max_y - 1) / supertile_h_in_pixels;
}
for (int y = min_y_supertile; y <= max_y_supertile; y++) {
for (int x = min_x_supertile; x <= max_x_supertile; x++) {