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

Builder ffmpegsos-solaris10-sparc Build #12619

Results:

Failed shell_2 shell_3 shell_4 shell_5

SourceStamp:

Projectffmpeg
Repositoryhttps://git.ffmpeg.org/ffmpeg.git
Branchmaster
Revisionabb15241388c18d1d2587dd8eb06f5eb50e2ff6a
Got Revisionabb15241388c18d1d2587dd8eb06f5eb50e2ff6a
Changes8 changes

BuildSlave:

unstable10s

Reason:

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

Steps and Logfiles:

  1. git update ( 7 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 ( 2 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 12619 Build
codebase Build
got_revision abb15241388c18d1d2587dd8eb06f5eb50e2ff6a Git
project ffmpeg Build
repository https://git.ffmpeg.org/ffmpeg.git Build
revision abb15241388c18d1d2587dd8eb06f5eb50e2ff6a 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. Niklas Haas

Timing:

StartWed Dec 24 17:41:38 2025
EndWed Dec 24 17:42:00 2025
Elapsed21 secs

All Changes:

:

  1. Change #253206

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 61eca588dc6ac2066919ad56b09beb1f53aa35d4

    Comments

    swscale/ops: move ff_sws_op_list_update_comps() to ops.c
    I think this is ultimately a better home, since the semantics of this are
    not really tied to optimization itself; and because I want to make it an
    explicitly suported part of the user-facing API (rather than just an
    internal-use field).
    
    The secondary motivating reason here is that I intend to use internal
    helpers of `ops.c` inside the next commit. (Though this is a weak reason
    on its own, and not sufficient to justify this move by itself.)

    Changed files

    • libswscale/ops.c
    • libswscale/ops_optimizer.c
  2. Change #253207

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 9586b813730b32d52f9088c4365313bc1dfbba97

    Comments

    swscale/ops: don't strip SwsComps from SWS_OP_READ
    The current behavior of assuming the value range implicitly on SWS_OP_READ
    has a number of serious drawbacks and shortcomings:
    
    - It ignored the effects of SWS_OP_RSHIFT, such as for p010 and related
      MSB-aligned formats. (This is actually a bug)
    
    - It adds a needless dependency on the "purely informative" src/dst fields
      inside SwsOpList.
    
    - It is difficult to reason about when acted upon by SWS_OP_SWAP_BYTES, and
      the existing hack of simply ignoring SWAP_BYTES on the value range is not
      a very good solution here.
    
    Instead, we need a more principled way for the op list generating code
    to communicate extra metadata about the operations read to the optimizer.
    
    I think the simplest way of doing this is to allow the SwsComps field attached
    to SWS_OP_READ to carry additional, user-provided information about the values
    read.
    
    This requires changing ff_sws_op_list_update_comps() slightly to not completely
    overwrite SwsComps on SWS_OP_READ, but instead merge the implicit information
    with the explictly provided one.

    Changed files

    • libswscale/ops.c
    • libswscale/ops.h
  3. Change #253208

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 5a6602e95938c353eef39bbe5b30bbb720d1c708

    Comments

    swscale/format: add pixel range metadata to SWS_OP_READ

    Changed files

    • libswscale/format.c
  4. Change #253209

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision 5f1be98f62c1fb692e19c4c13ca342fa6606d733

    Comments

    swscale/ops: add SWS_COMP_SWAPPED
    This flag keeps track of whether a pixel is currently byte-swapped or
    not. Not needed by current backends, but informative and useful for
    catching potential endianness errors.
    
    Updates a lot of FATE tests with a cosmetic diff like this:
    
     rgb24 -> gray16be:
       [ u8 XXXX -> +++X] SWS_OP_READ         : 3 elem(s) packed >> 0
       [ u8 ...X -> +++X] SWS_OP_CONVERT      : u8 -> f32
       [f32 ...X -> .++X] SWS_OP_LINEAR       : dot3 [...]
       [f32 .XXX -> +++X] SWS_OP_CONVERT      : f32 -> u16
    -  [u16 .XXX -> +++X] SWS_OP_SWAP_BYTES
    -  [u16 .XXX -> +++X] SWS_OP_WRITE        : 1 elem(s) planar >> 0
    -    (X = unused, + = exact, 0 = zero)
    +  [u16 .XXX -> zzzX] SWS_OP_SWAP_BYTES
    +  [u16 .XXX -> zzzX] SWS_OP_WRITE        : 1 elem(s) planar >> 0
    +    (X = unused, z = byteswapped, + = exact, 0 = zero)
    
    (The choice of `z` to represent swapped integers is arbitrary, but I think
    it's visually evocative and distinct from the other symbols)

    Changed files

    • libswscale/format.c
    • libswscale/ops.c
    • libswscale/ops.h
    • tests/ref/fate/sws-ops-list
  5. Change #253210

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision a0032fb40fef1a8bf83530c5d4f8a294a7782674

    Comments

    swscale/ops: use switch/case for updating SwsComps ranges
    A bit more readable and makes it clear what the special cases are.

    Changed files

    • libswscale/ops.c
  6. Change #253211

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision ede8318e9f2f1f1f17f8be5ff4aa7ef7b0e0883e

    Comments

    swscale/ops: explicitly reset value range after SWS_OP_UNPACK
    The current logic implicitly pulled the new value range out of SwsComps using
    ff_sws_apply_op_q(), but this was quite ill-formed and not very robust. In
    particular, it only worked because of the implicit assumption that the value
    range was always set to 0b1111...111.
    
    This actually poses a serious problem for 32-bit packed formats, whose
    value range actually does not fit into AVRational. In the past, it only
    worked because the value would implicitly overflow to -1, which SWS_OP_UNPACK
    would then correctly extract the bits out from again.
    
    In general, it's cleaner (and sufficient) to just explicitly reset the value
    range on SWS_OP_UNPACK again.

    Changed files

    • libswscale/ops.c
  7. Change #253212

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision c0f49db53dd409f48b20ee6a828402ccce03dee5

    Comments

    swscale/ops: remove broken value range assumption hack
    This information is now pre-filled automatically for SWS_OP_READ when
    relevant.
    
    yuv444p10msbbe -> rgb24:
       [u16 XXXX -> +++X] SWS_OP_READ         : 3 elem(s) planar >> 0
       [u16 ...X -> +++X] SWS_OP_SWAP_BYTES
       [u16 ...X -> +++X] SWS_OP_RSHIFT       : >> 6
       [u16 ...X -> +++X] SWS_OP_CONVERT      : u16 -> f32
       [f32 ...X -> ...X] SWS_OP_LINEAR       : matrix3+off3 [...]
       [f32 ...X -> ...X] SWS_OP_DITHER       : 16x16 matrix + {0 3 2 5}
       [f32 ...X -> ...X] SWS_OP_MAX          : {0 0 0 0} <= x
    +  [f32 ...X -> ...X] SWS_OP_MIN          : x <= {255 255 255 _}
       [f32 ...X -> +++X] SWS_OP_CONVERT      : f32 -> u8
       [ u8 ...X -> +++X] SWS_OP_WRITE        : 3 elem(s) packed >> 0
        (X = unused, + = exact, 0 = zero)
    
    (This clamp is needed and was incorrectly optimized away before, because the
    `SWS_OP_RSHIFT` incorrectly distorted the value range assertion)

    Changed files

    • libswscale/ops.c
    • tests/ref/fate/sws-ops-list
  8. Change #253213

    Category ffmpeg
    Changed by Niklas Haas <gitohnoyoudont@haasn.dev>
    Changed at Wed 24 Dec 2025 17:37:22
    Repository https://git.ffmpeg.org/ffmpeg.git
    Project ffmpeg
    Branch master
    Revision abb15241388c18d1d2587dd8eb06f5eb50e2ff6a

    Comments

    Revert "swscale/ops: clarify SwsOpList.src/dst semantics"
    This reverts commit c94e8afe5d195fb08c441fbe3f8c2295081fcf8b.
    
    These are now actually purely informational.

    Changed files

    • libswscale/ops.h