[Libva] [Bug 58176] [gst-vaapi] gi4_video.bits decoding failed (MPEG2)

[Li, Jocelyn]

Added Xiaowei.

From: Yuan, Feng
Sent: Thursday, January 31, 2013 11:09 AM
To: Li, Jocelyn; Beauchesne, Gwenole; Xiang, Haihao
Cc: Zhao, Halley; Zhong, CongX; libva at lists.freedesktop.org
Subject: RE: [Bug 58176] [gst-vaapi] gi4_video.bits decoding failed (MPEG2)

Copy to libva maillist, it’s more about gstreamer-codec-parser/gst-vaapi/libva/driver related issues.

Hi all,
     It’s an issue about how to process the zigzagged quantization table between gstreamer level and driver level.
     Libva defined API of
typedef struct _VAIQMatrixBufferMPEG2
{
    int load_intra_quantiser_matrix;
    int load_non_intra_quantiser_matrix;
    int load_chroma_intra_quantiser_matrix;
    int load_chroma_non_intra_quantiser_matrix;
    unsigned char intra_quantiser_matrix[64];
    unsigned char non_intra_quantiser_matrix[64];
    unsigned char chroma_intra_quantiser_matrix[64];
    unsigned char chroma_non_intra_quantiser_matrix[64];
} VAIQMatrixBufferMPEG2;
      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.
     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?

Thanks,
Wind


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)

Hi Gwenole and Haihao,

We need your comments on the solution to fix this issue.

Thanks,
Jocelyn

From: bugzilla-daemon at freedesktop.org<mailto:bugzilla-daemon at freedesktop.org> [mailto:bugzilla-daemon at freedesktop.org]
Sent: Wednesday, January 30, 2013 5:19 PM
To: Li, Jocelyn
Subject: [Bug 58176] [gst-vaapi] gi4_video.bits decoding failed (MPEG2)

Comment # 5<https://bugs.freedesktop.org/show_bug.cgi?id=58176#c5> on bug 58176<https://bugs.freedesktop.org/show_bug.cgi?id=58176> from Zhong Cong<mailto:congx.zhong at intel.com>

Created attachment 73914<attachment.cgi?id=73914> [details]<attachment.cgi?id=73914&action=edit> [review]<page.cgi?id=splinter.html&bug=58176&attachment=73914>

This patch remove inverse zigzag in gstvaapi codecparser



gstvaapi codecparse handles the data with inverse zigzag, and the intel-driver

alse asks for original data to inverse zigzag. Here comes the

contradiction.This patch remove inverse zigzag in gstvaapi codecparser,and it

can solve this issue.

________________________________
You are receiving this mail because:

  *   You are watching the assignee of the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libva/attachments/20130131/b11e9a51/attachment-0001.html>