Seungha Yang [Mon, 2 Aug 2021 08:46:37 +0000 (17:46 +0900)]
examples: playback-test: Fix for UI resize scenario on Windows
Application needs to notify videosink element of video widget resize
via gst_video_overlay_set_render_rectangle() since WM_SIZE event
wouldn't be notified.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1115>
Olivier Crête [Thu, 16 Apr 2020 03:41:52 +0000 (23:41 -0400)]
audio: Merge simd libs into the main one
Actually extract the .o objects from the convience libraries and put
them into the main one. Without this, they will just be referenced by
the .pc file, but it will be unusable because they are not installed.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1122>
Jordan Petridis [Tue, 12 Oct 2021 19:09:35 +0000 (22:09 +0300)]
ci: have the fedora docker build depend on the trigger job
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1120>
Nirbheek Chauhan [Tue, 12 Oct 2021 21:28:29 +0000 (02:58 +0530)]
meson: Remove duplicate definition of 'examples' option
This is a hard error starting with Meson 0.60.0rc1, due to
https://github.com/mesonbuild/meson/pull/9184#issuecomment-
941603358
The upstream change might get reverted before release, but we should
of course fix this regardless.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1127>
Olivier Crête [Fri, 10 Sep 2021 22:03:55 +0000 (18:03 -0400)]
rtopuspay: Set marker bit inside RTP packet too
At the end of a talk spurt, not only set the marker flag on the
GstBuffer, but also set the bit inside the RTP header as recommended
by the RFC.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1124>
Bastien Nocera [Mon, 11 Oct 2021 12:27:24 +0000 (14:27 +0200)]
gtksink: Fetch the default "widget" value in the docs
There's really no interesting "widget" value that could be shown in the
docs, so use the GST_PARAM_DOC_SHOW_DEFAULT flag to avoid showing
another value.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Bastien Nocera [Mon, 11 Oct 2021 08:22:41 +0000 (10:22 +0200)]
gtksink: Avoid errors fetching widget property
Avoid errors when fetching the "widget" property and GTK initialisation
fails, such as when running in a non-graphical environment.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 17:27:30 +0000 (13:27 -0400)]
gtksink: Return reference to GtkWidget in the acquire function
This should ensure thread safety.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Thu, 3 Sep 2020 22:27:19 +0000 (18:27 -0400)]
gtkglsink: Add rotate-method property
This mostly just takes code out of glimagesink and applies it here.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 29 Aug 2017 21:21:05 +0000 (17:21 -0400)]
glutils: Export affine transformation functions for gtkglsink
Also remove duplicated copy of those functions from the gl plugin
With contributions from Bastien Nocera
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 19:55:15 +0000 (15:55 -0400)]
glvideoflip: Replace GstVideoFlipMethod -> GstVideoOrientationMethod
It's the same enum, just drop the renamed copy. But keep the registered
GEnum as it is part of the API.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 19:51:36 +0000 (15:51 -0400)]
glvideoflip: Use the API to parse the image orientation
This will reduce the code duplication a little.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 19:51:13 +0000 (15:51 -0400)]
glimagesink: Use the API to parse the image orientation
This will reduce the code duplication a little.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 19:50:59 +0000 (15:50 -0400)]
video: Add API to parse the image orientation from a GstTagList
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 19:39:29 +0000 (15:39 -0400)]
glimagesink: Replace GstGLRotateMethod with GstVideoOrientationMethod
It's the same enum, just drop the renamed copy. But keep the registered
GEnum as it is part of the API.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Olivier Crête [Tue, 4 May 2021 19:28:25 +0000 (15:28 -0400)]
video: Put nicer documentation in GstVideoOrientationMethod
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1088>
Sebastian Dröge [Tue, 12 Oct 2021 16:01:37 +0000 (19:01 +0300)]
splitmuxsink: Don't assert on the input side if no GOP is available when shutting down
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1119>
Benjamin Gaignard [Fri, 1 Oct 2021 13:10:14 +0000 (15:10 +0200)]
move-script: fix type in dateutil package name
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1080>
Thibault Saunier [Tue, 12 Oct 2021 13:10:05 +0000 (10:10 -0300)]
tests: Mark check.gstreamer.gst_gstbin.test_watch_for_state_change flaky
See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/778
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Mon, 11 Oct 2021 15:03:02 +0000 (12:03 -0300)]
ges:tests: Mark check.gst-plugins-base.validate.giosrc.read-growing-file as flaky
See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/776
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Mon, 11 Oct 2021 14:45:50 +0000 (11:45 -0300)]
ges:tests: Mark ges 'check_edit_in_frames_with_framerate_mistmatch as flaky
See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/775
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Mon, 11 Oct 2021 14:05:36 +0000 (11:05 -0300)]
tests:webrtcbin: Make known flaky webrtcbin as such
See https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/773
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Fri, 8 Oct 2021 15:09:47 +0000 (12:09 -0300)]
validate:launcher: Do not dump on failure test we will retry
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Fri, 8 Oct 2021 14:54:22 +0000 (11:54 -0300)]
gst-env: Ignore SIGINT when running on the CI server
We get spurious sigint and this is just a small temporary workaround
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Sun, 26 Sep 2021 02:51:52 +0000 (23:51 -0300)]
tests: aggregator: Take `TIMEOUT_FACTOR` env var into account
This env var is set in the CI so we grow the timeout as required
(when running in valgrind for example).
Trying to avoid hitting wrong timeout in valgrind job like in:
https://gitlab.freedesktop.org/tpm/gstreamer/-/jobs/
14009456
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Fri, 8 Oct 2021 13:48:30 +0000 (10:48 -0300)]
ci: Split base and core valgrind on different runners
Those are very slow compared to other jobs and it should mitigate the problem
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Fri, 8 Oct 2021 13:44:50 +0000 (10:44 -0300)]
tests: Blacklist tests that are currently producing many CI failures
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Fri, 8 Oct 2021 12:09:21 +0000 (09:09 -0300)]
validate:launcher: Allow more flexible handling of flaky tests
And retry twice flaky tests
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Fri, 8 Oct 2021 00:28:12 +0000 (21:28 -0300)]
validate:launcher: Fix progress reporting with --n-runs
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Thu, 7 Oct 2021 22:43:58 +0000 (19:43 -0300)]
validate:tests: Mark rtsp tests as generally flaky
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Thu, 7 Oct 2021 22:43:30 +0000 (19:43 -0300)]
ges:tests: Mark nested timelines as generally flaky
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Thu, 7 Oct 2021 22:42:10 +0000 (19:42 -0300)]
validate:launcher: Support retry on failure when using --n-runs
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Thu, 7 Oct 2021 22:41:07 +0000 (19:41 -0300)]
validate:launcher: Fix potential invalid index access
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Thibault Saunier [Wed, 6 Oct 2021 20:21:58 +0000 (17:21 -0300)]
validate:launcher: Fix adding logs to xunit
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1113>
Tim-Philipp Müller [Sun, 26 Sep 2021 17:05:31 +0000 (18:05 +0100)]
gst-play: pick up minus and plus also from navigation events
Makes it easier to test playback rate changes with the video
window being in focus.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/928>
Nirbheek Chauhan [Tue, 12 Oct 2021 06:13:16 +0000 (11:43 +0530)]
libsoup: Bump to 2.74 to fix gssapi build failure on macOS
See: https://gitlab.gnome.org/GNOME/libsoup/-/merge_requests/255
We also need to disable sysprof by default because it only builds on
native Linux. If someone really wants it, they can enable the option
on the command-line by passing -Dlibsoup:sysprof=enabled
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1114>
Mengkejiergeli Ba [Wed, 22 Sep 2021 06:50:40 +0000 (14:50 +0800)]
msdkenc: fix vp9enc initialization fail
MediaSDK does not support to handle extbuff with id
MFX_EXTBUFF_VIDEO_SIGNAL_INFO for mjpegenc and vp9enc. Hence, need to
exclude mjpeg and vp9 when passing color properties to MediaSDK during
msdkenc initialization.
Fix issue: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/764
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1101>
Seungha Yang [Sun, 10 Oct 2021 08:04:13 +0000 (17:04 +0900)]
examples: d3d11: Add a desktop capture example
... with d3d11 desktop capture device provider
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1103>
Seungha Yang [Tue, 5 Oct 2021 12:49:38 +0000 (21:49 +0900)]
d3d11: Add device provider for d3d11desktopdupsrc
... and add support for multi-GPU/multi-monitor
By using newly added "monitor-handle" property, user can specify a
monitor to be captured via HMONITOR handle.
Fixes: https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/issues/1673
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1103>
Stéphane Cerveau [Mon, 11 Oct 2021 13:23:08 +0000 (15:23 +0200)]
ttml: fix log init
The log system should be init before calling a log
Fix regression after:
https://gitlab.freedesktop.org/gstreamer/gst-plugins-bad/-/merge_requests/2038
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1112>
Stéphane Cerveau [Wed, 29 Sep 2021 12:41:31 +0000 (14:41 +0200)]
move-mrs-script: add url option
You can give as option:
--url
https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/-/merge_requests/1
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/960>
Stéphane Cerveau [Wed, 29 Sep 2021 11:35:09 +0000 (13:35 +0200)]
move-mrs-script: add a list only options
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/960>
Víctor Manuel Jáquez Leal [Mon, 4 Oct 2021 11:30:37 +0000 (13:30 +0200)]
tests: va: Add VA buffer copy tests.
It should only work for raw buffers, but fails on dmabuf since it
should have a drm modifier.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Wed, 6 Oct 2021 13:20:50 +0000 (15:20 +0200)]
va: allocator: dmabuf: Use GstVaSurfaceCopy, if possible.
If dmabuf-based buffer to copy contains only one memory, and there are
memories available in the allocator's pool, it's possible a fast
memory copy using GstVaSurfaceCopy, regardless the drm modifier.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Tue, 5 Oct 2021 13:21:01 +0000 (15:21 +0200)]
va: allocator: Use GstVaSurfaceCopy.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Tue, 5 Oct 2021 11:36:56 +0000 (13:36 +0200)]
va: Add GstVaSurfaceCopy class.
This new class is a helper for fast/tricky copy of surfaces. First it
tries to copy using the function vaCopy in libva 1.12. If it fails, or
it's not available, a GstVaFilter is tried to be instantiated with the
allocator's parameters, and if succeed, it's used for copying the
source surface.
This is required for dmabuf surfaces with drm modifier.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Tue, 5 Oct 2021 11:21:00 +0000 (13:21 +0200)]
va: filter: Enable to pass VASurfaceID in GstVaSample.
Initially GstVaSample processed its GstBuffer member to get the
VASurfaceID. But it might cases where we already have the VASurfaceID
to process by the filter.
This patch enables the possibility to pass the surfaces rather than
the buffers. In order to validate the surfaces a function to check
surfaces were added.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Thu, 7 Oct 2021 19:51:55 +0000 (21:51 +0200)]
va: Split VA memory handling in different files.
Take out the VA memory wrappers from gstvallocator.c to an external
file exposing the functions.
This is going to be needed for the copy helper object.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Tue, 5 Oct 2021 04:54:25 +0000 (06:54 +0200)]
va: allocator: Add missing header file.
Added stdint.h because uintptr_t is used.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Víctor Manuel Jáquez Leal [Mon, 4 Oct 2021 19:31:53 +0000 (21:31 +0200)]
va: display: Add gst_va_display_has_vpp()
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1023>
Benjamin Gaignard [Thu, 30 Sep 2021 12:59:31 +0000 (14:59 +0200)]
v4l2codecs: Align v4l2-controls header with kernel 5.15-rc3
Update v4l2-controls to be aligned with kernel 5.15-rc3.
Fix VP8 decoder to use the correct field name.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1081>
Jordan Petridis [Wed, 6 Oct 2021 10:25:12 +0000 (13:25 +0300)]
ci: remove outdated rule for gstreamer-full-minimal
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1065>
Jordan Petridis [Wed, 6 Oct 2021 10:23:50 +0000 (13:23 +0300)]
ci: update rules that pointed to gst-ci subproject
The files are now moved into ci/ at the root of the repo.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1065>
Jan Schmidt [Fri, 8 Oct 2021 18:39:38 +0000 (05:39 +1100)]
uridecodebin3/urisourcebin: Reusability fixes
Improvements to uridecodebin3 and urisourcebin so that they are
reusable across a PAUSED->READY->PAUSED transition.
Disconnect and release decodebin3 request pads when urisourcebin
removes src pads.
In urisourcebin, make sure to remove src pads that are exposed
directly (raw pads and static typefind srcpads) when
cleaning up.
Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/768
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1100>
Jan Schmidt [Sat, 9 Oct 2021 14:24:44 +0000 (01:24 +1100)]
playbin3: Always register 'playbin3' element.
If the USE_PLAYBIN3=1 env var is set, we want to replace
playbin with playbin3, but separate to that, we always
want to register the 'playbin3' element so that applications
which explicitly use playbin3 work regardless of the env var.
This fixes `USE_PLAYBIN3=1 gst-validate-launcher`, for example.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1102>
Seungha Yang [Fri, 8 Oct 2021 14:07:32 +0000 (23:07 +0900)]
mediafoundation: mfvideoenc: Use DXGI adapter LUID
Make use of new DXGI adapter LUID based device context sharing.
Note that we were using DXGI adapter LUID to open MFT already.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1098>
Seungha Yang [Fri, 8 Oct 2021 13:37:20 +0000 (22:37 +0900)]
d3d11: d3d11decoder: Use DXGI adapter LUID
... instead of index of DXGI adapter.
The order of IDXGIAdapter1 enumerated via IDXGIFactory1::EnumAdapters1
can be varying even there's no rebooting in case that GPU preference order
is updated by user (for example, it can be done by using NVIDIA Control Panel
in case of multi-GPU laptop system) and eGPU is another possible case.
So, for an element which requires fixed target GPU requirement,
index based device enumeration is unreliable.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1098>
Seungha Yang [Fri, 8 Oct 2021 12:39:44 +0000 (21:39 +0900)]
d3d11: d3d11utils: Add support for DXGI Adapter LUID based D3D11 device context sharing
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1098>
Seungha Yang [Fri, 8 Oct 2021 10:41:22 +0000 (19:41 +0900)]
d3d11: d3d11device: Add gst_d3d11_device_new_{for_adapter_luid,wrapped}
* gst_d3d11_device_new_for_adapter_luid()
Used for creating D3D11 device for a DXGI adapter (i.e., GPU)
corresponding to a LUID (Locally Unique Identifier).
This method can be useful for interop with other APIs such as
Direct3D12, MediaFoundation, CUDA, etc.
* gst_d3d11_device_new_wrapped()
Allows creating a new GstD3D11Device object by using already
configured ID3D11Device. This is conceptually equivalent to
gst_gl_context_new_wrapped()
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1098>
Seungha Yang [Fri, 8 Oct 2021 08:16:02 +0000 (17:16 +0900)]
d3d11: d3d11device: Remove "allow-tearing" property
Plugin can query DXGI_FEATURE_PRESENT_ALLOW_TEARING without d3d11device
help
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1098>
Sebastian Dröge [Fri, 8 Oct 2021 18:14:52 +0000 (21:14 +0300)]
webrtcbin: Use the same promise reply structure name everywhere
This was an inconsistent mix of different names in the past. The name
has no meaning at all so let's set all to "application/x-gst-promise".
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1099>
Jan Schmidt [Mon, 4 Oct 2021 18:43:13 +0000 (05:43 +1100)]
playbin3: Avoid group deactivation deadlock.
Change locking around group deactivation to avoid deadlocks
when shutting down exactly as a buffering message arrives.
The PLAYBIN3_LOCK now protects the active field of the
source group. Everything else is still protected by the
source-group-lock.
Also properly protect group switching operations with
the PLAYBIN3_LOCK everywhere.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1049>
Vivia Nikolaidou [Fri, 8 Oct 2021 12:44:37 +0000 (15:44 +0300)]
chopmydata: Fix FIXMEs in gst_element_class_set_static_metadata
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1097>
Stéphane Cerveau [Fri, 8 Oct 2021 11:20:37 +0000 (13:20 +0200)]
FAQ: add mention to downstream cherry-pick
Give an help to cherry-pick an upstream patch to
downstream repository (old stable version)
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1095>
Seungha Yang [Tue, 5 Oct 2021 18:19:30 +0000 (03:19 +0900)]
d3d11: d3d11desktopdupsrc: Add support for non-D3D11 downstream element
By this commit, application doesn't need to configure d3d11download
element for software pipeline which will make things simpler
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1082>
Thibault Saunier [Tue, 5 Oct 2021 21:50:21 +0000 (18:50 -0300)]
validate:scenario: Allow waiting signal searching for element factory
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1064>
Marek Vasut [Fri, 5 Jun 2020 14:20:18 +0000 (16:20 +0200)]
jpegdec: Support libjpeg-turbo colorspace conversion
The libjpeg-turbo has a built-in support for performing colorspace
conversion. The performance of this conversion is much better than
doing the same separately using videoconvert. Implement support for
this conversion to RGBx/xRGB/BGRx/xBGR formats. Other formats can
be easily added later.
- The decoding of various pixel formats can be tested and compared to
non-libjpeg-turbo decoding as follows:
for gfmt in {RGB,BGR}{,x} x{RGB,BGR} ; do
echo "$gfmt"
gst-launch-1.0 -q \
videotestsrc pattern=colors ! \
video/x-raw,format=${gfmt} ! \
fakesink dump=true | \
head -n 200 | tail -n 1
gst-launch-1.0 -q --gst-plugin-path=build/ext/jpeg/ \
videotestsrc pattern=colors ! \
video/x-raw,format=${gfmt} ! \
jpegenc ! \
jpegdec ! \
video/x-raw,format=${gfmt} ! \
fakesink dump=true | \
head -n 200 | tail -n 1
done
Result looks as follows, i.e. comparable:
RGB
00000c70 (0x7f7736fbdd10): 05 33 19 05 33 26 05 33 33 05 33 40 05 33 4c 05 .3..3&.33.3@.3L.
00000c70 (0x7f389e8f7d10): 05 32 17 04 32 22 04 32 31 04 32 3e 04 32 4a 04 .2..2".21.2>.2J.
RGBx
00000c70 (0x7f79efd0ad10): cc 07 22 ff d9 07 22 ff e6 07 22 ff f3 07 22 ff .."..."..."...".
00000c70 (0x7fb6989f3d10): cd 06 22 00 d9 06 22 00 e6 06 22 00 f4 06 22 00 .."..."..."...".
BGR
00000c70 (0x7fa0a6c42d10): 05 0c 33 05 19 33 05 26 33 05 33 33 05 40 33 05 ..3..3.&3.33.@3.
00000c70 (0x7fc74165fd10): 05 08 32 04 17 32 04 22 32 04 31 32 04 3e 32 04 ..2..2."2.12.>2.
BGRx
00000c70 (0x7fbf399f1d10): 22 07 cc ff 22 07 d9 ff 22 07 e6 ff 22 07 f3 ff "..."..."..."...
00000c70 (0x7f50e3d1cd10): 22 06 cd 00 22 06 d9 00 22 06 e6 00 22 06 f4 00 "..."..."..."...
xRGB
00000c70 (0x7f0b950a2d10): ff cc 07 22 ff d9 07 22 ff e6 07 22 ff f3 07 22 ..."..."..."..."
00000c70 (0x7f4416b8dd10): 00 cd 06 22 00 d9 06 22 00 e6 06 22 00 f4 06 22 ..."..."..."..."
xBGR
00000c70 (0x7f237d74dd10): ff 22 07 cc ff 22 07 d9 ff 22 07 e6 ff 22 07 f3 ."..."..."..."..
00000c70 (0x7f095547dd10): 00 22 06 cd 00 22 06 d9 00 22 06 e6 00 22 06 f4 ."..."..."..."..
^^ ^^ ^^ ^^
Notice how the alpha channel is set to arbitrary value in case of the
libjpeg-turbo decoding into RGBx/BGRx/xRGB/xBGR pixel formats. This is
documented in libjpeg-turbo README.md as follows:
"
When using the RGBX, BGRX, XBGR, and XRGB colorspaces during decompression, the
X byte is undefined, and in order to ensure the best performance, libjpeg-turbo
can set that byte to whatever value it wishes.
"
- The interlaced num_fields=2 mjpeg stream can be generated and
tested as follows (this does require mjpegtools):
$ gst-launch-1.0 videotestsrc num-buffers=10 ! jpegenc ! multifilesink location=in%04d.jpg
$ jpeg2yuv -f 25 -I t -L 0 -j in%04d.jpg | yuv2lav -f avi -o result.avi
...
$ gst-launch-1.0 --gst-plugin-path=build/ext/jpeg/ filesrc location=result.avi ! \
avidemux ! jpegdec ! video/x-raw,format=RGBx ! videoconvert ! autovideosink
Signed-off-by: Marek Vasut <marex@denx.de>
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1091>
Sebastian Dröge [Thu, 7 Oct 2021 10:00:10 +0000 (13:00 +0300)]
rtsp-media: Unprepare suspended medias too
Previously suspended medias immediately reached the UNPREPARED state
without going through the media's unprepare() vfunc. This didn't allow
the media subclass to do any additional cleanup, and for example the
shutdown-eos property of GstRTSPMedia was ignored.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1090>
Seungha Yang [Fri, 1 Oct 2021 14:36:39 +0000 (23:36 +0900)]
tests: gstmeta: Fix failure on Windows
Windows doesn't support fork so every test will be performed in
one process. So the test_meta_custom_transform() is being
failed because "test-custom" custom meta is being used/defined in
another test test_meta_custom() as well.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1086>
Seungha Yang [Wed, 6 Oct 2021 13:06:44 +0000 (22:06 +0900)]
codecs: Use GST_VIDEO_DECODER_ERROR() only for decoding error case
The GST_VIDEO_DECODER_ERROR() should be used only for robust/error-resilient
decoding purpose. Any other error codes such as not-negotiated or flushing
should be returned without modified for upstream to be able to handle
it immediately. (for example, application might want to try other
decoder element on not-negotiated)
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1070>
Seungha Yang [Wed, 6 Oct 2021 16:54:29 +0000 (01:54 +0900)]
codecs: h264decoder: Update for remaining gboolean to GstFlowReturn port
Fix for spurious/spammy warning and wrong function return type
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1085>
Sebastian Dröge [Wed, 6 Oct 2021 15:19:29 +0000 (18:19 +0300)]
rtsp-media: Only unprepare a media if it was not already unpreparing anyway
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1083>
Nirbheek Chauhan [Thu, 23 Sep 2021 11:37:09 +0000 (17:07 +0530)]
meson: Fix gst-launch build on UWP
The APIs we need from WinMM for higher timer resolution are only
available for non-UWP apps. The winmm library itself is still
available, so we need to disable the check.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/890>
Nirbheek Chauhan [Thu, 23 Sep 2021 12:06:20 +0000 (17:36 +0530)]
meson: Fix build with -Dopenjpeg=disabled
tests/check/meson.build uses the openjpeg_dep variable
unconditionally, and the subdir_done() is useless anyway, since the
plugin is only built if openjpeg_dep.found() is true. Fixes:
..\tests\check\meson.build:23:0: ERROR: Unknown variable "openjpeg_dep".
In particular, this fixes the build on UWP since we disable openjpeg
explicitly in Cerbero when building for UWP.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1069>
Guillaume Desmottes [Fri, 1 Oct 2021 12:35:06 +0000 (14:35 +0200)]
bad: gs: update README
- add one missing dep
- change install path to match monorepo
- fix current dirs
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1008>
Ognyan Tonchev [Sun, 3 Oct 2021 21:25:23 +0000 (23:25 +0200)]
rtsp-client: make sure sessmedia will not get freed while used
handle_*_request() functions were all retrieving the session media from
the session by calling gst_rtsp_session_get_media () which is a transfer-none
call. If a session timeout happens at that time, the session media may get freed
making the pointer invalid..
Fixes #757
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1053>
Thibault Saunier [Sat, 2 Oct 2021 14:58:56 +0000 (11:58 -0300)]
validate:launcher: Sync asset only when passing --sync
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1020>
Xavier Claessens [Tue, 5 Oct 2021 15:47:17 +0000 (11:47 -0400)]
ci: Fix typo in cerbero trigger
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1056>
Xavier Claessens [Tue, 5 Oct 2021 15:20:32 +0000 (11:20 -0400)]
trigger_cerbero_pipeline.py: Do not hardcode gitlab instance
When GStreamer is forked into a private GitLab instance we should
trigger cerbero into that private instance too. Otherwise the token
won't be accepted.
Also do not hardcode the cerbero project ID because it is instance
specific.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1056>
Víctor Manuel Jáquez Leal [Fri, 17 Sep 2021 11:02:38 +0000 (13:02 +0200)]
va: Use macro rather than VAMemory feature string.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1061>
Jordan Petridis [Tue, 5 Oct 2021 20:28:57 +0000 (23:28 +0300)]
ci: remove duplicate stage key in documentation job
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Xavier Claessens [Tue, 5 Oct 2021 00:44:15 +0000 (20:44 -0400)]
ci: Remove "check allow-edit" job because gitlab does it by default
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Xavier Claessens [Tue, 5 Oct 2021 00:24:58 +0000 (20:24 -0400)]
ci: Update fedora image for cerbero trigger
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Xavier Claessens [Tue, 5 Oct 2021 12:51:08 +0000 (08:51 -0400)]
ci: Fix gst-indent path
It used to be downloaded into PATH, but we can now instead run it from
git. Also move it to top source dir instead of gstreamer subproject.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Xavier Claessens [Tue, 5 Oct 2021 00:25:54 +0000 (20:25 -0400)]
ci: Fix subprojects caching in Windows image
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Jordan Petridis [Mon, 27 Sep 2021 07:03:31 +0000 (10:03 +0300)]
ci/docker/fedora: update rustc and meson versions
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Jordan Petridis [Mon, 27 Sep 2021 06:41:10 +0000 (09:41 +0300)]
ci: port to fd.o citemplates
https://gitlab.freedesktop.org/freedesktop/ci-templates
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Jordan Petridis [Mon, 27 Sep 2021 06:42:18 +0000 (09:42 +0300)]
ci: remove duplicate keys
They are redefined/copied just below
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/931>
Nicolas Dufresne [Tue, 5 Oct 2021 13:27:50 +0000 (09:27 -0400)]
ci: Add a dummy trigger job/stage
This is to avoid triggering docker images for no reason, and bring back
running gst-indent always.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1054>
Nicolas Dufresne [Tue, 5 Oct 2021 13:06:32 +0000 (09:06 -0400)]
ci: Document the difference between the two doc jobs
Naming is hard, so I opted for a small comment to explain this one.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1054>
Sebastian Dröge [Tue, 5 Oct 2021 16:37:40 +0000 (19:37 +0300)]
rtsp-media: Also mark receive-only (RECORD) medias as prepared when unsuspending
Previously the status was only changed for other medias.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1058>
Tim-Philipp Müller [Tue, 5 Oct 2021 12:26:52 +0000 (13:26 +0100)]
docs: update "Building from Source using Meson" for mono repo
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1052>
Sebastian Dröge [Tue, 21 Sep 2021 10:37:35 +0000 (13:37 +0300)]
splitmuxsink: Keep track of the pending input GOPs in a queue
This cleans up input GOP handling and makes it possible to handle more
than 2 pending GOPs, which could happen before if keyframes are arriving
with too short of a distance between them.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1005>
Sebastian Dröge [Thu, 16 Sep 2021 16:36:27 +0000 (19:36 +0300)]
splitmuxsink: Handle frame reordering due to B frames better
Instead of assuming that the PTS of a keyframe is the lowest PTS of a
GOP, wait until the DTS has passed this PTS and take the minimum PTS up
to that point. That way the minimum PTS of a GOP can be determined, at
least for closed GOP streams. Open GOP streams still can't be handled
properly.
By knowing the minimum PTS of each GOP, keyframes can be requested at
the correct time relative to the GOP (and thus fragment) start and
fragment overflow calculations can calculate the correct durations of
the GOPs.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1005>
Sebastian Dröge [Thu, 16 Sep 2021 11:07:27 +0000 (14:07 +0300)]
splitmuxsink: Reset timecodes when resetting the sink
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1005>
Sebastian Dröge [Thu, 16 Sep 2021 10:11:20 +0000 (13:11 +0300)]
splitmuxsink: Use GST_CLOCK_STIME_NONE instead of 0 to initialize max out running time
Otherwise streams with only DTS would misbehave while it is negative.
For outputting everything immediately at EOS, use G_MAXINT64 instead
which is bigger-or-equal to any other running time.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1005>
Sebastian Dröge [Thu, 16 Sep 2021 10:02:29 +0000 (13:02 +0300)]
splitmuxsink: Only update max in/out running time if it's actually bigger
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1005>
Sebastian Dröge [Thu, 16 Sep 2021 09:53:12 +0000 (12:53 +0300)]
splitmuxsink: Only count keyframes for the reference context, consistently
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1005>
Jordan Petridis [Wed, 29 Sep 2021 12:15:03 +0000 (15:15 +0300)]
ci: Rework workflow rules to prevent duplicate pipelines
Currently gitlab spawns 2 pipelines, one for the branch and
another for the merge request.
Update our rules so that only one of them is spawned depending
on the situation.
https://docs.gitlab.com/ee/ci/yaml/index.html#switch-between-branch-pipelines-and-merge-request-pipelines
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/958>
Robert Rosengren [Fri, 24 Sep 2021 13:24:15 +0000 (15:24 +0200)]
docs/running: Describe GST_REGISTRY_MODE
Change-Id: I45e9fe90137d4a8306c3a5e4f636fa43425b978f
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/825>