summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* nvenc: better error handlingAnton Khirnov2016-01-12
| | | | Return proper error codes and print more descriptive error messages.
* movenc-test: stop setting the GLOBAL_HEADER codec flagAnton Khirnov2016-01-12
| | | | | This test does no encoding, setting the flag was done just to silence the warning removed in the previous commit.
* mux: drop the warning about global headersAnton Khirnov2016-01-12
| | | | | The AVStream codec context is often not (and should not be) the actual encoding context, so this warning will be spurious in many cases.
* lavc: Use get_bitsz where neededAndreas Cadhalpun2016-01-11
| | | | Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* lavc: Add get_bitsz()Andreas Cadhalpun2016-01-11
| | | | | | get_bit variant supporting 0-bits reads. Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* sunrastenc: Properly load codec private optionsVittorio Giovara2016-01-11
| | | | | Initialize the private class field and give the class a more appropriate name. Add a class member to the codec context.
* get_bits: Support max_depth > 2 in GET_RL_VLC_INTERNALKieran Kunhya2016-01-11
|
* lavu: add AV_CEIL_RSHIFT and use it in various placesClément Bœsch2016-01-11
| | | | Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
* swfdec: support compressed swfClément Bœsch2016-01-11
| | | | Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
* asfdec: Fix typo in commentAlexandra Hájková2016-01-11
| | | | Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* APIchanges: Add missing av_pix_fmt_get_chroma_sub_sample entryLuca Barbato2016-01-11
|
* arm: Fix vfp dead code elimination with have_vfp_vmMartin Storsjö2016-01-08
| | | | | | | | | This fixes builds with --disable-vfp. Checking for the armv6 cpu flag is incorrect, since vfpv2 isn't armv6 specific. Signed-off-by: Martin Storsjö <martin@martin.st>
* checkasm: Check register clobbering on aarch64Martin Storsjö2016-01-07
| | | | | | | This is disabled on iOS, since iOS uses a slightly different ABI for vararg parameters. Signed-off-by: Martin Storsjö <martin@martin.st>
* checkasm: Check register clobbering on armMartin Storsjö2016-01-07
| | | | | | | | | | | Use two separate functions, depending on whether VFP/NEON is available. This is set to require armv5te - it uses blx, which is only available since armv5t, but we don't have a separate configure item for that. (It also uses ldrd, which requires armv5te, but this could be avoided if necessary.) Signed-off-by: Martin Storsjö <martin@martin.st>
* avpacket: fix size check in packet_allocAndreas Cadhalpun2016-01-07
| | | | | | | | | | | The previous check only caught sizes from -AV_INPUT_BUFFER_PADDING_SIZE to -1. This fixes ubsan runtime error: signed integer overflow: 2147483647 + 32 cannot be represented in type 'int' Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Anton Khirnov <anton@khirnov.net>
* asfdec: Remove unused function parametersAlexandra Hájková2016-01-07
| | | | Signed-off-by: Anton Khirnov <anton@khirnov.net>
* asfdec: reject size > INT64_MAX in asf_read_unknownAndreas Cadhalpun2016-01-07
| | | | | | | | | | | Both avio_skip and detect_unknown_subobject use int64_t for the size parameter. This fixes a segmentation fault due to infinite recursion. Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Alexandra Hájková <alexandra.khirnova@gmail.com> Signed-off-by: Anton Khirnov <anton@khirnov.net>
* asfdec: only set asf_pkt->data_size after sanity checksAndreas Cadhalpun2016-01-07
| | | | | | | | | Otherwise invalid values are used unchecked in the next run. This can cause NULL pointer dereferencing. Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Alexandra Hájková <alexandra.khirnova@gmail.com> Signed-off-by: Anton Khirnov <anton@khirnov.net>
* qsvenc: split encoding frames and reading from the async FIFOAnton Khirnov2016-01-07
| | | | | | This makes sure all the frames are returned at the end. Found-By: Maxym Dmytrychenko <maxym.dmytrychenko@intel.com>
* qsvenc: properly handle the warning from MFXVideoCORE_SyncOperationMaxym Dmytrychenko2016-01-07
| | | | | | Same as what is done in 3b6473b43eb69fc3faaf69f7fd0b83b51db7607f. Signed-off-by: Anton Khirnov <anton@khirnov.net>
* avconv: Drop an impossible checkLuca Barbato2016-01-04
| | | | | | EAGAIN is already managed in poll_filters(). Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* avconv: Simplify poll_filters() return value checkLuca Barbato2016-01-04
| | | | Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* h264_refs: Remove broken trace debug codeDiego Biurrun2016-01-03
|
* lavc: Add missing #includes for ff_dlog()Diego Biurrun2016-01-03
|
* qsvdec: properly handle the warning from MFXVideoCORE_SyncOperationMaxym Dmytrychenko2016-01-01
| | | | Signed-off-by: Anton Khirnov <anton@khirnov.net>
* qsvenc_hevc: improve the default settingsMaxym Dmytrychenko2016-01-01
| | | | | | | Use 8 B-frames and modify the GOP size to be a multiple of the B-frame count. Signed-off-by: Anton Khirnov <anton@khirnov.net>
* h264: improve behaviour with invalid reference listsAnton Khirnov2016-01-01
| | | | | | | | | | | | | Before 741b494fa8cd28a7d096349bac183893c236e3f9, when the reference list modification description was invalid, the code would substitute the corresponding reference from the initial ("default") reference list. After that commit, it will just return an error. Since there are apparently invalid samples in the wild that used to play fine with the old code, it is a good idea to re-add some sort of error resilience here. So, when the reference list modification results in a missing frame, substitute a previous reference frame for it. The relevant sample again decodes fine with the same output as previously.
* dca: use defines for subband related constantsAlexandra Hájková2015-12-31
| | | | Signed-off-by: Janne Grunau <janne-libav@jannau.net>
* x86: use emms after ff_int32_to_float_fmul_scalar_sseJanne Grunau2015-12-30
| | | | | | | | | | | | | | | | | | | | | Intel's Instruction Set Reference (as of September 2015) clearly states that cvtpi2ps switches to MMX state. Actual CPUs do not switch if the source is a memory location. The Instruction Set Reference from 1999 (Order Number 243191) describes this behaviour but all later versions I've seen have make no distinction whether MMX registers or memory is used as source. The documentation for the matching SSE2 instruction to convert to double (cvtpi2pd) was fixed (see the valgrind bug https://bugs.kde.org/show_bug.cgi?id=210264). It will take time to get a clarification and fixes in place. In the meantime it makes sense to change ff_int32_to_float_fmul_scalar_sse to be correct according to the documentation. The vast majority of users will have SSE2 so a change to the SSE version has little effect. Fixes fate-checkasm on x86 valgrind targets. Valgrind 'bug' reported as https://bugs.kde.org/show_bug.cgi?id=357059
* checkasm: x86: post commit review fixesJanne Grunau2015-12-29
| | | | | | | Check the full FPU tag word instead of only the lower half and simplify the comparison. Use upper-case function base name as macro name to instantiate both checked_call variants.
* x86: zero extend the 32-bit length in int32_to_float_fmul_scalar implicitlyJanne Grunau2015-12-29
| | | | This reverts commit 5dfe4edad63971d669ae456b0bc40ef9364cca80.
* Remove stray line breaks from avpriv_{report_missing_feature|request_samples}Diego Biurrun2015-12-26
|
* rtpdec_jpeg: Coalesce redundant error checksDiego Biurrun2015-12-26
|
* dca: remove unused decode_hf function and quant_d tablesAlexandra Hájková2015-12-24
| | | | | They were superseded with their integer equivalents. Rename integer decode_hf to decode_hf.
* dca: change the core to work with integer coefficients.Alexandra Hájková2015-12-23
| | | | | | | | | | | | The DCA core decoder converts integer coefficients read from the bitstream to floats just after reading them (along with dequantization). All the other steps of the audio reconstruction are done with floats which makes the output for the DTS lossless extension (XLL) actually lossy. This patch changes the DCA core to work with integer coefficients until QMF. At this point the integer coefficients are converted to floats. The coefficients for the LFE channel (lfe_data) are not touched. This is the first step for the really lossless XLL decoding.
* dca: Add math helpers.Alexandra Hájková2015-12-23
| | | | They will be used by the integer core decoder.
* checkasm: add fmtconvert testsJanne Grunau2015-12-21
|
* checkasm: add synth_filter testJanne Grunau2015-12-21
|
* checkasm: add tests for dcadspJanne Grunau2015-12-21
|
* checkasm: add float comparison util functionsJanne Grunau2015-12-21
|
* x86: checkasm: check for or handle missing cleanup after MMX instructionsJanne Grunau2015-12-21
| | | | | | | | Not every asm routine is expected clear the MMX state after returning. It is however a requisite for testing floating point code in checkasm. Annotate functions requiring cleanup with declare_func_emms() and issue emms after the call. The remaining functions are checked for having a cleared MMX state after return.
* arm64: fix inverted register order in transpose_4x4HJanne Grunau2015-12-21
| | | | | | Fix related register order issue in ff_h264_idct_add_neon. Found-by: zjh8890 <243186085@qq.com>
* exr: fix out of bounds read in get_codeAndreas Cadhalpun2015-12-16
| | | | | | | | This macro unconditionally used out[-1], which causes an out of bounds read, if out is the very beginning of the buffer. Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* opus: Fix typo causing overflow in silk_stabilize_lsfAndreas Cadhalpun2015-12-16
| | | | | | | | | | Due to this typo max_center can be too large, causing nlsf to be set to too large values, which in turn can cause nlsf[i - 1] + min_delta[i] to overflow to a negative value, which is not allowed for nlsf and can cause an out of bounds read in silk_lsf2lpc. Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* arm: add ff_int32_to_float_fmul_array8_neonJanne Grunau2015-12-14
| | | | | | | | | | | | | | | | | | | | | | | Quite a bit faster than int32_to_float_fmul_array8_c calling ff_int32_to_float_fmul_scalar_neon through FmtConvertContext. Number of cycles per int32_to_float_fmul_array8 call while decoding padded.dts on exynos5422: before after change cortex-a7: 1270 951 -25% cortex-a15: 434 285 -34% checkasm --bench cycle counts: cortex-a15 cortex-a7 int32_to_float_fmul_array8_c: 1730.4 4384.5 int32_to_float_fmul_array8_neon_c: 571.5 1694.3 int32_to_float_fmul_array8_neon: 374.0 1448.8 Interesting are the differences between int32_to_float_fmul_array8_neon_c and int32_to_float_fmul_array8_neon. The former is current behaviour of calling ff_int32_to_float_fmul_scalar_neon repeatedly from the c function, The raw numbers differ since checkasm uses different lengths than the dca decoder.
* arm64: int32_to_float_fmul neon asmJanne Grunau2015-12-14
| | | | | | | | | | 3% faster dts decoding on a cortex-a57. cortex-a57 cortex-a53 int32_to_float_fmul_array8_c: 1270.9 4475.6 int32_to_float_fmul_array8_neon: 328.6 569.2 int32_to_float_fmul_scalar_c: 928.5 4119.6 int32_to_float_fmul_scalar_neon: 309.1 524.1
* arm64: port synth_filter_float_neon from armJanne Grunau2015-12-14
| | | | | | | | | | | | | | ~25% faster dts decoding overall. The checkasm CPU cycles numbers are not that useful since synth_filter_float() calls FFTContext.imdct_half(). cortex-a57 cortex-a53 synth_filter_float_c: 1866.2 3490.9 synth_filter_float_neon: 915.0 1531.5 With fftc.imdct_half forced to imdct_half_neon: cortex-a57 cortex-a53 synth_filter_float_c: 1718.4 3025.3 synth_filter_float_neon: 926.2 1530.1
* arm64: convert dcadsp neon asm from armJanne Grunau2015-12-14
| | | | | | | | | | | | ~2% faster dts decoding overall. cortex-a57 cortex-a53 dca_decode_hf_c: 474.8 1659.9 dca_decode_hf_neon: 225.2 301.1 dca_lfe_fir0_c: 913.2 1537.7 dca_lfe_fir0_neon: 286.8 451.9 dca_lfe_fir1_c: 848.7 1711.5 dca_lfe_fir1_neon: 387.1 506.4
* arm: add a cpu flag for the VFPv2 vector modeJanne Grunau2015-12-14
| | | | | | | | | | | | | | The vector mode was deprecated in ARMv7-A/VFPv3 and various cpu implementations do not support it in hardware. Vector mode code will depending the OS either be emulated in software or result in an illegal instruction on cpus which does not support it. This was not really problem in practice since NEON implementations of the same functions are preferred. It will however become a problem for checkasm which tests every cpu flag separately. Since this is a cpu feature newer cpu do not support anymore the behaviour of this flag differs from the other flags. It can be only activated by runtime cpu feature selection.
* x86_64: int32_to_float_fmul_scalar sign extend integer lengthJanne Grunau2015-12-14
|