Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1502059 > unrolled thread
| Started by | Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> |
|---|---|
| First post | 2016-10-17 16:50 +0200 |
| Last post | 2016-10-17 16:50 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH] perf annotate: Cleanup arch specific stuff Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> - 2016-10-17 16:50 +0200
| From | Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> |
|---|---|
| Date | 2016-10-17 16:50 +0200 |
| Subject | Re: [PATCH] perf annotate: Cleanup arch specific stuff |
| Message-ID | <sthoJ-4JB-7@gated-at.bofh.it> |
On Monday 10 October 2016 10:26 PM, Arnaldo Carvalho de Melo wrote: > Em Mon, Oct 10, 2016 at 10:19:28PM +0530, Ravi Bangoria escreveu: >> >> On Monday 10 October 2016 10:09 PM, Naveen N. Rao wrote: >>> On 2016/10/10 01:24PM, Arnaldo Carvalho de Melo wrote: >>>> Em Mon, Oct 10, 2016 at 07:29:02PM +0530, Ravi Bangoria escreveu: >>>>> Move arch specific stuff from util/annotate.c to their respective >>>>> files in util/annotate directory. >>>>> >>>>> No functionality changes. >>>>> >>>>> Signed-off-by: Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com> >>>>> --- >>>>> tools/perf/util/Build | 1 + >>>>> tools/perf/util/annotate.c | 259 +++---------------------------------- >>>>> tools/perf/util/annotate.h | 23 ++++ >>>>> tools/perf/util/annotate/Build | 3 + >>>>> tools/perf/util/annotate/arm.c | 50 +++++++ >>>>> tools/perf/util/annotate/powerpc.c | 63 +++++++++ >>>>> tools/perf/util/annotate/x86.c | 107 +++++++++++++++ >>>> We already have a per arch area: tools/perf/arch/ >>> I think this was done to support cross-arch annotate similar to the >>> remote unwind support with util/libunwind/ >> Yes, because tools/perf/arch/ will only include host arch code. > Ok, thanks for clarifying. Hi Arnaldo, Are you ok with this patchset. Please let me know if you want to respin it. -Ravi > - Arnaldo >
Back to top | Article view | linux.kernel
csiph-web