summaryrefslogtreecommitdiff
path: root/libavcodec
Commit message (Collapse)AuthorAge
...
* lavc: Move brd_scale to codec private optionsVittorio Giovara2016-01-21
| | | | | | | | This option is only used by mpegvideoenc, It is a very codec-specific option, so deprecate the global variant. Set proper limits to the maximum allowed values. Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
* lavc: Move b_frame_strategy and b_sensitivity to codec private optionsVittorio Giovara2016-01-21
| | | | | | | | | | The b_frame_strategy option is only used by mpegvideoenc, qsv, x264, and xavs, while b_sensitivity is only used by mpegvideoenc. These are very codec-specific options, so deprecate the global variants. Set proper limits to the maximum allowed values. Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
* riff: add YUYV FourCC (Drastic YUYV)Piotr Bandurski2016-01-21
|
* imgconvert: Add the proper API guards to a deprecated functionVittorio Giovara2016-01-21
|
* imgconvert: Move AVPicture-related static function to the deprecated sectionVittorio Giovara2016-01-21
|
* imgconvert: Move the shrink functions only where neededVittorio Giovara2016-01-21
|
* imgconvert: Drop outdated comment blockVittorio Giovara2016-01-21
|
* libkvazaar: Set frame rate as a rational numberArttu Ylä-Outinen2016-01-19
| | | | | | | | | | | | Updates libkvazaar to pass the exact frame rate to Kvazaar by setting the numerator and denominator separately instead of a single floating point number. The exact frame rate is needed for writing timing info to the bitstream. Requires Kvazaar version 0.8.1. Signed-off-by: Arttu Ylä-Outinen <arttu.yla-outinen@tut.fi> Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* lavc: Make sure that the effective timebase would not overflowArttu Ylä-Outinen2016-01-19
| | | | | | | In the unlikely situation the user decides to set ticks_per_frame and timebase to a value large enough to overflow. Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* mathops: mips: Correctly enable loongson-specific assemblyLuca Barbato2016-01-19
| | | | | The code wrongly assumed that the instructions used are supported on mips64, while it is supported only on loongson cpus.
* x86: build: Group all encoder objects togetherDiego Biurrun2016-01-18
|
* x86: ac3dsp: Drop forward declaration for nonexisting functionDiego Biurrun2016-01-18
|
* dca: fix misaligned access in ff_dca_convert_bitstreamAndreas Cadhalpun2016-01-15
| | | | | | | | | | | | The function is used on unaligned buffers (such as those provided by AVPacket), accessing them as uint16_t causes SIGBUS crashes on architectures like SPARC. This fixes ubsan runtime error: load of misaligned address for type 'const uint16_t', which requires 2 byte alignment Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
* xwddec: prevent overflow of lsize * avctx->heightAndreas Cadhalpun2016-01-13
| | | | | | | | | This is used to check if the input buffer is larger enough, so if this overflows it can cause a false negative leading to a segmentation fault in bytestream2_get_bufferu. Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Anton Khirnov <anton@khirnov.net>
* on2avc: limit number of bits to 30 in get_egolombAndreas Cadhalpun2016-01-13
| | | | | | | | | | More don't fit into the integer output. Also use get_bits_long, since get_bits only supports reading up to 25 bits, while get_bits_long supports the full integer range. Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> Signed-off-by: Anton Khirnov <anton@khirnov.net>
* nvenc: generate dts properlyAnton Khirnov2016-01-12
| | | | | | | | | | When there is a non-zero decoding delay due to reordering, the first dts should be lower than the first pts (since the first packet fed to the decoder does not produce any output). Use the same scheme used in mpegvideo_enc (which comes from x264 originally) -- wait for first two timestamps and extrapolate linearly to the past to produce the first dts value.
* nvenc: fix encoding with B-framesAnton Khirnov2016-01-12
| | | | | | | | | | | When B-frames are enabled and the encoder returns success, all currently pending buffers immediately become valid and can be returned to the caller. We can only return one packet at a time, so all the other pending buffers should be transferred to a new 'ready' fifo, from where they can be returned in subsequent calls (in which the encoder does not produce any new output). This bug was hidden by the incorrect testing of the encoder return value (the return value was overwritten before it was tested).
* nvenc: flush the encoder before closing it, as required by the docsAnton Khirnov2016-01-12
| | | | Otherwise, closing the encoder can crash.
* nvenc: rename encodersAnton Khirnov2016-01-12
| | | | | Change 'nvenc_<codec>' to '<codec>_nvenc', which is consistent with other similar decoders and encoders (QSV, MMAL).
* nvenc: better error handlingAnton Khirnov2016-01-12
| | | | Return proper error codes and print more descriptive error messages.
* 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>
* 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>
* 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>
* 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
* 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
|
* 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.
* 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
|
* lavc: make avpriv_mpa_decode_header private on next bumpAnton Khirnov2015-12-12
| | | | It's not used by anything outside of lavc anymore.