[PATCH][next] selftests: sched: Fix spelling mistake "hiearchy" -> "hierarchy"

[Colin Ian King]

There is a spelling mistake in a printf message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/sched/cs_prctl_test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/sched/cs_prctl_test.c 
b/tools/testing/selftests/sched/cs_prctl_test.c
index 3e1619b6bf2d..7b4fc02a0d05 100644
--- a/tools/testing/selftests/sched/cs_prctl_test.c
+++ b/tools/testing/selftests/sched/cs_prctl_test.c
@@ -276,7 +276,7 @@ int main(int argc, char *argv[])
if (setpgid(0, 0) != 0)
handle_error("process group");
 
-   printf("\n## Create a thread/process/process group hiearchy\n");
+   printf("\n## Create a thread/process/process group hierarchy\n");
create_processes(num_processes, num_threads, procs);
need_cleanup = 1;
disp_processes(num_processes, procs);
-- 
2.39.2

[PATCH][next] selftests/mm: Fix spelling mistake "succedded" -> "succeeded"

[Colin Ian King]

There is a spelling mistake in a ksft_exit_fail_msg message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/mm/vm_util.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/mm/vm_util.c 
b/tools/testing/selftests/mm/vm_util.c
index 4aeb8d5299ff..05736c615734 100644
--- a/tools/testing/selftests/mm/vm_util.c
+++ b/tools/testing/selftests/mm/vm_util.c
@@ -75,7 +75,7 @@ static bool pagemap_scan_supported(int fd, char *start)
/* Provide an invalid address in order to trigger EFAULT. */
ret = __pagemap_scan_get_categories(fd, start, (struct page_region *) 
~0UL);
if (ret == 0)
-   ksft_exit_fail_msg("PAGEMAP_SCAN succedded unexpectedly\n");
+   ksft_exit_fail_msg("PAGEMAP_SCAN succeeded unexpectedly\n");
 
supported = errno == EFAULT;
 
-- 
2.39.2

[PATCH][NEXT] selftests/bpf: Fix spelling mistake "get_signaure_size" -> "get_signature_size"

[Colin Ian King]

There is a spelling mistake in an ASSERT_GT message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/bpf/prog_tests/verify_pkcs7_sig.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/verify_pkcs7_sig.c 
b/tools/testing/selftests/bpf/prog_tests/verify_pkcs7_sig.c
index 6c90372b772d..ab0f02faa80c 100644
--- a/tools/testing/selftests/bpf/prog_tests/verify_pkcs7_sig.c
+++ b/tools/testing/selftests/bpf/prog_tests/verify_pkcs7_sig.c
@@ -498,7 +498,7 @@ static void test_pkcs7_sig_fsverity(void)
if (!ASSERT_OK_PTR(skel, "test_sig_in_xattr__open"))
goto out;
ret = get_signature_size(sig_path);
-   if (!ASSERT_GT(ret, 0, "get_signaure_size"))
+   if (!ASSERT_GT(ret, 0, "get_signature_size"))
goto out;
skel->bss->sig_size = ret;
skel->bss->user_keyring_serial = syscall(__NR_request_key, "keyring",
-- 
2.39.2

[PATCH][next] powerpc/selftests: Fix spelling mistake "EACCESS" -> "EACCES"

[Colin Ian King]

There is a spelling mistake of the EACCES error name, fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c 
b/tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c
index d5436de5b8ed..f56c15a11e2f 100644
--- a/tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c
+++ b/tools/testing/selftests/powerpc/papr_sysparm/papr_sysparm.c
@@ -177,7 +177,7 @@ static const struct sysparm_test sysparm_tests[] = {
},
{
.function = set_with_ro_fd,
-   .description = "PAPR_IOC_SYSPARM_SET returns EACCESS on 
read-only fd",
+   .description = "PAPR_IOC_SYSPARM_SET returns EACCES on 
read-only fd",
},
 };
 
-- 
2.39.2

[PATCH][next] selftests/net: Fix various spelling mistakes in TCP-AO tests

[Colin Ian King]

There are a handful of spelling mistakes in test messages in the
TCP-AIO selftests. Fix these.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/net/tcp_ao/connect-deny.c  | 2 +-
 tools/testing/selftests/net/tcp_ao/lib/proc.c  | 4 ++--
 tools/testing/selftests/net/tcp_ao/setsockopt-closed.c | 2 +-
 tools/testing/selftests/net/tcp_ao/unsigned-md5.c  | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tools/testing/selftests/net/tcp_ao/connect-deny.c 
b/tools/testing/selftests/net/tcp_ao/connect-deny.c
index 1ca78040d8b7..185a2f6e5ff3 100644
--- a/tools/testing/selftests/net/tcp_ao/connect-deny.c
+++ b/tools/testing/selftests/net/tcp_ao/connect-deny.c
@@ -55,7 +55,7 @@ static void try_accept(const char *tst_name, unsigned int 
port, const char *pwd,
err = test_wait_fd(lsk, timeout, 0);
if (err == -ETIMEDOUT) {
if (!fault(TIMEOUT))
-   test_fail("timeouted for accept()");
+   test_fail("timed out for accept()");
} else if (err < 0) {
test_error("test_wait_fd()");
} else {
diff --git a/tools/testing/selftests/net/tcp_ao/lib/proc.c 
b/tools/testing/selftests/net/tcp_ao/lib/proc.c
index 2322f4d4676d..2fb6dd8adba6 100644
--- a/tools/testing/selftests/net/tcp_ao/lib/proc.c
+++ b/tools/testing/selftests/net/tcp_ao/lib/proc.c
@@ -227,7 +227,7 @@ void netstat_print_diff(struct netstat *nsa, struct netstat 
*nsb)
}
 
if (nsb->counters_nr < nsa->counters_nr)
-   test_error("Unexpected: some counters dissapeared!");
+   test_error("Unexpected: some counters disappeared!");
 
for (j = 0, i = 0; i < nsb->counters_nr; i++) {
if (strcmp(nsb->counters[i].name, 
nsa->counters[j].name)) {
@@ -244,7 +244,7 @@ void netstat_print_diff(struct netstat *nsa, struct netstat 
*nsb)
j++;
}
if (j != nsa->counters_nr)
-   test_error("Unexpected: some counters dissapeared!");
+   test_error("Unexpected: some counters disappeared!");
 
nsb = nsb->next;
nsa = nsa->next;
diff --git a/tools/testing/selftests/net/tcp_ao/setsockopt-closed.c 
b/tools/testing/selftests/net/tcp_ao/setsockopt-closed.c
index 7e4601b3f6a3..a329f42f40ce 100644
--- a/tools/testing/selftests/net/tcp_ao/setsockopt-closed.c
+++ b/tools/testing/selftests/net/tcp_ao/setsockopt-closed.c
@@ -427,7 +427,7 @@ static void test_einval_del_key(void)
 
sk = prepare_defs(TCP_AO_DEL_KEY, &del);
del.set_current = 1;
-   setsockopt_checked(sk, TCP_AO_DEL_KEY, &del, ENOENT, "set non-exising 
current key");
+   setsockopt_checked(sk, TCP_AO_DEL_KEY, &del, ENOENT, "set non-existing 
current key");
 
sk = prepare_defs(TCP_AO_DEL_KEY, &del);
del.set_rnext = 1;
diff --git a/tools/testing/selftests/net/tcp_ao/unsigned-md5.c 
b/tools/testing/selftests/net/tcp_ao/unsigned-md5.c
index 7cffde02d2be..14addfd46468 100644
--- a/tools/testing/selftests/net/tcp_ao/unsigned-md5.c
+++ b/tools/testing/selftests/net/tcp_ao/unsigned-md5.c
@@ -72,7 +72,7 @@ static void try_accept(const char *tst_name, unsigned int 
port,
err = test_wait_fd(lsk, timeout, 0);
if (err == -ETIMEDOUT) {
if (!fault(TIMEOUT))
-   test_fail("timeouted for accept()");
+   test_fail("timed out for accept()");
} else if (err < 0) {
test_error("test_wait_fd()");
} else {
-- 
2.39.2

[PATCH][next] selftests: sched: Fix spelling mistake "hiearchy" -> "hierarchy"

[Colin Ian King]

There is a spelling mistake in a printed message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/sched/cs_prctl_test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/sched/cs_prctl_test.c 
b/tools/testing/selftests/sched/cs_prctl_test.c
index 7ba057154343..62fba7356af2 100644
--- a/tools/testing/selftests/sched/cs_prctl_test.c
+++ b/tools/testing/selftests/sched/cs_prctl_test.c
@@ -276,7 +276,7 @@ int main(int argc, char *argv[])
if (setpgid(0, 0) != 0)
handle_error("process group");
 
-   printf("\n## Create a thread/process/process group hiearchy\n");
+   printf("\n## Create a thread/process/process group hierarchy\n");
create_processes(num_processes, num_threads, procs);
need_cleanup = 1;
disp_processes(num_processes, procs);
-- 
2.39.2

[PATCH][next] KVM: selftests: Fix spelling mistake "trigged" -> "triggered"

[Colin Ian King]

There are spelling mistakes in __GUEST_ASSERT messages. Fix them.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/kvm/aarch64/arch_timer.c | 2 +-
 tools/testing/selftests/kvm/riscv/arch_timer.c   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/kvm/aarch64/arch_timer.c 
b/tools/testing/selftests/kvm/aarch64/arch_timer.c
index ddba2c2fb5de..16ac74d07d68 100644
--- a/tools/testing/selftests/kvm/aarch64/arch_timer.c
+++ b/tools/testing/selftests/kvm/aarch64/arch_timer.c
@@ -136,7 +136,7 @@ static void guest_run_stage(struct test_vcpu_shared_data 
*shared_data,
irq_iter = READ_ONCE(shared_data->nr_iter);
__GUEST_ASSERT(config_iter + 1 == irq_iter,
"config_iter + 1 = 0x%lx, irq_iter = 0x%lx.\n"
-   "  Guest timer interrupt was not trigged within 
the specified\n"
+   "  Guest timer interrupt was not triggered 
within the specified\n"
"  interval, try to increase the error margin 
by [-e] option.\n",
config_iter + 1, irq_iter);
}
diff --git a/tools/testing/selftests/kvm/riscv/arch_timer.c 
b/tools/testing/selftests/kvm/riscv/arch_timer.c
index e22848f747c0..0f9cabd99fd4 100644
--- a/tools/testing/selftests/kvm/riscv/arch_timer.c
+++ b/tools/testing/selftests/kvm/riscv/arch_timer.c
@@ -60,7 +60,7 @@ static void guest_run(struct test_vcpu_shared_data 
*shared_data)
irq_iter = READ_ONCE(shared_data->nr_iter);
__GUEST_ASSERT(config_iter + 1 == irq_iter,
"config_iter + 1 = 0x%x, irq_iter = 0x%x.\n"
-   "  Guest timer interrupt was not trigged within 
the specified\n"
+   "  Guest timer interrupt was not triggered 
within the specified\n"
"  interval, try to increase the error margin 
by [-e] option.\n",
config_iter + 1, irq_iter);
}
-- 
2.39.2

[PATCH][next] selftests/bpf: Remove second semicolon

[Colin Ian King]

There are statements with two semicolons. Remove the second one, it
is redundant.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/bpf/benchs/bench_local_storage_create.c | 2 +-
 tools/testing/selftests/bpf/progs/iters.c   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c 
b/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c
index b36de42ee4d9..e2ff8ea1cb79 100644
--- a/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c
+++ b/tools/testing/selftests/bpf/benchs/bench_local_storage_create.c
@@ -186,7 +186,7 @@ static void *task_producer(void *input)
 
for (i = 0; i < batch_sz; i++) {
if (!pthd_results[i])
-   pthread_join(pthds[i], NULL);;
+   pthread_join(pthds[i], NULL);
}
}
 
diff --git a/tools/testing/selftests/bpf/progs/iters.c 
b/tools/testing/selftests/bpf/progs/iters.c
index 3db416606f2f..fe65e0952a1e 100644
--- a/tools/testing/selftests/bpf/progs/iters.c
+++ b/tools/testing/selftests/bpf/progs/iters.c
@@ -673,7 +673,7 @@ static __noinline void fill(struct bpf_iter_num *it, int 
*arr, __u32 n, int mul)
 
 static __noinline int sum(struct bpf_iter_num *it, int *arr, __u32 n)
 {
-   int *t, i, sum = 0;;
+   int *t, i, sum = 0;
 
while ((t = bpf_iter_num_next(it))) {
i = *t;
-- 
2.39.2

[PATCH][next] KVM: selftests: Remove second semicolon

[Colin Ian King]

There is a statement with two semicolons. Remove the second one, it
is redundant.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c 
b/tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c
index d2ea0435f4f7..7d707d8068a4 100644
--- a/tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c
+++ b/tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c
@@ -125,7 +125,7 @@ struct compat_vcpu_runstate_info {
uint32_t state;
uint64_t state_entry_time;
uint64_t time[5];
-} __attribute__((__packed__));;
+} __attribute__((__packed__));
 
 struct arch_vcpu_info {
unsigned long cr2;
-- 
2.39.2

[PATCH][next] selftests/perf_events: Fix spelling mistake "sycnhronize" -> "synchronize"

[Colin Ian King]

There is a spelling mistake in an error message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/perf_events/watermark_signal.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/perf_events/watermark_signal.c 
b/tools/testing/selftests/perf_events/watermark_signal.c
index 49dc1e831174..e03fe1b9bba2 100644
--- a/tools/testing/selftests/perf_events/watermark_signal.c
+++ b/tools/testing/selftests/perf_events/watermark_signal.c
@@ -75,7 +75,7 @@ TEST(watermark_signal)
if (waitpid(child, &child_status, WSTOPPED) != child ||
!(WIFSTOPPED(child_status) && WSTOPSIG(child_status) == SIGSTOP)) {
fprintf(stderr,
-   "failed to sycnhronize with child errno=%d status=%x\n",
+   "failed to synchronize with child errno=%d status=%x\n",
errno,
child_status);
goto cleanup;
-- 
2.39.2

[PATCH][next] selftest/mm: Fix spelling mistake "skiped" -> "skipped"

[Colin Ian King]

There is a spelling mistake in a ksft_test_result_skip message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/mm/ksm_functional_tests.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/mm/ksm_functional_tests.c 
b/tools/testing/selftests/mm/ksm_functional_tests.c
index 2d277620fad2..db845dca8d19 100644
--- a/tools/testing/selftests/mm/ksm_functional_tests.c
+++ b/tools/testing/selftests/mm/ksm_functional_tests.c
@@ -502,7 +502,7 @@ static void test_child_ksm_err(int status)
else if (status == -2)
ksft_test_result_fail("Merge in child failed\n");
else if (status == -3)
-   ksft_test_result_skip("Merge in child skiped\n");
+   ksft_test_result_skip("Merge in child skipped\n");
 }
 
 /* Verify that prctl ksm flag is inherited. */
-- 
2.39.2

[PATCH][next] kselftest/arm64: Fix a couple of spelling mistakes

[Colin Ian King]

There are two spelling mistakes in some error messages. Fix them.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/arm64/fp/kernel-test.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/arm64/fp/kernel-test.c 
b/tools/testing/selftests/arm64/fp/kernel-test.c
index 50db26a3ed79..e8da3b4cbd23 100644
--- a/tools/testing/selftests/arm64/fp/kernel-test.c
+++ b/tools/testing/selftests/arm64/fp/kernel-test.c
@@ -233,7 +233,7 @@ static bool compute_digest(void *buf)
 reread:
ret = recv(sock, buf, digest_len, 0);
if (ret == 0) {
-   printf("No disgest returned\n");
+   printf("No digest returned\n");
return false;
}
if (ret != digest_len) {
@@ -308,7 +308,7 @@ int main(void)
 
while (true) {
if (!compute_digest(digest)) {
-   printf("Failed to coempute digest, iter=%d\n", iter);
+   printf("Failed to compute digest, iter=%d\n", iter);
return EXIT_FAILURE;
}
 
-- 
2.39.2

[PATCH][next] tools: selftests: riscv: Fix spelling mistake "regesters" -> "registers"

[Colin Ian King]

There are a couple of spelling mistakes in ksft_test_result_fail messages.
Fix them.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/riscv/vector/v_initval_nolibc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/riscv/vector/v_initval_nolibc.c 
b/tools/testing/selftests/riscv/vector/v_initval_nolibc.c
index 1dd94197da30..6838c561e4c9 100644
--- a/tools/testing/selftests/riscv/vector/v_initval_nolibc.c
+++ b/tools/testing/selftests/riscv/vector/v_initval_nolibc.c
@@ -49,14 +49,14 @@ int main(void)
ksft_print_msg("vl = %lu\n", vl);
 
if (datap[0] != 0x00 && datap[0] != 0xff) {
-   ksft_test_result_fail("v-regesters are not properly 
initialized\n");
+   ksft_test_result_fail("v-registers are not properly 
initialized\n");
dump(datap, vl * 4);
exit(-1);
}
 
for (i = 1; i < vl * 4; i++) {
if (datap[i] != datap[0]) {
-   ksft_test_result_fail("detect stale values on 
v-regesters\n");
+   ksft_test_result_fail("detect stale values on 
v-registers\n");
dump(datap, vl * 4);
exit(-2);
}
-- 
2.39.2

[PATCH][next] selftests/bpf: Fix spelling mistake "{GET|SET}SOCKOP" -> "{GET|SET}SOCKOPT"

[Colin Ian King]

There are two spelling mistakes in .descr literal strings. Fix them.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/bpf/prog_tests/sockopt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/bpf/prog_tests/sockopt.c 
b/tools/testing/selftests/bpf/prog_tests/sockopt.c
index eaac83a7f388..ecf5a7a047a4 100644
--- a/tools/testing/selftests/bpf/prog_tests/sockopt.c
+++ b/tools/testing/selftests/bpf/prog_tests/sockopt.c
@@ -933,7 +933,7 @@ static struct sockopt_test {
/*  prog_type   */
 
{
-   .descr = "can attach only BPF_CGROUP_SETSOCKOP",
+   .descr = "can attach only BPF_CGROUP_SETSOCKOPT",
.insns = {
/* return 1 */
BPF_MOV64_IMM(BPF_REG_0, 1),
@@ -947,7 +947,7 @@ static struct sockopt_test {
},
 
{
-   .descr = "can attach only BPF_CGROUP_GETSOCKOP",
+   .descr = "can attach only BPF_CGROUP_GETSOCKOPT",
.insns = {
/* return 1 */
BPF_MOV64_IMM(BPF_REG_0, 1),
-- 
2.39.2

[PATCH][next] selftests/powerpc/dexcr: Fix spelling mistake "predicition" -> "prediction"

[Colin Ian King]

There is a spelling mistake in the help message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/powerpc/dexcr/chdexcr.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/powerpc/dexcr/chdexcr.c 
b/tools/testing/selftests/powerpc/dexcr/chdexcr.c
index bda44630cada..c548d7a5bb9b 100644
--- a/tools/testing/selftests/powerpc/dexcr/chdexcr.c
+++ b/tools/testing/selftests/powerpc/dexcr/chdexcr.c
@@ -26,7 +26,7 @@ static void help(void)
   "\n"
   "The normal option sets the aspect in the DEXCR. The --no- 
variant\n"
   "clears that aspect. For example, --ibrtpd sets the IBRTPD 
aspect bit,\n"
-  "so indirect branch predicition will be disabled in the provided 
program.\n"
+  "so indirect branch prediction will be disabled in the provided 
program.\n"
   "Conversely, --no-ibrtpd clears the aspect bit, so indirect 
branch\n"
   "prediction may occur.\n"
   "\n"
-- 
2.39.2

[PATCH] selftest: epoll_busy_poll: Fix spelling mistake "couldnt" -> "couldn't"

[Colin Ian King]

There is a spelling mistake in a TH_LOG message. Fix it.

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/net/epoll_busy_poll.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/net/epoll_busy_poll.c 
b/tools/testing/selftests/net/epoll_busy_poll.c
index 9dd2830fd67c..16e457c2f877 100644
--- a/tools/testing/selftests/net/epoll_busy_poll.c
+++ b/tools/testing/selftests/net/epoll_busy_poll.c
@@ -232,7 +232,7 @@ TEST_F(epoll_busy_poll, test_set_invalid)
 
ret = cap_set_flag(self->caps, CAP_EFFECTIVE, 1, net_admin, CAP_CLEAR);
EXPECT_EQ(0, ret)
-   TH_LOG("couldnt clear CAP_NET_ADMIN");
+   TH_LOG("couldn't clear CAP_NET_ADMIN");
 
ret = cap_set_proc(self->caps);
EXPECT_EQ(0, ret)
-- 
2.39.2

[PATCH][next] selftests: mqueue: initialize array buf before using it

[Colin Ian King]

Currently array buf is not being initialized and so garbage values
on the stack are being used in the mq_send calls. Initialize the
values in the array to zero.

Cleans up cppcheck warning:
mq_perf_tests.c:334:25: error: Uninitialized variable: buff [uninitvar]

Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/mqueue/mq_perf_tests.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/mqueue/mq_perf_tests.c 
b/tools/testing/selftests/mqueue/mq_perf_tests.c
index 5c16159d0bcd..bd561dc785d8 100644
--- a/tools/testing/selftests/mqueue/mq_perf_tests.c
+++ b/tools/testing/selftests/mqueue/mq_perf_tests.c
@@ -322,7 +322,7 @@ void *fake_cont_thread(void *arg)
 
 void *cont_thread(void *arg)
 {
-   char buff[MSG_SIZE];
+   char buff[MSG_SIZE] = { };
int i, priority;
 
for (i = 0; i < num_cpus_to_pin; i++)
-- 
2.39.2

[PATCH][next] selftests: kvm: fix shift of 32 bit unsigned int more than 32 bits

[Colin Ian King]

Currrentl a 32 bit 1u value is being shifted more than 32 bits causing
overflow and incorrect checking of bits 32-63. Fix this by using the
BIT_ULL macro for shifting bits.

Detected by cppcheck:
sev_init2_tests.c:108:34: error: Shifting 32-bit value by 63 bits is
undefined behaviour [shiftTooManyBits]

Fixes: dfc083a181ba ("selftests: kvm: add tests for KVM_SEV_INIT2")
Signed-off-by: Colin Ian King 
---
 tools/testing/selftests/kvm/x86_64/sev_init2_tests.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c 
b/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c
index 7a4a61be119b..ea09f7a06aa4 100644
--- a/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c
+++ b/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c
@@ -105,11 +105,11 @@ void test_features(uint32_t vm_type, uint64_t 
supported_features)
int i;
 
for (i = 0; i < 64; i++) {
-   if (!(supported_features & (1u << i)))
+   if (!(supported_features & BIT_ULL(i)))
test_init2_invalid(vm_type,
&(struct kvm_sev_init){ .vmsa_features = 
BIT_ULL(i) },
"unknown feature");
-   else if (KNOWN_FEATURES & (1u << i))
+   else if (KNOWN_FEATURES & BIT_ULL(u))
test_init2(vm_type,
&(struct kvm_sev_init){ .vmsa_features = 
BIT_ULL(i) });
}
-- 
2.39.2

[PATCH][next][V2] selftests: kvm: fix shift of 32 bit unsigned int more than 32 bits

[Colin Ian King]

Currrentl a 32 bit 1u value is being shifted more than 32 bits causing
overflow and incorrect checking of bits 32-63. Fix this by using the
BIT_ULL macro for shifting bits.

Detected by cppcheck:
sev_init2_tests.c:108:34: error: Shifting 32-bit value by 63 bits is
undefined behaviour [shiftTooManyBits]

Fixes: dfc083a181ba ("selftests: kvm: add tests for KVM_SEV_INIT2")
Signed-off-by: Colin Ian King 

---

V2: Fix incorrect variable in 2nd BIT_ULL(), kudos to Dan Carpenter for
catching this error.

---
 tools/testing/selftests/kvm/x86_64/sev_init2_tests.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c 
b/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c
index 7a4a61be119b..ea09f7a06aa4 100644
--- a/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c
+++ b/tools/testing/selftests/kvm/x86_64/sev_init2_tests.c
@@ -105,11 +105,11 @@ void test_features(uint32_t vm_type, uint64_t 
supported_features)
int i;
 
for (i = 0; i < 64; i++) {
-   if (!(supported_features & (1u << i)))
+   if (!(supported_features & BIT_ULL(i)))
test_init2_invalid(vm_type,
&(struct kvm_sev_init){ .vmsa_features = 
BIT_ULL(i) },
"unknown feature");
-   else if (KNOWN_FEATURES & (1u << i))
+   else if (KNOWN_FEATURES & BIT_ULL(i))
test_init2(vm_type,
&(struct kvm_sev_init){ .vmsa_features = 
BIT_ULL(i) });
}
-- 
2.39.2