<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=us-ascii"> <meta name="Generator" content="Microsoft Word 12 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> See bugzilla here: <a href="https://bugzilla.gnome.org/show_bug.cgi?id=684341"> https://bugzilla.gnome.org/show_bug.cgi?id=684341</a> <o:p></o:p> <o:p> </o:p> <o:p> </o:p> Patch: <o:p></o:p> <o:p> </o:p> From 89c2e93aaef107f503d40d2b545288f6cbebc9ff Mon Sep 17 00:00:00 2001<o:p></o:p> From: Zhao Halley <halley.zhao@intel.com><o:p></o:p> Date: Wed, 19 Sep 2012 09:24:58 +0800<o:p></o:p> Subject: [PATCH] codecparsers: vc1: fix ttmbf when vstransform is not set<o:p></o:p> <o:p> </o:p> spec 8.3.6.2.1 Transform Type Selection<o:p></o:p> If variable-sized transform coding is not enabled,<o:p></o:p> then the 8x8 transform shall be used for all blocks.<o:p></o:p> <o:p> </o:p> I have to say the above spec isn't expressed clearly:<o:p></o:p> ttfrm is mentioned clearly, but not ttmbf.<o:p></o:p> anyway, since ttfrm is only specified when ttmbs is set,<o:p></o:p> we can deduce that ttmbf should set to 1.<o:p></o:p> it is also consistent with ffmpeg vc1 decoder and<o:p></o:p> hw behaviour (Intel GEN gfx)<o:p></o:p> <o:p> </o:p> <a href="https://bugzilla.gnome.org/show_bug.cgi?id=684341">https://bugzilla.gnome.org/show_bug.cgi?id=684341</a> <o:p></o:p> ---<o:p></o:p> gst-libs/gst/codecparsers/gstvc1parser.c | 16 ++++++++++++++++<o:p></o:p> 1 file changed, 16 insertions(+)<o:p></o:p> mode change 100644 => 100755 gst-libs/gst/codecparsers/gstvc1parser.c<o:p></o:p> <o:p> </o:p> diff --git a/gst-libs/gst/codecparsers/gstvc1parser.c b/gst-libs/gst/codecparsers/gstvc1parser.c<o:p></o:p> old mode 100644<o:p></o:p> new mode 100755<o:p></o:p> index bc94823..aa8cb4a<o:p></o:p> --- a/gst-libs/gst/codecparsers/gstvc1parser.c<o:p></o:p> +++ b/gst-libs/gst/codecparsers/gstvc1parser.c<o:p></o:p> @@ -1166,6 +1166,10 @@ parse_frame_header_advanced (GstBitReader * br, GstVC1FrameHdr * framehdr,<o:p></o:p> READ_UINT8 (br, pic->ttfrm, 2);<o:p></o:p> }<o:p></o:p> }<o:p></o:p> + else { // 8.3.6.2.1<o:p></o:p> + pic->ttmbf = 1;<o:p></o:p> + pic->ttfrm = 0;<o:p></o:p> + }<o:p></o:p> <o:p></o:p> framehdr->transacfrm = get_unary (br, 0, 2);<o:p></o:p> READ_UINT8 (br, framehdr->transdctab, 1);<o:p></o:p> @@ -1278,6 +1282,10 @@ parse_frame_header_advanced (GstBitReader * br, GstVC1FrameHdr * framehdr,<o:p></o:p> READ_UINT8 (br, pic->ttfrm, 2);<o:p></o:p> }<o:p></o:p> }<o:p></o:p> + else { // 8.3.6.2.1<o:p></o:p> + pic->ttmbf = 1;<o:p></o:p> + pic->ttfrm = 0;<o:p></o:p> + }<o:p></o:p> <o:p></o:p> framehdr->transacfrm = get_unary (br, 0, 2);<o:p></o:p> READ_UINT8 (br, framehdr->transdctab, 1);<o:p></o:p> @@ -1462,6 +1470,10 @@ parse_frame_header (GstBitReader * br, GstVC1FrameHdr * framehdr,<o:p></o:p> GST_DEBUG ("ttfrm %u", pic->ttfrm);<o:p></o:p> }<o:p></o:p> }<o:p></o:p> + else { // 8.3.6.2.1<o:p></o:p> + pic->ttmbf = 1;<o:p></o:p> + pic->ttfrm = 0;<o:p></o:p> + }<o:p></o:p> <o:p></o:p> framehdr->transacfrm = get_unary (br, 0, 2);<o:p></o:p> READ_UINT8 (br, framehdr->transdctab, 1);<o:p></o:p> @@ -1494,6 +1506,10 @@ parse_frame_header (GstBitReader * br, GstVC1FrameHdr * framehdr,<o:p></o:p> READ_UINT8 (br, pic->ttfrm, 2);<o:p></o:p> }<o:p></o:p> }<o:p></o:p> + else { // 8.3.6.2.1<o:p></o:p> + pic->ttmbf = 1;<o:p></o:p> + pic->ttfrm = 0;<o:p></o:p> + }<o:p></o:p> <o:p></o:p> framehdr->transacfrm = get_unary (br, 0, 2);<o:p></o:p> READ_UINT8 (br, framehdr->transdctab, 1);<o:p></o:p> -- <o:p></o:p> 1.7.9.5<o:p></o:p> <o:p> </o:p> </div> </body> </html>