Maybe we should do this:
Looks good to me.
As a side note, while testing this I noticed that the make perf run
goes a lot faster if you set GIT_PERF_REPEAT_COUNT=1. This is bad for
actually measuring things, but probably fine for profile feedback. I
don't use the profile builds myself,
Fix the bitrotted profile feedback support.
Changes to v1:
- Remove obsolete comment
- Remove controversal diff script.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at
From: Andi Kleen a...@linux.intel.com
Use BASIC_CFLAGS instead of CFLAGS to set up the profile feedback
option in the Makefile.
This allows still overriding CFLAGS on the make command line
without disabling profile feedback.
Signed-off-by: Andi Kleen a...@linux.intel.com
---
Makefile | 4
From: Andi Kleen a...@linux.intel.com
Profile feedback always failed for me with -jN. The problem
was that there was no implicit ordering between the profile generate
stage and the profile use stage. So some objects in the later stage
would be linked with profile generate objects, and fail due
From: Andi Kleen a...@linux.intel.com
Signed-off-by: Andi Kleen a...@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
From: Andi Kleen a...@linux.intel.com
Profile feedback sets -DNO_NORETURN, which causes the compat
header file to go into a default #else block. That #else
block defines away __attribute__(). Doing so causes all
kinds of problems with the Linux and gcc system headers:
in particular it makes
On Mon, Jul 07, 2014 at 02:06:57PM -0700, Junio C Hamano wrote:
Andi Kleen a...@firstfloor.org writes:
From: Andi Kleen a...@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
a justification why the geometric mean is used here would increase my
confident significantly.
It's just a standard way to summarize sets of benchmarks. For example SPEC
uses the same approach.
Anyways the script is not essential to the rest of the profile feedback
feature. Just ignore it if
From: Andi Kleen a...@linux.intel.com
Use BASIC_CFLAGS instead of CFLAGS to set up the profile feedback
option in the Makefile.
This allows still overriding CFLAGS on the make command line
without disabling profile feedback.
Signed-off-by: Andi Kleen a...@linux.intel.com
---
Makefile | 4
From: Andi Kleen a...@linux.intel.com
Profile feedback sets -DNO_NORETURN, which causes the compat
header file to go into a default #else block. That #else
block defines away __attribute__(). Doing so causes all
kinds of problems with the Linux and gcc system headers:
in particular it makes
From: Andi Kleen a...@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?
Signed-off-by: Andi Kleen a...@linux.intel.com
---
Makefile | 1 +
1 file changed, 1 insertion(+)
diff --git
The profile feedback build support had bitrotted. This patchkit fixes
it and improves it slightly by adding a new profile-fast target.
-Andi
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at
From: Andi Kleen a...@linux.intel.com
Profile feedback always failed for me with -jN. The problem
was that there was no implicit ordering between the profile generate
stage and the profile use stage. So some objects in the later stage
would be linked with profile generate objects, and fail due
From: Andi Kleen a...@linux.intel.com
Signed-off-by: Andi Kleen a...@linux.intel.com
---
diff-res | 26 ++
1 file changed, 26 insertions(+)
create mode 100755 diff-res
diff --git a/diff-res b/diff-res
new file mode 100755
index 000..90d57be
--- /dev/null
+++ b/diff
Can you please test if this patch fixes it?
-Andi
Don't compare linux processor index with APICID
Fixes boot up lockups on some machines where CPU apic ids
don't start with 0
Signed-off-by: Andi Kleen [EMAIL PROTECTED]
Index: linux/arch/x86_64/kernel/smpboot.c
15 matches
Mail list logo