platform/upstream/gstreamer.git
22 months agortpjitterbuffer: Make it more explicit that update_rtx_timers() takes ownership of...
Sebastian Dröge [Fri, 2 Sep 2022 09:20:58 +0000 (12:20 +0300)]
rtpjitterbuffer: Make it more explicit that update_rtx_timers() takes ownership of the passed in timer

It is not valid anymore afterwards and must not be used, otherwise an
already freed pointer might be used.

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

22 months agortpjitterbuffer: Don't shadow variable
Sebastian Dröge [Fri, 2 Sep 2022 09:20:30 +0000 (12:20 +0300)]
rtpjitterbuffer: Don't shadow variable

While this didn't cause any problems in this context it is simply
confusing.

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

22 months agortpjitterbuffer: Change RTX timer availability checks to assertions
Sebastian Dröge [Fri, 2 Sep 2022 09:19:26 +0000 (12:19 +0300)]
rtpjitterbuffer: Change RTX timer availability checks to assertions

It's impossible to end up in the corresponding code without a timer for
RTX packets because otherwise it would be an unsolicited RTX packet and
we would've already returned early.

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

22 months agortpjitterbuffer: Only unschedule timers for late packets if they're not RTX packets...
Sebastian Dröge [Fri, 2 Sep 2022 09:17:39 +0000 (12:17 +0300)]
rtpjitterbuffer: Only unschedule timers for late packets if they're not RTX packets and only once

Timers for RTX packets are dealt with later in update_rtx_timers(), and
timers for non-RTX packets would potentially also be unscheduled a
second time from there so avoid that.

Also don't shadow the timer variable from the outer scope but instead
make use of it directly.

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

22 months agosubprojects: update openh264 wrap to v2.3.0
Tim-Philipp Müller [Wed, 31 Aug 2022 23:14:52 +0000 (00:14 +0100)]
subprojects: update openh264 wrap to v2.3.0

.. and use tarball instead of git checkout.

Also bump ci image tags so the images cache the new tarball.

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

22 months agogst-rtsp-server: Fix pushing backlog to client
Bruce Liang [Wed, 24 Aug 2022 11:50:19 +0000 (19:50 +0800)]
gst-rtsp-server: Fix pushing backlog to client

Check back pressure of a stream transport before popping buffer from its backlog.

If the stream transport is not experiencing back pressure, the buffer can be popped from backlog and pushed to client.

Fixes:#1298

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

22 months agomeson: Subprjects can define both "plugins" and "gst_plugins"
Xavier Claessens [Fri, 2 Sep 2022 14:46:45 +0000 (10:46 -0400)]
meson: Subprjects can define both "plugins" and "gst_plugins"

If "gst_plugins" is defined we can ignore the value of legacy "plugins"
variable, subprojects could be using it for something else, which is the
case of gst-plugin-rs.

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

22 months agortsp-server: stream: Don't loop forever if binding to the multicast address fails
Sebastian Dröge [Fri, 2 Sep 2022 13:31:54 +0000 (16:31 +0300)]
rtsp-server: stream: Don't loop forever if binding to the multicast address fails

The address/port is pre-defined by the caller of the function, so
retrying is only going to loop forever.

Ideally the multicast address should be checked after allocating but
this doesn't happen currently, so it's better to error out cleanly then
to loop forever trying the same address.

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

22 months agova: Fix log message when registering H264 encoder.
Florian Zwoch [Fri, 2 Sep 2022 09:55:20 +0000 (09:55 +0000)]
va: Fix log message when registering H264 encoder.

The log message would report an error for the H264 decoder when
registering failed, but we tried to register the H264 encoder instead.

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

22 months agowebrtcbin: Allow locked mlines with no caps, as the last ones
Olivier Crête [Tue, 5 Jul 2022 21:14:37 +0000 (17:14 -0400)]
webrtcbin: Allow locked mlines with no caps, as the last ones

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

22 months agowebrtcbin: Reject creating an offer if a locked mline has no caps
Olivier Crête [Mon, 16 May 2022 22:05:25 +0000 (18:05 -0400)]
webrtcbin: Reject creating an offer if a locked mline has no caps

This avoids getting in a bunch of corner cases. We'd have to insert
a "rejected" line from the start as a place-holder to get around this,
but the rest of the code just becomes more complicated, so just
disallow it for now.

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

22 months agowebrtcbin: Store pending mid to make create-offer idempotent
Olivier Crête [Mon, 16 May 2022 21:17:13 +0000 (17:17 -0400)]
webrtcbin: Store pending mid to make create-offer idempotent

If the mid is not stored in the transceiver, but it is stored in
last_offer, then a further create-offer call will just ignore that
transceiver.

Also include unit test for ensure it doesn't regress.

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

22 months agortph265: keep delta unit flag
Patricia Muscalu [Tue, 5 Jul 2022 14:15:19 +0000 (16:15 +0200)]
rtph265: keep delta unit flag

Without this patch all buffers that pass the payloader
are marked as non-delta-unit buffers.

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

22 months agomsdkvpp: Add va memory when fixating src caps
Mengkejiergeli Ba [Thu, 26 May 2022 10:17:16 +0000 (18:17 +0800)]
msdkvpp: Add va memory when fixating src caps

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

22 months agomsdkvpp : Use va pool at linux path and system pool for windows
Mengkejiergeli Ba [Wed, 8 Jun 2022 04:31:56 +0000 (12:31 +0800)]
msdkvpp : Use va pool at linux path and system pool for windows

We use msdkvpp's own pool to allocate buffers instead of external frame
allocator mfxFrameAllocator.

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

22 months agomsdkvpp: Import buffer to msdk_surface
Mengkejiergeli Ba [Mon, 9 May 2022 10:34:38 +0000 (18:34 +0800)]
msdkvpp: Import buffer to msdk_surface

If the buffer is not msdk_buffer, we can try to directly import the
attached memory (i.e. va mem and dmabuf mem) by applying the common
uitl function: import_to_msdk_function ().

Here add a flag "from_qdata" in GstMsdkSurface to handle the cropping case,
we should avoid updating the crop values when msdk_surface is from the
memory's qdata, because the crop info from this surface is the already
updated one.

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

22 months agomsdkvpp: Add va caps at sink and src pad
Mengkejiergeli Ba [Mon, 9 May 2022 10:19:01 +0000 (18:19 +0800)]
msdkvpp: Add va caps at sink and src pad

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

22 months agomsdkenc: Apply common util func to import mem as msdk_surface
Mengkejiergeli Ba [Thu, 25 Aug 2022 09:28:36 +0000 (17:28 +0800)]
msdkenc: Apply common util func to import mem as msdk_surface

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

22 months agomsdk: Add help functions to get mfxFrameSurface1 from GstBuffer and wrap it as GstMsd...
Mengkejiergeli Ba [Fri, 27 May 2022 09:56:33 +0000 (17:56 +0800)]
msdk: Add help functions to get mfxFrameSurface1 from GstBuffer and wrap it as GstMsdkSurface

Note that the memory abstraction for system memory is for windows path.

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

22 months agomsdkenc: Use va pool on linux and system pool for windows
Mengkejiergeli Ba [Thu, 19 May 2022 11:18:21 +0000 (19:18 +0800)]
msdkenc: Use va pool on linux and system pool for windows

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

22 months agomsdkenc: Directly import dmabuf memory as mfx surface
Mengkejiergeli Ba [Thu, 19 May 2022 10:57:50 +0000 (18:57 +0800)]
msdkenc: Directly import dmabuf memory as mfx surface

When input buffer is of dmabuf memory but not a msdk buffer (i.e., the
allocator is not msdk_allocator), then we can try to get fd of this mem,
create the corresponding va surface and wrap it as mfx surface.

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

22 months agomsdkenc: Directly import va memory as mfx surface
Mengkejiergeli Ba [Thu, 21 Apr 2022 09:27:09 +0000 (17:27 +0800)]
msdkenc: Directly import va memory as mfx surface

If input buffer is of va memory, we can directly obtain va surface from
the buffer and wrap it as mfx surface.

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

22 months agomsdk: Use va libs API to get VA surface from buffer
Mengkejiergeli Ba [Thu, 3 Mar 2022 02:07:19 +0000 (10:07 +0800)]
msdk: Use va libs API to get VA surface from buffer

Direct apply gst_va_buffer_get_surface to get VASurface from a buffer,
so remove corresponding functions which are not used.

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

22 months agomeson: Call pkgconfig.generate in the loop where we declare plugins dependencies
Thibault Saunier [Thu, 1 Sep 2022 19:11:31 +0000 (15:11 -0400)]
meson: Call pkgconfig.generate in the loop where we declare plugins dependencies

Removing some copy pasted code

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

22 months agomeson: Namespace the plugins_doc_dep/libraries variables
Thibault Saunier [Thu, 1 Sep 2022 15:51:48 +0000 (11:51 -0400)]
meson: Namespace the plugins_doc_dep/libraries variables

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

22 months agocuda: meson: Fix generating GstCuda gir file against gstreamer-full
Thibault Saunier [Thu, 1 Sep 2022 14:28:13 +0000 (10:28 -0400)]
cuda: meson: Fix generating GstCuda gir file against gstreamer-full

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

22 months agomeson: Rename plugins list and make them "dependency" objects
Thibault Saunier [Wed, 31 Aug 2022 22:44:14 +0000 (18:44 -0400)]
meson: Rename plugins list and make them "dependency" objects

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

22 months agortpjitterbuffer: remove lost timer for out of order packets
Raul Tambre [Thu, 18 Aug 2022 14:08:51 +0000 (17:08 +0300)]
rtpjitterbuffer: remove lost timer for out of order packets

When receiving old packets remove the running lost timer if present.
This fixes incorrect reporting of a lost packet even if it arrived in time.

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

22 months agogstaudioconvert: doc: Fix mix-matrix example
Ádám Balázs [Wed, 31 Aug 2022 14:50:38 +0000 (14:50 +0000)]
gstaudioconvert: doc: Fix mix-matrix example

mix-matrix should contain float elements, modify the example to do so

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

22 months agogstpluginloader: Don't hang on short reads/writes
Martin Dørum [Wed, 31 Aug 2022 12:23:59 +0000 (14:23 +0200)]
gstpluginloader: Don't hang on short reads/writes

If read_one or write_one was called but the stream closed before it could
read/write a whole packet, read_one/write_one would hang indefinitely,
consuming 100% CPU. This commit fixes that by treating a short read/write
as an error.

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

22 months agovalidate: Fix gir generation
Thibault Saunier [Wed, 31 Aug 2022 14:31:40 +0000 (10:31 -0400)]
validate: Fix gir generation

It broke in https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2555
where we stopped parsing some .c files

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

22 months agovalidate: don't look for the common supression file
Jordan Petridis [Thu, 4 Aug 2022 18:12:08 +0000 (21:12 +0300)]
validate: don't look for the common supression file

We no longer have a common submodule

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

22 months agogst_init: Initialize static plugins just before dynamic plugins
Xavier Claessens [Wed, 31 Aug 2022 13:15:08 +0000 (09:15 -0400)]
gst_init: Initialize static plugins just before dynamic plugins

All plugins needs to be initialized after `gst_initialized = TRUE;`
otherwise they could complain that gst_init() has not been called.

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

22 months agomeson: Make sure devenv uses tools linked on gst-full
Xavier Claessens [Wed, 31 Aug 2022 01:15:25 +0000 (21:15 -0400)]
meson: Make sure devenv uses tools linked on gst-full

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

22 months agortpvp8depay: If configured to wait for keyframes after packet loss, also do that...
Sebastian Dröge [Fri, 26 Aug 2022 15:42:44 +0000 (18:42 +0300)]
rtpvp8depay: If configured to wait for keyframes after packet loss, also do that if incomplete frames are detected

This can happen if the data inside the packets is incomplete without the
seqnums being discontinuous because of ULPFEC being used.

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

22 months agoexamples: webrtc: mp-sendrecv: Add missing semicolon
Sebastian Dröge [Wed, 31 Aug 2022 07:57:39 +0000 (10:57 +0300)]
examples: webrtc: mp-sendrecv: Add missing semicolon

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

22 months agoexamples/webrtc: add some missing unrefs
Alireza Miryazdi [Tue, 30 Aug 2022 21:11:22 +0000 (21:11 +0000)]
examples/webrtc: add some missing unrefs

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

22 months agod3d11memory: Fix GstAllocator::mem_copy() implementation
Seungha Yang [Mon, 29 Aug 2022 20:02:31 +0000 (05:02 +0900)]
d3d11memory: Fix GstAllocator::mem_copy() implementation

We were checking possible bind flags for the DXGI format
of the source texture but that's never applied to
the destination texture desc.

Just use the already configured bind (and misc) flags of source texture
for the destination texture allocation without additional check.

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

22 months agoaudiovisualizer: fix buffer mapping to not increase refcount
Rafael Caricio [Tue, 30 Aug 2022 12:29:41 +0000 (14:29 +0200)]
audiovisualizer: fix buffer mapping to not increase refcount

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

22 months agodocs: disable in static build
Stéphane Cerveau [Tue, 31 May 2022 13:18:03 +0000 (15:18 +0200)]
docs: disable in static build

Following gst-plugins-base, disable docs if static_build
in:
 - gstreamer
 - gst-plugins-good
 - gst-plugins-ugly
 - gst-libav
 - gstreamer-vaapi

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

22 months agocurlbasesink: gst_curl_base_sink_transfer_thread_close is internal
Robert Rosengren [Wed, 24 Aug 2022 06:22:49 +0000 (08:22 +0200)]
curlbasesink: gst_curl_base_sink_transfer_thread_close is internal

gst_curl_base_sink_transfer_thread_close is moved from external header
to be static function, as it has no users.

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

22 months agocurlhttpsink: Only set MIME as content-type if not set by property
Robert Rosengren [Thu, 7 Jul 2022 07:20:04 +0000 (09:20 +0200)]
curlhttpsink: Only set MIME as content-type if not set by property

Setting the content-type property shall override internally detected MIME
types, to make it possible to do as following example (where audio/basic to be
used prior to audio/x-mulaw):

gst-launch-1.0 ... ! mulawenc ! audio/x-mulaw,rate=8000,channels=1 !
  curlhttpsink location=<url> content-type=audio/basic

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

22 months agod3d11: Update library doc
Seungha Yang [Thu, 25 Aug 2022 15:16:56 +0000 (00:16 +0900)]
d3d11: Update library doc

* Private header name is changed to gstd3d11-private.h to follow
  naming convention
* Add Since mark everywhere
* Update member variable names to be consistent with the other
  object implementations in this library
* Correct outdated documentation

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

22 months agodocker/windows: Pass a couple installargs to python3
Jordan Petridis [Thu, 18 Aug 2022 12:25:08 +0000 (15:25 +0300)]
docker/windows: Pass a couple installargs to python3

From these, only PrependPath isn't enabled by default,
but its nice to explicitly enable the ones we need anyway.

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

22 months agovalue: Use g_critical() when trying to serialize things that can't be
Olivier Crête [Wed, 24 Aug 2022 16:42:12 +0000 (12:42 -0400)]
value: Use g_critical() when trying to serialize things that can't be

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

22 months agogstvalue: Don't loop forever when serializing invalid flag
Olivier Crête [Mon, 1 Aug 2022 18:00:20 +0000 (14:00 -0400)]
gstvalue: Don't loop forever when serializing invalid flag

The serialization code would loop forever if an invalid flag was sent into it.

With unit test for this corner case.

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

22 months agobin: Don't propagate state change errors of elements in locked state
Sebastian Dröge [Tue, 23 Aug 2022 16:40:54 +0000 (19:40 +0300)]
bin: Don't propagate state change errors of elements in locked state

Theoretically having elements in locked state should not have any effect
at all when the surrounding bin is doing state changes. However
previously a state change error of a locked element would cause the
bin's state change to also fail, which is clearly not intended.

State change failures of locked elements are to be handled by whoever
set the element to locked state. By always returning them here it is
impossible for the owner of the element to handle state change failures
gracefully without potentially affecting the whole pipeline's state
changes.

Non-failure returns are still returned as-is as the distinction between
ASYNC/NO_PREROLL/SUCCESS has big consequences on the state changes of
the bin and overall pipeline. Theoretically SUCCESS should also be
returned in all cases but I can't estimate the effects this would have
on the overall pipeline.

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

22 months agox(v)imagesink: Don't create invalid GstNavigationModifierType by simply passing throu...
Sebastian Dröge [Tue, 23 Aug 2022 16:12:17 +0000 (19:12 +0300)]
x(v)imagesink: Don't create invalid GstNavigationModifierType by simply passing through X11 event states

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

22 months agoavfvideosrc: Fix wrong default framerate value
Piotr Brzeziński [Thu, 25 Aug 2022 12:24:25 +0000 (14:24 +0200)]
avfvideosrc: Fix wrong default framerate value

Current default G_MAXINT is not a correct value under any circumstances.
This creates an issue with screen capture, during which we currently do
not get any framerate info causing G_MAXINT to show up, where elements
downstream can possibly misbehave - for example, `vtenc` causes
a kernel panic.
Replace with 30/1 to avoid such scenarios.

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

22 months agojpegdecoder: return the real error of decode_scan and decode_frame.
He Junyan [Thu, 25 Aug 2022 07:28:21 +0000 (15:28 +0800)]
jpegdecoder: return the real error of decode_scan and decode_frame.

The current handle_frame() does not return the real error that happens
in decode_scan and decode_frame, which makes the pipeline continue with
the error and may trigger asserting later.

We also return the error when decode_quant_table or decode_huffman_table
fails.

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

22 months agoexamples: d3d11videosink: Add present signal example
Seungha Yang [Fri, 19 Aug 2022 12:54:48 +0000 (21:54 +0900)]
examples: d3d11videosink: Add present signal example

Add an example to show the usage of present singal.
In this example, a text overlay with alpha blended background
will be rendered on swapchain's backbuffer by using
Direct3D11, Direct2D, and DirectWrite APIs.

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

22 months agod3d11videosink: Add display-format property
Seungha Yang [Tue, 23 Aug 2022 15:13:21 +0000 (00:13 +0900)]
d3d11videosink: Add display-format property

Make swapchian's display format configurable, since some DXGI formats
we can use for swapchain are not API interop compatible.
For instance, BGRA format should be used for Direct2D interop.

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

22 months agod3d11videosink: Add present signal
Seungha Yang [Fri, 19 Aug 2022 11:25:31 +0000 (20:25 +0900)]
d3d11videosink: Add present signal

The "present" signal will be emitted just before the
IDXGISwapChain::Present() call. The client can perform additional
GPU operation with given GstD3D11Device object and
ID3D11RenderTargetView handle. Or, the client can read back
the scene to be displayed on window using the signal.

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

22 months agov4l2allocator: Fix invalid imported dmabuf fd
Jianhui Dai [Wed, 24 Aug 2022 09:30:34 +0000 (17:30 +0800)]
v4l2allocator: Fix invalid imported dmabuf fd

Fix a typo that set userptr to dmabuf fd. It leads to failure of
dmabuf-import io-mode.

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

22 months agoplayback: add onvif metadata caps to raw caps
Mathieu Duponchelle [Fri, 22 Jul 2022 11:41:17 +0000 (13:41 +0200)]
playback: add onvif metadata caps to raw caps

+ remove encoding from x-onvif-metadata caps output by qtdemux

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

22 months agovideoconvert: fix passthrough on equivalent transfer
Mathieu Duponchelle [Tue, 23 Aug 2022 14:02:31 +0000 (16:02 +0200)]
videoconvert: fix passthrough on equivalent transfer

When the input info and output info are equal, except for the
transfer functions that are *not* equivalent, we need to set up
a converter as we won't be passthrough.

Fixes an assertion in that case.

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

22 months agoisoff: Fix earliest pts field parse issue
zhiyuan.liu [Tue, 23 Aug 2022 10:28:30 +0000 (10:28 +0000)]
isoff: Fix earliest pts field parse issue

earliest pts will be covered by first_offset field on version 0 case.

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

22 months agodmabuf: Make it clear that the FORMAT is a DRM fourcc
Olivier Crête [Wed, 15 Jun 2022 21:32:06 +0000 (17:32 -0400)]
dmabuf: Make it clear that the FORMAT is a DRM fourcc

Some of the formats in GstVideoFormat are a combination
of a DRM format and modifier, making them unclear.

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

22 months agoh265decoder: Update SpsMaxLatencyPictures properly
Seungha Yang [Thu, 18 Aug 2022 19:49:53 +0000 (04:49 +0900)]
h265decoder: Update SpsMaxLatencyPictures properly

The SpsMaxLatencyPictures value never gets back to zero even if
it's needed. Update the value properly.

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

22 months agoh265decoder: Reorganize debug printing
Seungha Yang [Thu, 18 Aug 2022 19:44:48 +0000 (04:44 +0900)]
h265decoder: Reorganize debug printing

As of the MR
https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2575
gst_h265_decoder_process_sps() is called per slice and it causes
noisy debug message print.
Reorganize related and the other debug printings at once.

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

22 months agoopenh264: Register debug categories earlier
Philippe Normand [Sat, 20 Aug 2022 15:15:15 +0000 (16:15 +0100)]
openh264: Register debug categories earlier

Otherwise the GST_ERROR message logged in case of ABI mismatch would be done on
an uninitialized category.

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

22 months agoopenh264enc: Fix constrained-high encoding
Philippe Normand [Sat, 20 Aug 2022 15:57:27 +0000 (16:57 +0100)]
openh264enc: Fix constrained-high encoding

constrained-high is high without B-frames, there is no EProfileIdc for this, so
assume high instead of hitting an assert down the line.

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

22 months agowebrtc:ice: Fix candidate stats related APIs for bindings
Thibault Saunier [Fri, 19 Aug 2022 13:07:50 +0000 (09:07 -0400)]
webrtc:ice: Fix candidate stats related APIs for bindings

null-terminated arrays of structures is not usable.

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

22 months agova: h265dec: Enable the scc_main_4:4:4_10 profile.
He Junyan [Thu, 18 Aug 2022 05:27:02 +0000 (13:27 +0800)]
va: h265dec: Enable the scc_main_4:4:4_10 profile.

We should enable this profile which is already allown in vaprofile.c
after libva 1.8 version.

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

22 months agoadaptivedemux2: Fix download helper with libsoup 3.0.x
Jan Alexander Steffens (heftig) [Thu, 18 Aug 2022 14:36:45 +0000 (16:36 +0200)]
adaptivedemux2: Fix download helper with libsoup 3.0.x

libsoup 3.0.x dispatches using a single source attached when the session
is created, so we need to create the session with the same context that
our download thread is later using.

2.74 or 3.1 will dispatch a response using the context which sent the
request. However, for any context other than the one that created the
session, this will also create and destroy sources, so there's still
some slight performance benefit.

Fixes: https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/1384
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2913>

22 months agokmssink: add fd property
U. Artie Eoff [Wed, 6 Jul 2022 13:08:17 +0000 (09:08 -0400)]
kmssink: add fd property

This allows an application to provide their own opened DRM device
fd handle to kmssink.  For example, an application can lease
multiple fd's from a DRM master to display on different CRTC
outputs at the same time with multiple kmssink instances.

Specifying the fd property is not allowed when driver-name
and/or bus-id properties are specified.

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

22 months agod3d11: Use CRITICAL_SECTION instead of GRecMutex
Seungha Yang [Thu, 18 Aug 2022 15:05:17 +0000 (00:05 +0900)]
d3d11: Use CRITICAL_SECTION instead of GRecMutex

The GRecMutex abstraction (and heap allocation happens in GLib)
is unnecessary for this plugin.

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

22 months agoadaptivedemux2: Implement send_event() vfunc
Jan Schmidt [Wed, 17 Aug 2022 23:48:15 +0000 (09:48 +1000)]
adaptivedemux2: Implement send_event() vfunc

Handle select-streams and seek events in an element
level send_event() vfunc, so they can be received
before any source pads are created.

This allows preferred streams to be selected before
segment downloading starts.

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

22 months agowebrtc: doc: Cleanup Since markers
Thibault Saunier [Wed, 17 Aug 2022 15:18:54 +0000 (11:18 -0400)]
webrtc: doc: Cleanup Since markers

We hide some newly public symbols when moving some objects to the library
which was not really correct in the end. Properly add the Since markers
as needed.

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

22 months agowebrtc:ice: Reindent header
Thibault Saunier [Tue, 16 Aug 2022 12:59:00 +0000 (08:59 -0400)]
webrtc:ice: Reindent header

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

22 months agowebrtc:ice: Mark protected fields as such
Thibault Saunier [Tue, 16 Aug 2022 01:27:06 +0000 (21:27 -0400)]
webrtc:ice: Mark protected fields as such

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

22 months agowebrtc: Fix documentaton moving symbols in the right pages
Thibault Saunier [Tue, 16 Aug 2022 01:25:25 +0000 (21:25 -0400)]
webrtc: Fix documentaton moving symbols in the right pages

As those symbols are documented in a 'fwd' header smart indexing in
hotdoc wasn't working.

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

22 months agowebrtc:ice: Avoid using GArray in public API
Thibault Saunier [Mon, 15 Aug 2022 15:52:50 +0000 (11:52 -0400)]
webrtc:ice: Avoid using GArray in public API

And use plain null terminated C arrays instead.

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

22 months agowebrtc:ice: Fix some annotations
Thibault Saunier [Mon, 15 Aug 2022 15:37:57 +0000 (11:37 -0400)]
webrtc:ice: Fix some annotations

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

22 months agowebrtc: Make GstWebRTCIceCandidateStats a Boxed type
Thibault Saunier [Mon, 15 Aug 2022 15:36:25 +0000 (11:36 -0400)]
webrtc: Make GstWebRTCIceCandidateStats a Boxed type

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

22 months agowebrtc:ice: Add padding to structures
Thibault Saunier [Sun, 14 Aug 2022 21:38:24 +0000 (17:38 -0400)]
webrtc:ice: Add padding to structures

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

22 months agocuda: Hide runtime compiler related header and symbols
Seungha Yang [Sun, 14 Aug 2022 12:49:27 +0000 (21:49 +0900)]
cuda: Hide runtime compiler related header and symbols

That's already abstracted via gst_cuda_nvrtc_compile() method
and therefore, we do not need to expose such symbols yet.

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

22 months agoaggregator: Improve debug output to better understand why pads are not ready or can...
Sebastian Dröge [Wed, 17 Aug 2022 13:27:36 +0000 (16:27 +0300)]
aggregator: Improve debug output to better understand why pads are not ready or can't accept more data

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

22 months agovajpegdec: Fix memory leak.
Víctor Manuel Jáquez Leal [Wed, 17 Aug 2022 10:22:38 +0000 (12:22 +0200)]
vajpegdec: Fix memory leak.

Free value content.

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

22 months agovajpegdec: Fix 4:2:2 for i965.
Víctor Manuel Jáquez Leal [Tue, 16 Aug 2022 18:05:15 +0000 (20:05 +0200)]
vajpegdec: Fix 4:2:2 for i965.

Since i965 uses NV12 either for 4:2:0 and 4:2:2 chroma (using an
internal color conversion) the chroma validation has to be shortcut.

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

22 months agova: libs: Add Y42B format.
Víctor Manuel Jáquez Leal [Tue, 16 Aug 2022 17:50:49 +0000 (19:50 +0200)]
va: libs: Add Y42B format.

Enable 4:2:2 JPEG decoding in iHD.

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

22 months agova: Validate JPEG subsampling configurations.
Víctor Manuel Jáquez Leal [Tue, 16 Aug 2022 14:06:20 +0000 (16:06 +0200)]
va: Validate JPEG subsampling configurations.

There are Mesa Gallium drivers that report subsampling but without
any pixel format. So, strip out these subsamplings.

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

22 months agova: gst_va_create_raw_caps_from_config() may return NULL.
Víctor Manuel Jáquez Leal [Tue, 16 Aug 2022 12:45:46 +0000 (14:45 +0200)]
va: gst_va_create_raw_caps_from_config() may return NULL.

This patch verifies if the function returns NULL in the caller.

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

22 months agodecklink: Validate video input format
Eric Knapp [Wed, 17 Aug 2022 15:42:09 +0000 (11:42 -0400)]
decklink: Validate video input format

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

22 months agodecklink: Auto-detect 10-bit YUV
Eric Knapp [Mon, 15 Aug 2022 15:23:56 +0000 (11:23 -0400)]
decklink: Auto-detect 10-bit YUV

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

22 months agoexamples: gtk: Fix MSVC build
Seungha Yang [Fri, 12 Aug 2022 13:18:22 +0000 (22:18 +0900)]
examples: gtk: Fix MSVC build

gtk-play.c(370): error C2375: 'rewind_button_clicked_cb': redefinition;
different linkage

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

22 months agobase: video-converter: add accessors for input and output formats
Guillaume Desmottes [Wed, 17 Aug 2022 10:53:02 +0000 (12:53 +0200)]
base: video-converter: add accessors for input and output formats

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

22 months agovaapi: plugin: Handle when no encoders/decoders available.
Víctor Manuel Jáquez Leal [Thu, 11 Aug 2022 19:22:12 +0000 (21:22 +0200)]
vaapi: plugin: Handle when no encoders/decoders available.

Nouveau driver currently only exposes postproc entry. But
vaapidecodebin is registered independent if there are decoders or not,
exposing a segmentation fault.

This patch removes the encoder/decoder/codec arrays if no entries are
found, and if no decoders are found vaapidecodebin is not
registered. Also for vaapipostproc if no postproc entry is found.

Also, if general decoder, used by vaapidecodebin, doesn't have a sink
pad string, don't register the glib type.

Fixes: #1349
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2865>

22 months agodoc: Fix typo
Krystian Wojtas [Wed, 17 Aug 2022 06:00:12 +0000 (08:00 +0200)]
doc: Fix typo

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

22 months agogst-play: missing cleanup for g_autoptr
Daniel Morin [Mon, 15 Aug 2022 02:31:29 +0000 (22:31 -0400)]
gst-play: missing cleanup for g_autoptr

Without this change cleanup function for g_autoptr is not defined for
GstPlayMediaInfo, GstPlaySignalAdapter, GstPlayVideoRenderer,
GstPlayVideoOverlayVideoRenderer and GstPlayVisualization. Cleanup
function was defined in gstplay.h, but missing in other header files.

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

22 months agosplitmuxsrc: Stop pad task before cleanup
Jan Schmidt [Tue, 16 Aug 2022 23:11:52 +0000 (09:11 +1000)]
splitmuxsrc: Stop pad task before cleanup

When stopping the element, make sure the pad task
is stopped before destroying the part readers.

Closes a race where the pad task might access
a freed pointer.

Also add a guard against this sort of thing
by holding a ref to the reader in the pad loop.

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

22 months agoqtdemux: Avoid crash on reconfiguring.
Jan Schmidt [Fri, 12 Aug 2022 10:20:43 +0000 (20:20 +1000)]
qtdemux: Avoid crash on reconfiguring.

When reconfiguring a stream that never created
an output pad, don't access a NULL GstPad pointer.

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

22 months agoqtdemux: Set parsed=true on ONVIF Timed Metadata caps
Sebastian Dröge [Tue, 16 Aug 2022 16:01:19 +0000 (19:01 +0300)]
qtdemux: Set parsed=true on ONVIF Timed Metadata caps

Inside MP4 the metadata must be properly parsed into frames and in
order.

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

22 months agomeson: d3d11: Fix missing header install
Seungha Yang [Tue, 16 Aug 2022 12:01:47 +0000 (21:01 +0900)]
meson: d3d11: Fix missing header install

Install gstd3d11compile.h header file

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

22 months agov4l2bufferpool: Fix debug trace
Nicolas Dufresne [Mon, 15 Aug 2022 18:30:50 +0000 (14:30 -0400)]
v4l2bufferpool: Fix debug trace

The tracing of the buffer pointer was done on the secondary pointer, which
does not match with other traces of the same buffer. This made the trace
confusing and less useful.

Fixes #1379

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

22 months agoadaptivedemux2: Fix for period switching in live streams
Jan Schmidt [Fri, 12 Aug 2022 11:57:25 +0000 (21:57 +1000)]
adaptivedemux2: Fix for period switching in live streams

When playing live, it's possible that one stream reaches
the end of the available playback window and goes to sleep
waiting for a manifest update, and the manifest update
introduces a new period. In that case, the sleeping
stream needs to wake up and go 'properly' EOS before we
can advance the input to the new period.

Accordingly, make sure that a stream's last_ret value
is not marked as EOS if it's just sleeping waiting for a live
manifest update.

Also fix the output loop to go back and re-check if it's
time to switch to the next period after dequeuing and
discarding an EOS event.

https://livesim.dashif.org/livesim/periods_20/testpic_2s/Manifest.mpd

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

22 months agovideoconvert,videoscale: Do conversion in videoconvert and scaling in videoscale
Seungha Yang [Fri, 5 Aug 2022 11:52:19 +0000 (20:52 +0900)]
videoconvert,videoscale: Do conversion in videoconvert and scaling in videoscale

Keep behaving the same as before videoconvertscale port

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

22 months agoRevert "videoconvertscale: Add properties to disable scaling/converting in videoconve...
Seungha Yang [Fri, 5 Aug 2022 11:16:00 +0000 (20:16 +0900)]
Revert "videoconvertscale: Add properties to disable scaling/converting in videoconvert/videoscale"

This reverts commit cd7a91cef1c6a2e24d440126b7f2ab543fb205c5.

Reverting properties, scaling in videoconvert and converting in
videoscale will be disabled by the other commit

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

22 months agomssdemux2: Use gsturi structure to form fragment urls
Hosang Lee [Thu, 11 Aug 2022 00:34:58 +0000 (09:34 +0900)]
mssdemux2: Use gsturi structure to form fragment urls

Utilize gsturi to form fragment url paths.
A token query may contain the string "manifest" and this would lead
to improper url creations.

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

22 months agometa: Set the parent refcount of the GstStructure correctly
Nirbheek Chauhan [Mon, 15 Aug 2022 14:37:09 +0000 (20:07 +0530)]
meta: Set the parent refcount of the GstStructure correctly

The parent refcount is of the *transformed* buffer, not the input
buffer.

Also update the docs to clarify that @transbuf is the transformed
buffer, and not the buffer on which a transformation is being
performed.

Due to this bug, modifying the structure of a meta that has been
copied to another buffer fails with:

gst_structure_set: assertion 'IS_MUTABLE (structure) || field == NULL' failed

Add a test for the same.

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