After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 771528 - vaapivp8enc segfault
vaapivp8enc segfault
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer-vaapi
git master
Other Linux
: Normal normal
: 1.9.90
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-09-16 10:27 UTC by Florent Thiéry
Modified: 2016-09-30 07:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
encoder: vp8: Increase the allocation size for coded buffer (1.26 KB, patch)
2016-09-16 12:53 UTC, sreerenj
committed Details | Review

Description Florent Thiéry 2016-09-16 10:27:51 UTC
1) generate sample

gst-launch-1.0 videotestsrc num-buffers=242 pattern=snow ! video/x-raw\,\ format\=\(string\)I420\,\ width\=\(int\)1920\,\ height\=\(int\)1080\,\ framerate\=\(fraction\)30/1 ! filesink location=/tmp/buf.raw

2) run encode pipeline
gst-launch-1.0 filesrc location=/tmp/buf.raw blocksize=3110400 ! video/x-raw\,\ format\=\(string\)I420\,\ width\=\(int\)1920\,\ height\=\(int\)1080\,\ framerate\=\(fraction\)30/1 ! queue name=encoder ! vaapivp8enc ! fakesink

libva info: VA-API version 0.39.2
libva info: va_getDriverName() returns 0
libva info: Trying to open /usr/lib/dri/i965_drv_video.so
libva info: Found init function __vaDriverInit_0_39
libva info: va_openDriver() returns 0
Le pipeline est en phase de PREROLL…
Contexte de l’élément « vaapiencodevp8-0 » : gst.vaapi.Display=context, gst.vaapi.Display=(GstVaapiDisplay)NULL;
Le pipeline a terminé la phase PREROLL…
Passage du pipeline à la phase PLAYING…
New clock: GstSystemClock
Caught SIGSEGV
  • #0 poll
    from /usr/lib/libc.so.6
  • #1 ??
    from /usr/lib/libglib-2.0.so.0
  • #2 g_main_loop_run
    from /usr/lib/libglib-2.0.so.0
  • #3 gst_bus_poll
    from /usr/lib/libgstreamer-1.0.so.0
  • #4 ??
  • #5 ??
  • #6 __libc_start_main
    from /usr/lib/libc.so.6
  • #7 ??

Comment 1 sreerenj 2016-09-16 12:49:46 UTC
Hm, we are not getting enough compression for some frames and eventually end up with compressed frame size more than what we allocated...let me prepare a quick fix...
Comment 2 sreerenj 2016-09-16 12:53:58 UTC
Created attachment 335697 [details] [review]
encoder: vp8: Increase the allocation size for coded buffer


Florent, could you please try with this patch?
Comment 3 Florent Thiéry 2016-09-16 14:23:17 UTC
Seems to fix it, thanks !
Comment 4 sreerenj 2016-09-21 07:06:13 UTC
Review of attachment 335697 [details] [review]:

commit 0fae277d34cd2db89c9c7be45f41684f1040052f
Author: Sreerenj Balachandran <sreerenj.balachandran@intel.com>
Date:   Wed Sep 21 09:55:53 2016 +0300
Pused,
    encoder: vp8: Increase the allocation size for coded buffer
    
    We are not getting enough compression for some streams and
    encoded frame end up with more size than allocated.
    Assuming a compression ratio of 4, which should be good enough
    for holding the frames.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=771528
Comment 5 sreerenj 2016-09-21 07:07:39 UTC
Closing.