diff mbox series

[v2] Update string/test-memmove.c to cover 16KB copy

Message ID 20210820140709.1045930-1-hjl.tools@gmail.com
State Committed
Commit c333dcf8d8f9e6e46475d9eff24bd5394b5d3d9e
Headers show
Series [v2] Update string/test-memmove.c to cover 16KB copy | expand

Checks

Context Check Description
dj/TryBot-apply_patch success Patch applied to master at the time it was sent
dj/TryBot-32bit fail Patch series failed to apply

Commit Message

H.J. Lu Aug. 20, 2021, 2:07 p.m. UTC
---
 string/test-memmove.c | 48 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

Comments

Noah Goldstein Aug. 20, 2021, 4:11 p.m. UTC | #1
On Fri, Aug 20, 2021 at 10:07 AM H.J. Lu <hjl.tools@gmail.com> wrote:

> ---
>  string/test-memmove.c | 48 +++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 48 insertions(+)
>
> diff --git a/string/test-memmove.c b/string/test-memmove.c
> index b271248b1d..670094c9dc 100644
> --- a/string/test-memmove.c
> +++ b/string/test-memmove.c
> @@ -312,6 +312,50 @@ do_test2 (size_t offset)
>    munmap ((void *) large_buf, size);
>  }
>
> +static void
> +do_test3 (size_t bytes_move, size_t offset)
> +{
> +  size_t size = bytes_move * 3;
> +  uint32_t *buf;
> +
> +  buf = mmap (NULL, size, PROT_READ | PROT_WRITE,
> +             MAP_PRIVATE | MAP_ANON, -1, 0);
> +
> +  if (buf == MAP_FAILED)
> +    error (EXIT_UNSUPPORTED, errno, "mmap failed");
> +
> +  size_t arr_size = bytes_move / sizeof (uint32_t);
> +  size_t i;
> +
> +  FOR_EACH_IMPL (impl, 0)
> +    {
> +      for (i = 0; i < arr_size; i++)
> +        buf[i] = (uint32_t) i;
> +
> +      uint32_t *dst = &buf[arr_size + offset];
> +
> +#ifdef TEST_BCOPY
> +      CALL (impl, (char *) buf, (char *) dst, bytes_move);
> +#else
> +      CALL (impl, (char *) dst, (char *) buf, bytes_move);
> +#endif
> +
> +      for (i = 0; i < arr_size; i++)
> +       {
> +         if (dst[i] != (uint32_t) i)
> +           {
> +             error (0, 0,
> +                    "Wrong result in function %s dst \"%p\" src \"%p\"
> offset \"%zd\"",
> +                    impl->name, dst, buf, i);
> +             ret = 1;
> +             break;
> +           }
> +       }
> +    }
> +
> +  munmap ((void *) buf, size);
> +}
> +
>  int
>  test_main (void)
>  {
> @@ -356,6 +400,10 @@ test_main (void)
>    do_test2 (0x200000);
>    do_test2 (0x4000000 - 1);
>    do_test2 (0x4000000);
> +
> +  /* Copy 16KB data.  */
> +  do_test3 (16384, 3);
> +
>    return ret;
>  }
>
> --
> 2.31.1
> LGTM
diff mbox series

Patch

diff --git a/string/test-memmove.c b/string/test-memmove.c
index b271248b1d..670094c9dc 100644
--- a/string/test-memmove.c
+++ b/string/test-memmove.c
@@ -312,6 +312,50 @@  do_test2 (size_t offset)
   munmap ((void *) large_buf, size);
 }
 
+static void
+do_test3 (size_t bytes_move, size_t offset)
+{
+  size_t size = bytes_move * 3;
+  uint32_t *buf;
+
+  buf = mmap (NULL, size, PROT_READ | PROT_WRITE,
+	      MAP_PRIVATE | MAP_ANON, -1, 0);
+
+  if (buf == MAP_FAILED)
+    error (EXIT_UNSUPPORTED, errno, "mmap failed");
+
+  size_t arr_size = bytes_move / sizeof (uint32_t);
+  size_t i;
+
+  FOR_EACH_IMPL (impl, 0)
+    {
+      for (i = 0; i < arr_size; i++)
+        buf[i] = (uint32_t) i;
+
+      uint32_t *dst = &buf[arr_size + offset];
+
+#ifdef TEST_BCOPY
+      CALL (impl, (char *) buf, (char *) dst, bytes_move);
+#else
+      CALL (impl, (char *) dst, (char *) buf, bytes_move);
+#endif
+
+      for (i = 0; i < arr_size; i++)
+	{
+	  if (dst[i] != (uint32_t) i)
+	    {
+	      error (0, 0,
+		     "Wrong result in function %s dst \"%p\" src \"%p\" offset \"%zd\"",
+		     impl->name, dst, buf, i);
+	      ret = 1;
+	      break;
+	    }
+	}
+    }
+
+  munmap ((void *) buf, size);
+}
+
 int
 test_main (void)
 {
@@ -356,6 +400,10 @@  test_main (void)
   do_test2 (0x200000);
   do_test2 (0x4000000 - 1);
   do_test2 (0x4000000);
+
+  /* Copy 16KB data.  */
+  do_test3 (16384, 3);
+
   return ret;
 }