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 704848 - Failed with creating wrong WxH surface
Failed with creating wrong WxH surface
Status: RESOLVED FIXED
Product: gstreamer-vaapi
Classification: Other
Component: general
git master
Other Windows
: Normal normal
: ---
Assigned To: gstreamer-vaapi maintainer(s)
gstreamer-vaapi maintainer(s)
: 704849 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2013-07-25 05:22 UTC by Yang Lianyue
Modified: 2013-09-09 07:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Yang Lianyue 2013-07-25 05:22:05 UTC 
Environment:
--------------------------
libva:           (master)d2dbc3f69c69e5933e7b3da429c0fb0cae5b98b0
intel_driver:    (master)8bf807539c1790d6eee531373131672d38c82b31
Gstreamer:       (1.0)   80431b497d2b1530b9f4d2988956ba5669ee2eac
gst-plugins-base:(1.0)   d11092d478aa9118d0b6aebcfebeedd06e7bc12e
gst-plugins-bad: (1.0)   cbd83678103e9d8417e2e4905b9fa5e8d20c547a
gst-plugins-good:(1.0)   82c342f83bfdc8e57ee29e72aad848ba200443fc
gst-plugins-ugly:(1.0)   b31e84d5f7c4c6eddcf1b955d131d223fa1929c0
Gst-vaapi:       (master)0b1a97cbf93846eb7cc03d46d0eb8a04244fb8ab


Comments:
--------------------------
For files: 
sony-ct1.bits 
sony-ct2.bits 
sony-ct3.bs 
sony-ct4.bs
MPEG2 conformance video url: 
"Fedora.FrameWork/MPEG2_conformance/bitstreams/" under Media repo.

mpegvideoparse read the right WxH, but when create Surface, WxH is wrong.

For example, sony-ct1.bits 352x224, I print WxH by vaPutSurface, print srcw = 120 srch = 120, and the display video is not right. 

export GST_DEBUG=*base*:5
basesink gstbasesink.c:3025:gst_base_sink_event:<vaapisink0>^[[00m received event 0x7f23d8004300 caps event at time 99:99:99.999999999: GstEventCaps, caps=(GstCaps)video/x-surface, type=(string)vaapi, opengl=(boolean)false, width=(int)352, height=(int)224, framerate=(fraction)30/1, pixel-aspect-ratio=(fraction)4/3, interlace-mode=(string)interleaved;
Comment 1 André Klapper 2013-07-25 06:13:32 UTC 
*** Bug 704849 has been marked as a duplicate of this bug. ***
Comment 2 Gwenole Beauchesne 2013-08-26 14:20:05 UTC 
This is fixed in git master branch. Thanks.
Comment 3 Gwenole Beauchesne 2013-08-26 14:20:19 UTC 
commit 292b2f51b4233b75f6f94e0d55d78dc593b261b1
Author: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
Date:   Mon Aug 26 16:15:49 2013 +0200

    mpeg2: disable video cropping as picture_display_extension() is missing.
    
    Disable video cropping in MPEG-2 codec because it is partially implemented
    and actually because nobody implements it that way, and the standard spec
    does not specify the display process either anyway.
    
    Most notably, there are two possible use cases for sequence_display_extension()
    horizontal_display_size & vertical_display_size: (i) guesstimating the
    pixel-aspect-ratio, or (ii) implement some kind of span & scan process
    in conjunction with picture_display_extension() information.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=704848
Comment 4 congx.zhong 2013-09-09 07:06:13 UTC 
OK,the patch of this issue is merged to this branch. Pls try again.