[Libva] codec parser issues between gst-plugins-bad and gstreamer-vaapi

[Beauchesne, Gwenole]

Hi,

gst-vaapi pulls in newer code from gst-plugins-bad master, not 0.10 as features and more important bug fixes that may break compatibility happen there. gst-vaapi codecparsers will live forever for 0.10 based branches, and will be removed for 1.x integration. Please try the attached patch.

Thanks,
Gwenole.
________________________________________
From: Yuan, Feng
Sent: 10 December 2012 09:17
To: Beauchesne, Gwenole
Cc: libva at lists.freedesktop.org; Li, Jocelyn; Zhong, CongX
Subject: codec parser issues between gst-plugins-bad and gstreamer-vaapi

Hi Gwenole,
     We saw codec parser moved into gstreamer-vaapi as a submodule for a few weeks. PRC QA team reported some bugs of playbin2 errors but manual pipeline works good.  e.g. https://bugs.freedesktop.org/show_bug.cgi?id=55305. After Cong tracked this bug, he found the issue was caused by codec parser which may get confused when APIs exist both in gst-plugins-bad and gst-vaapi.
     Plugin <h264parse>, which is a gst-plugins-bad element, it calls API *gst_h264_parser_parse_sps* in gsth264parser.c to parse SPS data which should be in libgstcodecparser.so. The problem is that this API also exist in gstream-vaapi as submodule which was a built-in in libgstvaapi-0.10.so.  This API may get unexpected result since *GstH264SPS* has different definitions between gst-plugins-bad and gstreamer-vaapi.
     This issue blocked us, it seems we have to rebuild gst-plugins-bad every time by force copy codec-parser from gst-vaapi to gst-plugins-bad. Do you have any better idea to solve this issue?
      BTW, Why codec-parser was moved into gst-vaapi?Is it the reason that gst-plugins-bad accept patches slow or something? How long would codec-parser be in gst-vaapi, Temporally or always?

Thanks,
Wind
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gst.vaapi.hide.codecparsers.patch
Type: text/x-patch
Size: 420 bytes
Desc: gst.vaapi.hide.codecparsers.patch
URL: <http://lists.freedesktop.org/archives/libva/attachments/20121210/577eed00/attachment.bin>