Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Builder ffmpegsos-solaris10-sparc Build #13244

Results:

Failed shell_2 shell_3 shell_4 shell_5

SourceStamp:

Projectffmpeg
Repositoryhttps://git.ffmpeg.org/ffmpeg.git
Branchmaster
Revision32678dcc8833718ce5158af2b0d65397aa51c05f
Got Revision32678dcc8833718ce5158af2b0d65397aa51c05f
Changes13 changes

BuildSlave:

unstable10s

Reason:

The SingleBranchScheduler scheduler named 'schedule-ffmpegsos-solaris10-sparc' triggered this build

Steps and Logfiles:

  1. git update ( 21 secs )
    1. stdio
  2. shell 'gsed -i ...' ( 0 secs )
    1. stdio
  3. shell_1 'gsed -i ...' ( 0 secs )
    1. stdio
  4. shell_2 'gsed -i ...' failed ( 0 secs )
    1. stdio
  5. shell_3 './configure --samples="../../../ffmpeg/fate-suite" ...' failed ( 9 secs )
    1. stdio
    2. config.log
  6. shell_4 'gmake fate-rsync' failed ( 0 secs )
    1. stdio
  7. shell_5 '../../../ffmpeg/fate.sh ../../../ffmpeg/fate_config_sos.sh' failed ( 0 secs )
    1. stdio
    2. configure.log
    3. compile.log
    4. test.log

Build Properties:

NameValueSource
branch master Build
builddir /export/home/buildbot-unstable10s/slave/ffmpegsos-solaris10-sparc slave
buildername ffmpegsos-solaris10-sparc Builder
buildnumber 13244 Build
codebase Build
got_revision 32678dcc8833718ce5158af2b0d65397aa51c05f Git
project ffmpeg Build
repository https://git.ffmpeg.org/ffmpeg.git Build
revision 32678dcc8833718ce5158af2b0d65397aa51c05f Build
scheduler schedule-ffmpegsos-solaris10-sparc Scheduler
slavename unstable10s BuildSlave
workdir /export/home/buildbot-unstable10s/slave/ffmpegsos-solaris10-sparc slave (deprecated)

Forced Build Properties:

NameLabelValue

Responsible Users:

  1. Andreas Rheinhardt

Timing:

StartMon Apr 13 13:35:35 2026
EndMon Apr 13 13:36:08 2026
Elapsed32 secs

All Changes:

:

  1. Change #264276

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision eed0830a0cc33e9e7537983210fe71f0ac90f498

    Comments

    avcodec/snowdata: Don't use 8 bits for six bits data
    This has been done in 561a18d3ba07382dffdf583dda13f6954109b116
    in order to avoid shifts, yet this rationale no longer applies
    since d593e329832a432777218eb92469bad10594a3c5. So shift them back;
    this is in preparation for using these coefficients together with
    pmaddubsw.
    
    Hint: 561a18d3ba07382dffdf583dda13f6954109b116 also added a block
    guarded by "if(LOG2_OBMC_MAX == 8". I changed the condition to remove
    this check (i.e. kept the block) which should not change the output
    at all. Yet all FATE tests pass if the block is completely
    removed. I don't know if this block is necessary at all.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snow.c
    • libavcodec/snow.h
    • libavcodec/snowdata.h
    • libavcodec/snowenc.c
    • libavcodec/x86/snowdsp.c
  2. Change #264277

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 13d621cc7cd24ff1811b240aa81860c65366dcce

    Comments

    avcodec/snow: Disable dead code in ff_snow_inner_add_yblock()
    It is only used with add != 0 (and the assembly functions
    only support this case).
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snow.c
  3. Change #264278

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision fd77f00a8ff8f0558fef1bf7670c34b009414f2b

    Comments

    avcodec/snow: Avoid always-true branch
    The input lines used in ff_snow_inner_add_yblock()
    must always be set (because their values are used).
    The MMX assembly always relied on this.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snow.c
  4. Change #264279

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 5f373872c06ff8978b2996b2532bb08e2ba131fa

    Comments

    avcodec/x86/snow_dwt: Avoid slice_buffer in inner_add_yblock
    It is unnecessary and avoids the src_y parameter;
    it also makes this function more ASM-friendly.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snow.c
    • libavcodec/snow.h
    • libavcodec/snow_dwt.h
    • libavcodec/x86/snowdsp.c
  5. Change #264280

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 28d0a5091af0102526e4833ff640d8d772ddac8a

    Comments

    avcodec/snow_dwt: Remove pointless forward declaration
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snow_dwt.h
  6. Change #264281

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 764e0219468d008b957fd410c3b208337a9d9056

    Comments

    avcodec/snowdata: Add explicit alignment for obmc tables
    This is in preparation for adding SSSE3 assembly.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snowdata.h
  7. Change #264282

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:46:24
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision e0ed3fa834a8d7854e58190146d930ec824043d2

    Comments

    tests/checkasm: Add snowdsp test
    Only inner_add_yblock for now.
    Hint: Said function uses a pointer to an array of pointers as parameter.
    The MMX version clobbers the array in such a way that calling the
    function repeatedly with the same arguments (as happens inside bench_new())
    leads to buffer overflows and segfaults. Therefore CALL4 had to be
    overridden to restore the original pointers. This workaround will be
    removed soon when the MMX version is removed.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • tests/checkasm/Makefile
    • tests/checkasm/checkasm.c
    • tests/checkasm/checkasm.h
    • tests/checkasm/snowdsp.c
    • tests/fate/checkasm.mak
  8. Change #264283

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:51:34
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 4f30bd6fbaa1ce683114bf565dec8498d2f0bfcb

    Comments

    tests/checkasm/llvidencdsp: Fix nonsense randomization
    The first loop was never entered due to a precedence problem;
    the second loop initialized everything, although it was not intended
    that way.
    This has been added in 56b8769a1c26dac8fe51319f0c2b94de67a64217.
    Sorry for this.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • tests/checkasm/llvidencdsp.c
  9. Change #264284

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:51:35
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 2fdccaf7d6009b60cd839cdb82088bc078034327

    Comments

    tests/checkasm/mpegvideo_unquantize: Fix precedence problem
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • tests/checkasm/mpegvideo_unquantize.c
  10. Change #264285

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:51:35
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 5c830fccf4bd30481c3ccc03b8c99bd8791c8796

    Comments

    avcodec/x86/snowdsp: Add SSSE3 inner_add_yblock
    Compared to the MMX version, this version benefits from wider
    registers and pmaddubsw. It also has fewer unnecessary loads
    and stores: On x64, the MMX version has 12 unnecessary GPR loads
    and 6 stores in each line when width is eight; for width 16,
    there are 17 unnecessary GPR loads and six stores per line.
    Even the 32bit SSSE3 version only has six loads and zero stores
    per line more than the x64 version. Furthermore, in contrast
    to the MMX version, the SSSE3 version also does not clobber
    the array of block pointers given to it.
    
    Benchmarks:
    inner_add_yblock_2_c:                                   29.2 ( 1.00x)
    inner_add_yblock_2_mmx:                                 32.5 ( 0.90x)
    inner_add_yblock_2_ssse3:                               28.6 ( 1.02x)
    inner_add_yblock_4_c:                                   85.2 ( 1.00x)
    inner_add_yblock_4_mmx:                                 89.2 ( 0.96x)
    inner_add_yblock_4_ssse3:                               84.5 ( 1.01x)
    inner_add_yblock_8_c:                                  302.0 ( 1.00x)
    inner_add_yblock_8_mmx:                                 77.0 ( 3.92x)
    inner_add_yblock_8_ssse3:                               30.6 ( 9.85x)
    inner_add_yblock_16_c:                                1164.7 ( 1.00x)
    inner_add_yblock_16_mmx:                               260.4 ( 4.47x)
    inner_add_yblock_16_ssse3:                              82.3 (14.15x)
    
    Both the MMX and SSSE3 versions leave the size 2 and 4 cases
    to ff_snow_inner_add_yblock_c() (but the MMX version has
    a prologue at the beginning that it needs to undo before
    the call, leading to the higher overhead for these sizes).
    I don't know why the SSSE3 version is marginally faster than
    the C version in these cases.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/snow.c
    • libavcodec/snow_dwt.c
    • libavcodec/snow_dwt.h
    • libavcodec/x86/Makefile
    • libavcodec/x86/snowdsp.asm
    • libavcodec/x86/snowdsp.c
    • libavcodec/x86/snowdsp_init.c
  11. Change #264286

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:53:17
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 338dc256422701134457c6326f5604f528c2f381

    Comments

    avcodec/x86/snowdsp_init: Remove MMXEXT, SSE2 inner_add_yblock versions
    They have been superseded by SSSE3; the SSE2 version was even disabled
    (and segfaults if enabled).
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/x86/snowdsp_init.c
    • tests/checkasm/snowdsp.c
  12. Change #264287

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:56:01
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision bd2964e6110c676fe9e2bbc607fe4f470d80b400

    Comments

    avcodec/x86/snowdsp_init: Use standard init pattern
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/x86/snowdsp_init.c
  13. Change #264288

    Category ffmpeg
    Changed by Andreas Rheinhardt <andreas.rheinhardtohnoyoudont@outlook.com>
    Changed at Mon 13 Apr 2026 12:56:35
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 32678dcc8833718ce5158af2b0d65397aa51c05f

    Comments

    avcodec/x86/snowdsp_init: Remove disabled SSE2 functions
    Disabled in 3e0f7126b53b395d9e79df57b2e626eb99ad846b
    (almost 20 years ago) and no one fixed them, so remove them.
    
    Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>

    Changed files

    • libavcodec/x86/snowdsp_init.c