[Mesa-dev] [PATCH 16/16] gallium: remove unused PIPE_DUMP_* defines
Marek Olšák
maraeo at gmail.com
Sun Aug 20 16:11:51 UTC 2017
For patches 12-16:
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Marek
On Wed, Aug 16, 2017 at 1:05 PM, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
> From: Nicolai Hähnle <nicolai.haehnle at amd.com>
>
> ---
> src/gallium/drivers/ddebug/dd_draw.c | 21 ++++-----------------
> src/gallium/drivers/radeonsi/si_debug.c | 8 +++-----
> src/gallium/include/pipe/p_defines.h | 3 ---
> 3 files changed, 7 insertions(+), 25 deletions(-)
>
> diff --git a/src/gallium/drivers/ddebug/dd_draw.c b/src/gallium/drivers/ddebug/dd_draw.c
> index aa38c561c57..4bf13bd7e46 100644
> --- a/src/gallium/drivers/ddebug/dd_draw.c
> +++ b/src/gallium/drivers/ddebug/dd_draw.c
> @@ -620,10 +620,7 @@ dd_flush_and_handle_hang(struct dd_context *dctx,
> if (f) {
> fprintf(f, "dd: %s.\n", cause);
> dd_dump_driver_state(dctx, f,
> - PIPE_DUMP_DEVICE_STATUS_REGISTERS |
> - PIPE_DUMP_CURRENT_STATES |
> - PIPE_DUMP_CURRENT_SHADERS |
> - PIPE_DUMP_LAST_COMMAND_BUFFER);
> + PIPE_DUMP_DEVICE_STATUS_REGISTERS);
> dd_dump_dmesg(f);
> dd_close_file_stream(f);
> }
> @@ -1118,10 +1115,7 @@ dd_after_draw(struct dd_context *dctx, struct dd_call *call)
> if (!dscreen->no_flush &&
> dd_flush_and_check_hang(dctx, NULL, 0)) {
> dd_write_report(dctx, call,
> - PIPE_DUMP_DEVICE_STATUS_REGISTERS |
> - PIPE_DUMP_CURRENT_STATES |
> - PIPE_DUMP_CURRENT_SHADERS |
> - PIPE_DUMP_LAST_COMMAND_BUFFER,
> + PIPE_DUMP_DEVICE_STATUS_REGISTERS,
> true);
>
> /* Terminate the process to prevent future hangs. */
> @@ -1136,19 +1130,12 @@ dd_after_draw(struct dd_context *dctx, struct dd_call *call)
> case DD_DUMP_ALL_CALLS:
> if (!dscreen->no_flush)
> pipe->flush(pipe, NULL, 0);
> - dd_write_report(dctx, call,
> - PIPE_DUMP_CURRENT_STATES |
> - PIPE_DUMP_CURRENT_SHADERS |
> - PIPE_DUMP_LAST_COMMAND_BUFFER,
> - false);
> + dd_write_report(dctx, call, 0, false);
> break;
> case DD_DUMP_APITRACE_CALL:
> if (dscreen->apitrace_dump_call ==
> dctx->draw_state.apitrace_call_number) {
> - dd_write_report(dctx, call,
> - PIPE_DUMP_CURRENT_STATES |
> - PIPE_DUMP_CURRENT_SHADERS,
> - false);
> + dd_write_report(dctx, call, 0, false);
> /* No need to continue. */
> exit(0);
> } else {
> diff --git a/src/gallium/drivers/radeonsi/si_debug.c b/src/gallium/drivers/radeonsi/si_debug.c
> index 8dd1e1329c3..7ad28a23dfc 100644
> --- a/src/gallium/drivers/radeonsi/si_debug.c
> +++ b/src/gallium/drivers/radeonsi/si_debug.c
> @@ -1058,11 +1058,9 @@ static void si_dump_debug_state(struct pipe_context *ctx, FILE *f,
> if (flags & PIPE_DUMP_DEVICE_STATUS_REGISTERS) {
> si_dump_debug_registers(sctx, f);
>
> - if (flags & PIPE_DUMP_CURRENT_SHADERS) {
> - si_dump_annotated_shaders(sctx, f);
> - si_dump_command("Active waves (raw data)", "umr -wa | column -t", f);
> - si_dump_command("Wave information", "umr -O bits -wa", f);
> - }
> + si_dump_annotated_shaders(sctx, f);
> + si_dump_command("Active waves (raw data)", "umr -wa | column -t", f);
> + si_dump_command("Wave information", "umr -O bits -wa", f);
> }
> }
>
> diff --git a/src/gallium/include/pipe/p_defines.h b/src/gallium/include/pipe/p_defines.h
> index 8609aefb986..da7d5da7347 100644
> --- a/src/gallium/include/pipe/p_defines.h
> +++ b/src/gallium/include/pipe/p_defines.h
> @@ -358,9 +358,6 @@ enum pipe_flush_flags
> * Flags for pipe_context::dump_debug_state.
> */
> #define PIPE_DUMP_DEVICE_STATUS_REGISTERS (1 << 0)
> -#define PIPE_DUMP_CURRENT_STATES (1 << 1)
> -#define PIPE_DUMP_CURRENT_SHADERS (1 << 2)
> -#define PIPE_DUMP_LAST_COMMAND_BUFFER (1 << 3)
>
> /**
> * Create a compute-only context. Use in pipe_screen::context_create.
> --
> 2.11.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list