summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* omxvideoenc: Unref frame of codec config bufferShinya Saito2019-07-111-0/+1
| | | | After handling codec config, codec frame should be unreffed.
* omxvideo: check difference between frame and requested tsGuillaume Desmottes2019-06-171-1/+12
| | | | | This has proven to be very useful when debugging to detect bugs where we match the wrong gst frame with an output OMX buffer.
* omxvideo: add debug infos to find_nearest_frame()Guillaume Desmottes2019-06-174-4/+12
| | | | | | Those debug infos have proved to be very helpful when debugging timestamp issues. They are often linked to gst-omx picking the wrong frame when trying to map from OMX.
* omxvideodec: Deactivate negotiated pool when output own bufferShinya Saito2019-06-071-0/+2
| | | | | If decoder outputs internal buffer and not use OMX_UseBuffer, downstream bufferpool should be stopped.
* omxh264enc: Add 'ref-frames' propertyShinya Saito2019-06-042-1/+24
| | | | | | Add a property to control the number of frames for reference. Min and max value is based on OpenMAX IL 1.2.0 Specification.
* meson: Bump minimal GLib version to 2.44Niels De Graef2019-06-032-2/+2
| | | | | | | This means we can use some newer features and get rid of some boilerplate code using the G_DECLARE_* macros. As discussed on IRC, 2.44 is old enough by now to start depending on it.
* doc: Build documentation of hotdocThibault Saunier2019-05-137-0/+81
|
* gstomx: remove gst_omx_buffer_set_omx_buf/get_omx_bufGeorge Kiagiadakis2019-04-252-21/+0
| | | | They are no longer used anywhere
* omxbufferpool: refactor to allow memory sharingGeorge Kiagiadakis2019-04-257-265/+849
| | | | | | | | | | | | | | | | | | | | | | One big restriction of the OMX buffer pool has always been that the GstMemory objects were flagged with NO_SHARE. This was because the buffer pool needed to be sure that when a buffer returned to the pool, it would be safe to release the OMX buffer back to OpenMAX. With this change, this is no longer a restriction. What this commit introduces is a new allocator that allows us to track the GstMemory objects independently. Now, when a buffer returns to the pool, it is not necessary for the memory to be released as well. We simply track the memory's ref count in the allocator and we return the OMX buffer back when the memory's ref count drops to 0. The reason for doing this is to allow implementing zero-copy transfers in situations where we may need to copy or map a certain region of the buffer. For instance, omxh264enc ! h264parse should be possible to be zero-copy by using an OMX buffer pool between them.
* omxbufferpool: fix memory mapping with offsetGuillaume Desmottes2019-04-231-1/+1
| | | | | | | | | gst_memory_map() is already adding the offset to the mapped pointer. Doing it in the memory implementation was resulting in the offset being accounted twice. It doesn't matter yet as we are only creating memory without offset for now but it will once we'll start sharing OMX memories.
* Back to developmentTim-Philipp Müller2019-04-193-5/+5
|
* Release 1.16.01.16.0Tim-Philipp Müller2019-04-196-87/+247
|
* Fixes build with omx >= 1.2.0Julien Isorce2019-04-161-12/+19
| | | | | | | | | gstomx.c:1405:10: error: ‘OMX_IndexParamCustomContentPipe’ undeclared (first use in this function) case OMX_IndexParamCustomContentPipe Some enums have been deprecated in 1.2.0 https://gitlab.freedesktop.org/gstreamer/gst-omx/issues/27
* testegl: properly detect and use rpi specific libsGuillaume Desmottes2019-04-123-5/+12
| | | | | Use pkg-config to detect and configure rpi specific libs used in testegl rather than hardcoding their flags.
* testelg: include eglext.hGuillaume Desmottes2019-04-121-0/+1
| | | | Neded for the declaration of eglSaneChooseConfigBRCM().
* testegl: move up pi specific macrosGuillaume Desmottes2019-04-121-9/+9
| | | | | We need to define __VCCOREVER__ and disable redundant-decls before including the egl.h from the pi.
* Release 1.15.901.15.90Tim-Philipp Müller2019-04-116-47/+176
|
* meson: add -Wundef as additional warning flagTim-Philipp Müller2019-04-101-0/+1
|
* omx: fix autotools build for generic targetTim-Philipp Müller2019-04-101-1/+1
| | | | | | | gstomxvideoenc.c:2874:7: error: "USE_OMX_TARGET_ZYNQ_USCALE_PLUS" is not defined, evaluates to 0 [-Werror=undef] #elif USE_OMX_TARGET_ZYNQ_USCALE_PLUS Works on meson because it doesn't use -Wundef
* Add Gitlab CI configurationJordan Petridis2019-04-061-0/+1
| | | | | | | | | | | This commit adds a .gitlab-ci.yml file, which uses a feature to fetch the config from a centralized repository. The intent is to have all the gstreamer modules use the same configuration. The configuration is currently hosted at the gst-ci repository under the gitlab/ci_template.yml path. Part of https://gitlab.freedesktop.org/gstreamer/gstreamer-project/issues/29
* omx: disable OMX_API_TRACE code if gst debug is disabledGuillaume Desmottes2019-03-261-0/+6
| | | | | No need to create debug structs which won't be used as DEBUG macros are no-op.
* omx: log Get/SetParameter/Config callsGuillaume Desmottes2019-03-261-0/+476
| | | | Extend OMX_API_TRACE by logging component configuration calls.
* omxbufferpool: don't use CAT_PERFORMANCE if pool will copyGuillaume Desmottes2019-03-261-4/+1
| | | | | | | | | | This was the single place where this category was used in gst-omx so most users, including me, are generally not turning it and were missing this important information from logs. The copying code uses gst_video_frame_copy() which is already logging with CAT_PERFORMANCE so we can still have this information when using only this debug category.
* omxvideodec: add debug if proposed pool can't provide enough buffersGuillaume Desmottes2019-03-261-0/+2
| | | | | We were silently ignoring the pool which was pretty confusing when debugging.
* omx: Add hardware classifiers to encoders/decodersCharlie Turner2019-03-2618-18/+18
|
* meson: sync warnings flags with -goodGuillaume Desmottes2019-03-251-0/+60
| | | | | | | Add more warnings flags and disabled unused variable warnings if gst debug system is disabled. Copied from gst-plugins-good/meson.build
* Back to developmentTim-Philipp Müller2019-03-044-8/+8
|
* Release 1.15.21.15.2Tim-Philipp Müller2019-02-266-101/+288
|
* meson: add options to disable examples, tests and tools and bump meson ↵Tim-Philipp Müller2019-02-142-7/+21
| | | | requirement
* omxvideo: use GST_VIDEO_CAPS_MAKE() for template capsGuillaume Desmottes2019-01-254-6/+8
| | | | | Simplify the code and so we advertise the formats actually supported by gst-omx.
* omxvideoenc: validate cpb-size and initial-delayGuillaume Desmottes2019-01-251-11/+19
| | | | cpb-size cannot be smaller than initial-delay.
* omxvideodec: Remove duplicated QoS codeNicolas Dufresne2019-01-251-10/+1
| | | | | The 'finish' function do the exact same check / drop, there is no need to duplicate this here.
* omxvideodec: Remove dead codeNicolas Dufresne2019-01-252-16/+0
| | | | | The omxvideodec base class have a totally unused prepare_frame() vritual function, remove it.
* omxvideoenc: add adaptive gop-mode optionVarunkumar Allagadapa2019-01-251-0/+2
| | | | Added adaptive gop-mode option to ZYNQ_USCALE_PLUS encoder properties
* omxvideoenc: Add dynamic framerate supportNicolas Dufresne2019-01-251-0/+52
| | | | | Instead of going through a full reset, try and change the framerate config on the encoder when only the framerate have change.
* Release 1.15.11.15.1Tim-Philipp Müller2019-01-176-58/+1702
|
* omxvideoenc: Add dynamic IDR insertion support on zynqVarunkumar Allagadapa2019-01-091-0/+9
| | | | As the pi, the zynq has its own API to request keyframe.
* omxbufferpool: fix race when releasing input buffersGuillaume Desmottes2019-01-083-1/+13
| | | | | | | | | If buffers were released from the pool while gst_omx_video_enc_handle_frame() was waiting for new buffers, gst_omx_port_acquire_buffer() was never awaken as the buffers weren't released through OMX's messaging system. GQueue isn't thread safe so also protect it with the lock mutex.
* omxbufferpool: fix early input buffer releaseGuillaume Desmottes2018-12-214-11/+1
| | | | | | | | | | | | | | | | | We used to track the 'allocating' status on the pool. It is used while allocating so output buffers aren't passed right away to OMX and input ones are not re-added to the pending queue. This was causing a bug when exporting buffers to v4l2src. On start v4l2src acquires a buffer, read its stride and release it right away. As no buffer was received by the encoder element at this point, 'allocating' was still on TRUE and so the the buffer wasn't put back to the pending queue and, as result, no longer available to the pool. Fix this by checking the active status of the pool instead of manually tracking it down. The pool is considered as active at the very end of the activation process so we're good when buffers are released during the activation.
* Automatic update of common submoduleThibault Saunier2018-12-051-0/+0
| | | | From ed78bee to 59cb678
* omx: fix OMX_EventBufferFlag OMX_API_TRACE structGuillaume Desmottes2018-11-261-1/+2
| | | | The GType was missing from the second field of the struct.
* Update git locations to gitlabMatthew Waters2018-11-052-3/+3
|
* omx: rename OMX_PERFORMANCE debug cat to OMX_API_TRACEGuillaume Desmottes2018-09-261-20/+20
| | | | | | | This debug category can now be used to track more OMX calls and events so best to rename it to something more generic. https://bugzilla.gnome.org/show_bug.cgi?id=797171
* omx: log OMX commands with OMX_PERFORMANCE debug categoryGuillaume Desmottes2018-09-191-0/+55
| | | | | | | It has been useful to have a clear raw and structured view of the gst <-> OMX exchanges when debugging. https://bugzilla.gnome.org/show_bug.cgi?id=797171
* omx: factor out gst_omx_component_send_command()Guillaume Desmottes2018-09-191-5/+18
| | | | | | No semantic change. I'm going to add extra debug in this function. https://bugzilla.gnome.org/show_bug.cgi?id=797171
* omx: log OMX events with OMX_PERFORMANCE debug categoryGuillaume Desmottes2018-09-191-0/+141
| | | | | | | It has been useful to have a clear raw and structured view of the gst <-> OMX exchanges when debugging. https://bugzilla.gnome.org/show_bug.cgi?id=797171
* omx: rename log_omx_performance() to log_omx_performance_buffer()Guillaume Desmottes2018-09-191-6/+6
| | | | | | I'm about to log more things under this category https://bugzilla.gnome.org/show_bug.cgi?id=797171
* omxvideoenc: Remove spurious lockingNicolas Dufresne2018-09-101-7/+3
| | | | | | | | The method we call in the context of pushing a buffer are all thread safe. Holding a lock would prevent input buffers from being queued while pushing. https://bugzilla.gnome.org/show_bug.cgi?id=715192
* omxvideoenc: Remove unneeded size checkNicolas Dufresne2018-09-101-11/+7
| | | | | | | We only enter this branch if nFilledLen > 0, there is not need to check again. https://bugzilla.gnome.org/show_bug.cgi?id=715192
* omxvideodec: Remove spurious unlock in error caseNicolas Dufresne2018-09-101-3/+1
| | | | | | | This was forgotton in previous patch. We no long hold the lock when goto invalid_buffer is called. https://bugzilla.gnome.org/show_bug.cgi?id=715192