Messages in this thread | | | Date | Mon, 14 Nov 2016 11:46:12 +0100 | From | Peter Zijlstra <> | Subject | Re: [PATCH 1/3] perf: add PERF_RECORD_NAMESPACES to include namespaces related info |
| |
On Mon, Nov 14, 2016 at 04:02:30PM +0530, Hari Bathini wrote:
> >> PERF_RECORD_SWITCH_CPU_WIDE = 15, > >>+ /* > >>+ * struct { > >>+ * struct perf_event_header header; > >>+ * > >>+ * u32 pid, tid; > >>+ * u64 time; > >>+ * u32 uts_ns_inum; > >>+ * u32 ipc_ns_inum; > >>+ * u32 mnt_ns_inum; > >>+ * u32 pid_ns_inum; > >>+ * u32 net_ns_inum; > >>+ * u32 cgroup_ns_inum; > >>+ * u32 user_ns_inum; > >>+ * struct sample_id sample_id; > >>+ * }; > >>+ */ > >>+ PERF_RECORD_NAMESPACES = 16, > >So this format is not extensible, that is, if someone adds yet another > >namespace, we'll need to introduce PERF_RECORD_NAMESPACES2. > > > >Is there a 'natural' and exposed namespace index that we can use to > >change it like: > > > > u32 nr_nss; > > u32 namespace[nr_nss]; > > > >? > > Nothing of that sort exists, currently. > Maybe, time to introduce with this patch-set..?
Would be good, but you'll have to sort that with the namespace folks.
| |