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

chore: fix instant panic on some case #3308

Merged
merged 1 commit into from
Feb 21, 2022

Conversation

driftluo
Copy link
Collaborator

@driftluo driftluo commented Feb 17, 2022

What problem does this PR solve?

ref: rust-lang/rust#89926

upgrade tokio/tentacle/parking_lot

tokio:

### Fixed

- time: prevent panicking in `sleep` with large durations ([#4495])
- time: eliminate potential panics in `Instant` arithmetic on platforms where
  `Instant::now` is not monotonic ([#4461])
- io: fix `DuplexStream` not participating in cooperative yielding ([#4478])
- rt: fix potential double panic when dropping a `JoinHandle` ([#4430])

### Changed

- update minimum supported Rust version to 1.49 ([#4457])
- update `parking_lot` dependency to v0.12.0 ([#4459])
- update `mio` dependency to v0.8 ([#4449])
- rt: remove an unnecessary lock in the blocking pool ([#4436])
- rt: remove an unnecessary enum in the basic scheduler ([#4462])
- time: use bit manipulation instead of modulo to improve performance ([#4480])
- net: use `std::future::Ready` instead of our own `Ready` future ([#4271])
- replace deprecated `atomic::spin_loop_hint` with `hint::spin_loop` ([#4491])
- fix miri failures in intrusive linked lists ([#4397]) 

Check List

Tests

  • Unit test
  • Integration test

Release note

Title Only: Include only the PR title in the release note.

@driftluo driftluo requested a review from a team as a code owner February 17, 2022 08:29
@driftluo driftluo requested review from zhangsoledad and removed request for a team February 17, 2022 08:29
@zhangsoledad zhangsoledad added s:ready-to-merge Status: Waiting to be merged. s:waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. labels Feb 21, 2022
@zhangsoledad
Copy link
Member

@driftluo conflicts

@zhangsoledad zhangsoledad removed the s:waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. label Feb 21, 2022
@zhangsoledad
Copy link
Member

bors r+

@bors bors bot merged commit 20b50b5 into nervosnetwork:develop Feb 21, 2022
@driftluo driftluo deleted the instant-panic branch February 21, 2022 10:12
@driftluo driftluo removed the s:ready-to-merge Status: Waiting to be merged. label Jun 6, 2022
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