If incoming caps have a spatial-video-info structure,
attempt to parse it and write out an sv3d atom to the
video track info

Write stereo info to the st3d atom as well if we have some.

Add some support for parsing cubemap and equirectangular
projection information from atoms as specified by
https://github.com/google/spatial-media/blob/master/docs/spherical-video-v2-rfc.md

Doesn't support the mesh projection mode yet, and needs
some API support added to libgstvideo.

We always answer it successfully no matter what.

The default return value in the function is FALSE even if the code below
sets it again to FALSE.

Part-of: <gstreamer/gst-plugins-good!654>

Part-of: <gstreamer/gst-plugins-good!653>

Previously imagefreeze would always operate as non-live element and
output frames as fast as possible according to the configured segment
(via SEEK events) and the negotiated framerate from start to stop or the
other way around.

With the new live mode (enabled via the is-live property) it would only
output frames in PLAYING. Frames would be output according to the
negotiated framerate unless it would be too late, in which case it would
jump ahead and skip over the requirement amount of frames.

This makes it possible to actually use imagefreeze in live pipelines
without having to manually ensure somehow that it would start outputting
at the current running time and without still risking to fall behind
without recovery.

Part-of: <gstreamer/gst-plugins-good!653>

We never run as a live element, even if upstream is live, and never
output any buffers with latency but immediately generate buffers as
fast as we can according to the negotiated framerate.

Passing the query upstream would proxy whatever mode of operation
upstream has, which has nothing to do with how we produce buffers.

Part-of: <gstreamer/gst-plugins-good!653>

Nicolas Dufresne authored Jun 25, 2020 and GStreamer Marge Bot committed Jun 28, 2020

The pool orphaning function was colling internal _stop() virtual function
implementation. This is not thread safe, as a private lock inside the buffer
pool is supposed to be held. Fix this by keeping delayed _stop() and orphaning
the GstV4L2Allocator instead (REQBUFS(0)).

Then, protect the orphaned boolean with the object lock for the case a buffer
is being released after we have orphaned the buffer. That would otherwise
cause a QBUF to happen while the queue is no longer owned by the buffer pool.
This boolean is otherwise used and set from the streaming lock, or after
threads have been stopped (final cleanup).

Part-of: <gstreamer/gst-plugins-good!648>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 28, 2020

When the buffer pool is importing buffer, it will requeue num_allocated on
streamon. As this value was not set for DMABuf import and USERPTR, no buffer
was queued  back.

Part-of: <gstreamer/gst-plugins-good!651>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 28, 2020

This reverts commit 6bf9f4bd.

Part-of: <gstreamer/gst-plugins-good!651>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 28, 2020

This reverts commit 94e323c1.

Fixes #754

Part-of: <gstreamer/gst-plugins-good!651>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 28, 2020

On streamon, we need to resurrect (queue back) some buffers, as during
flushign seek we'd endup with an empty queued. We initially started with
resurrecting as many as we could without blocking, but that miss-behaved with
dynamic CREATE_BUFS, causing the pool to grow dramatically. This was limited
by the number of allocated buffers, but this still tried to resurrect too many
buffers for the first run, as activating the pool will queued buffers.

In this patch, we calculte the missing detal in the queue and only try and
resurrect that amount of buffers.

Part-of: <gstreamer/gst-plugins-good!651>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 28, 2020

Avoid offering a pool if it's not needed or if it's still active.
This works around the fact the we only have one pool in V4L2.

Part-of: <gstreamer/gst-plugins-good!651>

Instead of querying the Qt include path from the dependency or from
qmake, rely on the qt5qml_dep to set the include path to QtGui
correctly, and look for the header inside the private includedir.

Then we can use that path to include the header directly.

Reported in gst-plugins-base#780 (comment 548092)

Part-of: <!640>

This is an extra check that's only needed for working around Linux
distribution packaging. `moc` is not required in the cross file.

Part-of: <gstreamer/gst-plugins-good!640>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 26, 2020

If the allocator is no longer active, it means the memory has already
been freed, calling REQBUF(0) would make no sense.

Part-of: <gstreamer/gst-plugins-good!649>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 26, 2020

The gst_buffer_pool_set_flushing() warns when that function is called
on an inactive pool. Avoid the warning by checking the state, this is
similar to what we do in gst_v4l2_object_unlock().

Part-of: <gstreamer/gst-plugins-good!649>

Nicolas Dufresne authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 26, 2020

The check was too strict causing spurious warning. Now check for <= so that 0
sized buffer do not cause a warning.

Part-of: <gstreamer/gst-plugins-good!649>

Nicolas Dufresne authored Jun 25, 2020 and GStreamer Marge Bot committed Jun 26, 2020

Part-of: <gstreamer/gst-plugins-good!649>

Tim-Philipp Müller authored Jun 26, 2020 and GStreamer Marge Bot committed Jun 26, 2020

Make explicit that the returned strings need to be g_free()-able.

Fixes #753

Part-of: <gstreamer/gst-plugins-good!652>

The codec structure is optional and not used for fwht test codec. This
was leading to a crash dereferencing NULL pointer.

Part-of: <gstreamer/gst-plugins-good!650>

Håvard Graff authored May 03, 2020 and GStreamer Marge Bot committed Jun 26, 2020

Part-of: <!646>

Håvard Graff authored Jun 17, 2020 and Tim-Philipp Müller committed Jun 26, 2020

Fixes #750

Part-of: <gstreamer/gst-plugins-good!645>

Nirbheek Chauhan authored Jun 24, 2020 and GStreamer Marge Bot committed Jun 25, 2020

We already do this for the plugin.

gst-plugins-base#780 (comment 548179)

Part-of: <!642>

Part-of: <!644>

When generating the cache we inspect the base class through
an instance of one of its subclasses. We don't want potential
assignments in subclasses initialization to leak into the
base class documentation

Part-of: <gstreamer/gst-plugins-good!641>

When generating the cache we inspect the base class through
an instance of one of its subclasses. We don't want potential
assignments in subclasses initialization to leak into the
base class documentation

Part-of: <gstreamer/gst-plugins-good!641>

ORC_RESTRICT may not be defined in yadif.c and cause build error.

Part-of: <gstreamer/gst-plugins-good!637>

Håvard Graff authored Jun 06, 2019 and GStreamer Marge Bot committed Jun 20, 2020

Part-of: <gstreamer/gst-plugins-good!410>

Some default values include our version string, like
user agent strings.

Add host cpu type check as we would enable asm only for x86_64

Part-of: <gstreamer/gst-plugins-good!636>