-
Notifications
You must be signed in to change notification settings - Fork 173
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
Change the netperf test version and source patch #2668
Conversation
40af15b
to
7e7c76e
Compare
Hi @PaulYuuu Could you help me review it ? Thanks a lot. |
|
I have solved this issue, please help me review it again, thanks. |
If possible, it would be better to delete the unused netperf source tar( |
Hi @PaulYuuu I am not sure if this will affect others. |
1.Change the netperf test version 2.Change the path to get netperf Signed-off-by: Lei Yang <[email protected]>
As discussed at #2629, there are no tests to use them. However, we should not maintain too many netperf versions, so this step should be done now or in the future. You can also submit another PR to fix it, it's up to you. |
Hi @PaulYuuu I think I should sent a new patch to fix this problem, this is more targeted. And I need to check with performance colleagues on the path they use. |
Okay, then I have no other questions, please provide test results, thanks. |
Test Pass(except rhel bug) |
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.
Also passed on aarch64.
ACK
ID:1934952
Signed-off-by: Lei Yang [email protected]