From: Mathieu Poirier <mathieu.poirier@linaro.org> To: acme@kernel.org, peterz@infradead.org, gregkh@linuxfoundation.org Cc: mingo@redhat.com, tglx@linutronix.de, alexander.shishkin@linux.intel.com, schwidefsky@de.ibm.com, heiko.carstens@de.ibm.com, will.deacon@arm.com, mark.rutland@arm.com, jolsa@redhat.com, namhyung@kernel.org, adrian.hunter@intel.com, ast@kernel.org, hpa@zytor.com, suzuki.poulose@arm.com, linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: [PATCH 0/7] perf: Communicate sink via event::attr:config2 Date: Tue, 15 Jan 2019 16:07:35 -0700 [thread overview] Message-ID: <20190115230742.13730-1-mathieu.poirier@linaro.org> (raw) From: "Mathieu Poirier]" <mathieu.poirier@linaro.org> This set is a refurbished version of this one [1]. I dropped the version count and changed the name because a new approach is taken. The end result is the same though, that is to allow multiple sources to select the same sink for a session which is a prerequisite for the CoreSight support of CPU-wide trace scenarios. Here the sink ID is communicated to the kernel by way of the event's configuration attribure (event::attr::config2). The ID itself is an hash of the sink's name, something that is readily available and doesn't require the addition of a new information field in the coresight_device structure. With this new strategy the mechanic used to communicate sink selection to the kernel via sysfs is no longer needed and removed from the code base. Everthing has been applied and tested on 5.0-rc2. Regards, Mathieu PS: Greg, I'm drawing your attention to patch 02 where I seem to be the first to use function sysfs_add_file_to_group() in this way. [1]. https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1818488.html Mathieu Poirier (6): perf/aux: Make perf_event accessible to setup_aux() coresight: perf: Add "sinks" group to PMU directory coresight: Use event attributes for sink selection perf tools: Use event attributes to send sink information to kernel perf tools: Removing CoreSight set_drv_config() API perf tools: Remove PMU::set_drv_config API linaro (1): perf pmu: Moving EVENT_SOURCE_DEVICE_PATH to PMU header file arch/s390/kernel/perf_cpum_sf.c | 6 +- arch/x86/events/intel/bts.c | 4 +- arch/x86/events/intel/pt.c | 5 +- .../hwtracing/coresight/coresight-etm-perf.c | 65 ++++++++++--- .../hwtracing/coresight/coresight-etm-perf.h | 1 + drivers/hwtracing/coresight/coresight-priv.h | 1 + drivers/hwtracing/coresight/coresight.c | 56 +++++++++++ drivers/perf/arm_spe_pmu.c | 6 +- include/linux/perf_event.h | 2 +- kernel/events/ring_buffer.c | 2 +- tools/perf/arch/arm/util/cs-etm.c | 94 ++++++++----------- tools/perf/arch/arm/util/cs-etm.h | 3 - tools/perf/arch/arm/util/pmu.c | 2 - tools/perf/builtin-record.c | 10 -- tools/perf/builtin-stat.c | 9 -- tools/perf/builtin-top.c | 13 --- tools/perf/util/Build | 1 - tools/perf/util/drv_configs.c | 78 --------------- tools/perf/util/drv_configs.h | 26 ----- tools/perf/util/pmu.c | 2 - tools/perf/util/pmu.h | 2 +- 21 files changed, 163 insertions(+), 225 deletions(-) delete mode 100644 tools/perf/util/drv_configs.c delete mode 100644 tools/perf/util/drv_configs.h -- 2.17.1
WARNING: multiple messages have this Message-ID (diff)
From: Mathieu Poirier <mathieu.poirier@linaro.org> To: acme@kernel.org, peterz@infradead.org, gregkh@linuxfoundation.org Cc: mark.rutland@arm.com, linux-s390@vger.kernel.org, suzuki.poulose@arm.com, alexander.shishkin@linux.intel.com, will.deacon@arm.com, heiko.carstens@de.ibm.com, adrian.hunter@intel.com, ast@kernel.org, mingo@redhat.com, linux-arm-kernel@lists.infradead.org, hpa@zytor.com, schwidefsky@de.ibm.com, namhyung@kernel.org, tglx@linutronix.de, jolsa@redhat.com, linux-kernel@vger.kernel.org Subject: [PATCH 0/7] perf: Communicate sink via event::attr:config2 Date: Tue, 15 Jan 2019 16:07:35 -0700 [thread overview] Message-ID: <20190115230742.13730-1-mathieu.poirier@linaro.org> (raw) From: "Mathieu Poirier]" <mathieu.poirier@linaro.org> This set is a refurbished version of this one [1]. I dropped the version count and changed the name because a new approach is taken. The end result is the same though, that is to allow multiple sources to select the same sink for a session which is a prerequisite for the CoreSight support of CPU-wide trace scenarios. Here the sink ID is communicated to the kernel by way of the event's configuration attribure (event::attr::config2). The ID itself is an hash of the sink's name, something that is readily available and doesn't require the addition of a new information field in the coresight_device structure. With this new strategy the mechanic used to communicate sink selection to the kernel via sysfs is no longer needed and removed from the code base. Everthing has been applied and tested on 5.0-rc2. Regards, Mathieu PS: Greg, I'm drawing your attention to patch 02 where I seem to be the first to use function sysfs_add_file_to_group() in this way. [1]. https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1818488.html Mathieu Poirier (6): perf/aux: Make perf_event accessible to setup_aux() coresight: perf: Add "sinks" group to PMU directory coresight: Use event attributes for sink selection perf tools: Use event attributes to send sink information to kernel perf tools: Removing CoreSight set_drv_config() API perf tools: Remove PMU::set_drv_config API linaro (1): perf pmu: Moving EVENT_SOURCE_DEVICE_PATH to PMU header file arch/s390/kernel/perf_cpum_sf.c | 6 +- arch/x86/events/intel/bts.c | 4 +- arch/x86/events/intel/pt.c | 5 +- .../hwtracing/coresight/coresight-etm-perf.c | 65 ++++++++++--- .../hwtracing/coresight/coresight-etm-perf.h | 1 + drivers/hwtracing/coresight/coresight-priv.h | 1 + drivers/hwtracing/coresight/coresight.c | 56 +++++++++++ drivers/perf/arm_spe_pmu.c | 6 +- include/linux/perf_event.h | 2 +- kernel/events/ring_buffer.c | 2 +- tools/perf/arch/arm/util/cs-etm.c | 94 ++++++++----------- tools/perf/arch/arm/util/cs-etm.h | 3 - tools/perf/arch/arm/util/pmu.c | 2 - tools/perf/builtin-record.c | 10 -- tools/perf/builtin-stat.c | 9 -- tools/perf/builtin-top.c | 13 --- tools/perf/util/Build | 1 - tools/perf/util/drv_configs.c | 78 --------------- tools/perf/util/drv_configs.h | 26 ----- tools/perf/util/pmu.c | 2 - tools/perf/util/pmu.h | 2 +- 21 files changed, 163 insertions(+), 225 deletions(-) delete mode 100644 tools/perf/util/drv_configs.c delete mode 100644 tools/perf/util/drv_configs.h -- 2.17.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next reply other threads:[~2019-01-15 23:07 UTC|newest] Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-01-15 23:07 Mathieu Poirier [this message] 2019-01-15 23:07 ` [PATCH 0/7] perf: Communicate sink via event::attr:config2 Mathieu Poirier 2019-01-15 23:07 ` [PATCH 1/7] perf/aux: Make perf_event accessible to setup_aux() Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier 2019-01-16 10:21 ` Suzuki K Poulose 2019-01-16 10:21 ` Suzuki K Poulose 2019-01-15 23:07 ` [PATCH 2/7] coresight: perf: Add "sinks" group to PMU directory Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier 2019-01-16 10:29 ` Suzuki K Poulose 2019-01-16 10:29 ` Suzuki K Poulose 2019-01-16 23:43 ` Mathieu Poirier 2019-01-16 23:43 ` Mathieu Poirier 2019-01-17 11:55 ` Suzuki K Poulose 2019-01-17 11:55 ` Suzuki K Poulose 2019-01-16 15:39 ` Greg KH 2019-01-16 15:39 ` Greg KH 2019-01-16 16:14 ` Mathieu Poirier 2019-01-16 16:14 ` Mathieu Poirier 2019-01-16 16:33 ` Greg KH 2019-01-16 16:33 ` Greg KH 2019-01-16 16:38 ` Mathieu Poirier 2019-01-16 16:38 ` Mathieu Poirier 2019-01-16 17:12 ` Greg KH 2019-01-16 17:12 ` Greg KH 2019-01-16 17:30 ` Mathieu Poirier 2019-01-16 17:30 ` Mathieu Poirier 2019-01-15 23:07 ` [PATCH 3/7] coresight: Use event attributes for sink selection Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier 2019-01-16 10:36 ` Suzuki K Poulose 2019-01-16 10:36 ` Suzuki K Poulose 2019-01-16 23:57 ` Mathieu Poirier 2019-01-16 23:57 ` Mathieu Poirier 2019-01-17 17:33 ` Suzuki K Poulose 2019-01-17 17:33 ` Suzuki K Poulose 2019-01-23 21:03 ` Peter Zijlstra 2019-01-23 21:03 ` Peter Zijlstra 2019-01-23 21:05 ` Peter Zijlstra 2019-01-23 21:05 ` Peter Zijlstra 2019-01-15 23:07 ` [PATCH 4/7] perf pmu: Moving EVENT_SOURCE_DEVICE_PATH to PMU header file Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier 2019-01-16 10:37 ` Suzuki K Poulose 2019-01-16 10:37 ` Suzuki K Poulose 2019-01-15 23:07 ` [PATCH 5/7] perf tools: Use event attributes to send sink information to kernel Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier 2019-01-15 23:07 ` [PATCH 6/7] perf tools: Removing CoreSight set_drv_config() API Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier 2019-01-15 23:07 ` [PATCH 7/7] perf tools: Remove PMU::set_drv_config API Mathieu Poirier 2019-01-15 23:07 ` Mathieu Poirier
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=20190115230742.13730-1-mathieu.poirier@linaro.org \ --to=mathieu.poirier@linaro.org \ --cc=acme@kernel.org \ --cc=adrian.hunter@intel.com \ --cc=alexander.shishkin@linux.intel.com \ --cc=ast@kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=heiko.carstens@de.ibm.com \ --cc=hpa@zytor.com \ --cc=jolsa@redhat.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-s390@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mingo@redhat.com \ --cc=namhyung@kernel.org \ --cc=peterz@infradead.org \ --cc=schwidefsky@de.ibm.com \ --cc=suzuki.poulose@arm.com \ --cc=tglx@linutronix.de \ --cc=will.deacon@arm.com \ /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: linkBe 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.