Commit 1bf1d86f authored by Matt Roper's avatar Matt Roper Committed by Rodrigo Vivi

drm/xe: Don't emit extra MI_BATCH_BUFFER_END in WA batchbuffer

The MI_BATCH_BUFFER_END is already added automatically by
__xe_bb_create_job(); including it in the construction of the workaround
batchbuffer results in an unnecessary duplicate.

Link: https://lore.kernel.org/r/20230329173334.4015124-4-matthew.d.roper@intel.comSigned-off-by: default avatarMatt Roper <matthew.d.roper@intel.com>
Reviewed-by: default avatarJosé Roberto de Souza <jose.souza@intel.com>
Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent 9b36f7af
......@@ -203,8 +203,6 @@ static int emit_wa_job(struct xe_gt *gt, struct xe_engine *e)
bb->cs[bb->len++] = entry->set_bits;
}
}
bb->cs[bb->len++] = MI_NOOP;
bb->cs[bb->len++] = MI_BATCH_BUFFER_END;
batch_ofs = xe_bo_ggtt_addr(gt->kernel_bb_pool.bo);
job = xe_bb_create_wa_job(e, bb, batch_ofs);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment