FFmpeg

Commit Graph

Author	SHA1	Message	Date
Andreas Rheinhardt	d163eefd47	avcodec/me_cmp, motion_est: Sanitize permissible cmp_funcs Several of the potential choices of comparison functions need an initialized MpegEncContext (initialized for encoding, not only ff_mpv_common_init()) or they crash when called. Modify ff_set_cmp() to check for this. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	7 months ago
Andreas Rheinhardt	b1a31b32ab	avcodec/me_cmp,dvenc,mpegvideo: Move ildct_cmp to its users MECmpContext.ildct_cmp is an array of function pointers that are not set by ff_me_cmp_init(), but that are set by users to one of the other arrays via ff_set_cmp(). Remove these pointers from MECmpContext and add pointers for the actually used functions to its users. (The DV encoder already did so.) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	7 months ago
Andreas Rheinhardt	cd2e46a350	avcodec/me_cmp, mpegvideo: Move frame_skip_cmp to MpegEncContext MECmpContext has several arrays of function pointers that are not set by ff_me_cmp_init(), but that are set by users to one of the other arrays via ff_set_cmp(). One of these other users is mpegvideo_enc; it is the only user of MECmpContext.frame_skip_cmp and it only uses one of these function pointers at all. This commit therefore moves this function pointer to MpegEncContext; and removes the array from MECmpContext. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	7 months ago
Andreas Rheinhardt	182e647a64	avcodec/me_cmp, motion_est: Move me_(pre_)?_cmp etc. to MotionEstContext MECmpContext has several arrays of function pointers that are not set by ff_me_cmp_init(), but that are set by users to one of the other arrays via ff_set_cmp(). One of these other users is the motion estimation API. It uses MECmpContext.(me_pre\|me\|me_sub\|mb)_cmp. It is basically the only user of these arrays. This commit therefore moves these arrays to MotionEstContext; this has the additional advantage of making motion_est.c more independent from MpegEncContext. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	7 months ago
Andreas Rheinhardt	1367ef198a	avcodec/me_cmp: Constify ff_set_cmp() Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	7 months ago
sunyuechi	b41e115dde	lavc/me_cmp: R-V V pix_abs C908: pix_abs_0_0_c: 534.0 pix_abs_0_0_rvv_i32: 136.2 pix_abs_1_0_c: 287.7 pix_abs_1_0_rvv_i32: 125.2 sad_0_c: 534.0 sad_0_rvv_i32: 136.2 sad_1_c: 287.7 sad_1_rvv_i32: 125.2 Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>	11 months ago
James Almer	7c6e26a184	avcodec/mp_cmp: reject invalid comparison function values Fixes tickets #10306 and #10318. Signed-off-by: James Almer <jamrial@gmail.com>	2 years ago
Andreas Rheinhardt	b9133bce04	avcodec/me_cmp: Mark ff_square_tab as hidden ff_square_tab is always used with an offset; if this table is marked as hidden, the compiler can infer that it and therefore also ff_square_tab + 256 have a fixed offset from the code. This allows to avoid performing "+ 256" at runtime by baking it into the offset from the code to the table. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	2 years ago
Andreas Rheinhardt	abb85429f3	avcodec/me_cmp: Constify me_cmp_func buffer parameters Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>	2 years ago
Swinney, Jonathan	c471cc7474	lavc/aarch64: motion estimation functions in neon - ff_pix_abs16_neon - ff_pix_abs16_xy2_neon In direct micro benchmarks of these ff functions verses their C implementations, these functions performed as follows on AWS Graviton 3. ff_pix_abs16_neon: pix_abs_0_0_c: 141.1 pix_abs_0_0_neon: 19.6 ff_pix_abs16_xy2_neon: pix_abs_0_3_c: 269.1 pix_abs_0_3_neon: 39.3 Tested with: ./tests/checkasm/checkasm --test=motion --bench --disable-linux-perf Signed-off-by: Jonathan Swinney <jswinney@amazon.com> Signed-off-by: Martin Storsjö <martin@martin.st>	3 years ago
Andreas Rheinhardt	b347384173	avcodec/me_cmp: Remove ff_check_alignment() The usage of a static variable presents a potential for data races and means that this function can't be used in init functions of codecs with FF_CODEC_CAP_INIT_THREADSAFE (unless of course one presumes that everything is alright in which case the error is not triggered; but then the whole function is pointless...). This makes the Snow decoder init-threadsafe as it already claims. Notice that this function has been removed in 2014 by Libav in commit `9103185bd1`, because only some codepaths are checked this way and because it only affects legacy compilers. The latter is of course even more true today. Reviewed-by: Anton Khirnov <anton@khirnov.net> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>	4 years ago
Muhammad Faiz	caa4bd7a9f	avcodec/me_cmp: remove ff_me_cmp_init_static() Precalculate and constify ff_square_tab. Reviewed-by: James Almer <jamrial@gmail.com> Signed-off-by: Muhammad Faiz <mfcc64@gmail.com>	7 years ago
Michael Niedermayer	d9883ded34	avcodec/me_cmp: Fix median_sad size Fixes out of array read Fixes: COV1396255 Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>	8 years ago
Stanislav Dolganov	4edd74bd7c	avcodec/me_cmp: add median SAD compare function Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>	8 years ago
Shivraj Patil	709bb45c66	avcodec/mips: MSA (MIPS-SIMD-Arch) optimizations for me_cmp functions This patch adds MSA (MIPS-SIMD-Arch) optimizations for me_cmp functions in new file me_cmp_msa.c Signed-off-by: Shivraj Patil <shivraj.patil@imgtec.com> Signed-off-by: Michael Niedermayer <michaelni@gmx.at>	10 years ago
Vittorio Giovara	9c12c6ff95	motion_est: convert stride to ptrdiff_t CC: libav-stable@libav.org Bug-Id: CID 700556 / CID 700557 / CID 700558	10 years ago
Diego Biurrun	2d60444331	dsputil: Split motion estimation compare bits off into their own context	11 years ago
Diego Biurrun	1173320249	dsputil: Drop unused bit_depth parameter from all init functions	11 years ago
Diego Biurrun	f46bb608d9	dsputil: Split off pixel block routines into their own context	11 years ago
Diego Biurrun	a9aee08d90	dsputil: Split off FDCT bits into their own context	11 years ago
Diego Biurrun	3c650efb81	dsputil: Move draw_edges() to mpegvideoencdsp	11 years ago
Diego Biurrun	c166148409	dsputil: Move pix_sum, pix_norm1, shrink function pointers to mpegvideoenc	11 years ago
Diego Biurrun	8d686ca59d	dsputil: Split off *_8x8basis to a separate context	11 years ago
Diego Biurrun	e3fcb14347	dsputil: Split off IDCT bits into their own context	11 years ago
Diego Biurrun	d2869aea04	dsputil: Move MMX/SSE2-optimized IDCT bits to the x86 subdirectory	11 years ago
Diego Biurrun	fab9df63a3	dsputil: Split off global motion compensation bits into a separate context	11 years ago
Diego Biurrun	c67b449beb	dsputil: Split bswap*_buf() off into a separate context	11 years ago
Diego Biurrun	9a9e2f1c8a	dsputil: Split audio operations off into a separate context	11 years ago
Diego Biurrun	e74433a8e6	dsputil: Split clear_block/fill_block off into a separate context	11 years ago
Diego Biurrun	880e2aa236	Remove all Blackfin architecture optimizations Blackfin is a painful platform to work with, no test machines are available and the range of multimedia applications is dubious. Thus it only represents a maintenance burden.	11 years ago
Diego Biurrun	368f50359e	dsputil: Split off quarterpel bits into their own context	11 years ago
Diego Biurrun	cb52a17cb6	dsputil: Move Bink-specific add_pixels8 to binkdsp	11 years ago
Diego Biurrun	054013a0fc	dsputil: Move APE-specific bits into apedsp	11 years ago
Diego Biurrun	256da0770e	dsputil: Move mspel_pixels_tab to the only place it is used	11 years ago
Diego Biurrun	3832a65901	dsputil: Move ff_alternate_*_scan tables to mpegvideo	11 years ago
Diego Biurrun	65d5d58658	dsputil: Move SVQ1 encoding specific bits into svq1enc	11 years ago
Diego Biurrun	512f3ffe9b	dsputil: Split off HuffYUV encoding bits into their own context Also shorten HuffYUV context member names to avoid clutter.	11 years ago
Diego Biurrun	0d439fbede	dsputil: Split off HuffYUV decoding bits into their own context Also shorten HuffYUV context member names to avoid clutter.	11 years ago
Diego Biurrun	5f92204370	dsputil: Move DV-specific ff_zigzag248_direct table to dvdata	11 years ago
Diego Biurrun	da5be23525	dsputil: Move RV40-specific bits into rv40dsp	11 years ago
Diego Biurrun	57f09608e1	dsputil: Move thirdpel-related bits into their own context	11 years ago
Diego Biurrun	322a1dda97	dsputil: Refactor duplicated CALL_2X_PIXELS / PIXELS16 macros	11 years ago
Diego Biurrun	f9279ee749	dsputil: Move ff_zigzag_direct and ff_crop_tab declarations to mathops.h	11 years ago
Diego Biurrun	82bb304801	dsputil: Use correct type in me_cmp_func function pointer	11 years ago
Diego Biurrun	5169e68895	dsputil: Propagate bit depth information to all (sub)init functions This avoids recalculating the value over and over again.	11 years ago
Diego Biurrun	05563ccacc	dsputil: cosmetics: Lose camelCase on ff_cropTbl and ff_squareTbl names Also switch from "tbl" to "tab" name suffixes.	11 years ago
Diego Biurrun	503dda7d4f	dsputil.h: K&R formatting cosmetics	11 years ago
Diego Biurrun	8199bb7be0	dsputil.h: Clean up comments	11 years ago
Diego Biurrun	ed9625eb62	dsputil: Move intreadwrite.h #include from header file to .c files	11 years ago
Diego Biurrun	9103185bd1	dsputil: Drop ff_check_alignment() function The function is supposed to confirm that the compiler provided enough alignment, but in practice it is only run in certain code paths and insufficient alignment problems are restricted to legacy compilers.	11 years ago

19 Commits (81a360a5ed9e7dcc841e791c2779f09cbd99b5d3)