commit 90f61610cbc67b905e2995b3d1f122e137c7a7bc
author Jing SUN <jing.a.sun@intel.com> Tue Apr 22 13:16:15 2014 +0800
committer buildslave <sys_buildbot@intel.com> Wed Apr 23 13:18:37 2014 +0000
tree 649098eb339f06b2e71b423406cd1cf675d411ec
parent 85337018f040843583d1baec473e7b8f89f4ff75

Corrected JPEG encoding's libva log trace.

BZ: 189365

Issue: The format of JPEG encoding's log
dumped by libva trace was messy.

Solution: Corrected that log trace.

Change-Id: If06c6928b5e31b2ff84610d08bced57dc4cafc77
Signed-off-by: Jing SUN <jing.a.sun@intel.com>

va/va_trace.c

diff --git a/va/va_trace.c b/va/va_trace.c
index aaa55ac..92f354d 100755
--- a/va/va_trace.c
+++ b/va/va_trace.c
@@ -1003,17 +1003,17 @@
         for (j = 0; j < 16; ++j) {
             sprintf(tmp + strlen(tmp), "%u ", p->huffman_table[i].num_ac_codes[j]);
         }
-        va_TraceMsg(trace_ctx,"\t\tnum_dc_codes =%s\n", tmp);
+        va_TraceMsg(trace_ctx,"\t\tnum_ac_codes =%s\n", tmp);
         memset(tmp, 0, sizeof tmp);
         for (j = 0; j < 162; ++j) {
             sprintf(tmp + strlen(tmp), "%u ", p->huffman_table[i].ac_values[j]);
         }
-        va_TraceMsg(trace_ctx,"\t\tnum_dc_codes =%s\n", tmp);
+        va_TraceMsg(trace_ctx,"\t\tac_values =%s\n", tmp);
         memset(tmp, 0, sizeof tmp);
         for (j = 0; j < 2; ++j) {
             sprintf(tmp + strlen(tmp), "%u ", p->huffman_table[i].pad[j]);
         }
-        va_TraceMsg(trace_ctx,"\t\tnum_dc_codes =%s\n", tmp);
+        va_TraceMsg(trace_ctx,"\t\tpad =%s\n", tmp);
     }
 }
 
@@ -2344,16 +2344,18 @@
     va_TraceMsg(trace_ctx, "\tsample_bit_depth = %d\n", p->sample_bit_depth);
     va_TraceMsg(trace_ctx, "\tnum_scan = %d\n", p->num_scan);
     va_TraceMsg(trace_ctx, "\tnum_components = %d\n", p->num_components);
-    va_TraceMsg(trace_ctx, "\tcomponent_id[] = ");
-    for (i=0; i<4; i++)
-        va_TraceMsg(trace_ctx, "%d\t", p->component_id[i]);
-    va_TraceMsg(trace_ctx, "\n");
-    va_TraceMsg(trace_ctx, "\tquantiser_table_selector[] = ");
-    for (i=0; i<4; i++)
-        va_TraceMsg(trace_ctx, "%d\t", p->quantiser_table_selector[i]);
-    va_TraceMsg(trace_ctx, "\n");
-    va_TraceMsg(trace_ctx, "\tquality = %d\n", p->picture_height);
-    
+    for (i=0; i<p->num_components; i++)
+        va_TraceMsg(trace_ctx, "\tcomponent_id[%d] = %d\n", i, p->component_id[i]);
+
+    if (p->quality > 0)
+        va_TraceMsg(trace_ctx, "\tquality = %d\n", p->quality);
+    else
+        va_TraceMsg(trace_ctx, "\tquantiser_table_selector[] = %d %d %d %d\n",
+                    p->quantiser_table_selector[0],
+                    p->quantiser_table_selector[1],
+                    p->quantiser_table_selector[2],
+                    p->quantiser_table_selector[3]);
+
     va_TraceMsg(trace_ctx, NULL);
 
     return;
@@ -2372,25 +2374,38 @@
     DPY2TRACECTX(dpy);
     
     va_TraceMsg(trace_ctx, "\t--VAQMatrixBufferJPEG\n");
-    va_TraceMsg(trace_ctx, "\tload_lum_quantiser_matrix = %d", p->load_lum_quantiser_matrix);
+    va_TraceMsg(trace_ctx, "\tload_lum_quantiser_matrix = %d\n", p->load_lum_quantiser_matrix);
     if (p->load_lum_quantiser_matrix) {
         int i;
-        for (i = 0; i < 64; i++) {
-            if ((i % 8) == 0)
-                va_TraceMsg(trace_ctx, "\n\t");
-            va_TraceMsg(trace_ctx, "\t0x%02x", p->lum_quantiser_matrix[i]);
+        for (i = 0; i < 8; i++) {
+            va_TraceMsg(trace_ctx,
+                        "\t\t0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x\n",
+                        p->lum_quantiser_matrix[i*8],
+                        p->lum_quantiser_matrix[i*8 + 1],
+                        p->lum_quantiser_matrix[i*8 + 2],
+                        p->lum_quantiser_matrix[i*8 + 3],
+                        p->lum_quantiser_matrix[i*8 + 4],
+                        p->lum_quantiser_matrix[i*8 + 5],
+                        p->lum_quantiser_matrix[i*8 + 6],
+                        p->lum_quantiser_matrix[i*8 + 7]);
         }
-        va_TraceMsg(trace_ctx, "\n");
     }
-    va_TraceMsg(trace_ctx, "\tload_chroma_quantiser_matrix = %08x\n", p->load_chroma_quantiser_matrix);
+
+    va_TraceMsg(trace_ctx, "\tload_chroma_quantiser_matrix = %d\n", p->load_chroma_quantiser_matrix);
     if (p->load_chroma_quantiser_matrix) {
         int i;
-        for (i = 0; i < 64; i++) {
-            if ((i % 8) == 0)
-                va_TraceMsg(trace_ctx, "\n\t");
-            va_TraceMsg(trace_ctx, "\t0x%02x", p->chroma_quantiser_matrix[i]);
+        for (i = 0; i < 8; i++) {
+            va_TraceMsg(trace_ctx,
+                        "\t\t0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x 0x%02x\n",
+                        p->chroma_quantiser_matrix[i*8],
+                        p->chroma_quantiser_matrix[i*8 + 1],
+                        p->chroma_quantiser_matrix[i*8 + 2],
+                        p->chroma_quantiser_matrix[i*8 + 3],
+                        p->chroma_quantiser_matrix[i*8 + 4],
+                        p->chroma_quantiser_matrix[i*8 + 5],
+                        p->chroma_quantiser_matrix[i*8 + 6],
+                        p->chroma_quantiser_matrix[i*8 + 7]);
         }
-        va_TraceMsg(trace_ctx, "\n");
     }
     
     va_TraceMsg(trace_ctx, NULL);