aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFenghua Yu <fenghua.yu@intel.com>2021-03-17 02:22:40 +0000
committerShuah Khan <skhan@linuxfoundation.org>2021-04-02 13:53:02 -0600
commitd7af3d0d515cbdf63b6c3398a3c15ecb1bc2bd38 (patch)
tree20af0598bcc057a31f7cd6ae6cebeb7f0d1ed41b
parentf5f16ae4fae9d4d51aa365a0e1d84d368bef53ea (diff)
downloadstable-d7af3d0d515cbdf63b6c3398a3c15ecb1bc2bd38.tar.gz
stable-d7af3d0d515cbdf63b6c3398a3c15ecb1bc2bd38.tar.xz
stable-d7af3d0d515cbdf63b6c3398a3c15ecb1bc2bd38.zip
selftests/resctrl: Fix missing options "-n" and "-p"
resctrl test suite accepts command line arguments (like -b, -t, -n and -p) as documented in the help. But passing -n and -p throws an invalid option error. This happens because -n and -p are missing in the list of characters that getopt() recognizes as valid arguments. Hence, they are treated as invalid options. Fix this by adding them to the list of characters that getopt() recognizes as valid arguments. Please note that the main() function already has the logic to deal with the values passed as part of these arguments and hence no changes are needed there. Tested-by: Babu Moger <babu.moger@amd.com> Signed-off-by: Fenghua Yu <fenghua.yu@intel.com> Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
-rw-r--r--tools/testing/selftests/resctrl/resctrl_tests.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/resctrl/resctrl_tests.c b/tools/testing/selftests/resctrl/resctrl_tests.c
index 4b109a59f72d..ac2269610aa9 100644
--- a/tools/testing/selftests/resctrl/resctrl_tests.c
+++ b/tools/testing/selftests/resctrl/resctrl_tests.c
@@ -73,7 +73,7 @@ int main(int argc, char **argv)
}
}
- while ((c = getopt(argc_new, argv, "ht:b:")) != -1) {
+ while ((c = getopt(argc_new, argv, "ht:b:n:p:")) != -1) {
char *token;
switch (c) {