-
Notifications
You must be signed in to change notification settings - Fork 5.1k
[release/7.0] Use CLOCK_BOOTTIME to calculate BootTime on linux (#67589) #83966
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
[release/7.0] Use CLOCK_BOOTTIME to calculate BootTime on linux (#67589) #83966
Conversation
…23.1 (dotnet#83860) Microsoft.NET.Workload.Emscripten.net6.Manifest-7.0.100 , Microsoft.NET.Workload.Emscripten.net7.Manifest-7.0.100 From Version 7.0.4 -> To Version 7.0.5 Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com> Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com> Co-authored-by: Simon Rozsival <simon@rozsival.com>
Tagging subscribers to this area: @dotnet/area-system-diagnostics-process Issue DetailsBackport of #67589 to release/7.0 /cc @am11 @adamsitnik Customer impactA customer reported that they can't use the This bug as already been fixed in .NET 8 but we didn't backport it to .NET 7.0 at that time. Fixes #83816 TestingManual testing and unit tests. RiskLow. The fix is well tested.
|
The CI failures seem related:
|
We would need to backport #75334 as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you for backporting both PRs @simonrozsival !
|
I'm retargeting this PR to the new Repo maintainers will now be allowed to merge their own servicing PR as long as it meets the requirements:
The new process is described here: runtime/docs/project/library-servicing.md. The infra team will be actively monitoring servicing PRs to ensure all requirements are met and to help with any issues. Let me know if you have any questions. |
…d 20230323.1 (dotnet#83860)" This reverts commit 6a533c1.
Thanks, @carlossanlop. |
|
@carlossanlop I believe all the requirements for merging the backport PR have been met so I'm merging it now. |
Backport of #67589 and #75334 to release/7.0
/cc @am11 @adamsitnik
Customer impact
A customer reported that they can't use the
Process.StartTime
API in their Android app (#83816). The problem is that recent Android versions don't allow reading the/proc/stat
file anymore. Instead, we use theclock_gettime
libc function.This bug as already been fixed in .NET 8 but we didn't backport it to .NET 7.0 at that time.
Fixes #83816
Testing
Manual testing and unit tests.
Risk
Low. The fix is well tested.