Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756260Ab2F0O2Y (ORCPT ); Wed, 27 Jun 2012 10:28:24 -0400 Received: from mail-wg0-f42.google.com ([74.125.82.42]:64423 "EHLO mail-wg0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751903Ab2F0O2X convert rfc822-to-8bit (ORCPT ); Wed, 27 Jun 2012 10:28:23 -0400 MIME-Version: 1.0 In-Reply-To: References: <1339944948-3193-1-git-send-email-namhyung@kernel.org> <20120618090501.GB28824@gmail.com> <87mx3zhekz.fsf@sejong.aot.lge.com> Date: Wed, 27 Jun 2012 23:28:22 +0900 Message-ID: Subject: Re: [PATCH] perf tools: Fix an error on perf-bench document From: Hitoshi Mitake To: Namhyung Kim Cc: Ingo Molnar , Arnaldo Carvalho de Melo , Ingo Molnar , Peter Zijlstra , LKML Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2584 Lines: 97 On Mon, Jun 25, 2012 at 12:59 AM, Hitoshi Mitake wrote: > On Tue, Jun 19, 2012 at 5:48 PM, Namhyung Kim wrote: >> Hi, Ingo >> >> On Mon, 18 Jun 2012 11:05:01 +0200, Ingo Molnar wrote: >>> There's several typos all across the perf bench documentation: >>> >>> 1) >>> >>> ? ? ? ?This perf bench command is general framework for benchmark suites. >>> >>> s/is general/is a general >>> >> >> Will fix. >> >> >>> 2) >>> >>> The output of 'perf bench --help' outputs weird looking >>> formatting characters like: >>> >>> ? ? ? ? ? ?.ft C >>> ? ? ? ? ? ?% perf bench sched pipe ? ? ? ? ? ? ? ? ? ? ?# with no style specified >>> ? ? ? ? ? ?(executing 1000000 pipe operations between two tasks) >>> ? ? ? ? ? ? ? ? ? ?Total time:5.855 sec >>> ? ? ? ? ? ? ? ? ? ? ? ? ? ?5.855061 usecs/op >>> ? ? ? ? ? ? ? ? ? ? ? ? ? ?170792 ops/sec >>> ? ? ? ? ? ?.ft >>> >> >> I have no idea on this, maybe a problem of asciidoc? >> >> >>> 3) >>> >>> This: >>> >>> ? ? ? ? ? ?all: test all suite (pseudo suite) >>> >>> s/all suite (psudo suite)/all benchmark suites >>> >>> (this should be fixed for perf bench mem as well.) >>> >> >> Will fix. >> >> >>> 4) >>> >>> perf bench mem memcpy -h outputs: >>> >>> ? ? -l, --length <1MB> ? ?Specify length of memory to copy. available unit: B, MB, GB (upper and lower) >>> >>> bad capitalization: s/. a/. A >>> >>> s/unit/units >>> >> >> Will fix too. >> >> >>> 5) >>> >>> This: >>> >>> ? ? -c, --clock ? ? ? ? ? Use CPU clock for measuring >>> >>> should probably clarify why the CPU clock matters and what >>> measurement is done otherwise. >>> >> >> This looks a bit weird. If -c switch was given, it'd use the cycles >> event not the cpu-clock event. If omitted, it'd use the gtod(). >> > > Very sorry, I misused the terms clock and cycles. The memset uses > hardware cycles event, the above naming and description with "clock" > is very bad... > I will write and send a patch for fix the problem. I wrote the simple fix for eliminatng wrong usage of "clock": https://github.com/mitake/linux/commit/b7836d41deff430c16058ca729120dccfa41e74e This patch assumes the change made by the Namhyung's patch. So I'm planning to send this after the Namhyung's patch is applied to tip tree. Or should the patches be unified before applied? -- Hitoshi Mitake h.mitake@gmail.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/