path: root/tools
diff options
authorMike Kravetz <>2016-06-24 14:48:40 -0700
committerLinus Torvalds <>2016-06-24 17:23:52 -0700
commita7b50abc90afb2e3c27e1bd212643cc53eaf0b60 (patch)
treef077b8755dc655d1d95f4ad44d425cc2d92fdb4c /tools
parent9df10fb7b80bc2f540956ba01b5e7ee1012001a5 (diff)
selftests/vm/compaction_test: fix write to restore nr_hugepages
The write at the end of the test to restore nr_hugepages to its previous value is failing. This is because it is trying to write the number of bytes in the char array as opposed to the number of bytes in the string. Link: Signed-off-by: Mike Kravetz <> Cc: Shuah Khan <> Cc: Sri Jayaramappa <> Cc: Eric B Munson <> Signed-off-by: Andrew Morton <> Signed-off-by: Linus Torvalds <>
Diffstat (limited to 'tools')
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/testing/selftests/vm/compaction_test.c b/tools/testing/selftests/vm/compaction_test.c
index 932ff577ffc0..00c4f65d12da 100644
--- a/tools/testing/selftests/vm/compaction_test.c
+++ b/tools/testing/selftests/vm/compaction_test.c
@@ -136,7 +136,7 @@ int check_compaction(unsigned long mem_free, unsigned int hugepage_size)
printf("No of huge pages allocated = %d\n",
- if (write(fd, initial_nr_hugepages, sizeof(initial_nr_hugepages))
+ if (write(fd, initial_nr_hugepages, strlen(initial_nr_hugepages))
!= strlen(initial_nr_hugepages)) {
perror("Failed to write to /proc/sys/vm/nr_hugepages\n");
goto close_fd;