-
Notifications
You must be signed in to change notification settings - Fork 82
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
host_exerciser updates #2355
host_exerciser updates #2355
Conversation
- Read the AFU frequency from a new CSR. - Add tests for PCIe atomic functions, including validating that each function is performed properly. - Fill the source buffer with random data instead of the same pattern over and over so that read errors are more likely to be found. - Make the log level work. Dump a few lines of the buffers in some modes.
@@ -331,6 +356,10 @@ class host_exerciser : public test_afu { | |||
// Configures test rollover or test termination | |||
app_.add_option("--continuousmode", he_continuousmode_, "test rollover or test termination")->default_val("false"); | |||
|
|||
// Atomic function | |||
app_.add_option("--atomic", he_req_atomic_func_, "atomic requests (only permitted in combination with lpbk/cl_1)") |
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.
Should he_req_atomic_func_ be initialized in the constructor?
@@ -346,12 +375,14 @@ class host_exerciser : public test_afu { | |||
app_.add_option("--contmodetime", he_contmodetime_, | |||
"Continuous mode time in seconds")->default_val("0"); | |||
|
|||
app_.add_option("--clock-mhz", he_clock_mhz_, |
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.
Should he_clock_mhz_ be initialized in the constructor?
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.
Isn't that what default_val is for? That's how all the other params are handled, too.
|
||
// In atomic mode, at most the first 8 bytes of each line will be | ||
// updated and copied. In the source buffer, write a function of | ||
// the value at the start of each line to the second postion so |
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.
s/postion/position/
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.
Fixed typo. Please approve again.
625d825
host_exerciser updates - Read the AFU frequency from a new CSR. - Add tests for PCIe atomic functions, including validating that each function is performed properly. - Fill the source buffer with random data instead of the same pattern over and over so that read errors are more likely to be found. - Make the log level work. Dump a few lines of the buffers in some modes.
host_exerciser updates - Read the AFU frequency from a new CSR. - Add tests for PCIe atomic functions, including validating that each function is performed properly. - Fill the source buffer with random data instead of the same pattern over and over so that read errors are more likely to be found. - Make the log level work. Dump a few lines of the buffers in some modes. Co-authored-by: Michael Adler <Michael.Adler@intel.com>
is performed properly.
and over so that read errors are more likely to be found.