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.

Sign up for GitHub

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

Jump to bottom

use better random source #938

Merged
merged 1 commit into from
Apr 10, 2024
Merged

use better random source #938

merged 1 commit into from
Apr 10, 2024

Conversation

donghaobo
Copy link
Contributor

use better random source

@ywc689
Copy link
Collaborator

ywc689 commented Feb 27, 2024

Could you please evaluate the performance differences between the old and the new random sources?

@donghaobo
Copy link
Contributor Author

Could you please evaluate the performance differences between the old and the new random sources?

generate_random_key is only called at startup, there is no difference in packet processing.
open read and close take about 300us

@ywc689 ywc689 added pr/codes-reviewed-ok code review passed and no problem found pr/accepted the pr passed all review stages and await to be merged labels Mar 6, 2024
@ywc689 ywc689 merged commit cead683 into iqiyi:devel Apr 10, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ywc689 ywc689 ywc689 approved these changes

Assignees
No one assigned
Labels
pr/accepted the pr passed all review stages and await to be merged pr/codes-reviewed-ok code review passed and no problem found
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@donghaobo @ywc689