Skip to content

[non-normative] Change default build settings#5020

Merged
leolzhao merged 5 commits into
AOMediaCodec:av2-normativefrom
urvangjoshi:disable_simd_threads
Jun 3, 2026
Merged

[non-normative] Change default build settings#5020
leolzhao merged 5 commits into
AOMediaCodec:av2-normativefrom
urvangjoshi:disable_simd_threads

Conversation

@urvangjoshi
Copy link
Copy Markdown
Contributor

@urvangjoshi urvangjoshi commented Jun 1, 2026

This patch changes the default build settings to disable both multi-threading and SIMD on the normative branch, as requested by the AOMedia Codec Working Group.

Also, two pre-requisite changes:

  • Static analyzer warning fixes in C-only build
  • Disable a multi-thread unit test when CONFIG_MULTITHREAD is off
  • When CONFIG_MULTITHREAD=0, force unit tests to use 1 thread
  • av2_update_states_c(): Avoid integer wrap-around

#5023 #5022 #5025

Note: Needs to be merged without squashing to merge commits separately.

@urvangjoshi
Copy link
Copy Markdown
Contributor Author

There seem to be some CI failures. I'll check in a bit @leolzhao

@urvangjoshi urvangjoshi force-pushed the disable_simd_threads branch from efb1d80 to 3daf433 Compare June 1, 2026 21:58
@urvangjoshi urvangjoshi force-pushed the disable_simd_threads branch from 3daf433 to 6ac0340 Compare June 1, 2026 22:59
@urvangjoshi urvangjoshi marked this pull request as draft June 2, 2026 21:07
@urvangjoshi
Copy link
Copy Markdown
Contributor Author

A number of *MultiLayer* tests seem to have encode/decode mismatch in C-only build.
Converted PR to draft while I investigate.

@urvangjoshi urvangjoshi force-pushed the disable_simd_threads branch 2 times, most recently from f0b813b to 63aca58 Compare June 2, 2026 23:14
Also,
- Remove multi-threaded tests from compilation
- Ensure that encoder API doesn't allow specifying more than 1
thread.
This patch changes the default build settings to disable
both multi-threading and SIMD on the normative branch, as
requested by the AOMedia Codec Working Group.
@urvangjoshi urvangjoshi force-pushed the disable_simd_threads branch from 63aca58 to ae55ddd Compare June 3, 2026 17:28
@urvangjoshi urvangjoshi marked this pull request as ready for review June 3, 2026 18:45
@urvangjoshi
Copy link
Copy Markdown
Contributor Author

@leolzhao : I have fixed the issues found by CI into separate commits. Pls review again.
Note: Use "Rebase and merge" to merge commits without squashing.

@leolzhao leolzhao merged commit afc656b into AOMediaCodec:av2-normative Jun 3, 2026
109 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants