benchtests: Add small sizes (<= 64) to bench-bzero-walk.c

Message ID 20220224174727.3595-1-hjl.tools@gmail.com
State Committed
Commit 89377d41d73b59a5b8c781fe08df99df0a470fac
Headers
Series benchtests: Add small sizes (<= 64) to bench-bzero-walk.c |

Checks

Context Check Description
dj/TryBot-apply_patch success Patch applied to master at the time it was sent
dj/TryBot-32bit success Build for i686

Commit Message

H.J. Lu Feb. 24, 2022, 5:47 p.m. UTC
  Small sizes (<= 64) represent large portion of memset usages with zero
value.  Add sizes (<= 64) to bench-bzero-walk.c to cover small sizes.
---
 benchtests/bench-bzero-walk.c | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

Sunil Pandey Feb. 24, 2022, 6:28 p.m. UTC | #1
LGTM

Reviewed-by: Sunil K Pandey <skpgkp2@gmail.com>

Thanks,
Sunil



On Thu, Feb 24, 2022 at 9:47 AM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> Small sizes (<= 64) represent large portion of memset usages with zero
> value.  Add sizes (<= 64) to bench-bzero-walk.c to cover small sizes.
> ---
>  benchtests/bench-bzero-walk.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/benchtests/bench-bzero-walk.c b/benchtests/bench-bzero-walk.c
> index 00226f80b8..4984f6e190 100644
> --- a/benchtests/bench-bzero-walk.c
> +++ b/benchtests/bench-bzero-walk.c
> @@ -113,6 +113,9 @@ test_main (void)
>    json_array_end (&json_ctx);
>
>    json_array_begin (&json_ctx, "results");
> +  for (i = 1; i <= 64; i++)
> +    do_test (&json_ctx, i);
> +
>    for (i = START_SIZE; i <= MIN_PAGE_SIZE; i <<= 1)
>      {
>        do_test (&json_ctx, i);
> --
> 2.35.1
>
  

Patch

diff --git a/benchtests/bench-bzero-walk.c b/benchtests/bench-bzero-walk.c
index 00226f80b8..4984f6e190 100644
--- a/benchtests/bench-bzero-walk.c
+++ b/benchtests/bench-bzero-walk.c
@@ -113,6 +113,9 @@  test_main (void)
   json_array_end (&json_ctx);
 
   json_array_begin (&json_ctx, "results");
+  for (i = 1; i <= 64; i++)
+    do_test (&json_ctx, i);
+
   for (i = START_SIZE; i <= MIN_PAGE_SIZE; i <<= 1)
     {
       do_test (&json_ctx, i);