<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 14 (filtered medium)"> <base href="https://bugs.freedesktop.org/"><style></style> </head> <body lang="ZH-CN" link="blue" vlink="purple"> <div class="WordSection1"> Added Xiaowei.<o:p></o:p> <o:p> </o:p> <div> <div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Yuan, Feng Sent: Thursday, January 31, 2013 11:09 AM To: Li, Jocelyn; Beauchesne, Gwenole; Xiang, Haihao Cc: Zhao, Halley; Zhong, CongX; libva@lists.freedesktop.org Subject: RE: [Bug 58176] [gst-vaapi] gi4_video.bits decoding failed (MPEG2)<o:p></o:p> </div> </div> <o:p> </o:p> Copy to libva maillist, it’s more about gstreamer-codec-parser/gst-vaapi/libva/driver related issues.<o:p></o:p> <o:p> </o:p> Hi all,<o:p></o:p> It’s an issue about how to process the zigzagged quantization table between gstreamer level and driver level.<o:p></o:p> Libva defined API of <o:p></o:p> typedef struct _VAIQMatrixBufferMPEG2<o:p></o:p> {<o:p></o:p> int load_intra_quantiser_matrix;<o:p></o:p> int load_non_intra_quantiser_matrix;<o:p></o:p> int load_chroma_intra_quantiser_matrix;<o:p></o:p> int load_chroma_non_intra_quantiser_matrix;<o:p></o:p> unsigned char intra_quantiser_matrix[64];<o:p></o:p> unsigned char non_intra_quantiser_matrix[64];<o:p></o:p> unsigned char chroma_intra_quantiser_matrix[64];<o:p></o:p> unsigned char chroma_non_intra_quantiser_matrix[64];<o:p></o:p> } VAIQMatrixBufferMPEG2;<o:p></o:p> Question is how to fill <xxx_intra_quantiser_matrix>, is it zigzag ordered or scan ordered? Mpeg2 data store the quantization table in zigzag order. Currently gstreamer-codec-parser would parse all zigzagged quantization table from MPEG2 data and converting into scanning order. Intel-Driver would do the same thing again. The result certainly would be incorrect.<o:p></o:p> Ffmpeg would fill the quantiser_matrix by zigzag order, do we need to change gstreamer-codec-parse to read data into zigzag order? Does any other package/lib already using gstreamer-codec-parser(mpeg2)? Which package should be the right role to do the conversion from zigzag to scanning order?<o:p></o:p> <o:p> </o:p> Thanks,<o:p></o:p> Wind<o:p></o:p> <o:p> </o:p> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt"> <div> <div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: Li, Jocelyn Sent: Thursday, January 31, 2013 10:24 AM To: Beauchesne, Gwenole; Xiang, Haihao Cc: Zhao, Halley; Yuan, Feng; Zhong, CongX Subject: FW: [Bug 58176] [gst-vaapi] gi4_video.bits decoding failed (MPEG2)<o:p></o:p> </div> </div> <o:p> </o:p> Hi Gwenole and Haihao,<o:p></o:p> <o:p> </o:p> We need your comments on the solution to fix this issue.<o:p></o:p> <o:p> </o:p> Thanks,<o:p></o:p> Jocelyn<o:p></o:p> <o:p> </o:p> <div> <div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm"> From: <a href="mailto:bugzilla-daemon@freedesktop.org">bugzilla-daemon@freedesktop.org</a> [<a href="mailto:bugzilla-daemon@freedesktop.org">mailto:bugzilla-daemon@freedesktop.org</a>] Sent: Wednesday, January 30, 2013 5:19 PM To: Li, Jocelyn Subject: [Bug 58176] [gst-vaapi] gi4_video.bits decoding failed (MPEG2)<o:p></o:p> </div> </div> <o:p> </o:p> <div> <a href="https://bugs.freedesktop.org/show_bug.cgi?id=58176#c5" title="NEW --- - [gst-vaapi] gi4_video.bits decoding failed (MPEG2)">Comment # 5</a> on <a href="https://bugs.freedesktop.org/show_bug.cgi?id=58176" title="NEW --- - [gst-vaapi] gi4_video.bits decoding failed (MPEG2)"> bug 58176</a> from <a href="mailto:congx.zhong@intel.com" title="Zhong Cong <congx.zhong@intel.com>">Zhong Cong</a> <o:p></o:p> <pre>Created <a name="attach_73914"></a><a href="attachment.cgi?id=73914" title="This patch remove inverse zigzag in gstvaapi codecparser">attachment 73914</a> <a href="attachment.cgi?id=73914&action=edit" title="This patch remove inverse zigzag in gstvaapi codecparser">[details]</a> <a href="page.cgi?id=splinter.html&bug=58176&attachment=73914">[review]</a><o:p></o:p></pre> <pre>This patch remove inverse zigzag in gstvaapi codecparser<o:p></o:p></pre> <pre><o:p> </o:p></pre> <pre>gstvaapi codecparse handles the data with inverse zigzag, and the intel-driver<o:p></o:p></pre> <pre>alse asks for original data to inverse zigzag. Here comes the<o:p></o:p></pre> <pre>contradiction.This patch remove inverse zigzag in gstvaapi codecparser,and it<o:p></o:p></pre> <pre>can solve this issue.<o:p></o:p></pre> </div> <div class="MsoNormal" align="center" style="text-align:center"> <hr size="2" width="100%" align="center"> </div> You are receiving this mail because: <o:p> </o:p> <ul type="disc"> <li class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;mso-list:l0 level1 lfo3"> You are watching the assignee of the bug.<o:p></o:p></li></ul> </div> </div> </body> </html>