lib-test-string_helpersc-add-string_get_size-tests-v5

- Single quotes in pr_warn() to see empty strings [Andy Shevchenko].
- Check for test validity compile-time [Rasmus Villemoes, Andy Shevchenko].
- All __test_string_get_size() arguments are const.

Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Cc: James Bottomley <JBottomley@Odin.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Cc: "K. Y. Srinivasan" <kys@microsoft.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/lib/test-string_helpers.c b/lib/test-string_helpers.c
index ee67adab13463c374ec4063763f6d9a2609c5b0b..98866a770770c8bba0acf7bdbacea4699d9f14bf 100644 (file)
--- a/lib/test-string_helpers.c
+++ b/lib/test-string_helpers.c
@@ -326,21 +326,30 @@ out:
        kfree(out_test);
 }
 
-static __init void test_string_get_size_one(u64 size, u64 blk_size,
-                                           const enum string_size_units units,
-                                           const char *exp_result)
+#define string_get_size_maxbuf 16
+#define test_string_get_size_one(size, blk_size, units, exp_result)            \
+       do {                                                                   \
+               BUILD_BUG_ON(sizeof(exp_result) >= string_get_size_maxbuf);    \
+               __test_string_get_size((size), (blk_size), (units),            \
+                                      (exp_result));                          \
+       } while (0)
+
+
+static __init void __test_string_get_size(const u64 size, const u64 blk_size,
+                                         const enum string_size_units units,
+                                         const char *exp_result)
 {
-       char buf[256];
+       char buf[string_get_size_maxbuf];
 
        string_get_size(size, blk_size, units, buf, sizeof(buf));
-       if (!strncmp(buf, exp_result, min(sizeof(buf), strlen(exp_result))))
+       if (!memcmp(buf, exp_result, strlen(exp_result) + 1))
                return;
 
+       buf[sizeof(buf) - 1] = '\0';
        pr_warn("Test 'test_string_get_size_one' failed!\n");
        pr_warn("string_get_size(size = %llu, blk_size = %llu, units = %d\n",
                size, blk_size, units);
-       pr_warn("expected: %s, got %s\n", exp_result, buf);
-
+       pr_warn("expected: '%s', got '%s'\n", exp_result, buf);
 }
 
 static __init void test_string_get_size(void)