Bug 58064 - [gst-vaapi] Core dumped when decording MR3_TANDBERG_B.264
Summary: [gst-vaapi] Core dumped when decording MR3_TANDBERG_B.264
Status: VERIFIED FIXED
Alias: None
Product: libva
Classification: Unclassified
Component: intel (show other bugs)
Version: unspecified
Hardware: Other All
: medium major
Assignee: Wind Yuan
QA Contact: Ouping Zhang
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-10 03:36 UTC by Du Yan
Modified: 2014-01-02 05:30 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

Attachments
0001-h264-fix-reference-list-count-less-than-num_ref (1.75 KB, text/plain)
2013-02-28 08:32 UTC, Wind Yuan 		Details
0002-h264-remove-reference-if-picture-frame_num-is-same (1.59 KB, text/plain)
2013-02-28 08:33 UTC, Wind Yuan 		Details
0003-h264-support-process-for-gaps-in-frame_num (6.29 KB, text/plain)
2013-02-28 08:33 UTC, Wind Yuan 		Details
h264-dec-workaround-to-enable-baseline-profile (1.16 KB, patch)
2014-01-02 05:30 UTC, Wind Yuan 		Details | Splinter Review
View All

Description Du Yan 2012-12-10 03:36:07 UTC 
Environment:
--------------
Platform: SNB IVB
Libva: staging
commit 8865fd148a4444b26c35197b3176f87fb2e85393
Intel-driver: staging
commit c0ef9d99df37ae45589fecb898727be495e50304
Gst-vaapi: qa
commit df1eb6e01e1bf76e23c8eb7d219f15804b109d2d


Bug Info:
--------------
Decoding below files will cause coredump.
MR3_TANDBERG_B.264
MR4_TANDBERG_C.264
MR5_TANDBERG_C.264


Reproduce steps:
----------------
1. xinit&
2. gst-launch-0.10 filesrc location=/home/AVC_conformance/Base_Ext_Main_profile/MR3_TANDBERG_B.264 ! h264parse ! vaapidecode ! vaapisink sync=false
Comment 1 Du Yan 2012-12-28 07:52:36 UTC 
Issue can be reproduced on HSW
Comment 2 Wind Yuan 2013-02-28 08:32:50 UTC 
Created attachment 75667 [details]
0001-h264-fix-reference-list-count-less-than-num_ref
Comment 3 Wind Yuan 2013-02-28 08:33:21 UTC 
Created attachment 75668 [details]
0002-h264-remove-reference-if-picture-frame_num-is-same
Comment 4 Wind Yuan 2013-02-28 08:33:51 UTC 
Created attachment 75669 [details]
0003-h264-support-process-for-gaps-in-frame_num
Comment 5 Wind Yuan 2013-02-28 08:41:05 UTC 
0001...patch is to fix the coredump issue.
RefPicListx_count is less than the num_ref_listx because there's not enough reference pictures in dpb.
After 0001 installed, the coredump is gone but last few frames displayed with mosaic blocks. The reason is gstreamer-vaapi doesn't process gaps_in_frame_num_value_allow_flag. 
002...patch and 0003...patch is going to process 'non-exist' pictures and slide the reference window.
Comment 6 Du Yan 2013-03-04 02:40:35 UTC 
gst-vaapi: qa branch 41940904625f7a3dbdf88d4d1f9fe2df4c434286

Issue can't be reproduced. close it.
Comment 7 zhenxiang.li 2014-01-02 01:46:24 UTC 
It can be reproduced again, both on PRC's tree and Gwenole's tree
Comment 8 Wind Yuan 2014-01-02 05:27:48 UTC 
(In reply to comment #7)
> It can be reproduced again, both on PRC's tree and Gwenole's tree
MR3_TANDBERG_B.264 is a baseline profile. since baseline removed from gst-vaapi. so this file can't be tested now. I'll post a workaround to test baseline here.
Comment 9 Wind Yuan 2014-01-02 05:30:10 UTC 
Created attachment 91403 [details] [review]
h264-dec-workaround-to-enable-baseline-profile

workaround to re-enable H.264 baseline profile to do more tests. At least this bug need enable baseline to test gaps in frame_num

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.