Builder ffmpegsos-solaris10-sparc Build #12619
Results:
Failed shell_2 shell_3 shell_4 shell_5
SourceStamp:
| Project | ffmpeg |
| Repository | https://git.ffmpeg.org/ffmpeg.git |
| Branch | master |
| Revision | abb15241388c18d1d2587dd8eb06f5eb50e2ff6a |
| Got Revision | abb15241388c18d1d2587dd8eb06f5eb50e2ff6a |
| Changes | 8 changes |
BuildSlave:
unstable10sReason:
The SingleBranchScheduler scheduler named 'schedule-ffmpegsos-solaris10-sparc' triggered this build
Steps and Logfiles:
-
git update ( 7 secs )
-
shell 'gsed -i ...' ( 0 secs )
-
shell_1 'gsed -i ...' ( 0 secs )
-
shell_2 'gsed -i ...' failed ( 0 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_sos.sh' failed ( 2 secs )
Build Properties:
| Name | Value | Source |
|---|---|---|
| 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:
| Name | Label | Value |
|---|
Responsible Users:
- Niklas Haasgit@haasn.dev
Timing:
| Start | Wed Dec 24 17:41:38 2025 |
| End | Wed Dec 24 17:42:00 2025 |
| Elapsed | 21 secs |
All Changes:
:
Change #253206
Category ffmpeg Changed by Niklas Haas <git@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
Change #253207
Category ffmpeg Changed by Niklas Haas <git@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
Change #253208
Category ffmpeg Changed by Niklas Haas <git@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
Change #253209
Category ffmpeg Changed by Niklas Haas <git@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
Change #253210
Category ffmpeg Changed by Niklas Haas <git@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
Change #253211
Category ffmpeg Changed by Niklas Haas <git@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
Change #253212
Category ffmpeg Changed by Niklas Haas <git@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
Change #253213
Category ffmpeg Changed by Niklas Haas <git@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