RE: [PATCH RFC 02/10] perf,tools: Support new sort type --socket

From	"Liang, Kan" <kan.liang@intel.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH RFC 02/10] perf,tools: Support new sort type --socket
Date	2015-08-27 20:30 +0200
Message-ID	<q2a5X-41H-5@gated-at.bofh.it> (permalink)
References	(5 earlier) <q10V4-1Sr-17@gated-at.bofh.it> <q10V4-1Sr-15@gated-at.bofh.it> <q136y-4Vw-7@gated-at.bofh.it> <q15L4-tl-29@gated-at.bofh.it> <q2a5X-41H-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

> 
> On Mon, Aug 24, 2015 at 09:30:21PM +0200, Jiri Olsa wrote:
> > On Mon, Aug 24, 2015 at 04:47:12PM +0000, Liang, Kan wrote:
> > >
> > >
> > > > On Mon, Aug 24, 2015 at 02:22:08PM +0000, Liang, Kan wrote:
> > > > > >
> > > > > > On Fri, Aug 21, 2015 at 08:25:24PM +0000, Liang, Kan wrote:
> > > > > >
> > > > > > SNIP
> > > > > >
> > > > > > > >
> > > > > > > > we need global topology information in perf.data and use
> > > > > > > > the mapping from there, we can't use current server info
> > > > > > > >
> > > > > > > > we currently store core_siblings_list and
> > > > > > > > thread_siblings_list, in topology FEATURE, which is
> > > > > > > > probably not enough
> > > > > > > >
> > > > > > >
> > > > > > > core_siblings_list  includes the cpu list in the same socket.
> > > > > > > thread_siblings_list includes the cpu list in the same core.
> > > > > > > numa_nodes includes the cpu list for each node.
> > > > > > >
> > > > > > > It looks we have enough data from topology FEATURE.
> > > > > >
> > > > > > hum, haven't hecked deeply.. how will you get core id for cpu?
> > > > > >
> > > > >
> > > > > from thread_siblings_list.
> > > > >  I just noticed that svg_build_topology_map did the similar
> > > > > thing to get topology map for timechart from perf header.
> > > >
> > > > could you please provide both functions then cpu -> core, cpu ->
> > > > socket
> > > >
> > >
> > > Do you mean something like this?
> > > Store cpu->socket and cpu->core in perf_session_env.
> >
> > yep, seems ok
> 
> any chance you could send out this one? I'd need to use it in my other stuff
> 

Ok I will send out this patch as a separate patch first.


--
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/

Thread

Re: [PATCH RFC 02/10] perf,tools: Support new sort type --socket Jiri Olsa <jolsa@redhat.com> - 2015-08-20 11:10 +0200
  RE: [PATCH RFC 02/10] perf,tools: Support new sort type --socket "Liang, Kan" <kan.liang@intel.com> - 2015-08-21 22:30 +0200
    Re: [PATCH RFC 02/10] perf,tools: Support new sort type --socket Jiri Olsa <jolsa@redhat.com> - 2015-08-24 00:10 +0200
      Re: [PATCH RFC 02/10] perf,tools: Support new sort type --socket Jiri Olsa <jolsa@redhat.com> - 2015-08-24 16:30 +0200
        RE: [PATCH RFC 02/10] perf,tools: Support new sort type --socket "Liang, Kan" <kan.liang@intel.com> - 2015-08-24 18:50 +0200
          Re: [PATCH RFC 02/10] perf,tools: Support new sort type --socket Jiri Olsa <jolsa@redhat.com> - 2015-08-24 21:40 +0200
            RE: [PATCH RFC 02/10] perf,tools: Support new sort type --socket "Liang, Kan" <kan.liang@intel.com> - 2015-08-27 20:30 +0200
            Re: [PATCH RFC 02/10] perf,tools: Support new sort type --socket Jiri Olsa <jolsa@redhat.com> - 2015-08-27 20:30 +0200
      RE: [PATCH RFC 02/10] perf,tools: Support new sort type --socket "Liang, Kan" <kan.liang@intel.com> - 2015-08-24 16:30 +0200

csiph-web