From: Noah Goldstein <goldstein.w.n@gmail.com>
To: Wilco Dijkstra <Wilco.Dijkstra@arm.com>
Cc: GNU C Library <libc-alpha@sourceware.org>
Subject: Re: [PATCH] Benchtests: Improve large memcpy/memset benchmarks
Date: Mon, 18 Mar 2024 14:34:36 -0500 [thread overview]
Message-ID: <CAFUsyfLLMcq3RZhJUWO-ueqvhAGhq1TFO6MxYxx6jW97r-oYOA@mail.gmail.com> (raw)
In-Reply-To: <CAFUsyfLEb0h7_YDz0HhM0s5hCkuaBaHks4rPWXWRKMSo5RxDoQ@mail.gmail.com>
On Mon, Mar 18, 2024 at 2:29 PM Noah Goldstein <goldstein.w.n@gmail.com> wrote:
>
> On Mon, Mar 18, 2024 at 10:08 AM Wilco Dijkstra <Wilco.Dijkstra@arm.com> wrote:
> >
> > Adjust sizes between 64KB and 16MB and iterations based on length.
> > Remove incorrect uses of alloc_bufs (since we're not interested in measuring
> > Linux clear_page time).
> >
> > OK for commit?
> >
> > ---
> >
> > diff --git a/benchtests/bench-bzero-large.c b/benchtests/bench-bzero-large.c
> > index 7076c0a17b00d76c26aa6adb4c3cc0aedcf69955..dea414ec8d1bb160409725996244178ee4fa93fd 100644
> > --- a/benchtests/bench-bzero-large.c
> > +++ b/benchtests/bench-bzero-large.c
> > @@ -22,9 +22,8 @@
> > #else
> > # define TEST_NAME "bzero"
> > #endif
> > -#define START_SIZE (128 * 1024)
> > -#define MIN_PAGE_SIZE (getpagesize () + 64 * 1024 * 1024)
> > -#define TIMEOUT (20 * 60)
> > +#define START_SIZE (64 * 1024)
> > +#define MIN_PAGE_SIZE (getpagesize () + 16 * 1024 * 1024)
> > #include "bench-string.h"
> >
> > #include "json-lib.h"
> > @@ -52,7 +51,7 @@ IMPL (memset_zero, 0)
> > static void
> > do_one_test (json_ctx_t *json_ctx, impl_t *impl, CHAR *s, size_t n)
> > {
> > - size_t i, iters = 16;
> > + size_t i, iters = (MIN_PAGE_SIZE * 64) / n;
> > timing_t start, stop, cur;
> >
> > TIMING_NOW (start);
> > @@ -74,20 +73,13 @@ do_one_test (json_ctx_t *json_ctx, impl_t *impl, CHAR *s, size_t n)
> > static void
> > do_test (json_ctx_t *json_ctx, size_t align, size_t len)
> > {
> > - align &= 63;
> > - if ((align + len) * sizeof (CHAR) > page_size)
> > - return;
> > -
> > json_element_object_begin (json_ctx);
> > json_attr_uint (json_ctx, "length", len);
> > json_attr_uint (json_ctx, "alignment", align);
> > json_array_begin (json_ctx, "timings");
> >
> > FOR_EACH_IMPL (impl, 0)
> > - {
> > - do_one_test (json_ctx, impl, (CHAR *) (buf1) + align, len);
> > - alloc_bufs ();
> > - }
> > + do_one_test (json_ctx, impl, (CHAR *) (buf1) + align, len);
> >
> > json_array_end (json_ctx);
> > json_element_object_end (json_ctx);
> > diff --git a/benchtests/bench-memcpy-large.c b/benchtests/bench-memcpy-large.c
> > index 7b2c5530af3883acc4b3895cb11667e6ff1a55ff..9e544a4729197f886f6cb092ec225004b586d197 100644
> > --- a/benchtests/bench-memcpy-large.c
> > +++ b/benchtests/bench-memcpy-large.c
> > @@ -19,10 +19,9 @@
> > #ifndef MEMCPY_RESULT
> > # define MEMCPY_RESULT(dst, len) dst
> > # define START_SIZE (64 * 1024)
> > -# define MIN_PAGE_SIZE (getpagesize () + 32 * 1024 * 1024)
> > +# define MIN_PAGE_SIZE (getpagesize () + 16 * 1024 * 1024)
> > # define TEST_MAIN
> > # define TEST_NAME "memcpy"
> > -# define TIMEOUT (20 * 60)
> > # include "bench-string.h"
> >
> > IMPL (memcpy, 1)
> > @@ -36,7 +35,7 @@ static void
> > do_one_test (json_ctx_t *json_ctx, impl_t *impl, char *dst, const char *src,
> > size_t len)
> > {
> > - size_t i, iters = 16;
> > + size_t i, iters = (MIN_PAGE_SIZE * 8) / len;
> > timing_t start, stop, cur;
> >
> > TIMING_NOW (start);
> > @@ -59,12 +58,7 @@ do_test (json_ctx_t *json_ctx, size_t align1, size_t align2, size_t len,
> > char *s1, *s2;
> > size_t repeats;
> > align1 &= 4095;
> > - if (align1 + len >= page_size)
> > - return;
> > -
> > align2 &= 4095;
> > - if (align2 + len >= page_size)
> > - return;
> >
> > s1 = (char *) (buf1 + align1);
> > s2 = (char *) (buf2 + align2);
> > diff --git a/benchtests/bench-memmove-large.c b/benchtests/bench-memmove-large.c
> > index a09dd3678848a3bf8612732439700eb8ef5d82ea..fd504653f681b172800cc34e054cc745180aa4fa 100644
> > --- a/benchtests/bench-memmove-large.c
> > +++ b/benchtests/bench-memmove-large.c
> > @@ -16,12 +16,10 @@
> > License along with the GNU C Library; if not, see
> > <https://www.gnu.org/licenses/>. */
> >
> > -#define BASE_PAGE_SIZE (1024 * 1024)
> > -#define START_SIZE (4 * 1024)
> > +#define START_SIZE (64 * 1024)
> > #define MIN_PAGE_SIZE (getpagesize () + 16 * 1024 * 1024)
> > #define TEST_MAIN
> > #define TEST_NAME "memmove"
> > -#define TIMEOUT (20 * 60)
> > #include "bench-string.h"
> > #include "json-lib.h"
> >
> > @@ -33,7 +31,7 @@ static void
> > do_one_test (json_ctx_t *json_ctx, impl_t *impl, char *dst, char *src,
> > size_t len)
> > {
> > - size_t i, iters = 16;
> > + size_t i, iters = (MIN_PAGE_SIZE * 8) / len;
> maybe MAX(16, (MIN_PAGE_SIZE * )/ len). Likewise above.
Also, can you include the iter count in the output?
> > timing_t start, stop, cur;
> >
> > TIMING_NOW (start);
> > @@ -54,13 +52,8 @@ do_test (json_ctx_t *json_ctx, size_t align1, size_t align2, size_t len)
> > size_t i, j;
> > char *s1, *s2;
> >
> > - align1 &= 127;
> > - if (align1 + len >= page_size)
> > - return;
> > -
> > - align2 &= 127;
> > - if (align2 + len >= page_size)
> > - return;
> > + align1 &= 4095;
> > + align2 &= 4095;
> >
> > s1 = (char *) (buf2 + align1);
> > s2 = (char *) (buf2 + align2);
> > diff --git a/benchtests/bench-memset-large.c b/benchtests/bench-memset-large.c
> > index a1f33245d4960c1e82fc441b5faf322204035202..c3f9ee0cd75d3cb30f8b3df15672f895c111fc28 100644
> > --- a/benchtests/bench-memset-large.c
> > +++ b/benchtests/bench-memset-large.c
> > @@ -18,9 +18,8 @@
> >
> > #define TEST_MAIN
> > #define TEST_NAME "memset"
> > -#define START_SIZE (128 * 1024)
> > -#define MIN_PAGE_SIZE (getpagesize () + 64 * 1024 * 1024)
> > -#define TIMEOUT (20 * 60)
> > +#define START_SIZE (64 * 1024)
> > +#define MIN_PAGE_SIZE (getpagesize () + 16 * 1024 * 1024)
> > #include "bench-string.h"
> >
> > #include "json-lib.h"
> > @@ -35,7 +34,7 @@ static void
> > do_one_test (json_ctx_t *json_ctx, impl_t *impl, CHAR *s,
> > int c __attribute ((unused)), size_t n)
> > {
> > - size_t i, iters = 16;
> > + size_t i, iters = (MIN_PAGE_SIZE * 64) / n;
> > timing_t start, stop, cur;
> >
> > TIMING_NOW (start);
> > @@ -53,10 +52,6 @@ do_one_test (json_ctx_t *json_ctx, impl_t *impl, CHAR *s,
> > static void
> > do_test (json_ctx_t *json_ctx, size_t align, int c, size_t len)
> > {
> > - align &= 63;
> > - if ((align + len) * sizeof (CHAR) > page_size)
> > - return;
> > -
> > json_element_object_begin (json_ctx);
> > json_attr_uint (json_ctx, "length", len);
> > json_attr_uint (json_ctx, "alignment", align);
> > @@ -64,10 +59,7 @@ do_test (json_ctx_t *json_ctx, size_t align, int c, size_t len)
> > json_array_begin (json_ctx, "timings");
> >
> > FOR_EACH_IMPL (impl, 0)
> > - {
> > - do_one_test (json_ctx, impl, (CHAR *) (buf1) + align, c, len);
> > - alloc_bufs ();
> > - }
> > + do_one_test (json_ctx, impl, (CHAR *) (buf1) + align, c, len);
> >
> > json_array_end (json_ctx);
> > json_element_object_end (json_ctx);
> >
next prev parent reply other threads:[~2024-03-18 19:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-18 15:08 Wilco Dijkstra
2024-03-18 19:29 ` Noah Goldstein
2024-03-18 19:34 ` Noah Goldstein [this message]
2024-03-27 17:02 ` Wilco Dijkstra
2024-03-28 19:28 ` Noah Goldstein
-- strict thread matches above, loose matches on Subject: below --
2023-03-23 11:41 Wilco Dijkstra
2023-03-23 18:17 ` Noah Goldstein
2023-03-23 23:01 ` Wilco Dijkstra
2023-03-24 16:29 ` Noah Goldstein
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFUsyfLLMcq3RZhJUWO-ueqvhAGhq1TFO6MxYxx6jW97r-oYOA@mail.gmail.com \
--to=goldstein.w.n@gmail.com \
--cc=Wilco.Dijkstra@arm.com \
--cc=libc-alpha@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).