Skip to content

Commit b00599b

Browse files
Dmitry RogozhkinXinfengZhang
authored andcommitted
trace: add missing trace fields for VAProcPipelineParameterBuffer
Signed-off-by: Dmitry Rogozhkin <dmitry.v.rogozhkin@gmail.com>
1 parent 29fbbd8 commit b00599b

File tree

2 files changed

+57
-2
lines changed

2 files changed

+57
-2
lines changed

va/va_trace.c

Lines changed: 56 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6606,7 +6606,62 @@ va_TraceVAProcPipelineParameterBuffer(
66066606
}
66076607
}
66086608

6609-
/* FIXME: add other info later */
6609+
va_TraceMsg(trace_ctx, "\t rotation_state = 0x%08x\n", p->rotation_state);
6610+
6611+
if (p->blend_state) {
6612+
va_TraceMsg(trace_ctx, "\t blend_state\n");
6613+
va_TraceMsg(trace_ctx, "\t flags = 0x%08x\n", p->blend_state->flags);
6614+
va_TraceMsg(trace_ctx, "\t global_alpha = %f\n", p->blend_state->global_alpha);
6615+
va_TraceMsg(trace_ctx, "\t min_luma = %f\n", p->blend_state->min_luma);
6616+
va_TraceMsg(trace_ctx, "\t max_luma = %f\n", p->blend_state->max_luma);
6617+
} else {
6618+
va_TraceMsg(trace_ctx, "\t blend_state = (NULL)\n");
6619+
}
6620+
6621+
va_TraceMsg(trace_ctx, "\t mirror_state = 0x%08x\n", p->mirror_state);
6622+
va_TraceMsg(trace_ctx, "\t num_additional_outputs = %d\n", p->num_additional_outputs);
6623+
6624+
if (p->num_additional_outputs) {
6625+
va_TraceMsg(trace_ctx, "\t additional_outputs\n");
6626+
6627+
if (p->additional_outputs) {
6628+
/* only dump the first 5 additional outputs */
6629+
for (i = 0; i < p->num_additional_outputs && i < 5; i++) {
6630+
va_TraceMsg(trace_ctx, "\t additional_outputs[%d] = 0x%08x\n", i, p->additional_outputs[i]);
6631+
}
6632+
} else {
6633+
for (i = 0; i < p->num_additional_outputs && i < 5; i++) {
6634+
va_TraceMsg(trace_ctx, "\t additional_outputs[%d] = (NULL)\n", i);
6635+
}
6636+
}
6637+
}
6638+
6639+
va_TraceMsg(trace_ctx, "\t input_surface_flag = 0x%08x\n", p->input_surface_flag);
6640+
va_TraceMsg(trace_ctx, "\t output_surface_flag = 0x%08x\n", p->output_surface_flag);
6641+
6642+
va_TraceMsg(trace_ctx, "\t input_color_properties\n");
6643+
va_TraceMsg(trace_ctx, "\t chroma_sample_location = 0x%02x\n", p->input_color_properties.chroma_sample_location);
6644+
va_TraceMsg(trace_ctx, "\t color_range = %d\n", p->input_color_properties.color_range);
6645+
va_TraceMsg(trace_ctx, "\t colour_primaries = %d\n", p->input_color_properties.colour_primaries);
6646+
va_TraceMsg(trace_ctx, "\t transfer_characteristics = %d\n", p->input_color_properties.transfer_characteristics);
6647+
va_TraceMsg(trace_ctx, "\t matrix_coefficients = %d\n", p->input_color_properties.matrix_coefficients);
6648+
6649+
va_TraceMsg(trace_ctx, "\t output_color_properties\n");
6650+
va_TraceMsg(trace_ctx, "\t chroma_sample_location = 0x%02x\n", p->output_color_properties.chroma_sample_location);
6651+
va_TraceMsg(trace_ctx, "\t color_range = %d\n", p->output_color_properties.color_range);
6652+
va_TraceMsg(trace_ctx, "\t colour_primaries = %d\n", p->output_color_properties.colour_primaries);
6653+
va_TraceMsg(trace_ctx, "\t transfer_characteristics = %d\n", p->output_color_properties.transfer_characteristics);
6654+
va_TraceMsg(trace_ctx, "\t matrix_coefficients = %d\n", p->output_color_properties.matrix_coefficients);
6655+
6656+
va_TraceMsg(trace_ctx, "\t processing_mode = %d\n", p->processing_mode);
6657+
6658+
if (p->output_hdr_metadata) {
6659+
va_TraceMsg(trace_ctx, "\t output_hdr_metadata\n");
6660+
va_TraceMsg(trace_ctx, "\t metadata_type = %d\n", p->output_hdr_metadata->metadata_type);
6661+
va_TraceMsg(trace_ctx, "\t metadata_size = %d\n", p->output_hdr_metadata->metadata_size);
6662+
} else {
6663+
va_TraceMsg(trace_ctx, "\t output_hdr_metadata = (NULL)\n");
6664+
}
66106665

66116666
va_TraceMsg(trace_ctx, NULL);
66126667
}

va/va_vpp.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -909,7 +909,7 @@ typedef struct _VAProcPipelineParameterBuffer {
909909
* be queried with vaQueryVideoProcPipelineCaps().
910910
*
911911
* If this is set to VAProcColorStandardExplicit, the color properties
912-
* are specified explicitly in surface_color_properties instead.
912+
* are specified explicitly in input_color_properties instead.
913913
*/
914914
VAProcColorStandardType surface_color_standard;
915915
/**

0 commit comments

Comments
 (0)