Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix big endian issue in Interlocked test cases #101429

Merged
merged 2 commits into from
Apr 25, 2024

Conversation

giritrivedi
Copy link
Contributor

Fix big indian issue wiht BitConverted as part of intercloked test cases.
These were causing failures in runtime_community pipeline

Fix big indian issue wiht BitConverted as part of intercloked
test cases.
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Apr 23, 2024
Copy link
Contributor

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

@giritrivedi
Copy link
Contributor Author

/azp run runtime-community

Copy link

Commenter does not have sufficient privileges for PR 101429 in repo dotnet/runtime

Fix big indian issue wiht BitConverted as part of intercloked test cases.
These were causing failures in runtime_community pipeline
@giritrivedi
Copy link
Contributor Author

@akoeplinger , can you run "runtime community" pipeline on this. Lets see if the fix holds good for System.Threads.tests failures seen as part of #101380

@akoeplinger
Copy link
Member

/azp run runtime-community

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@giritrivedi
Copy link
Contributor Author

@akoeplinger Thanks for initiating the run.
The run CI looks green.
Do you mind enabling the disabled test cases which were causing hang and check if the fix still holds good ?

@akoeplinger akoeplinger marked this pull request as ready for review April 25, 2024 07:08
@akoeplinger
Copy link
Member

/ba-g failures are not related, this change only affects big endian platforms

@akoeplinger akoeplinger merged commit c028937 into dotnet:main Apr 25, 2024
149 of 156 checks passed
@akoeplinger
Copy link
Member

akoeplinger commented Apr 25, 2024

Do you mind enabling the disabled test cases which were causing hang and check if the fix still holds good ?

Yep, I'm doing it in #101527 so that we don't need to restart CI if it doesn't.

@giritrivedi giritrivedi deleted the SystemThreads_CiFix branch April 29, 2024 10:13
matouskozak pushed a commit to matouskozak/runtime that referenced this pull request Apr 30, 2024
Fix big indian issue wiht BitConverted as part of intercloked test cases.
These were causing failures in runtime_community pipeline

---------

Co-authored-by: root <root@t8375036.lnxero1.boe>
michaelgsharp pushed a commit to michaelgsharp/runtime that referenced this pull request May 9, 2024
Fix big indian issue wiht BitConverted as part of intercloked test cases.
These were causing failures in runtime_community pipeline

---------

Co-authored-by: root <root@t8375036.lnxero1.boe>
Ruihan-Yin pushed a commit to Ruihan-Yin/runtime that referenced this pull request May 30, 2024
Fix big indian issue wiht BitConverted as part of intercloked test cases.
These were causing failures in runtime_community pipeline

---------

Co-authored-by: root <root@t8375036.lnxero1.boe>
@github-actions github-actions bot locked and limited conversation to collaborators May 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Threading community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants