443af7c29d
[ Upstream commit 48ed4e799e8fbebae838dca404a8527763d41191 ] The MBM and MBA tests need to discover the event and umask with which to configure the performance event used to measure read memory bandwidth. This is done by parsing the /sys/bus/event_source/devices/uncore_imc_<imc instance>/events/cas_count_read file for each iMC instance that contains the formatted output: "event=<event>,umask=<umask>" Parsing of cas_count_read contents is done by initializing an array of MAX_TOKENS elements with tokens (deliminated by "=,") from this file. Remove the unnecessary append of a delimiter to the string needing to be parsed. Per the strtok() man page: "delimiter bytes at the start or end of the string are ignored". This has no impact on the token placement within the array. After initialization, the actual event and umask is determined by parsing the tokens directly following the "event" and "umask" tokens respectively. Iterating through the array up to index "i < MAX_TOKENS" but then accessing index "i + 1" risks array overrun during the final iteration. Avoid array overrun by ensuring that the index used within for loop will always be valid. Fixes: 1d3f08687d76 ("selftests/resctrl: Read memory bandwidth from perf IMC counter and from resctrl file system") Signed-off-by: Reinette Chatre <reinette.chatre@intel.com> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org> |
||
---|---|---|
.. | ||
cache.c | ||
cat_test.c | ||
cqm_test.c | ||
fill_buf.c | ||
Makefile | ||
mba_test.c | ||
mbm_test.c | ||
README | ||
resctrl.h | ||
resctrl_tests.c | ||
resctrl_val.c | ||
resctrlfs.c |
resctrl_tests - resctrl file system test suit Authors: Fenghua Yu <fenghua.yu@intel.com> Sai Praneeth Prakhya <sai.praneeth.prakhya@intel.com>, resctrl_tests tests various resctrl functionalities and interfaces including both software and hardware. Currently it supports Memory Bandwidth Monitoring test and Memory Bandwidth 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. BUILD ----- Run "make" to build executable file "resctrl_tests". RUN --- 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: sudo ./resctrl_tests OVERVIEW OF EXECUTION --------------------- A test case has four stages: - setup: mount resctrl file system, create group, setup schemata, move test process pids to tasks, start benchmark. - execute: let benchmark run - verify: get resctrl data and verify the data with another source, e.g. perf event. - teardown: umount resctrl and clear temporary files. ARGUMENTS --------- Parameter '-h' shows usage information. usage: resctrl_tests [-h] [-b "benchmark_cmd [options]"] [-t test list] [-n no_of_bits] -b benchmark_cmd [options]: run specified benchmark for MBM, MBA and CQM default benchmark is builtin fill_buf -t test list: run tests specified in the test list, e.g. -t mbm,mba,cqm,cat -n no_of_bits: run cache tests using specified no of bits in cache bit mask -p cpu_no: specify CPU number to run the test. 1 is default -h: help