From: Junio C Hamano <gitster@pobox.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: git@vger.kernel.org, Andi Kleen <ak@linux.intel.com>
Subject: Re: [PATCH 3/5] Run the perf test suite for profile feedback too
Date: Mon, 07 Jul 2014 14:06:57 -0700 [thread overview]
Message-ID: <xmqq1ttwdgjy.fsf@gitster.dls.corp.google.com> (raw)
In-Reply-To: <1404517432-25185-4-git-send-email-andi@firstfloor.org> (Andi Kleen's message of "Fri, 4 Jul 2014 16:43:50 -0700")
Andi Kleen <andi@firstfloor.org> writes:
> From: Andi Kleen <ak@linux.intel.com>
>
> Open: If the perf test suite is representative enough it may
> be reasonable to only run that and skip the much longer full
> test suite. Thoughts?
I do not think it right now is representative, nor it was meant to
become so. The operations are those that people cared about and
tuned, and hopefully it would cover stuff actual end users care
about in the real life, though.
>
> Signed-off-by: Andi Kleen <ak@linux.intel.com>
> ---
> Makefile | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Makefile b/Makefile
> index a9770ac..ba64be9 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1647,6 +1647,7 @@ ifeq ($(filter all,$(MAKECMDGOALS)),all)
> all:: profile-clean
> $(MAKE) PROFILE=GEN all
> $(MAKE) PROFILE=GEN -j1 test
> + $(MAKE) PROFILE=GEN -j1 perf
> endif
> endif
next prev parent reply other threads:[~2014-07-07 21:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-04 23:43 Fix the bitrotted profile feedback build Andi Kleen
2014-07-04 23:43 ` [PATCH 1/5] Use BASIC_FLAGS for profile feedback Andi Kleen
2014-07-04 23:43 ` [PATCH 2/5] Don't define away __attribute__ on gcc Andi Kleen
2014-07-04 23:43 ` [PATCH 3/5] Run the perf test suite for profile feedback too Andi Kleen
2014-07-07 21:06 ` Junio C Hamano [this message]
2014-07-07 22:15 ` Andi Kleen
2014-07-04 23:43 ` [PATCH 4/5] Fix profile feedback with -jN and add profile-fast Andi Kleen
2014-07-04 23:43 ` [PATCH 5/5] Add a little script to compare two make perf runs Andi Kleen
2014-07-06 16:12 ` Bert Wesarg
2014-07-06 16:15 ` Andi Kleen
2014-07-06 16:46 ` Bert Wesarg
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=xmqq1ttwdgjy.fsf@gitster.dls.corp.google.com \
--to=gitster@pobox.com \
--cc=ak@linux.intel.com \
--cc=andi@firstfloor.org \
--cc=git@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.