summaryrefslogtreecommitdiffstats
path: root/meta
diff options
context:
space:
mode:
authorJussi Kukkonen <jussi.kukkonen@intel.com>2017-04-26 10:01:42 +0300
committerRichard Purdie <richard.purdie@linuxfoundation.org>2017-04-29 11:17:22 +0100
commit081e0a28f2ec4453b6f3e9a36ea1e45b80752672 (patch)
tree03d347d705110b0f647c71490b00f411d6f39394 /meta
parent1bc292fe253176963f304ed027e3ed32b00549b6 (diff)
downloadpoky-081e0a28f2ec4453b6f3e9a36ea1e45b80752672.tar.gz
gstreamer1.0-vaapi: Fix playback breaking bug
gstreamer-vaapi fails to play files with specific frame sizes because of buffer allocation issues. Fix is a backport. Fixes [YOCTO #11311]. (From OE-Core rev: e01eb9b37ba5512d6fd9893c70b1f4d766fab2ac) Signed-off-by: Jussi Kukkonen <jussi.kukkonen@intel.com> Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
-rw-r--r--meta/recipes-multimedia/gstreamer/gstreamer-vaapi/vaapivideobufferpool-create-allocator-if-needed.patch61
-rw-r--r--meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.10.4.bb2
2 files changed, 63 insertions, 0 deletions
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer-vaapi/vaapivideobufferpool-create-allocator-if-needed.patch b/meta/recipes-multimedia/gstreamer/gstreamer-vaapi/vaapivideobufferpool-create-allocator-if-needed.patch
new file mode 100644
index 0000000000..f666adc819
--- /dev/null
+++ b/meta/recipes-multimedia/gstreamer/gstreamer-vaapi/vaapivideobufferpool-create-allocator-if-needed.patch
@@ -0,0 +1,61 @@
1Signed-off-by: Jussi Kukkonen <jussi.kukkonen@intel.com>
2Upstream-Status: Backport [commit 59a731be6b in '1.10' branch]
3
4
5From 02a6002c3a80b3a5301c0943b1a1518bbdf439fc Mon Sep 17 00:00:00 2001
6From: =?UTF-8?q?V=C3=ADctor=20Manuel=20J=C3=A1quez=20Leal?=
7 <vjaquez@igalia.com>
8Date: Fri, 21 Apr 2017 19:07:18 +0200
9Subject: [PATCH] vaapivideobufferpool: create allocator if needed
10
11Backport to branch 1.10
12
13Sometimes a video decoder could set different buffer pool
14configurations, because their frame size changes. In this case we
15did not reconfigure the allocator.
16
17This patch enables this use case, creating a new allocator inside
18the VAAPI buffer pool if the caps changed, if it is not dmabuf-based.
19If so, it is just reconfigured, since it doesn't have a surface pool.
20
21https://bugzilla.gnome.org/show_bug.cgi?id=781577
22---
23 gst/vaapi/gstvaapivideobufferpool.c | 21 +++++++++++++++++++++
24 1 file changed, 21 insertions(+)
25
26diff --git a/gst/vaapi/gstvaapivideobufferpool.c b/gst/vaapi/gstvaapivideobufferpool.c
27index a3b9223f..9a50614b 100644
28--- a/gst/vaapi/gstvaapivideobufferpool.c
29+++ b/gst/vaapi/gstvaapivideobufferpool.c
30@@ -159,6 +159,27 @@ gst_vaapi_video_buffer_pool_set_config (GstBufferPool * pool,
31 gst_object_replace ((GstObject **) & priv->allocator, NULL);
32 priv->video_info = new_vip;
33
34+ {
35+ guint surface_alloc_flags;
36+ gboolean vinfo_changed = FALSE;
37+
38+ if (allocator) {
39+ const GstVideoInfo *alloc_vinfo =
40+ gst_allocator_get_vaapi_video_info (allocator, &surface_alloc_flags);
41+ vinfo_changed = gst_video_info_changed (alloc_vinfo, &new_vip);
42+ }
43+
44+ if (vinfo_changed && allocator && priv->use_dmabuf_memory) {
45+ gst_allocator_set_vaapi_video_info (allocator, &new_vip,
46+ surface_alloc_flags);
47+ } else if (!priv->use_dmabuf_memory && (vinfo_changed || !allocator)) {
48+ /* let's destroy the other allocator and create a new one */
49+ allocator = gst_vaapi_video_allocator_new (priv->display, &new_vip,
50+ surface_alloc_flags);
51+ gst_buffer_pool_config_set_allocator (config, allocator, NULL);
52+ }
53+ }
54+
55 if (!gst_buffer_pool_config_has_option (config,
56 GST_BUFFER_POOL_OPTION_VAAPI_VIDEO_META))
57 goto error_no_vaapi_video_meta_option;
58--
592.11.0
60
61
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.10.4.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.10.4.bb
index c4a884db73..44c66de442 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.10.4.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-vaapi_1.10.4.bb
@@ -2,4 +2,6 @@ require gstreamer1.0-vaapi.inc
2SRC_URI[md5sum] = "318af17f906637570b61dd7be9b5581d" 2SRC_URI[md5sum] = "318af17f906637570b61dd7be9b5581d"
3SRC_URI[sha256sum] = "03e690621594d9f9495d86c7dac8b8590b3a150462770ed070dc76f66a70de75" 3SRC_URI[sha256sum] = "03e690621594d9f9495d86c7dac8b8590b3a150462770ed070dc76f66a70de75"
4 4
5SRC_URI += "file://vaapivideobufferpool-create-allocator-if-needed.patch"
6
5DEPENDS += "gstreamer1.0 gstreamer1.0-plugins-base gstreamer1.0-plugins-bad" 7DEPENDS += "gstreamer1.0 gstreamer1.0-plugins-base gstreamer1.0-plugins-bad"