Skip to content

[23545] Avoid setting thread affinity of 0 in mac#5962

Merged
cferreiragonz merged 1 commit intomasterfrom
fix/avoid_setting_0_affinity_mac
Aug 1, 2025
Merged

[23545] Avoid setting thread affinity of 0 in mac#5962
cferreiragonz merged 1 commit intomasterfrom
fix/avoid_setting_0_affinity_mac

Conversation

@cferreiragonz
Copy link
Copy Markdown
Contributor

@cferreiragonz cferreiragonz commented Jul 30, 2025 •
edited
Loading

Description

This PR avoids setting thread affinity with values equal to zero in MAC.

@Mergifyio backport 3.3.x 3.2.x 2.14.x

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The code follows the style guidelines of this project.
  • N/A Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
    -N/A Any new/modified methods have been properly documented using Doxygen.
  • N/A Any new configuration API has an equivalent XML API (with the corresponding XSD extension)
  • Changes are backport compatible: they do NOT break ABI nor change library core behavior.
  • Changes are API compatible.
  • N/A New feature has been added to the versions.md file (if applicable).
  • N/A New feature has been documented/Current behavior is correctly described in the documentation.
  • Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • If this is a critical bug fix, backports to the critical-only supported branches have been requested.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

@cferreiragonz cferreiragonz added this to the v3.4.0 milestone Jul 30, 2025
@cferreiragonz
Refs #23545: Avoid setting thread affinity of 0 in mac
1379383 
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
@cferreiragonz cferreiragonz force-pushed the fix/avoid_setting_0_affinity_mac branch from a7c268c to 1379383 Compare July 31, 2025 06:27
@cferreiragonz cferreiragonz requested review from richiprosima and removed request for richiprosima July 31, 2025 06:27
@github-actions github-actions Bot added the ci-pending PR which CI is running label Jul 31, 2025
@cferreiragonz cferreiragonz merged commit bc89cda into master Aug 1, 2025
37 of 38 checks passed
@cferreiragonz cferreiragonz deleted the fix/avoid_setting_0_affinity_mac branch August 1, 2025 05:41
@cferreiragonz cferreiragonz added ready-to-merge Ready to be merged. CI and changes have been reviewed and approved. and removed ci-pending PR which CI is running labels Aug 1, 2025
@cferreiragonz cferreiragonz restored the fix/avoid_setting_0_affinity_mac branch August 1, 2025 05:44
@MiguelCompany
Copy link
Copy Markdown
Member

MiguelCompany commented Aug 1, 2025

@Mergifyio backport 3.3.x 3.2.x 2.14.x

@MiguelCompany MiguelCompany deleted the fix/avoid_setting_0_affinity_mac branch August 1, 2025 05:44
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Aug 1, 2025 •
edited
Loading

backport 3.3.x 3.2.x 2.14.x

✅ Backports have been created

Details

@MiguelCompany MiguelCompany restored the fix/avoid_setting_0_affinity_mac branch August 1, 2025 05:45
mergify Bot pushed a commit that referenced this pull request Aug 1, 2025
@cferreiragonz @mergify
[23545] Avoid setting thread affinity of 0 in mac (#5962)
d2e106d 
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit bc89cda)
mergify Bot pushed a commit that referenced this pull request Aug 1, 2025
@cferreiragonz @mergify
[23545] Avoid setting thread affinity of 0 in mac (#5962)
af91e2b 
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit bc89cda)
mergify Bot pushed a commit that referenced this pull request Aug 1, 2025
@cferreiragonz @mergify
[23545] Avoid setting thread affinity of 0 in mac (#5962)
7a1f893 
Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
(cherry picked from commit bc89cda)
This was referenced Aug 1, 2025
Merged
Merged
Merged
cferreiragonz added a commit that referenced this pull request Aug 27, 2025
@mergify @cferreiragonz
Avoid setting thread affinity of 0 in mac (#5962) (#5971)
110b0a7 
(cherry picked from commit bc89cda)

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
MiguelCompany pushed a commit that referenced this pull request Sep 2, 2025
@mergify @cferreiragonz
Avoid setting thread affinity of 0 in mac (#5962) (#5970)
0bc405c 
(cherry picked from commit bc89cda)

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
@MiguelCompany MiguelCompany deleted the fix/avoid_setting_0_affinity_mac branch September 12, 2025 09:00
MiguelCompany pushed a commit that referenced this pull request Sep 17, 2025
@mergify @cferreiragonz
Avoid setting thread affinity of 0 in mac (#5962) (#5969)
d8effe7 
(cherry picked from commit bc89cda)

Signed-off-by: cferreiragonz <carlosferreira@eprosima.com>
Co-authored-by: Carlos Ferreira González <carlosferreira@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Mario-DL Mario-DL Mario-DL approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ready-to-merge Ready to be merged. CI and changes have been reviewed and approved.

Projects

None yet

Milestone

v3.4.0 - Frozen ❄️

Development

Successfully merging this pull request may close these issues.

3 participants

@cferreiragonz @MiguelCompany @Mario-DL