Builder ffmpeg-solaris10-sparc Build #12773
Results:
Failed shell_2 shell_3 shell_4 shell_5
SourceStamp:
| Project | ffmpeg |
| Repository | https://git.ffmpeg.org/ffmpeg.git |
| Branch | master |
| Revision | c22c2c5e033e3f73df47d88e49df86206f298e46 |
| Got Revision | c22c2c5e033e3f73df47d88e49df86206f298e46 |
| Changes | 14 changes |
BuildSlave:
unstable10sReason:
The SingleBranchScheduler scheduler named 'schedule-ffmpeg-solaris10-sparc' triggered this build
Steps and Logfiles:
-
git update ( 20 secs )
-
shell 'gsed -i ...' ( 0 secs )
-
shell_1 'gsed -i ...' ( 0 secs )
-
shell_2 'gsed -i ...' failed ( 1 secs )
-
shell_3 './configure --samples="../../../ffmpeg/fate-suite" ...' failed ( 9 secs )
-
shell_4 'gmake fate-rsync' failed ( 0 secs )
-
shell_5 '../../../ffmpeg/fate.sh ../../../ffmpeg/fate_config.sh' failed ( 0 secs )
Build Properties:
| Name | Value | Source |
|---|---|---|
| branch | master | Build |
| builddir | /export/home/buildbot-unstable10s/slave/ffmpeg-solaris10-sparc | slave |
| buildername | ffmpeg-solaris10-sparc | Builder |
| buildnumber | 12773 | Build |
| codebase | Build | |
| got_revision | c22c2c5e033e3f73df47d88e49df86206f298e46 | Git |
| project | ffmpeg | Build |
| repository | https://git.ffmpeg.org/ffmpeg.git | Build |
| revision | c22c2c5e033e3f73df47d88e49df86206f298e46 | Build |
| scheduler | schedule-ffmpeg-solaris10-sparc | Scheduler |
| slavename | unstable10s | BuildSlave |
| workdir | /export/home/buildbot-unstable10s/slave/ffmpeg-solaris10-sparc | slave (deprecated) |
Forced Build Properties:
| Name | Label | Value |
|---|
Responsible Users:
- Andreas Rheinhardtandreas.rheinhardt@outlook.com
Timing:
| Start | Wed Dec 3 10:58:28 2025 |
| End | Wed Dec 3 10:59:01 2025 |
| Elapsed | 32 secs |
All Changes:
:
Change #250782
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:20:41 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 358c569b05bc6f9a107a5caebcc8da56e8bf9799 Comments
avcodec/mpegvideo_unquantize: Constify MPVContext pointee Also use MPVContext instead of MpegEncContext. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/arm/mpegvideo_arm.c
- libavcodec/arm/mpegvideo_armv5te.c
- libavcodec/mips/h263dsp_mips.h
- libavcodec/mips/mpegvideo_mips.h
- libavcodec/mips/mpegvideo_mmi.c
- libavcodec/mips/mpegvideo_msa.c
- libavcodec/mpeg4videodec.h
- libavcodec/mpegvideo.h
- libavcodec/mpegvideo_unquantize.c
- libavcodec/mpegvideo_unquantize.h
- libavcodec/neon/mpegvideo.c
- libavcodec/ppc/mpegvideo_altivec.c
- libavcodec/x86/mpegvideo.c
Change #250783
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:20:42 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 011ef7fc65fcbf2141adcec9ca805874bb0a6a16 Comments
avcodec/ppc/mpegvideo_altivec: Split intra/inter unquantizing Don't use a single function that checks mb_intra. Forgotten in d50635cd247e17fe16c63219b9ae80d45a8185b1. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/ppc/mpegvideo_altivec.c
Change #250784
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:20:42 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 5d41d3e21dff14058b283491480a7382daeb5da9 Comments
avcodec/ppc/mpegvideo_altivec: Reindent after the previous commit Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/ppc/mpegvideo_altivec.c
Change #250785
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:20:42 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision e7a629049f7e9be397b0acabe75beb207ad9dc21 Comments
avcodec/{arm,neon}/mpegvideo: Use intra scantable to unquant H263 intra Forgotten in 70a7df049c411d9247eb6075720c84196c3e55e8. Using the wrong scantable matters for codecs for which both scantables can differ, namely the MPEG-4 decoder and the WMV1/2 codecs. For WMV1 it can lead to wrong output in case the IDCT permutation is FF_IDCT_PERM_PARTTRANS, because in this case the entries of of the intra scantable's raster end are not always <= the corresponding entries of the inter scantable's raster end when the former is initialized via ff_wmv1_scantable[1] and the latter via ff_wmv1_scantable[0]. FF_IDCT_PERM_PARTTRANS is used iff the Neon IDCT is used (for both arm and aarch64).* Said IDCT is not used during FATE, so that this issue went unnoticed. WMV2 uses the same scantables, but uses a custom IDCT which always uses FF_IDCT_PERM_NONE for which the inter_scantable, so that the output is always correct for it. The scantable for MPEG-4 can change mid-stream (for the decoder), but since c41818dc5dc14eb944761204e7b0ac179a6dcd1a only the intra scantable is updated, so that both scantables can get out of sync. In such a case the unquantize intra functions could unquantize an incorrect number of coefficients. Using raster_end of the wrong scantable can also lead to an unnecessarily large amount of coefficients unquantized. *: FF_IDCT_PERM_SIMPLE and FF_IDCT_PERM_TRANSPOSE would also not work, but they are not used at all by arm and aarch64. Reviewed-by: Martin Storsjö <martin@martin.st> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>Changed files
- libavcodec/arm/asm-offsets.h
- libavcodec/arm/mpegvideo_arm.c
- libavcodec/arm/mpegvideo_armv5te.c
- libavcodec/arm/mpegvideo_neon.S
- libavcodec/mpegvideo.h
- libavcodec/neon/mpegvideo.c
Change #250786
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:39 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 581050a1755b335cb106ad1b6c8e5f6fa9c19bd0 Comments
tests/checkasm: Add mpegvideo unquantize test This adds a test for the mpegvideo unquantize functions. It has been written in order to be able to easily bench these functions. It should be noted that the random input fed to the tested functions is not necessarily representative of the stuff actually occuring in the wild. So benchmarks should be taken with a grain of salt; but comparisons between two functions that do not depend on branch predictions are valid (the usecase for this is to port the x86 mmx functions to use xmm registers). During testing I have found a bug in the arm/aarch64 neon optimizations when using the LIBMPEG2 permutation (used by FF_IDCT_INT): The code seems to be based on the presumption that the remainder of the number of coefficients to process is always <= 4 mod 16. The test therefore sometimes fails for these arches. Hint: I am not certain that 16 bits are enough for the intermediate values of all the computations involved; e.g. both FLV and MPEG-4 escape values can go beyond that after the corresponding multiplications. The input in this test is nevertheless designed to fit into 16 bits. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- tests/checkasm/Makefile
- tests/checkasm/checkasm.c
- tests/checkasm/checkasm.h
- tests/checkasm/mpegvideo_unquantize.c
- tests/fate/checkasm.mak
Change #250787
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:39 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 357fc5243c32300bba91c096488e86558beed4c8 Comments
avcodec/{arm,neon}/mpegvideo: Fix h263 unquantize functions These functions currently operate on the assumption that the number of coefficients to process is always of the form 16k+m with m<=4 or >8. Yet this is not true when the IDCT permutation is of type FF_IDCT_PERM_LIBMPEG2 (i.e. when FF_IDCT_INT is in use). Reviewed-by: Martin Storsjö <martin@martin.st> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>Changed files
- libavcodec/arm/mpegvideo_neon.S
- libavcodec/neon/mpegvideo.c
Change #250788
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 0f7cc6aeeacba070d6d4b76a9f3a4d4036c3bb0b Comments
avcodec/mpegvideo: Move ff_init_scantable() to mpegvideo_unquantize.c This is necessary so that the mpegvideo_unquantize checkasm test does not pull mpegvideo.o and then all of libavcodec into checkasm. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/mpegvideo.c
- libavcodec/mpegvideo_unquantize.c
Change #250789
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 6d56807a06ce06712c65f8fcbf2a9a444bf59353 Comments
avcodec/x86/mpegvideo: Use correct inline assembly constraints The H.263 unquantize functions modified an input parameter. (And they did so since this code was added in 7f3f5ec87bcbf244fce49ffdb476d4ae6e523af6. I am surprised that this didn't cause issues, particularly with the intra function.) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/x86/mpegvideo.c
Change #250790
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 1fa8ffc1db2b62e475545bc6b117215704f9e1d8 Comments
avcodec/x86/mpegvideo: Improve unquantizing MPEG-2 intra blocks Unquantizing involves calculating (block[j] * qscale * quant_matrix[j]) / 16 where / rounds towards zero. Arithmetic right shifts naturally round towards -inf, so the earlier code calculated the absolute value first, then used a right-shift and then negated the result if necessary. This commit uses a different procedure: It biases the product for negative values of block[j] by 0xf. The combination of this and the arithmetic right shift is the same as rounding towards zero. Furthermore, a write-only store to mm7 has been removed. Benchmarks: dct_unquantize_mpeg2_intra_c: 214.3 ( 1.00x) dct_unquantize_mpeg2_intra_mmx (old): 43.0 ( 4.98x) dct_unquantize_mpeg2_intra_mmx (new): 28.4 ( 7.56x) (The bitexact flag and the test for correctness have beem removed from checkasm for the benchmarks.) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>Changed files
- libavcodec/x86/mpegvideo.c
Change #250791
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision a9a23925dfcf781dedc9cb910dd3097dd6224104 Comments
avcodec/x86/mpegvideo: Don't duplicate register Currently several inline ASM blocks used a value as an input and rax as clobber register. The input value was just moved into the register which then served as loop counter. This is wasteful, as one can just use the value's register directly as loop counter. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/x86/mpegvideo.c
Change #250792
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 1cb987d25bf4c8214461e12b01864b23c9bae67c Comments
avcodec/x86/mpegvideo: Port dct_unquantize_h263_{intra,inter}_mmx to SSSE3 It benefits from wider registers and psignw. Benchmarks: dct_unquantize_h263_inter_c: 88.3 ( 1.00x) dct_unquantize_h263_inter_mmx: 24.7 ( 3.58x) dct_unquantize_h263_inter_ssse3: 9.3 ( 9.47x) dct_unquantize_h263_intra_c: 93.7 ( 1.00x) dct_unquantize_h263_intra_mmx: 30.6 ( 3.06x) dct_unquantize_h263_intra_ssse3: 16.5 ( 5.69x) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>Changed files
- libavcodec/x86/mpegvideo.c
Change #250793
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 60084b136916a4dcace41e75a3b873e77eebd648 Comments
avcodec/x86/mpegvideo: Port MPEG-1 unquantize functions to SSSE3 Benefits from wider registers and pabsw, psignw. Benchmarks: dct_unquantize_mpeg1_inter_c: 343.0 ( 1.00x) dct_unquantize_mpeg1_inter_mmx: 50.6 ( 6.78x) dct_unquantize_mpeg1_inter_ssse3: 17.2 (19.94x) dct_unquantize_mpeg1_intra_c: 352.1 ( 1.00x) dct_unquantize_mpeg1_intra_mmx: 48.8 ( 7.22x) dct_unquantize_mpeg1_intra_ssse3: 19.5 (18.03x) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/mpegvideo.h
- libavcodec/x86/mpegvideo.c
Change #250794
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 6e2153111d5ff3b21a5303b7c23dd29de8a3bda6 Comments
avcodec/x86/mpegvideo: Port dct_unquantize_mpeg2_inter_mmx to SSSE3 Benefits from wider registers, pabsw and psignw. Benchmarks: dct_unquantize_mpeg2_inter_c: 131.2 ( 1.00x) dct_unquantize_mpeg2_inter_mmx: 50.2 ( 2.62x) dct_unquantize_mpeg2_inter_ssse3: 20.5 ( 6.38x) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/x86/mpegvideo.c
Change #250795
Category ffmpeg Changed by Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Changed at Wed 03 Dec 2025 10:23:43 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision c22c2c5e033e3f73df47d88e49df86206f298e46 Comments
avcodec/mpegvideo: Port dct_unquantize_mpeg2_intra_mmx to SSE2 Benefits from wider registers. Benchmarks: dct_unquantize_mpeg2_intra_c: 228.2 ( 1.00x) dct_unquantize_mpeg2_intra_mmx: 28.2 ( 8.10x) dct_unquantize_mpeg2_intra_sse2: 18.4 (12.37x) Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Changed files
- libavcodec/x86/mpegvideo.c
- tests/checkasm/mpegvideo_unquantize.c