Hi Fenghua,
On Fri, Mar 04, 2022 at 07:38:33PM +0900, Shaopeng Tan wrote:
In this patch series, I make restrl_tests build/run using kselftest framework, but some users do not known how to build/run resctrl_tests using kseltest framework.
Please don't use "I" or "we" in commit messages. Also the grammar seems not right here.
Add manual of how to make resctrl_tests build/run using kselftest framework into README.
Maybe change the commit message to this:
resctrl_tests can be built or run using kselftests framework. Add description on how to do so in README.
Thanks for your advice. I will use it for commit log in next version.
Signed-off-by: Shaopeng Tan tan.shaopeng@jp.fujitsu.com
tools/testing/selftests/resctrl/README | 31 +++++++++++++++++++++++++- 1 file changed, 30 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/resctrl/README b/tools/testing/selftests/resctrl/README index 3d2bbd4fa3aa..268cf3f95bd5 100644 --- a/tools/testing/selftests/resctrl/README +++ b/tools/testing/selftests/resctrl/README @@ -12,9 +12,37 @@ Allocation test on Intel RDT hardware. More tests will
be added in the future.
And the test suit can be extended to cover AMD QoS and ARM MPAM hardware as well.
+resctrl_tests can be run with or without kselftest framework.
+USE KSELFTEST FRAMEWORK +-----------------------
+BUILD +-----
The "---" under titles are all same. This cannot tell readers clearly topic hierarchies.
How about this? +USE KSELFTEST FRAMEWORK +-----------------------
+* BUILD
+Execute the following command in top level directory of the kernel source.
+Build resctrl:
- $ make -C tools/testing/selftests TARGETS=resctrl
+RUN +---
How about this? +* RUN
+Run resctrl:
- $ make -C tools/testing/selftests TARGETS=resctrl run_tests
Run this as sudo or root.
- $ sudo make -C tools/testing/selftests TARGETS=resctrl run_tests
Thanks, I will take your above advice in next version.
+Using kselftest framework, the ./resctrl_tests will be run without any
parameters.
+More details about kselftest framework as follow. +Documentation/dev-tools/kselftest.rst
+NOT USE KSELFTEST FRAMEWORK +---------------------------
BUILD
+Execute the following command in this
directory(tools/testing/selftests/resctrl/).
Run "make" to build executable file "resctrl_tests".
RUN @@ -24,7 +52,8 @@ To use resctrl_tests, root or sudoer privileges are required. This is because the test needs to mount resctrl file system and change contents in the file system.
-Executing the test without any parameter will run all supported tests: +Executing the test without any parameter will run all supported tests. +It takes about 68 seconds on a Intel(R) Xeon(R) Gold 6254 CPU @ 3.10GHz.
resctrl will add more tests in the future. Running time may be longer in the future. I would suggest to remove the "It takes about 68 seconds..." line.
I added this sentence based on Shuah Khan's feedback, but I didn't consider extending this test. I will delete this sentence in next version.
sudo ./resctrl_tests
-- 2.27.0
Best regards, Tan Shaopeng