Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add System.IO.Hashing dependency to windowsdesktop transport pack#103695

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
ViktorHofer merged 1 commit intomainfromViktorHofer-patch-2
Jun 19, 2024

Conversation

@ViktorHofer
Copy link
Member

System.IO.Hashing is a dependency of System.Formats.Nrbf.

Didn't realize that until just now. Continuation of91f1481

System.IO.Hashing is a dependency of System.Formats.Nrbf.Didn't realize that until just now. Continuation of91f1481
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info inarea-owners.md if you want to be subscribed.

Copy link
Member

@MichalStrehovskyMichalStrehovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I'd consider bypassing the CI and just slamming it in and canceling whatever official build is in progress so that we can finish the flow today. Official build is going to be 4 hours.

ViktorHofer and adamsitnik reacted with thumbs up emoji
@ViktorHofer
Copy link
MemberAuthor

/ba-g this is a safe change (I double checked it) and we need this flowing asap for P6.

@ViktorHoferViktorHofer merged commit153296b intomainJun 19, 2024
@ViktorHoferViktorHofer deleted the ViktorHofer-patch-2 branchJune 19, 2024 11:16
@ViktorHofer
Copy link
MemberAuthor

PR and official builds taking 4 hours is just crazy, isn't it? We should bring this up and invest in reducing this to at least half.

@MichalStrehovsky
Copy link
Member

PR and official builds taking 4 hours is just crazy, isn't it? We should bring this up and invest in reducing this to at least half.

Yep. I canceled the official one that was in progress. It was only 20 minutes in, so it would take solid 8 hours to get a good build of dotnet/runtime out. Now we should have one in 4.

ViktorHofer reacted with thumbs up emoji

@am11
Copy link
Member

Maybe official build should use a better pool with 32-64 cores machines? Linux distros builders, e.g., use those kind of beefy machines for official builds. Stands to reason company which owns Azure cloud can handle a pool of dozen such machines for .NET. 😉

@ViktorHofer
Copy link
MemberAuthor

Agree on that. But the other angle is that we can build runtime so much faster than what the official build does today. In the current version of the VMR (without join steps) runtime builds in less than 30 minutes. I expect that this time will go up when we add join point legs and signing but the entire VMR will still be faster than the current runtime official build.

@MichalStrehovsky
Copy link
Member

https://dnceng.visualstudio.com/internal/_build/results?buildId=2477368&view=results

The PGO legs died on nuget restore issues. Anyone know if there's a way to rerun it while the build is still in progress? Or wait until it fails and then click rerun on the button that magically appears? Or just scrap the whole run?

am11 reacted with confused emoji

@stephentoub
Copy link
Member

System.IO.Hashing is a dependency of System.Formats.Nrbf.

Does it need to be?@adamsitnik, I see System.IO.Hashing being used in one place:

publicoverrideintGetHashCode(){intid=_id;#ifNETSpan<int>integers=new(refid);#elseSpan<int>integers=stackallocint[1]{id};#endifreturn(int)XxHash32.HashToUInt32(MemoryMarshal.AsBytes(integers));}

Why does this need to use XxHash32 rather than just being:

publicoverrideintGetHashCode()=>_id;

?

stephentoub added a commit to stephentoub/runtime that referenced this pull requestJun 19, 2024
stephentoub added a commit that referenced this pull requestJun 19, 2024
* Revert "Add System.IO.Hashing dependency to windowsdesktop transport pack (#103695)"This reverts commit153296b.* Remove System.IO.Hashing dependency from Nrbf* Use HashCode
@github-actionsgithub-actionsbot locked and limited conversation to collaboratorsJul 20, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@MichalStrehovskyMichalStrehovskyMichalStrehovsky approved these changes

Assignees

@ViktorHoferViktorHofer

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ViktorHofer@MichalStrehovsky@am11@stephentoub

[8]ページ先頭

©2009-2025 Movatter.jp