platform/upstream/gstreamer.git
3 years agovaapi: use gst_clear_object instead of g_clear_object
Víctor Manuel Jáquez Leal [Sat, 12 Dec 2020 09:30:41 +0000 (10:30 +0100)]
vaapi: use gst_clear_object instead of g_clear_object

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/406>

3 years agoplugins: encode: unlock the stream lock before _flush()
He Junyan [Tue, 8 Dec 2020 05:34:35 +0000 (13:34 +0800)]
plugins: encode: unlock the stream lock before _flush()

The current encoder will hang when EOS comes. When we call the
gst_vaapi_encoder_encode_and_queue(), we should release the stream
lock, just like what we do in gst_vaapiencode_handle_frame().

The deadlock happens when: The input thread holding the stream lock
is using gst_vaapi_encoder_create_coded_buffer() to acquire a coded
buffer, while the output thread which holding the coded buffer resource
is acquiring the stream lock in _push_frame() to push the data to
down stream element.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/404>

3 years agolibs: encoder: H264: Fix one assert in get_pending_reordered().
He Junyan [Tue, 8 Dec 2020 16:04:33 +0000 (00:04 +0800)]
libs: encoder: H264: Fix one assert in get_pending_reordered().

gst_vaapi_encoder_h264_get_pending_reordered() does not consider the
case for HIERARCHICAL_B mode. The pipeline:

gst-launch-1.0  videotestsrc num-buffers=48 ! vaapih264enc prediction-type=2 \
keyframe-period=32 ! fakesink

get a assert:

ERROR:../gst-libs/gst/vaapi/gstvaapiencoder_h264.c:1996:reflist1_init_hierarchical_b:
assertion failed: (count != 0)

The last few B frames are not fetched in correct order when HIERARCHICAL_B
is enabled.

We also fix a latent bug for normal mode. The g_queue_pop_tail() of B frames
make the last several frames encoded in reverse order. The NAL of last few
frames come in reverse order in the bit stream, though it can still output
the correct image.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/405>

3 years agolibs: encoder: H265: Add screen content coding extensions support.
He Junyan [Thu, 25 Jun 2020 08:25:21 +0000 (16:25 +0800)]
libs: encoder: H265: Add screen content coding extensions support.

In scc mode, the I frame can ref to itself and it needs the L0 reference
list enabled. So we should set the I frame to P_SLICE type. We do not need
to change the ref_pic_list0/1 passed to VA driver, just need to enable the
VAEncPictureParameterBufferHEVC->pps_curr_pic_ref_enabled_flag to notify
the driver consider the current frame as reference. For bits conformance,
the NumRpsCurrTempList0 should be incremented by one to include the current
picture as the reference frame. We manually do it when packing the slice header.

Command line like:
gst-launch-1.0  videotestsrc num-buffers=10 ! \
capsfilter caps=video/x-raw,format=NV12, framerate=30/1,width=640,height=360 ! \
vaapih265enc ! capsfilter caps=video/x-h265,profile="{ (string)screen-extended-main }" ! \
filesink location=out.265
Can be used to specify that the encoder should use SCC profiles.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/379>

3 years agoplugin: encode: vp9: Implement the set_config().
He Junyan [Sat, 11 Jul 2020 15:37:29 +0000 (23:37 +0800)]
plugin: encode: vp9: Implement the set_config().

We store the allowed profiles list to encoder in set_config().

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/380>

3 years agoplugin: encode: vp9: Add the profile into output caps.
He Junyan [Sat, 11 Jul 2020 15:39:40 +0000 (23:39 +0800)]
plugin: encode: vp9: Add the profile into output caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/380>

3 years agolibs: encoder: vp9: no need to ensure_hw_profile.
He Junyan [Sat, 11 Jul 2020 15:27:21 +0000 (23:27 +0800)]
libs: encoder: vp9: no need to ensure_hw_profile.

Once we decide the profile and can get the valid entrypoint for
that profile, hw must already support this profile/entrypoint pair.
No need to check it again in set_context_info().

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/380>

3 years agolibs: encoder: vp9: Improve the manner to decide the profile.
He Junyan [Sat, 11 Jul 2020 15:22:55 +0000 (23:22 +0800)]
libs: encoder: vp9: Improve the manner to decide the profile.

We should decide the VP9 encoder's profile based on the chroma and
depth of the input format, then make sure it is included in the
allowed list.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/380>

3 years agolibs: util: vpx: add get_chroma_format_idc for VP9
He Junyan [Sat, 11 Jul 2020 15:17:02 +0000 (23:17 +0800)]
libs: util: vpx: add get_chroma_format_idc for VP9

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/380>

3 years agolibs: encoder: vp9: Add allowed_profiles.
He Junyan [Sat, 11 Jul 2020 15:09:59 +0000 (23:09 +0800)]
libs: encoder: vp9: Add allowed_profiles.

We need the allowed_profiles to store the allowed profiles in down
stream's caps.
Command line like:
  vaapivp9enc ! capsfilter caps=video/x-vp9,profile="{ (string)1, \
    (string)3 }"
We need to store GST_VAAPI_PROFILE_VP9_1 and GST_VAAPI_PROFILE_VP9_3
in this list.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/380>

3 years agolibs: decoder: H265: Fix a typo in scc reference setting.
He Junyan [Mon, 30 Nov 2020 10:00:30 +0000 (18:00 +0800)]
libs: decoder: H265: Fix a typo in scc reference setting.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/402>

3 years agolibs: decoder: H265: Add MAIN_422_12 profile supporting.
He Junyan [Fri, 17 Jul 2020 10:00:30 +0000 (18:00 +0800)]
libs: decoder: H265: Add MAIN_422_12 profile supporting.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/390>

3 years agovideo-format: Add Y212_LE format.
He Junyan [Fri, 31 Jul 2020 06:38:42 +0000 (14:38 +0800)]
video-format: Add Y212_LE format.

It can be used as HEVC YUV_4:2:2 12bits stream's decoder output, and
also can be used as the input format for encoding HEVC YUV_4:2:2 12bits
stream.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/390>

3 years agolibs: decoder: H265: Add MAIN_444_12 profile supporting.
He Junyan [Thu, 30 Jul 2020 15:21:06 +0000 (23:21 +0800)]
libs: decoder: H265: Add MAIN_444_12 profile supporting.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/390>

3 years agovideo-format: Add Y412_LE format.
He Junyan [Thu, 30 Jul 2020 15:13:10 +0000 (23:13 +0800)]
video-format: Add Y412_LE format.

It can be used as HEVC YUV_4:4:4 12bits stream's decoder output, and
also can be used as the input format for encoding HEVC YUV_4:4:4 12bits
stream.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/390>

3 years agolibs: decoder: h265: fill missing predictor_palette_size field.
He Junyan [Thu, 17 Sep 2020 08:47:43 +0000 (16:47 +0800)]
libs: decoder: h265: fill missing predictor_palette_size field.

The predictor_palette_size of VAPictureParameterBufferHEVCScc is
forgotten and need to be filled when streams have palettes.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/396>

4 years agolibs: utils: h265: Use get_profile_from_sps to get profile.
He Junyan [Thu, 17 Sep 2020 07:35:11 +0000 (15:35 +0800)]
libs: utils: h265: Use get_profile_from_sps to get profile.

We now use gst_h265_get_profile_from_sps() to replace the old way
of gst_h265_profile_tier_level_get_profile() to get more precise
profile. The new function consider the unstandard cases and give
a more suitable profile decision.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/395>

4 years agolibs: decoder: vp9: 0xff segment pred probs if no temporal update
Víctor Manuel Jáquez Leal [Mon, 19 Oct 2020 11:46:44 +0000 (13:46 +0200)]
libs: decoder: vp9: 0xff segment pred probs if no temporal update

According to the spec (6.2.11 Segmentation params syntax)
segmentation_pred_prob[i] ast to be 0xff if not temporal_update.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/400>

4 years agolibs: decoder: vp9: avoid reference rewriting
Víctor Manuel Jáquez Leal [Mon, 19 Oct 2020 11:42:53 +0000 (13:42 +0200)]
libs: decoder: vp9: avoid reference rewriting

The removed code set all the reference frames to the current frame it is a key
one, but later, all the reference frames were rewritten with the decoded picture
buffers or VA_INVALID_SURFACE if they were not available.

Basically, all this time the first reference frame assignment has been ignored,
and it's not described by the spec, and this patch removes that code.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/400>

4 years agodecoder: don't reply src caps query with allowed if pad is fixed
Víctor Manuel Jáquez Leal [Sun, 20 Sep 2020 07:56:40 +0000 (09:56 +0200)]
decoder: don't reply src caps query with allowed if pad is fixed

If the pad is already fixed the caps query have to be reply with the
current fixed caps. Otherwise the query has to be replied with the
autogeneratd src caps.

This path fix this by falling back to the normal caps query processing
if the pad is already fixed. Otherwise it will fetch the allowed src
pad caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/397>

4 years agoplugins: decode: fix a DMA caps typo in ensure_allowed_srcpad_caps.
He Junyan [Mon, 14 Sep 2020 16:11:30 +0000 (00:11 +0800)]
plugins: decode: fix a DMA caps typo in ensure_allowed_srcpad_caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/392>

4 years agovaapisink: when updating the caps, reset rotation
Marc Leeman [Tue, 1 Sep 2020 07:31:33 +0000 (09:31 +0200)]
vaapisink: when updating the caps, reset rotation

When an element upstream changes settings (e.g. crop), new caps are sent
to vaapisink. When vaapisink was rotating the image, it needs to
re-evaluate if the sink needs to rotate the image.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/389>

4 years agoci: include template from gst-ci master branch again
Tim-Philipp Müller [Tue, 8 Sep 2020 16:31:02 +0000 (17:31 +0100)]
ci: include template from gst-ci master branch again

4 years agoBack to development
Tim-Philipp Müller [Tue, 8 Sep 2020 15:59:07 +0000 (16:59 +0100)]
Back to development

4 years agoRelease 1.18.0
Tim-Philipp Müller [Mon, 7 Sep 2020 23:09:51 +0000 (00:09 +0100)]
Release 1.18.0

4 years agoUpdate for gst_video_transfer_function_*() function renaming
Sebastian Dröge [Mon, 7 Sep 2020 09:15:43 +0000 (12:15 +0300)]
Update for gst_video_transfer_function_*() function renaming

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/391>

4 years agoRevert "libs: decoder: h264, h265: in context at least 16 reference surfaces"
Víctor Manuel Jáquez Leal [Sat, 22 Aug 2020 10:53:12 +0000 (12:53 +0200)]
Revert "libs: decoder: h264, h265: in context at least 16 reference surfaces"

This reverts commit b387081a4d77d3da202da72686ab40fb9c83ee1e as discussed in
https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/commit/b387081a4d77d3da202da72686ab40fb9c83ee1e

4 years agoRelease 1.17.90
Tim-Philipp Müller [Thu, 20 Aug 2020 15:16:25 +0000 (16:16 +0100)]
Release 1.17.90

4 years agolibs: surface: egl: guard memory type
Víctor Manuel Jáquez Leal [Mon, 17 Aug 2020 09:43:53 +0000 (11:43 +0200)]
libs: surface: egl: guard memory type

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/384>

4 years agoplugin: decode: Fix two mem leaks because of caps.
He Junyan [Mon, 17 Aug 2020 11:26:43 +0000 (19:26 +0800)]
plugin: decode: Fix two mem leaks because of caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/385>

4 years agoplugin: allocator: No need to ref allocator when create mem.
He Junyan [Sat, 15 Aug 2020 17:57:15 +0000 (01:57 +0800)]
plugin: allocator: No need to ref allocator when create mem.

We do not need to ref the allocator when creating GstVaapiVideoMemory
kind memory, and then release it in _free(). The framework already
does it for us.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/383>

4 years agovaapiencode: h264: ignore level without breaking negotiation
Víctor Manuel Jáquez Leal [Fri, 14 Aug 2020 08:42:51 +0000 (10:42 +0200)]
vaapiencode: h264: ignore level without breaking negotiation

Since commit 9f627ef2 if the user sets level in the encoder src caps
the caps negotiation is rejected.

But since the same commit the same encoder set the autoconfigured
level in caps. Some change in the base class might fixed the operation
order so now the caps are set and later negotiated.

This patch removes the level check.

Fixes: #273
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/382>

4 years agolibs: egl: surface: export EGLImage as DMABuf if GEM not supported
Víctor Manuel Jáquez Leal [Tue, 9 Jul 2019 17:17:48 +0000 (19:17 +0200)]
libs: egl: surface: export EGLImage as DMABuf if GEM not supported

This code path is used when frames are rendered as textures through
GstVideoGLTextureUploadMeta with EGL, mainly under Wayland.

Originally the EGLImage was exported as GEM, which was handled by
Intel drivers, but Gallium ones cannot create VA surfaces from
GEM buffers, only DMABuf.

This patch checks the memory types supported by VA driver to choose
the render the EGLImages from GEM or DMABuf, because GEM is still
better where supported.

DMABuf is well handled either by intel-vaapi-driver and gallium.

Fixes: #137
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/122>

4 years agolibs: filter: gst_vaapi_filter_get_memory_types()
Víctor Manuel Jáquez Leal [Tue, 26 May 2020 14:18:32 +0000 (16:18 +0200)]
libs: filter: gst_vaapi_filter_get_memory_types()

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/122>

4 years agoplugins: remove gst_vaapi_plugin_base_get_allowed_srcpad_raw_caps()
Víctor Manuel Jáquez Leal [Wed, 12 Aug 2020 16:48:59 +0000 (18:48 +0200)]
plugins: remove gst_vaapi_plugin_base_get_allowed_srcpad_raw_caps()

Since nobody uses it, just remove it.

Thus extract_allowed_surface_formats() is refactored to attend only
gst_vaapi_plugin_base_get_allowed_sinkpad_raw_caps().

Now a surface is created when the image chorma is different from the
previous one. And if the driver has the quirk, it outputs all the
supported image formats without trying them.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/381>

4 years agovaapidecode: expose raw src caps with same chroma
Víctor Manuel Jáquez Leal [Wed, 12 Aug 2020 15:50:50 +0000 (17:50 +0200)]
vaapidecode: expose raw src caps with same chroma

The try-and-error approach for getting the possible image formats from
a surface has brought several problems in different drivers, from
crashes to drop in performance.

Instead of that we change the algorithm to determine the possible
image formats based in the surface chroma: only those available image
formats with same chroma are exposed as possible raw caps.

Do this is important to avoid performance degrading in raw sinks
which doesn't handle NV12 but it does YV12 or I420.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/381>

4 years agolibs: util: h265: use common parser API to get vaapi profiles.
He Junyan [Fri, 10 Jul 2020 09:05:38 +0000 (17:05 +0800)]
libs: util: h265: use common parser API to get vaapi profiles.

We can reuse H265 parser's API to recognize the correct profile and
then just need to convert them to VAAPI profiles.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/356>

4 years agobuild: update for gl pkg-config file split
Víctor Manuel Jáquez Leal [Fri, 7 Aug 2020 14:41:49 +0000 (16:41 +0200)]
build: update for gl pkg-config file split

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/378>

4 years agoh264dec: mark remaining frames as unreference before exec_picture_refs_modification
Xu Guangxin [Thu, 6 Aug 2020 04:51:27 +0000 (12:51 +0800)]
h264dec: mark remaining frames as unreference before exec_picture_refs_modification

8.2.4.2 required this. Some clips will crash if we do not fill the reference list like this.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/376>

4 years agolibs: encoder: H265: Enable Main 12 profile support.
He Junyan [Fri, 31 Jul 2020 10:22:46 +0000 (18:22 +0800)]
libs: encoder: H265: Enable Main 12 profile support.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/375>

4 years agovideo format: Fix P012_LE's chrome type typo.
He Junyan [Fri, 31 Jul 2020 11:17:39 +0000 (19:17 +0800)]
video format: Fix P012_LE's chrome type typo.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/375>

4 years agogstvaapiminiobject.c: fix clang 10 warnings
Jordan Petridis [Tue, 4 Aug 2020 18:15:01 +0000 (21:15 +0300)]
gstvaapiminiobject.c:   fix clang 10 warnings

the typesystem checks in g_atomic_pointer_compare_and_exchange
seem to trigger some false positives with clang 10

similar to gstreamer!584

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/377>

4 years agolibs: window: wayland: destroy all wayland buffers during finalize
Michael Olbrich [Fri, 31 Jul 2020 09:07:23 +0000 (11:07 +0200)]
libs: window: wayland: destroy all wayland buffers during finalize

Some buffers and the associated FrameState state may still be pending at
that point. If the wayland connection is shared, then messages for the
buffer may still arrive. However, the associated event queue is already
deleted. So the result is a crash.

With a private connection the associated memory is leaked instead.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agotest: vaapicontext: fix draw callback with multiple videos
Michael Olbrich [Thu, 18 Jun 2020 18:25:18 +0000 (20:25 +0200)]
test: vaapicontext: fix draw callback with multiple videos

The callback is called for both windows. So make sure that
gst_video_overlay_set_render_rectangle() is called for the correct one.
Otherwise, the left video will be randomly moved behind the right video.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agotest: vaapicontext: support wayland display
Michael Olbrich [Fri, 19 Jun 2020 07:23:52 +0000 (09:23 +0200)]
test: vaapicontext: support wayland display

On Wayland, The whole gtk window is one Wayland surface. So
gtk_widget_get_window() must be called on the top-level widget.
For any other widget the following gdk_window_ensure_native() may create a
new top-level Wayland surface that is never visible.

As a result, the coordinates passed to
gst_video_overlay_set_render_rectangle() must be relativ to the top-level
window. Otherwise the video is placed incorrectly.

Original-Patch-By: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agotest: vaapicontext: use playbin to test files
Víctor Manuel Jáquez Leal [Fri, 1 Dec 2017 19:18:28 +0000 (20:18 +0100)]
test: vaapicontext: use playbin to test files

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agotest: vaapicontext: add PLAY and NULL buttons
Víctor Manuel Jáquez Leal [Wed, 29 Nov 2017 10:11:39 +0000 (11:11 +0100)]
test: vaapicontext: add PLAY and NULL buttons

They only appear when only one sink is instanciated and their purpose
is to test the NULL-PLAY use case in context sharing.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agolibs: wayland: update the opaque region in set_render_rect
Michael Olbrich [Fri, 19 Jun 2020 19:26:52 +0000 (21:26 +0200)]
libs: wayland: update the opaque region in set_render_rect

gst_vaapi_window_wayland_set_render_rect() may be called from an arbitrary
thread. That thread may be responsible for making the window visible.

At that point another thread will block in gst_vaapi_window_wayland_sync()
because the frame callback will not be called until the window is visible.

If that happens, then acquiring the display lock in
gst_vaapi_window_wayland_set_render_rect() would result in a deadlock.

Cache the size of the opaque rectangle separately and create the opaque
region right before applying it to the surface.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agolibs: window: implements gst_vaapi_window_set_render_rectangle
Hyunjun Ko [Fri, 19 Jun 2020 07:21:16 +0000 (09:21 +0200)]
libs: window: implements gst_vaapi_window_set_render_rectangle

Implements new vmethod gst_vaapi_window_set_render_rectangle,
which is doing set the information of the rendered rectangle set by
user.
This is necessary on wayland at least to get exact information of
external surface.

And vaapisink calls this when gst_video_overlay_set_render_rectangle is
called.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agovaapisink: implements gst_vaapisink_wayland_create_window_from_handle()
Hyunjun Ko [Wed, 8 Nov 2017 04:23:39 +0000 (13:23 +0900)]
vaapisink: implements gst_vaapisink_wayland_create_window_from_handle()

Implements gst_vaapisink_wayland_create_window_from_handle() to support
using external wl_surface.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agolibs: wayland: implement video overlay API
Michael Olbrich [Fri, 19 Jun 2020 07:11:20 +0000 (09:11 +0200)]
libs: wayland: implement video overlay API

The Wayland sub-surfaces API is used to embed the video into an application
window.

See Appendix A. Wayland Protocol Specification as the following.

"""
The aim of sub-surfaces is to offload some of the compositing work
within a window from clients to the compositor. A prime example is
a video player with decorations and video in separate wl_surface
objects.

This should allow the compositor to pass YUV video buffer processing to
dedicated overlay hardware when possible.
"""

Added new method gst_vaapi_window_wayland_new_with_surface()

Original-Patch-By: Víctor Manuel Jáquez Leal <vjaquez@igalia.com>
                   Zhao Halley <halley.zhao@intel.com>
                   changzhix.wei@intel.com
                   Hyunjun Ko <zzoon@igalia.com>

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agodoc: libs: wayland: add 'transfer full' to the returnvalye of gst_vaapi_window_waylan...
Michael Olbrich [Fri, 19 Jun 2020 19:54:52 +0000 (21:54 +0200)]
doc: libs: wayland: add 'transfer full' to the returnvalye of gst_vaapi_window_wayland_new

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agolibs: display: always call close_display()
Michael Olbrich [Thu, 11 Jun 2020 06:25:57 +0000 (08:25 +0200)]
libs: display: always call close_display()

All close_display() have their own checks for use_foreign_display and only
destroy locally created objects in that case.
Without this objects other than the actuall foreign display itself are
leaked.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/342>

4 years agovideo-format: Add the missing P012_LE into GST_VAAPI_FORMATS_ALL.
He Junyan [Thu, 30 Jul 2020 15:37:10 +0000 (23:37 +0800)]
video-format: Add the missing P012_LE into GST_VAAPI_FORMATS_ALL.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/373>

4 years agoplugins: add gst_vaapi_caps_set_width_and_height_range()
Víctor Manuel Jáquez Leal [Mon, 25 May 2020 15:02:26 +0000 (17:02 +0200)]
plugins: add gst_vaapi_caps_set_width_and_height_range()

This utility function is called internally by
gst_vaapi_build_caps_from_formats() and can be used outside.

This function sets frame size and framerates ranges.

Also gst_vaapi_build_caps_from_formats() is simplified.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/374>

4 years agolibs: decoder: fix a crash issue when get_surface_formats.
He Junyan [Fri, 31 Jul 2020 07:27:38 +0000 (15:27 +0800)]
libs: decoder: fix a crash issue when get_surface_formats.

Some context does not report any valid format that we can support.
For example, the HEVC 444 12 bits decoder context, all the formats
it reports is not supported now, which make the formats list a NULL
array. We should check that pointer before we use it.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/372>

4 years agolibs: encoder: h265: choose the profile based on allowed list.
He Junyan [Fri, 3 Jul 2020 11:28:28 +0000 (19:28 +0800)]
libs: encoder: h265: choose the profile based on allowed list.

We can decide the profile in ensure_profile(), based on allowed list
passed by the encode. We also need to check whether the entrypoint is
available. Once it is decided, no need to check the hw entrypoint
them again.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/349>

4 years agoplugins: encode: h265: set all allowed profiles to encoder.
He Junyan [Wed, 29 Jul 2020 14:05:41 +0000 (22:05 +0800)]
plugins: encode: h265: set all allowed profiles to encoder.

We should collect all allowed profiles and pass them to the inside
encoder, rather than just calculate the max profile idc.
The allowed profiles should also be supported by the HW.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/349>

4 years agolibs: display: Add a helper function to get profiles by codec.
He Junyan [Wed, 29 Jul 2020 14:32:55 +0000 (22:32 +0800)]
libs: display: Add a helper function to get profiles by codec.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/349>

4 years agoplugins: encode: h265: collect all allowed profiles to encoder.
He Junyan [Thu, 2 Jul 2020 17:28:28 +0000 (01:28 +0800)]
plugins: encode: h265: collect all allowed profiles to encoder.

We should collect all allowed profiles and pass them to the inside
encoder, rather than just calculate the max profile idc.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/349>

4 years agolibs: encoder: h265: modify set_max_profile to set_allowed_profiles.
He Junyan [Thu, 2 Jul 2020 16:53:31 +0000 (00:53 +0800)]
libs: encoder: h265: modify set_max_profile to set_allowed_profiles.

In h265, bigger profile idc may not be compatible with the small profile
idc. And more important, there are multi profiles with the same profile
idc. Such as main-422-10, main-444 and main-444-10, they all have profile
idc 4.
So recording the max profile idc is not enough, the encoder needs to know
all allowed profiles when deciding the real profile.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/349>

4 years agolibs: encoder: h265: No need to check hw_max_profile.
He Junyan [Thu, 2 Jul 2020 15:33:31 +0000 (23:33 +0800)]
libs: encoder: h265: No need to check hw_max_profile.

In h265, higher profile idc number does not mean better compression
performance and may be not compatible with the lower profile idc.
So, it is not suitable to find the heighest idc for hw to ensure the
compatibility.

On the other side, when the entrypoint of the selected profile is valid,
it means the hw really support this profile, no need to check it again.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/349>

4 years agovaapipostproc: early return if fixate srcpad caps fails
Víctor Manuel Jáquez Leal [Wed, 8 Apr 2020 17:41:09 +0000 (19:41 +0200)]
vaapipostproc: early return if fixate srcpad caps fails

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/371>

4 years agovaapipluginutil: simplify gst_vaapi_find_preferred_caps_feature()
Víctor Manuel Jáquez Leal [Wed, 29 Jul 2020 11:39:44 +0000 (13:39 +0200)]
vaapipluginutil: simplify gst_vaapi_find_preferred_caps_feature()

Generalize the way how the preferred color format is chosen. Also
use new GStreamre API as syntatic sugar.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/370>

4 years agolibs: profile: Use get_codec_from_caps to get codec type.
He Junyan [Wed, 29 Jul 2020 06:22:18 +0000 (14:22 +0800)]
libs: profile: Use get_codec_from_caps to get codec type.

There is no need to get a profile from the caps and then convert
that profile into codec type. We can get the codec type by caps's
name easily.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/358>

4 years agolibs: profile: h265: Fix return value of from_codec_data_h265.
He Junyan [Sun, 12 Jul 2020 11:42:40 +0000 (19:42 +0800)]
libs: profile: h265: Fix return value of from_codec_data_h265.

profile_from_codec_data_h265() returns wrong GstVaapiProfile for h265.
The codec data of caps contain the profile IDC, but the mapping between
profile IDC and GstVaapiProfile is wrong.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/358>

4 years agoplugins: encode: vp9: Implement vp9's allowed_profiles() func.
He Junyan [Mon, 6 Jul 2020 16:46:23 +0000 (00:46 +0800)]
plugins: encode: vp9: Implement vp9's allowed_profiles() func.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/188>

4 years agoplugin: util: rename h26x_encoder_get_profiles_from_caps().
He Junyan [Wed, 8 Jul 2020 08:30:17 +0000 (16:30 +0800)]
plugin: util: rename h26x_encoder_get_profiles_from_caps().

Change its name to encoder_get_profiles_from_caps(). Other codecs such
as VP9 also needs to use this function.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/188>

4 years agolibs: utils: vpx: Add utils vpx to handle VP8/9 misc things.
He Junyan [Mon, 6 Jul 2020 15:35:12 +0000 (23:35 +0800)]
libs: utils: vpx: Add utils vpx to handle VP8/9 misc things.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/188>

4 years agolibs: display, context: handle broken jpeg decoder for i965 driver
Víctor Manuel Jáquez Leal [Wed, 29 Jul 2020 08:17:31 +0000 (10:17 +0200)]
libs: display, context: handle broken jpeg decoder for i965 driver

JPEG decoding in i965 driver is pretty much broken, and the driver is
deprecated which mean authors only accept trivial fixes.

Surfaces for JPEG decoder context in i965 only handle IMC3[1] color
format which is not a common format in GStreamer. It can export it to
I420 at mapping raw bytes, but DMABuf exporting is problematic.

This patch artificially adds NV12 to the context format list when it's
JPEG decoder for i965 and force the usage of old VA-API for surface
creation without specifying color format. Also it artificially
disables the DMABuf announcement.

1. https://docs.microsoft.com/en-us/windows/win32/medfound/recommended-8-bit-yuv-formats-for-video-rendering#420-formats-16-bits-per-pixel

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/369>

4 years agolibs: context: change function to internal code style
Víctor Manuel Jáquez Leal [Wed, 29 Jul 2020 10:02:50 +0000 (12:02 +0200)]
libs: context: change function to internal code style

Instead of a getter the function `get_preferred_format()` to
`ensure_preferred_format()` which aligns to the code style.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/369>

4 years agovaapidecode: always merge profile caps in sink caps
Víctor Manuel Jáquez Leal [Tue, 28 Jul 2020 18:00:09 +0000 (20:00 +0200)]
vaapidecode: always merge profile caps in sink caps

This commit fixes a regression of e962069d, where if the profile's
caps doesn't have a caps profile, it's ignored.

This patch add a conditional jump if the caps doesn't have a profile
field to merge it.

Fixes: #271
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/368>

4 years agobuild: request libdrm >= 2.4.98 and fallback
Víctor Manuel Jáquez Leal [Tue, 28 Jul 2020 10:22:40 +0000 (12:22 +0200)]
build: request libdrm >= 2.4.98 and fallback

Fixes: #270
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/367>

4 years agovaapidecode: dma caps only use reported color format
Víctor Manuel Jáquez Leal [Mon, 18 May 2020 15:32:27 +0000 (17:32 +0200)]
vaapidecode: dma caps only use reported color format

This fix pipelines without vaapipostproc after vaapi decoder, such as

gst-launch-1.0 filesrc location=~/file.mp4 ! parsebin ! vaapih264dec ! glimagesink

On EGL platforms, so DMABuf is used.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/366>

4 years agovaapidecode: use allowed srcpad caps for caps query
Víctor Manuel Jáquez Leal [Fri, 7 Feb 2020 16:10:45 +0000 (17:10 +0100)]
vaapidecode: use allowed srcpad caps for caps query

Instead of using just the template caps use the current allowed
srcpad caps, which is created considering the current decoder
context.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/366>

4 years agovaapidecode: build allowed srcpad caps from va context
Víctor Manuel Jáquez Leal [Wed, 22 Jan 2020 16:41:28 +0000 (17:41 +0100)]
vaapidecode: build allowed srcpad caps from va context

Instead of generating allowed srcpad caps with generic information,
now it takes the size an formats limits from the decoder's context.

This is possible since srcpad caps are generated after the internal
decoder is created.

The patch replaces gst_vaapi_decoder_get_surface_formats() with
gst_vaapi_decoder_get_suface_attributes().

From these attributes, formats are only used for VASurface memory
caps feature. For system memory caps feature, the old
gst_vaapi_plugin_get_allowed_srcpad_caps() is still used, since
i965 jpeg decoder cannot deliver mappable format for gstreamer.
And for the other caps features (dmabuf and texture upload) the
same static list are used.

This patch also adds DMABuf caps feature only if the context
supports that memory type. Nonetheless, we keep the pre-defined
formats since they are the subset of common derive formats formats
supported either by amd/gallium and both intel drivers, since,
when exporting the fd through vaAcquireBufferHandle()/
vaReleaseBufferHandle(), the formats of the derivable image cannot
be retriebable from the driver. Later we'll use the attribute
formats for the DMABuf feature too, when the code be ported to
vaExportSurfaceHandle().

Finally, the allowed srcpad caps are removed if the internal decoder
is destroyed, since context attribues will change.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/366>

4 years agovaapidecode: reorder src caps template
Víctor Manuel Jáquez Leal [Fri, 7 Feb 2020 15:50:52 +0000 (16:50 +0100)]
vaapidecode: reorder src caps template

Since negotiation depends on caps order, first is VA, then DMABuf,
later GLUploadTexture (deprecated) and finally raw.

Also, for decoders, the possible available color formats for DMABuf
is extended to all the possible VA color formats.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/366>

4 years agolibs: window: wayland: use dmabuf protocol if available
Michael Olbrich [Wed, 22 Jul 2020 08:01:41 +0000 (10:01 +0200)]
libs: window: wayland: use dmabuf protocol if available

Currently vaGetSurfaceBufferWl() is used to create wayland buffers.
Unfortunately this is not implemented by the 'media-driver' and Mesa VA-API
drivers. And the implementation provided by 'intel-vaapi-driver' is not
compatible with a Wayland server that uses the iris Mesa driver.

So create the Wayland buffers manually with the zwp_linux_dmabuf_v1 wayland
protocol. Formats and modifiers supported by the Wayland server are taken
into account. If necessary, VPP is enabled to convert the buffer into a
supported format.

Fall back to vaGetSurfaceBufferWl() if creating buffers via dambuf protocol
fails.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/346>

4 years agolibs: window: allow choosing the format for the vpp pool
Michael Olbrich [Tue, 21 Jul 2020 08:03:19 +0000 (10:03 +0200)]
libs: window: allow choosing the format for the vpp pool

Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/346>

4 years agovideo-format: add DRM formats to the mapping table
Michael Olbrich [Sun, 28 Jun 2020 15:42:29 +0000 (17:42 +0200)]
video-format: add DRM formats to the mapping table

This will be needed for the DMABuf protocol support to map DRM formats to
vaapi and gstreamer formats.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/346>

4 years agolibs: display: wayland: add basic dmabuf protocol support
Michael Olbrich [Wed, 22 Jul 2020 07:36:18 +0000 (09:36 +0200)]
libs: display: wayland: add basic dmabuf protocol support

This is just the basic infrastructure. Hook up the interface and collect
all supported formats.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/346>

4 years agolibs: window: wayland: wait for configure before committing the first buffer
Michael Olbrich [Mon, 6 Jul 2020 07:59:40 +0000 (09:59 +0200)]
libs: window: wayland: wait for configure before committing the first buffer

Committing the first buffer for a surface must not be done before
ack_configure() has been sent for the xdg_surface.

With weston, the commit will fail with "error 3: xdg_surface has never been
configured".

Wait in gst_vaapi_window_wayland_show() until configure is done to avoid
this.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/346>

4 years agolibs: encoder: h265: set no P frame automatically.
He Junyan [Wed, 1 Jul 2020 06:50:51 +0000 (14:50 +0800)]
libs: encoder: h265: set no P frame automatically.

The double reference lists may be required by drivers and there should
be no P frames in the of stream. The old way of converting P frames to
B frames is by setting `low-delay-b` property, which is unconvenient
and has bad user experience, since most of the users do not know when
to set this property, and if it is not set correctly, the encoding
pipeline fails or even hangs on some platforms. VA driver now provides
a attribute to query whether both reference lists must be un-NULL for
a profile/entrypoint pair.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/284>

4 years agolibs: encoder: h265: Deprecate the low-delay-b property.
He Junyan [Wed, 15 Apr 2020 08:26:55 +0000 (16:26 +0800)]
libs: encoder: h265: Deprecate the low-delay-b property.

In HEVC, P and B definitions are different from AVC: P frames have
just one reference list and so 1 MV, while B frames have two reference
lists and so 2 MVs. No matter B or P, ist reference lists can contain
forward/backward reference. So P and B can both have bi-directions
dependency, the difference is just their reference list
number (i.e. MV number). This is different from the AVC.

The *low delay b mode* refers to a special HEVC mode, in which the
stream just contain I and B frames, without P frames, and all B frames
only have forward direction dependencies (i.e. all inter frames have 2
reference lists but no backward reference in both lists).  This is
similar to AVC I/P mode, but changing the P to the forward dependent
B.

The `low-delay-b` property is now just used to simply convert all P
frames to B frames when driver does not support P frames (so both
reference lists have the same references frames). This is a little
different from the meaning of low delay b mode (the two ref lists may
have the different reference frames). And the driver now can report
whether it supports P frames correctly, so there is no need to use
this property and deprecate it.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/284>

4 years agopostproc: reconfigure after changing cropping values
Marc Leeman [Fri, 24 Jul 2020 10:54:31 +0000 (12:54 +0200)]
postproc: reconfigure after changing cropping values

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/365>

4 years agoplugin: encode: Add static caps for template documentation.
He Junyan [Thu, 9 Jul 2020 05:49:29 +0000 (13:49 +0800)]
plugin: encode: Add static caps for template documentation.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: vp9: Use the dynamically built src template caps.
He Junyan [Wed, 8 Jul 2020 11:03:14 +0000 (19:03 +0800)]
plugin: encode: vp9: Use the dynamically built src template caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: vp8: Use the dynamically built src template caps.
He Junyan [Wed, 8 Jul 2020 11:02:45 +0000 (19:02 +0800)]
plugin: encode: vp8: Use the dynamically built src template caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: jpeg: Use the dynamically built src template caps.
He Junyan [Wed, 8 Jul 2020 11:02:23 +0000 (19:02 +0800)]
plugin: encode: jpeg: Use the dynamically built src template caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: mpeg2: Use the dynamically built src template caps.
He Junyan [Wed, 8 Jul 2020 11:00:39 +0000 (19:00 +0800)]
plugin: encode: mpeg2: Use the dynamically built src template caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: h265: Use the dynamically built src template caps.
He Junyan [Wed, 8 Jul 2020 10:59:18 +0000 (18:59 +0800)]
plugin: encode: h265: Use the dynamically built src template caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: h264: Use the dynamically built src template caps.
He Junyan [Wed, 8 Jul 2020 10:57:26 +0000 (18:57 +0800)]
plugin: encode: h264: Use the dynamically built src template caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: encode: Store the coded caps in type's init data.
He Junyan [Wed, 8 Jul 2020 10:46:58 +0000 (18:46 +0800)]
plugin: encode: Store the coded caps in type's init data.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugin: util: add helper function build_template_coded_caps_by_codec()
He Junyan [Wed, 8 Jul 2020 10:30:00 +0000 (18:30 +0800)]
plugin: util: add helper function build_template_coded_caps_by_codec()

Like build_template_raw_caps_by_codec(), this function can detect and
build the caps for specified codec based on the query of the profiles.
The result is coded caps such as video/x-h265, video/x-h264. The result
can be used as the template of encode's src or decode's sink.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agoplugins: utils: rename build_template_caps_by_codec.
He Junyan [Tue, 7 Jul 2020 09:16:41 +0000 (17:16 +0800)]
plugins: utils: rename build_template_caps_by_codec.

Rename the function build_template_caps_by_codec() to the name of
build_template_raw_caps_by_codec(). It can be used to collect all
raw video formats for encode's sink and decode's src.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/354>

4 years agovaapidecode: merge common profiles before setting size range
Víctor Manuel Jáquez Leal [Tue, 21 Jul 2020 18:14:57 +0000 (20:14 +0200)]
vaapidecode: merge common profiles before setting size range

The synthetic profiles, such as H264 baseline, H265 intra, etc. are
added at the end of processing all available VA profiles. This
generated an non-optimal caps for negotiation, since the synthetic
profiles don't have frame size ranges.

This patch adds those possible synthetic profiles when the associated
profile is processed, with its frame size ranges.

Now allowed sink caps are simpler.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/363>

4 years agoplugin: util: Add the missing DMA buffer input in template caps.
He Junyan [Tue, 21 Jul 2020 14:05:08 +0000 (22:05 +0800)]
plugin: util: Add the missing DMA buffer input in template caps.

We pass the wrong parameter to gst_vaapi_build_caps_from_formats()
and lose the DMA feature in caps.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/362>

4 years agolibs: decoder: H265: Add MAIN_12 profile supporting.
He Junyan [Tue, 14 Jul 2020 10:13:56 +0000 (18:13 +0800)]
libs: decoder: H265: Add MAIN_12 profile supporting.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/359>

4 years agovideo-format: Add P012_LE format.
He Junyan [Thu, 9 Jul 2020 15:07:38 +0000 (23:07 +0800)]
video-format: Add P012_LE format.

It can be used as HEVC YUV_4:2:0 12bits stream's decoder output, and
also can be used as the input format for encoding HEVC YUV_4:2:0 12bits
stream.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/359>

4 years agoplugin: decode: correct ensure_allowed_sinkpad_caps's caps.
He Junyan [Thu, 16 Jul 2020 16:45:53 +0000 (00:45 +0800)]
plugin: decode: correct ensure_allowed_sinkpad_caps's caps.

The decode allowed caps returned by ensure_allowed_sinkpad_caps()
contains all profiles of the whole VAAPI, like:
  image/jpeg, width=(int)[ 0, 1638 4 ], height=(int)[ 0, 16384 ];
  video/mpeg, mpegversion=(int)2, profile=(string){ simple, main },
  width=(int)[ 0, 2048 ], height=(int)[ 0, 2048 ]; video/x-h264,
  profile=(string){ main, high, constrained-baseline }, width=(int)[ 0,
  4096 ], height=(int)[ 0, 4096 ]; video/x-h264, profile=(string){
  constrained-high, progressive-high, baseline }; video/x-h265,
  profile=(string){ main, main-intra }, width=(int)[ 0, 8192 ],
  height=(int)[ 0, 8192 ]; video/x-vp8, width=(int)[ 0, 4096 ],
  height=(int)[ 0, 4096 ]; video/x-wmv, wmvversion=(int)3,
  format=(string)WVC1, profile=(string)advanced, width=(int)[ 0, 3840 ],
  height=(int)[ 0, 3840 ]; video/x-wmv, wmvversion=(int)3,
  profile=(string){ simple, main }, width=(int)[ 0, 3840 ],
  height=(int)[ 0, 3840 ]

Which is verbose and may have latent problems. It should only contains
the profiles belong to its codec type. For example, h265 should only
return:
  video/x-h265, profile=(string){ main, main-intra },
  width=(int)[ 0, 8192 ], height=(int)[ 0, 8192 ]

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/361>