From 8a2352262e27bf7d3c31a20b72eb04772d56ff42 Mon Sep 17 00:00:00 2001 From: Chia-I Wu Date: Fri, 19 Sep 2014 15:24:23 +0800 Subject: [PATCH] ilo: rename ilo_cp_flush() "Flush" is used for too many things already: pipe resource flush, pipe context flush, pipe transfer region flush, and hardware pipeline flush. Rename it to ilo_cp_submit(). As such, ILO_DEBUG=flush is renamed to ILO_DEBUG=submit. --- src/gallium/drivers/ilo/ilo_3d.c | 8 ++++---- src/gallium/drivers/ilo/ilo_3d.h | 2 +- src/gallium/drivers/ilo/ilo_3d_pipeline.c | 6 +++--- src/gallium/drivers/ilo/ilo_blitter_blt.c | 4 ++-- src/gallium/drivers/ilo/ilo_common.h | 2 +- src/gallium/drivers/ilo/ilo_context.c | 10 +++++----- src/gallium/drivers/ilo/ilo_cp.c | 14 +++++++------- src/gallium/drivers/ilo/ilo_cp.h | 28 ++++++++++++++-------------- src/gallium/drivers/ilo/ilo_query.c | 2 +- src/gallium/drivers/ilo/ilo_screen.c | 2 +- src/gallium/drivers/ilo/ilo_transfer.c | 26 +++++++++++++------------- 11 files changed, 52 insertions(+), 52 deletions(-) diff --git a/src/gallium/drivers/ilo/ilo_3d.c b/src/gallium/drivers/ilo/ilo_3d.c index fbfaef1..2fe98a5 100644 --- a/src/gallium/drivers/ilo/ilo_3d.c +++ b/src/gallium/drivers/ilo/ilo_3d.c @@ -214,7 +214,7 @@ ilo_3d_reserve_for_query(struct ilo_3d *hw3d, struct ilo_query *q, /* XXX we should check the aperture size */ if (ilo_cp_space(hw3d->cp) < q->reg_cmd_size * 2) { - ilo_cp_flush(hw3d->cp, "out of space"); + ilo_cp_submit(hw3d->cp, "out of space"); assert(ilo_cp_space(hw3d->cp) >= q->reg_cmd_size * 2); } @@ -383,7 +383,7 @@ ilo_3d_process_query(struct ilo_context *ilo, struct ilo_query *q) * Hook for CP new-batch. */ void -ilo_3d_cp_flushed(struct ilo_3d *hw3d) +ilo_3d_cp_submitted(struct ilo_3d *hw3d) { if (ilo_debug & ILO_DEBUG_3D) ilo_builder_decode(&hw3d->cp->builder); @@ -489,7 +489,7 @@ draw_vbo(struct ilo_3d *hw3d, const struct ilo_context *ilo, } if (max_len > ilo_cp_space(hw3d->cp)) { - ilo_cp_flush(hw3d->cp, "out of space"); + ilo_cp_submit(hw3d->cp, "out of space"); need_flush = false; assert(max_len <= ilo_cp_space(hw3d->cp)); } @@ -810,7 +810,7 @@ ilo_texture_barrier(struct pipe_context *pipe) /* don't know why */ if (ilo_dev_gen(ilo->dev) >= ILO_GEN(7)) - ilo_cp_flush(hw3d->cp, "texture barrier"); + ilo_cp_submit(hw3d->cp, "texture barrier"); } static void diff --git a/src/gallium/drivers/ilo/ilo_3d.h b/src/gallium/drivers/ilo/ilo_3d.h index c0e2326..ccc57cd 100644 --- a/src/gallium/drivers/ilo/ilo_3d.h +++ b/src/gallium/drivers/ilo/ilo_3d.h @@ -66,7 +66,7 @@ void ilo_3d_destroy(struct ilo_3d *hw3d); void -ilo_3d_cp_flushed(struct ilo_3d *hw3d); +ilo_3d_cp_submitted(struct ilo_3d *hw3d); void ilo_3d_own_render_ring(struct ilo_3d *hw3d); diff --git a/src/gallium/drivers/ilo/ilo_3d_pipeline.c b/src/gallium/drivers/ilo/ilo_3d_pipeline.c index baadfa7..e56b129 100644 --- a/src/gallium/drivers/ilo/ilo_3d_pipeline.c +++ b/src/gallium/drivers/ilo/ilo_3d_pipeline.c @@ -189,7 +189,7 @@ ilo_3d_pipeline_emit_draw(struct ilo_3d_pipeline *p, /* flush and try again */ if (ilo_builder_batch_used(&p->cp->builder)) { - ilo_cp_flush(p->cp, "out of aperture"); + ilo_cp_submit(p->cp, "out of aperture"); continue; } @@ -273,7 +273,7 @@ ilo_3d_pipeline_emit_rectlist(struct ilo_3d_pipeline *p, ILO_3D_PIPELINE_RECTLIST, blitter); if (max_len > ilo_cp_space(p->cp)) - ilo_cp_flush(p->cp, "out of space"); + ilo_cp_submit(p->cp, "out of space"); while (true) { struct ilo_builder_snapshot snapshot; @@ -291,7 +291,7 @@ ilo_3d_pipeline_emit_rectlist(struct ilo_3d_pipeline *p, /* flush and try again */ if (ilo_builder_batch_used(&p->cp->builder)) { - ilo_cp_flush(p->cp, "out of aperture"); + ilo_cp_submit(p->cp, "out of aperture"); continue; } } diff --git a/src/gallium/drivers/ilo/ilo_blitter_blt.c b/src/gallium/drivers/ilo/ilo_blitter_blt.c index b55327b..aead0b2 100644 --- a/src/gallium/drivers/ilo/ilo_blitter_blt.c +++ b/src/gallium/drivers/ilo/ilo_blitter_blt.c @@ -60,7 +60,7 @@ ilo_blitter_blt_begin(struct ilo_blitter *blitter, int max_cmd_size, } if (!ilo_builder_validate(&ilo->cp->builder, count, aper_check)) - ilo_cp_flush(ilo->cp, "out of aperture"); + ilo_cp_submit(ilo->cp, "out of aperture"); /* set BCS_SWCTRL */ swctrl = 0x0; @@ -84,7 +84,7 @@ ilo_blitter_blt_begin(struct ilo_blitter *blitter, int max_cmd_size, max_cmd_size += (4 + 3) * 2; if (ilo_cp_space(ilo->cp) < max_cmd_size) { - ilo_cp_flush(ilo->cp, "out of space"); + ilo_cp_submit(ilo->cp, "out of space"); assert(ilo_cp_space(ilo->cp) >= max_cmd_size); } diff --git a/src/gallium/drivers/ilo/ilo_common.h b/src/gallium/drivers/ilo/ilo_common.h index a2e74bf..03f4224 100644 --- a/src/gallium/drivers/ilo/ilo_common.h +++ b/src/gallium/drivers/ilo/ilo_common.h @@ -62,7 +62,7 @@ enum ilo_debug { ILO_DEBUG_FS = 1 << 3, ILO_DEBUG_CS = 1 << 4, ILO_DEBUG_DRAW = ILO_DEBUG_HOT << 5, - ILO_DEBUG_FLUSH = 1 << 6, + ILO_DEBUG_SUBMIT = 1 << 6, /* flags that affect the behaviors of the driver */ ILO_DEBUG_NOHW = 1 << 20, diff --git a/src/gallium/drivers/ilo/ilo_context.c b/src/gallium/drivers/ilo/ilo_context.c index 9e6a068..04538a1 100644 --- a/src/gallium/drivers/ilo/ilo_context.c +++ b/src/gallium/drivers/ilo/ilo_context.c @@ -42,11 +42,11 @@ #include "ilo_context.h" static void -ilo_context_cp_flushed(struct ilo_cp *cp, void *data) +ilo_context_cp_submitted(struct ilo_cp *cp, void *data) { struct ilo_context *ilo = ilo_context(data); - ilo_3d_cp_flushed(ilo->hw3d); + ilo_3d_cp_submitted(ilo->hw3d); } static void @@ -56,7 +56,7 @@ ilo_flush(struct pipe_context *pipe, { struct ilo_context *ilo = ilo_context(pipe); - ilo_cp_flush(ilo->cp, + ilo_cp_submit(ilo->cp, (flags & PIPE_FLUSH_END_OF_FRAME) ? "frame end" : "user request"); if (f) { @@ -119,8 +119,8 @@ ilo_context_create(struct pipe_screen *screen, void *priv) return NULL; } - ilo_cp_set_flush_callback(ilo->cp, - ilo_context_cp_flushed, (void *) ilo); + ilo_cp_set_submit_callback(ilo->cp, + ilo_context_cp_submitted, (void *) ilo); ilo->base.screen = screen; ilo->base.priv = priv; diff --git a/src/gallium/drivers/ilo/ilo_cp.c b/src/gallium/drivers/ilo/ilo_cp.c index fd4f49e..f7411a9 100644 --- a/src/gallium/drivers/ilo/ilo_cp.c +++ b/src/gallium/drivers/ilo/ilo_cp.c @@ -50,8 +50,8 @@ ilo_cp_release_owner(struct ilo_cp *cp) * Set the parser owner. If this is a new owner or a new ring, the old owner * is released and the new owner's own() is called. * - * The parser may be implicitly flushed if there is a ring change or there is - * not enough space for the new owner. + * The parser may implicitly submit if there is a ring change or there is not + * enough space for the new owner. */ void ilo_cp_set_owner(struct ilo_cp *cp, enum intel_ring_type ring, @@ -61,7 +61,7 @@ ilo_cp_set_owner(struct ilo_cp *cp, enum intel_ring_type ring, owner = &ilo_cp_default_owner; if (cp->ring != ring) { - ilo_cp_flush(cp, "ring change"); + ilo_cp_submit(cp, "ring change"); cp->ring = ring; } @@ -70,7 +70,7 @@ ilo_cp_set_owner(struct ilo_cp *cp, enum intel_ring_type ring, /* multiply by 2 because there are own() and release() */ if (ilo_cp_space(cp) < owner->reserve * 2) { - ilo_cp_flush(cp, "new owner"); + ilo_cp_submit(cp, "new owner"); assert(ilo_cp_space(cp) >= owner->reserve * 2); } @@ -111,7 +111,7 @@ ilo_cp_end_batch(struct ilo_cp *cp, unsigned *used) * is empty, the callback is not invoked. */ void -ilo_cp_flush_internal(struct ilo_cp *cp) +ilo_cp_submit_internal(struct ilo_cp *cp) { const bool do_exec = !(ilo_debug & ILO_DEBUG_NOHW); struct intel_bo *bo; @@ -138,8 +138,8 @@ ilo_cp_flush_internal(struct ilo_cp *cp) cp->last_submitted_bo = bo; intel_bo_reference(cp->last_submitted_bo); - if (cp->flush_callback) - cp->flush_callback(cp, cp->flush_callback_data); + if (cp->submit_callback) + cp->submit_callback(cp, cp->submit_callback_data); } ilo_builder_begin(&cp->builder); diff --git a/src/gallium/drivers/ilo/ilo_cp.h b/src/gallium/drivers/ilo/ilo_cp.h index c313745..12d1a98 100644 --- a/src/gallium/drivers/ilo/ilo_cp.h +++ b/src/gallium/drivers/ilo/ilo_cp.h @@ -62,8 +62,8 @@ struct ilo_cp { struct ilo_shader_cache *shader_cache; struct intel_context *render_ctx; - ilo_cp_callback flush_callback; - void *flush_callback_data; + ilo_cp_callback submit_callback; + void *submit_callback_data; enum intel_ring_type ring; const struct ilo_cp_owner *owner; @@ -83,18 +83,18 @@ void ilo_cp_destroy(struct ilo_cp *cp); void -ilo_cp_flush_internal(struct ilo_cp *cp); +ilo_cp_submit_internal(struct ilo_cp *cp); static inline void -ilo_cp_flush(struct ilo_cp *cp, const char *reason) +ilo_cp_submit(struct ilo_cp *cp, const char *reason) { - if (ilo_debug & ILO_DEBUG_FLUSH) { - ilo_printf("cp flushed for %s because of %s: ", - (cp->ring == INTEL_RING_RENDER) ? "render" : "other", reason); + if (ilo_debug & ILO_DEBUG_SUBMIT) { + ilo_printf("submit batch buffer to %s ring because of %s: ", + (cp->ring == INTEL_RING_RENDER) ? "render" : "unknown", reason); ilo_builder_batch_print_stats(&cp->builder); } - ilo_cp_flush_internal(cp); + ilo_cp_submit_internal(cp); } void @@ -116,7 +116,7 @@ ilo_cp_space(struct ilo_cp *cp) } /** - * Set one-off flags. They will be cleared after flushing. + * Set one-off flags. They will be cleared after submission. */ static inline void ilo_cp_set_one_off_flags(struct ilo_cp *cp, unsigned flags) @@ -125,15 +125,15 @@ ilo_cp_set_one_off_flags(struct ilo_cp *cp, unsigned flags) } /** - * Set flush callback. The callback is invoked after the bo has been - * successfully executed, and before the bo is reallocated. + * Set submit callback. The callback is invoked after the bo has been + * successfully submitted, and before the bo is reallocated. */ static inline void -ilo_cp_set_flush_callback(struct ilo_cp *cp, ilo_cp_callback callback, +ilo_cp_set_submit_callback(struct ilo_cp *cp, ilo_cp_callback callback, void *data) { - cp->flush_callback = callback; - cp->flush_callback_data = data; + cp->submit_callback = callback; + cp->submit_callback_data = data; } #endif /* ILO_CP_H */ diff --git a/src/gallium/drivers/ilo/ilo_query.c b/src/gallium/drivers/ilo/ilo_query.c index c789397..812e056 100644 --- a/src/gallium/drivers/ilo/ilo_query.c +++ b/src/gallium/drivers/ilo/ilo_query.c @@ -186,7 +186,7 @@ ilo_get_query_result(struct pipe_context *pipe, struct pipe_query *query, if (q->bo) { if (ilo_builder_has_reloc(&ilo->cp->builder, q->bo)) - ilo_cp_flush(ilo->cp, "syncing for queries"); + ilo_cp_submit(ilo->cp, "syncing for queries"); if (!wait && intel_bo_is_busy(q->bo)) return false; diff --git a/src/gallium/drivers/ilo/ilo_screen.c b/src/gallium/drivers/ilo/ilo_screen.c index 82fda2c..8d17089 100644 --- a/src/gallium/drivers/ilo/ilo_screen.c +++ b/src/gallium/drivers/ilo/ilo_screen.c @@ -48,7 +48,7 @@ static const struct debug_named_value ilo_debug_flags[] = { { "fs", ILO_DEBUG_FS, "Dump fragment shaders" }, { "cs", ILO_DEBUG_CS, "Dump compute shaders" }, { "draw", ILO_DEBUG_DRAW, "Show draw information" }, - { "flush", ILO_DEBUG_FLUSH, "Show batch buffer flushes" }, + { "submit", ILO_DEBUG_SUBMIT, "Show batch buffer submissions" }, { "nohw", ILO_DEBUG_NOHW, "Do not send commands to HW" }, { "nocache", ILO_DEBUG_NOCACHE, "Always invalidate HW caches" }, { "nohiz", ILO_DEBUG_NOHIZ, "Disable HiZ" }, diff --git a/src/gallium/drivers/ilo/ilo_transfer.c b/src/gallium/drivers/ilo/ilo_transfer.c index f1186ff..3062bf5 100644 --- a/src/gallium/drivers/ilo/ilo_transfer.c +++ b/src/gallium/drivers/ilo/ilo_transfer.c @@ -1043,12 +1043,12 @@ copy_staging_resource(struct ilo_context *ilo, } static bool -is_bo_busy(struct ilo_context *ilo, struct intel_bo *bo, bool *need_flush) +is_bo_busy(struct ilo_context *ilo, struct intel_bo *bo, bool *need_submit) { const bool referenced = ilo_builder_has_reloc(&ilo->cp->builder, bo); - if (need_flush) - *need_flush = referenced; + if (need_submit) + *need_submit = referenced; if (referenced) return true; @@ -1064,22 +1064,22 @@ static bool choose_transfer_method(struct ilo_context *ilo, struct ilo_transfer *xfer) { struct pipe_resource *res = xfer->base.resource; - bool need_flush; + bool need_submit; if (!resource_get_transfer_method(res, &xfer->base, &xfer->method)) return false; /* see if we can avoid blocking */ - if (is_bo_busy(ilo, ilo_resource_get_bo(res), &need_flush)) { + if (is_bo_busy(ilo, ilo_resource_get_bo(res), &need_submit)) { bool resource_renamed; if (!xfer_unblock(xfer, &resource_renamed)) { if (xfer->base.usage & PIPE_TRANSFER_DONTBLOCK) return false; - /* flush to make bo really busy so that map() correctly blocks */ - if (need_flush) - ilo_cp_flush(ilo->cp, "syncing for transfers"); + /* submit to make bo really busy and map() correctly blocks */ + if (need_submit) + ilo_cp_submit(ilo->cp, "syncing for transfers"); } if (resource_renamed) @@ -1093,10 +1093,10 @@ static void buf_pwrite(struct ilo_context *ilo, struct ilo_buffer *buf, unsigned usage, int offset, int size, const void *data) { - bool need_flush; + bool need_submit; /* see if we can avoid blocking */ - if (is_bo_busy(ilo, buf->bo, &need_flush)) { + if (is_bo_busy(ilo, buf->bo, &need_submit)) { bool unblocked = false; if ((usage & PIPE_TRANSFER_DISCARD_WHOLE_RESOURCE) && @@ -1132,9 +1132,9 @@ buf_pwrite(struct ilo_context *ilo, struct ilo_buffer *buf, } } - /* flush to make bo really busy so that pwrite() correctly blocks */ - if (!unblocked && need_flush) - ilo_cp_flush(ilo->cp, "syncing for pwrites"); + /* submit to make bo really busy and pwrite() correctly blocks */ + if (!unblocked && need_submit) + ilo_cp_submit(ilo->cp, "syncing for pwrites"); } intel_bo_pwrite(buf->bo, offset, size, data); -- 2.7.4