Skip to content

Enable parallelism by default#278

Open
dag-erling wants to merge 1 commit into
freebsd:masterfrom
dag-erling:des/auto-parallelism
Open

Enable parallelism by default#278
dag-erling wants to merge 1 commit into
freebsd:masterfrom
dag-erling:des/auto-parallelism

Conversation

@dag-erling
Copy link
Copy Markdown
Member

If _SC_NPROCESSORS_ONLN is defined in <unistd.h> (as is the case in Solaris, Linux, and the four major BSDs), use it to set the default parallelism value to the number of available cores.

If _SC_NPROCESSORS_ONLN is defined in <unistd.h> (as is the case in
Solaris, Linux, and the four major BSDs), use it to set the default
parallelism value to the number of available cores.
@markjdb
Copy link
Copy Markdown
Member

markjdb commented Apr 24, 2026

There are still quite some tests which are flaky with parallelism enabled. With bricoler I've been trying to get test runs clean with 16-way parallelism. It's getting close, but there's more to do.

@dag-erling
Copy link
Copy Markdown
Member Author

There are still quite some tests which are flaky with parallelism enabled.

We can mark those is_exclusive until they're fixed.

@markjdb
Copy link
Copy Markdown
Member

markjdb commented Apr 25, 2026

There are still quite some tests which are flaky with parallelism enabled.

We can mark those is_exclusive until they're fixed.

Ok, but that has to happen before this change.

@ngie-eign
Copy link
Copy Markdown
Contributor

There are still quite some tests which are flaky with parallelism enabled.

We can mark those is_exclusive until they're fixed.

Ok, but that has to happen before this change.

Please keep in mind that this tool is used on multiple versions of FreeBSD and other OSes as well, and not all test suites that leverage this are in tree. You're better off changing the config in :main, instead of making this change globally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants