This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 2/2] benchtests: Add a new argument -t to read throughput results

From: Carlos O'Donell <carlos at redhat dot com>
To: Siddhesh Poyarekar <siddhesh at sourceware dot org>, libc-alpha at sourceware dot org
Date: Thu, 21 Sep 2017 12:31:26 -0600
Subject: Re: [PATCH 2/2] benchtests: Add a new argument -t to read throughput results
Authentication-results: sourceware.org; auth=none
References: <1505756414-12857-1-git-send-email-siddhesh@sourceware.org> <1505756414-12857-2-git-send-email-siddhesh@sourceware.org>

On 09/18/2017 11:40 AM, Siddhesh Poyarekar wrote:
> String benchmarks that store results as throughput rather than
> latencies will show positive improvements as negative.  Add a flag to
> fix the output of compare_strings.py in such cases.
> 
> 	* benchtests/scripts/compare_strings.py: New option -t.
... and you wouldn't need this patch if you'd not changed to throughput.

Can't you just post-process the data to get throughput for your fancy
graphs... or better yet add fancy graph support directly to benchtests ;-)

-- 
Cheers,
Carlos.

Follow-Ups:
- Re: [PATCH 2/2] benchtests: Add a new argument -t to read throughput results
  - From: Siddhesh Poyarekar

References:
- [PATCH 1/2] benchtests: Memory walking benchmark for memcpy
  - From: Siddhesh Poyarekar
- [PATCH 2/2] benchtests: Add a new argument -t to read throughput results
  - From: Siddhesh Poyarekar

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]