diff options
| author | John Hubbard <jhubbard@nvidia.com> | 2024-05-02 20:51:05 -0700 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2024-05-03 09:06:10 -1000 |
| commit | 8f6d24a5db2acac3f52a34e6df347ec131d231ab (patch) | |
| tree | a4fe116517ac54884608885b995b732e3b43dfac | |
| parent | 3309ca6f47f11b5d817ce1e5d8b2f1637b93243e (diff) | |
| download | cgroup-for-next.tar.gz | |
selftests/cgroup: fix uninitialized variables in test_zswap.cfor-next
First of all, in order to build with clang at all, one must first apply
Valentin Obst's build fix for LLVM [1]. Once that is done, then when
building with clang, via:
make LLVM=1 -C tools/testing/selftests
...clang finds and warning about some uninitialized variables. Fix these
by initializing them.
[1] https://lore.kernel.org/all/20240329-selftests-libmk-llvm-rfc-v1-1-2f9ed7d1c49f@valentinobst.de/
Signed-off-by: John Hubbard <jhubbard@nvidia.com>
Reviewed-by: Roman Gushchin <roman.gushchin@linux.dev>
Signed-off-by: Tejun Heo <tj@kernel.org>
| -rw-r--r-- | tools/testing/selftests/cgroup/test_zswap.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/cgroup/test_zswap.c b/tools/testing/selftests/cgroup/test_zswap.c index ef7f3954531738..fa571bfd243284 100644 --- a/tools/testing/selftests/cgroup/test_zswap.c +++ b/tools/testing/selftests/cgroup/test_zswap.c @@ -257,7 +257,7 @@ static int test_no_invasive_cgroup_shrink(const char *root) { int ret = KSFT_FAIL; size_t control_allocation_size = MB(10); - char *control_allocation, *wb_group = NULL, *control_group = NULL; + char *control_allocation = NULL, *wb_group = NULL, *control_group = NULL; wb_group = setup_test_group_1M(root, "per_memcg_wb_test1"); if (!wb_group) @@ -342,7 +342,7 @@ static int test_no_kmem_bypass(const char *root) struct sysinfo sys_info; int ret = KSFT_FAIL; int child_status; - char *test_group; + char *test_group = NULL; pid_t child_pid; /* Read sys info and compute test values accordingly */ |