Re: [PATCH] coresight: Moving framework and drivers to SPDX identifier

2018-04-17 Thread Alexander Shishkin
On Mon, Apr 16, 2018 at 02:51:02PM -0600, Mathieu Poirier wrote: > Moving all kernel side CoreSight framework and drivers to SPDX identifier. > > Signed-off-by: Mathieu Poirier This reminds me: there's drivers/hwtracing/stm/ftrace.c that I didn't convert during the previous round, do you want

[PATCH v2] perf: Suppress AUX/OVERWRITE records

2018-04-04 Thread Alexander Shishkin
there have userspace that depends on a constant stream of OVERWRITE records for a good reason, they'll have to let us know. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> Cc: Markus Metzger <markus.t.metz...@intel.com> Cc: Adrian Hunter <adrian.hun...@intel.com>

[PATCH v2] perf: Suppress AUX/OVERWRITE records

2018-04-04 Thread Alexander Shishkin
there have userspace that depends on a constant stream of OVERWRITE records for a good reason, they'll have to let us know. Signed-off-by: Alexander Shishkin Cc: Markus Metzger Cc: Adrian Hunter --- kernel/events/ring_buffer.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff

[tip:perf/core] perf/x86/pt, coresight: Clean up address filter structure

2018-03-29 Thread tip-bot for Alexander Shishkin
Commit-ID: 6ed70cf342de03c7b11cd4eb032705faeb29d284 Gitweb: https://git.kernel.org/tip/6ed70cf342de03c7b11cd4eb032705faeb29d284 Author: Alexander Shishkin <alexander.shish...@linux.intel.com> AuthorDate: Thu, 29 Mar 2018 15:06:48 +0300 Committer: Ingo Molnar <mi...@kernel.org>

[tip:perf/core] perf/x86/pt, coresight: Clean up address filter structure

2018-03-29 Thread tip-bot for Alexander Shishkin
Commit-ID: 6ed70cf342de03c7b11cd4eb032705faeb29d284 Gitweb: https://git.kernel.org/tip/6ed70cf342de03c7b11cd4eb032705faeb29d284 Author: Alexander Shishkin AuthorDate: Thu, 29 Mar 2018 15:06:48 +0300 Committer: Ingo Molnar CommitDate: Thu, 29 Mar 2018 16:07:22 +0200 perf/x86/pt

Re: [PATCH v3 1/3] perf/core: store context switch out type into Perf trace

2018-03-29 Thread Alexander Shishkin
On Mon, Mar 26, 2018 at 12:20:32PM +0300, Alexey Budankov wrote: > > Store thread context-switch-out event type into Perf trace as a part of > PERF_RECORD_SWITCH[_CPU_WIDE] records. > > Introduced types of switch-out events assumed to be > a) preempt: task->state == TASK_RUNNING and b)

Re: [PATCH v3 1/3] perf/core: store context switch out type into Perf trace

2018-03-29 Thread Alexander Shishkin
On Mon, Mar 26, 2018 at 12:20:32PM +0300, Alexey Budankov wrote: > > Store thread context-switch-out event type into Perf trace as a part of > PERF_RECORD_SWITCH[_CPU_WIDE] records. > > Introduced types of switch-out events assumed to be > a) preempt: task->state == TASK_RUNNING and b)

[PATCH v2] perf, pt, coresight: Clean up address filter structure

2018-03-29 Thread Alexander Shishkin
, so this patch replaces it with 'action' enum. The 'range' field is completely redundant (meaning that the filter is an address range as opposed to a single address trigger), as we can use zero size to mean the same thing. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com>

[PATCH v2] perf, pt, coresight: Clean up address filter structure

2018-03-29 Thread Alexander Shishkin
, so this patch replaces it with 'action' enum. The 'range' field is completely redundant (meaning that the filter is an address range as opposed to a single address trigger), as we can use zero size to mean the same thing. Signed-off-by: Alexander Shishkin Acked-by: Mathieu Poirier --- arch/x86

[GIT PULL 03/10] stm class: Make dummy's master/channel ranges configurable

2018-03-29 Thread Alexander Shishkin
To allow for more flexible testing of the stm class, make it possible to specify the ranges of masters and channels that the dummy_stm devices cover. This is done via module parameters. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/stm/dummy

[GIT PULL 03/10] stm class: Make dummy's master/channel ranges configurable

2018-03-29 Thread Alexander Shishkin
To allow for more flexible testing of the stm class, make it possible to specify the ranges of masters and channels that the dummy_stm devices cover. This is done via module parameters. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/stm/dummy_stm.c | 24

[GIT PULL 01/10] MAINTAINERS: Bestow upon myself the care for drivers/hwtracing

2018-03-29 Thread Alexander Shishkin
Although all components of drivers/hwtracing are Maintained, the directory itself isn't, which leads to confusion when people want to patch the Kconfig file, for example. This adds an entry to the MAINTAINERS file with myself as a maintainer. Signed-off-by: Alexander Shishkin <alexander.sh

[GIT PULL 01/10] MAINTAINERS: Bestow upon myself the care for drivers/hwtracing

2018-03-29 Thread Alexander Shishkin
Although all components of drivers/hwtracing are Maintained, the directory itself isn't, which leads to confusion when people want to patch the Kconfig file, for example. This adds an entry to the MAINTAINERS file with myself as a maintainer. Signed-off-by: Alexander Shishkin --- MAINTAINERS

[GIT PULL 05/10] intel_th: Use correct method of finding hub

2018-03-29 Thread Alexander Shishkin
ce") Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/intel_th/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwtracing/intel_th/core.c b/drivers/hwtracing/intel_th/core.c index bf1e4543c722..8e4f5fd

[GIT PULL 05/10] intel_th: Use correct method of finding hub

2018-03-29 Thread Alexander Shishkin
ce") Signed-off-by: Alexander Shishkin --- drivers/hwtracing/intel_th/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwtracing/intel_th/core.c b/drivers/hwtracing/intel_th/core.c index bf1e4543c722..8e4f5fd8a43e 100644 --- a/drivers/hwtracing/intel_th/

[GIT PULL 07/10] intel_th: Pick up irq number from resources

2018-03-29 Thread Alexander Shishkin
Platform devices pass their IRQs around as resources, so as a convenience for the glue layer code, allow them pass the IRQ to the core driver in the resources array. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/intel_th/core.c | 9 -

[GIT PULL 07/10] intel_th: Pick up irq number from resources

2018-03-29 Thread Alexander Shishkin
Platform devices pass their IRQs around as resources, so as a convenience for the glue layer code, allow them pass the IRQ to the core driver in the resources array. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/intel_th/core.c | 9 - 1 file changed, 8 insertions(+), 1

[GIT PULL 02/10] stm class: Add SPDX GPL-2.0 header to replace GPLv2 boilerplate

2018-03-29 Thread Alexander Shishkin
This adds SPDX GPL-2.0 header to to stm core files and removes the GPLv2 boilerplate text. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/stm/console.c | 10 +- drivers/hwtracing/stm/core.c | 10 +- drivers/hwtraci

[GIT PULL 02/10] stm class: Add SPDX GPL-2.0 header to replace GPLv2 boilerplate

2018-03-29 Thread Alexander Shishkin
This adds SPDX GPL-2.0 header to to stm core files and removes the GPLv2 boilerplate text. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/stm/console.c | 10 +- drivers/hwtracing/stm/core.c | 10 +- drivers/hwtracing/stm/dummy_stm.c | 10 +- drivers

[GIT PULL 06/10] intel_th: Don't touch switch routing in host mode

2018-03-29 Thread Alexander Shishkin
When the Trace Hub is operating in Host Debugger mode, it is up to the debugger to configure master routing even for the software sources. Do not do this in the driver in this case. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/intel_th/core

[GIT PULL 09/10] intel_th: Add ACPI glue layer

2018-03-29 Thread Alexander Shishkin
The Trace Hub devices now can be enumerated as ACPI devices, which translates into "Host Debugger mode". There are two IDs: one for PCH Trace Hub, and one for the uncore Trace Hub. These are expected to stay the same across all platforms. Signed-off-by: Alexander Shishkin <al

[GIT PULL 06/10] intel_th: Don't touch switch routing in host mode

2018-03-29 Thread Alexander Shishkin
When the Trace Hub is operating in Host Debugger mode, it is up to the debugger to configure master routing even for the software sources. Do not do this in the driver in this case. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/intel_th/core.c | 4 1 file changed, 4 insertions

[GIT PULL 09/10] intel_th: Add ACPI glue layer

2018-03-29 Thread Alexander Shishkin
The Trace Hub devices now can be enumerated as ACPI devices, which translates into "Host Debugger mode". There are two IDs: one for PCH Trace Hub, and one for the uncore Trace Hub. These are expected to stay the same across all platforms. Signed-off-by: Alexander Shishkin --- drivers

[GIT PULL 08/10] intel_th: Allow forcing host mode through drvdata

2018-03-29 Thread Alexander Shishkin
Some devices can only operate in host mode, so we need means of communicating this to the core driver on per-device basis. This adds a flag to drvdata to signal host-only capability to the core. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/in

[GIT PULL 08/10] intel_th: Allow forcing host mode through drvdata

2018-03-29 Thread Alexander Shishkin
Some devices can only operate in host mode, so we need means of communicating this to the core driver on per-device basis. This adds a flag to drvdata to signal host-only capability to the core. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/intel_th/core.c | 6 -- drivers

[GIT PULL 10/10] hwtracing: Add HW tracing support menu

2018-03-29 Thread Alexander Shishkin
ng listed at the top level of the Device Drivers menu. Signed-off-by: Randy Dunlap <rdun...@infradead.org> Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/Kconfig | 4 +--- drivers/hwtracing/Kconfig | 7 +++ 2 files changed, 8 insertions

[GIT PULL 10/10] hwtracing: Add HW tracing support menu

2018-03-29 Thread Alexander Shishkin
of the Device Drivers menu. Signed-off-by: Randy Dunlap Signed-off-by: Alexander Shishkin --- drivers/Kconfig | 4 +--- drivers/hwtracing/Kconfig | 7 +++ 2 files changed, 8 insertions(+), 3 deletions(-) create mode 100644 drivers/hwtracing/Kconfig diff --git a/drivers/Kconfig b/d

[GIT PULL 04/10] intel_th: Add SPDX GPL-2.0 header to replace GPLv2 boilerplate

2018-03-29 Thread Alexander Shishkin
This adds SPDX GPL-2.0 header to the Trace Hub driver and removes the GPLv2 boilerplate text. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- drivers/hwtracing/intel_th/core.c | 10 +- drivers/hwtracing/intel_th/debug.c| 10 +- drivers/hwt

[GIT PULL 04/10] intel_th: Add SPDX GPL-2.0 header to replace GPLv2 boilerplate

2018-03-29 Thread Alexander Shishkin
This adds SPDX GPL-2.0 header to the Trace Hub driver and removes the GPLv2 boilerplate text. Signed-off-by: Alexander Shishkin --- drivers/hwtracing/intel_th/core.c | 10 +- drivers/hwtracing/intel_th/debug.c| 10 +- drivers/hwtracing/intel_th/debug.h| 10

[GIT PULL 00/10] stm class/intel_th: Updates for v4.17

2018-03-29 Thread Alexander Shishkin
ore module parameters to dummy_stm for better test coverage ---- Alexander Shishkin (9): MAINTAINERS: Bestow upon myself the care for drivers/hwtracing stm class: Add SPDX GPL-2.0 header to replace GPLv2 boilerplate stm c

[GIT PULL 00/10] stm class/intel_th: Updates for v4.17

2018-03-29 Thread Alexander Shishkin
ore module parameters to dummy_stm for better test coverage ---- Alexander Shishkin (9): MAINTAINERS: Bestow upon myself the care for drivers/hwtracing stm class: Add SPDX GPL-2.0 header to replace GPLv2 boilerplate stm c

[PATCH] perf: Allow suppressing AUX records

2018-01-15 Thread Alexander Shishkin
userspace depends on AUX records in the overwrite mode, we preserve the original behavior and add an opt-in for the new behavior, wherein the 'useless' records get suppressed. This patch adds an attribute bit to the described effect. Signed-off-by: Alexander Shishkin <alexander.sh

[PATCH] perf: Allow suppressing AUX records

2018-01-15 Thread Alexander Shishkin
userspace depends on AUX records in the overwrite mode, we preserve the original behavior and add an opt-in for the new behavior, wherein the 'useless' records get suppressed. This patch adds an attribute bit to the described effect. Signed-off-by: Alexander Shishkin Cc: Markus Metzger Cc: Adrian

[PATCH v1 4/4] perf intel-pt, intel-bts: Suppress useless AUX records by default

2017-11-14 Thread Alexander Shishkin
This makes use of the shiny new attr::suppress_aux that suppresses the AUX records that don't carry any 'interesting' information for the decoders, that is PERF_RECORD_AUX[flag==OVERWRITE], which just stack up in the DATA buffer for no good reason. Signed-off-by: Alexander Shishkin

[PATCH v1 4/4] perf intel-pt, intel-bts: Suppress useless AUX records by default

2017-11-14 Thread Alexander Shishkin
This makes use of the shiny new attr::suppress_aux that suppresses the AUX records that don't carry any 'interesting' information for the decoders, that is PERF_RECORD_AUX[flag==OVERWRITE], which just stack up in the DATA buffer for no good reason. Signed-off-by: Alexander Shishkin --- tools

[PATCH v1 3/4] perf tools: Add 'suppress_aux' attribute bit definition and fallback

2017-11-14 Thread Alexander Shishkin
This adds support for suppress_aux, the switch that enables suppressing not-so-useful PERF_RECORD_AUX records. Also handle kernels where it's not supported. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- tools/perf/util/evsel.c | 9 + 1 file chan

[PATCH v1 3/4] perf tools: Add 'suppress_aux' attribute bit definition and fallback

2017-11-14 Thread Alexander Shishkin
This adds support for suppress_aux, the switch that enables suppressing not-so-useful PERF_RECORD_AUX records. Also handle kernels where it's not supported. Signed-off-by: Alexander Shishkin --- tools/perf/util/evsel.c | 9 + 1 file changed, 9 insertions(+) diff --git a/tools/perf/util

[PATCH v1 2/4] tools, perf_event.h: Synchronize

2017-11-14 Thread Alexander Shishkin
I've just added attr::suppress_aux bit to the kernel header, adding it to the tools' header too. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- tools/include/uapi/linux/perf_event.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/i

[PATCH v1 2/4] tools, perf_event.h: Synchronize

2017-11-14 Thread Alexander Shishkin
I've just added attr::suppress_aux bit to the kernel header, adding it to the tools' header too. Signed-off-by: Alexander Shishkin --- tools/include/uapi/linux/perf_event.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/include/uapi/linux/perf_event.h b/tools

[PATCH v1 1/4] perf: Allow suppressing AUX records

2017-11-14 Thread Alexander Shishkin
an attribute bit that enables suppressing such records. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> Cc: Markus Metzger <markus.t.metz...@intel.com> Cc: Adrian Hunter <adrian.hun...@intel.com> --- include/uapi/linux/perf_event.h | 3 ++- kernel/events/core

[PATCH v1 1/4] perf: Allow suppressing AUX records

2017-11-14 Thread Alexander Shishkin
an attribute bit that enables suppressing such records. Signed-off-by: Alexander Shishkin Cc: Markus Metzger Cc: Adrian Hunter --- include/uapi/linux/perf_event.h | 3 ++- kernel/events/core.c| 5 + kernel/events/ring_buffer.c | 12 ++-- 3 files changed, 17 insertions(+), 3

[PATCH v1 0/4] perf: Allow suppressing useless AUX records

2017-11-14 Thread Alexander Shishkin
periods of time, making the buildup of AUX records an unnecessary annoyance. Alexander Shishkin (4): perf: Allow suppressing AUX records tools, perf_event.h: Synchronize perf tools: Add 'suppress_aux' attribute bit definition and fallback perf intel-pt, intel-bts: Suppress useless AUX records

[PATCH v1 0/4] perf: Allow suppressing useless AUX records

2017-11-14 Thread Alexander Shishkin
periods of time, making the buildup of AUX records an unnecessary annoyance. Alexander Shishkin (4): perf: Allow suppressing AUX records tools, perf_event.h: Synchronize perf tools: Add 'suppress_aux' attribute bit definition and fallback perf intel-pt, intel-bts: Suppress useless AUX records

[tip:x86/urgent] x86/debug: Handle warnings before the notifier chain, to fix KGDB crash

2017-11-10 Thread tip-bot for Alexander Shishkin
Commit-ID: b8347c2196492f4e1cccde3d92fda1cc2cc7de7e Gitweb: https://git.kernel.org/tip/b8347c2196492f4e1cccde3d92fda1cc2cc7de7e Author: Alexander Shishkin <alexander.shish...@linux.intel.com> AuthorDate: Mon, 24 Jul 2017 13:04:28 +0300 Committer: Ingo Molnar <mi...@kernel.org>

[tip:x86/urgent] x86/debug: Handle warnings before the notifier chain, to fix KGDB crash

2017-11-10 Thread tip-bot for Alexander Shishkin
Commit-ID: b8347c2196492f4e1cccde3d92fda1cc2cc7de7e Gitweb: https://git.kernel.org/tip/b8347c2196492f4e1cccde3d92fda1cc2cc7de7e Author: Alexander Shishkin AuthorDate: Mon, 24 Jul 2017 13:04:28 +0300 Committer: Ingo Molnar CommitDate: Fri, 10 Nov 2017 08:04:19 +0100 x86/debug: Handle

[tip:x86/urgent] x86/debug: Handle warnings before the notifier chain, to fix KGDB crash

2017-11-09 Thread tip-bot for Alexander Shishkin
Commit-ID: a8d6c1bd62ffefb075c9d3570f07659e2a36ecb3 Gitweb: https://git.kernel.org/tip/a8d6c1bd62ffefb075c9d3570f07659e2a36ecb3 Author: Alexander Shishkin <alexander.shish...@linux.intel.com> AuthorDate: Mon, 24 Jul 2017 13:04:28 +0300 Committer: Ingo Molnar <mi...@kernel.org>

[tip:x86/urgent] x86/debug: Handle warnings before the notifier chain, to fix KGDB crash

2017-11-09 Thread tip-bot for Alexander Shishkin
Commit-ID: a8d6c1bd62ffefb075c9d3570f07659e2a36ecb3 Gitweb: https://git.kernel.org/tip/a8d6c1bd62ffefb075c9d3570f07659e2a36ecb3 Author: Alexander Shishkin AuthorDate: Mon, 24 Jul 2017 13:04:28 +0300 Committer: Ingo Molnar CommitDate: Fri, 10 Nov 2017 07:16:23 +0100 x86/debug: Handle

[PATCH 3/4] perf tools: Add 'suppress_aux' attribute bit definition and fallback

2017-11-09 Thread Alexander Shishkin
This adds support for suppress_aux, the switch that enables suppressing not-so-useful PERF_RECORD_AUX records. Also handle kernels where it's not supported. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- tools/perf/util/evsel.c | 9 + 1 file chan

[PATCH 3/4] perf tools: Add 'suppress_aux' attribute bit definition and fallback

2017-11-09 Thread Alexander Shishkin
This adds support for suppress_aux, the switch that enables suppressing not-so-useful PERF_RECORD_AUX records. Also handle kernels where it's not supported. Signed-off-by: Alexander Shishkin --- tools/perf/util/evsel.c | 9 + 1 file changed, 9 insertions(+) diff --git a/tools/perf/util

[PATCH 4/4] perf intel-pt, intel-bts: Suppress useless AUX records by default

2017-11-09 Thread Alexander Shishkin
This makes use of the shiny new attr::suppress_aux that suppresses the AUX records that don't carry any 'interesting' information for the decoders, that is PERF_RECORD_AUX[flag==OVERWRITE], which just stack up in the DATA buffer for no good reason. Signed-off-by: Alexander Shishkin

[PATCH 4/4] perf intel-pt, intel-bts: Suppress useless AUX records by default

2017-11-09 Thread Alexander Shishkin
This makes use of the shiny new attr::suppress_aux that suppresses the AUX records that don't carry any 'interesting' information for the decoders, that is PERF_RECORD_AUX[flag==OVERWRITE], which just stack up in the DATA buffer for no good reason. Signed-off-by: Alexander Shishkin --- tools

[PATCH 2/4] tools, perf_event.h: Synchronize

2017-11-09 Thread Alexander Shishkin
I've just added attr::suppress_aux bit to the kernel header, adding it to the tools' header too. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- tools/include/uapi/linux/perf_event.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/i

[PATCH 2/4] tools, perf_event.h: Synchronize

2017-11-09 Thread Alexander Shishkin
I've just added attr::suppress_aux bit to the kernel header, adding it to the tools' header too. Signed-off-by: Alexander Shishkin --- tools/include/uapi/linux/perf_event.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/include/uapi/linux/perf_event.h b/tools

[PATCH 1/4] perf: Allow suppressing AUX records

2017-11-09 Thread Alexander Shishkin
an attribute bit that enables suppressing such records. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> Cc: Markus Metzger <markus.t.metz...@intel.com> Cc: Adrian Hunter <adrian.hun...@intel.com> --- include/uapi/linux/perf_event.h | 3 ++- kernel/events/core

[PATCH 1/4] perf: Allow suppressing AUX records

2017-11-09 Thread Alexander Shishkin
an attribute bit that enables suppressing such records. Signed-off-by: Alexander Shishkin Cc: Markus Metzger Cc: Adrian Hunter --- include/uapi/linux/perf_event.h | 3 ++- kernel/events/core.c| 5 + kernel/events/ring_buffer.c | 12 ++-- 3 files changed, 17 insertions(+), 3

[PATCH 0/4] perf: Allow suppressing useless AUX records

2017-11-09 Thread Alexander Shishkin
. Alexander Shishkin (4): perf: Allow suppressing AUX records tools, perf_event.h: Synchronize perf tools: Add 'suppress_aux' attribute bit definition and fallback perf intel-pt, intel-bts: Suppress useless AUX records by default include/uapi/linux/perf_event.h | 3 ++- kernel

[PATCH 0/4] perf: Allow suppressing useless AUX records

2017-11-09 Thread Alexander Shishkin
. Alexander Shishkin (4): perf: Allow suppressing AUX records tools, perf_event.h: Synchronize perf tools: Add 'suppress_aux' attribute bit definition and fallback perf intel-pt, intel-bts: Suppress useless AUX records by default include/uapi/linux/perf_event.h | 3 ++- kernel

[PATCH v1] PCI: Fixup the RTIT_BAR of Intel TH on Denverton

2017-10-26 Thread Alexander Shishkin
hci_hcd :00:15.0: xHC not responding in xhci_irq, assume controller is > dead > xhci_hcd :00:15.0: HC died; cleaning up > usb 1-1: USB disconnect, device number 2 ... For this reason, we need to resize the RTIT_BAR on Denverton to its actual size, which in this case is 4MB. Sig

[PATCH v1] PCI: Fixup the RTIT_BAR of Intel TH on Denverton

2017-10-26 Thread Alexander Shishkin
hci_hcd :00:15.0: xHC not responding in xhci_irq, assume controller is > dead > xhci_hcd :00:15.0: HC died; cleaning up > usb 1-1: USB disconnect, device number 2 ... For this reason, we need to resize the RTIT_BAR on Denverton to its actual size, which in this case is 4MB. Sig

[tip:perf/urgent] perf/x86/intel/bts: Fix exclusive event reference leak

2017-10-24 Thread tip-bot for Alexander Shishkin
Commit-ID: 2eece390bf68ec8f733d7e4a3ba8a5ea350082ae Gitweb: https://git.kernel.org/tip/2eece390bf68ec8f733d7e4a3ba8a5ea350082ae Author: Alexander Shishkin <alexander.shish...@linux.intel.com> AuthorDate: Mon, 23 Oct 2017 15:35:33 +0300 Committer: Ingo Molnar <mi...@kernel.org>

[tip:perf/urgent] perf/x86/intel/bts: Fix exclusive event reference leak

2017-10-24 Thread tip-bot for Alexander Shishkin
Commit-ID: 2eece390bf68ec8f733d7e4a3ba8a5ea350082ae Gitweb: https://git.kernel.org/tip/2eece390bf68ec8f733d7e4a3ba8a5ea350082ae Author: Alexander Shishkin AuthorDate: Mon, 23 Oct 2017 15:35:33 +0300 Committer: Ingo Molnar CommitDate: Tue, 24 Oct 2017 13:19:27 +0200 perf/x86/intel/bts

[PATCH] perf/x86/intel/bts: Fix exclusive event reference leak

2017-10-23 Thread Alexander Shishkin
ges. After this, nobody in the system gets to use PT or LBR afterwards. This patch moves the privilege check to where it should have been in the first place. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> Fixes: d2878d642a4ed ("perf/x86/intel/bts: Disallow use

[PATCH] perf/x86/intel/bts: Fix exclusive event reference leak

2017-10-23 Thread Alexander Shishkin
ges. After this, nobody in the system gets to use PT or LBR afterwards. This patch moves the privilege check to where it should have been in the first place. Signed-off-by: Alexander Shishkin Fixes: d2878d642a4ed ("perf/x86/intel/bts: Disallow use by unprivileged users on paranoid systems&

Re: [PATCH] stm class: make config_item_type const

2017-10-16 Thread Alexander Shishkin
Bhumika Goyal writes: > On Thu, Oct 12, 2017 at 2:12 PM, Bhumika Goyal wrote: >> This is a followup patch for: >> https://patchwork.kernel.org/patch/649/ and >> https://lkml.org/lkml/2017/10/11/375 >> >> Make config_item_type structures const as they

Re: [PATCH] stm class: make config_item_type const

2017-10-16 Thread Alexander Shishkin
Bhumika Goyal writes: > On Thu, Oct 12, 2017 at 2:12 PM, Bhumika Goyal wrote: >> This is a followup patch for: >> https://patchwork.kernel.org/patch/649/ and >> https://lkml.org/lkml/2017/10/11/375 >> >> Make config_item_type structures const as they are either passed to a >> function

Re: [RFC PATCH 09/17] perf: Use shmemfs pages for userspace-only per-thread detached events

2017-10-06 Thread Alexander Shishkin
Peter Zijlstra <pet...@infradead.org> writes: > On Tue, Sep 05, 2017 at 04:30:18PM +0300, Alexander Shishkin wrote: >> In order to work around the problem of using up mlocked memory for the >> detached events, we can pin the ring buffer pages only while they are >>

Re: [RFC PATCH 09/17] perf: Use shmemfs pages for userspace-only per-thread detached events

2017-10-06 Thread Alexander Shishkin
Peter Zijlstra writes: > On Tue, Sep 05, 2017 at 04:30:18PM +0300, Alexander Shishkin wrote: >> In order to work around the problem of using up mlocked memory for the >> detached events, we can pin the ring buffer pages only while they are >> in use (that is, the event is AC

Re: [RFC PATCH 08/17] perf: Allow inheritance for detached events

2017-10-06 Thread Alexander Shishkin
Peter Zijlstra <pet...@infradead.org> writes: > On Tue, Sep 05, 2017 at 04:30:17PM +0300, Alexander Shishkin wrote: >> This enables inheritance for detached events. Unlike traditional events, >> these do not have parents: inheritance produces a new independent event >&

Re: [RFC PATCH 08/17] perf: Allow inheritance for detached events

2017-10-06 Thread Alexander Shishkin
Peter Zijlstra writes: > On Tue, Sep 05, 2017 at 04:30:17PM +0300, Alexander Shishkin wrote: >> This enables inheritance for detached events. Unlike traditional events, >> these do not have parents: inheritance produces a new independent event >> with the same attribute.

Re: [RFC PATCH 05/17] perf: Introduce detached events

2017-10-06 Thread Alexander Shishkin
Peter Zijlstra writes: > So I'm not opposed to the idea of creating events that live independent > from of file descriptors. And stuffing them in a filesystem makes sense. > However I'm not entire convinced on the details. > > The above has a number of problems: > > -

Re: [RFC PATCH 05/17] perf: Introduce detached events

2017-10-06 Thread Alexander Shishkin
Peter Zijlstra writes: > So I'm not opposed to the idea of creating events that live independent > from of file descriptors. And stuffing them in a filesystem makes sense. > However I'm not entire convinced on the details. > > The above has a number of problems: > > - there's a filesystem race;

[tip:perf/core] perf/core: Explain perf_sched_mutex

2017-09-29 Thread tip-bot for Alexander Shishkin
Commit-ID: 5bce9db1894c998c5b85a34036d679ea6517668f Gitweb: https://git.kernel.org/tip/5bce9db1894c998c5b85a34036d679ea6517668f Author: Alexander Shishkin <alexander.shish...@linux.intel.com> AuthorDate: Tue, 29 Aug 2017 17:01:03 +0300 Committer: Ingo Molnar <mi...@kernel.org>

[tip:perf/core] perf/core: Explain perf_sched_mutex

2017-09-29 Thread tip-bot for Alexander Shishkin
Commit-ID: 5bce9db1894c998c5b85a34036d679ea6517668f Gitweb: https://git.kernel.org/tip/5bce9db1894c998c5b85a34036d679ea6517668f Author: Alexander Shishkin AuthorDate: Tue, 29 Aug 2017 17:01:03 +0300 Committer: Ingo Molnar CommitDate: Fri, 29 Sep 2017 13:28:30 +0200 perf/core: Explain

[tip:perf/urgent] perf/aux: Only update ->aux_wakeup in non-overwrite mode

2017-09-29 Thread tip-bot for Alexander Shishkin
Commit-ID: 441430eb54a00586f95f1aefc48e0801bbd6a923 Gitweb: https://git.kernel.org/tip/441430eb54a00586f95f1aefc48e0801bbd6a923 Author: Alexander Shishkin <alexander.shish...@linux.intel.com> AuthorDate: Wed, 6 Sep 2017 19:08:11 +0300 Committer: Ingo Molnar <mi...@kernel.org>

[tip:perf/urgent] perf/aux: Only update ->aux_wakeup in non-overwrite mode

2017-09-29 Thread tip-bot for Alexander Shishkin
Commit-ID: 441430eb54a00586f95f1aefc48e0801bbd6a923 Gitweb: https://git.kernel.org/tip/441430eb54a00586f95f1aefc48e0801bbd6a923 Author: Alexander Shishkin AuthorDate: Wed, 6 Sep 2017 19:08:11 +0300 Committer: Ingo Molnar CommitDate: Fri, 29 Sep 2017 10:06:45 +0200 perf/aux: Only

Re: [GIT PULL 1/3] stm class: Fix a use-after-free

2017-09-22 Thread Alexander Shishkin
Greg KH <g...@kroah.com> writes: > On Tue, Sep 19, 2017 at 06:47:40PM +0300, Alexander Shishkin wrote: >> For reasons unknown, the stm_source removal path uses device_destroy() >> to kill the underlying device object. Because device_destroy() uses >> devt to loo

Re: [GIT PULL 1/3] stm class: Fix a use-after-free

2017-09-22 Thread Alexander Shishkin
Greg KH writes: > On Tue, Sep 19, 2017 at 06:47:40PM +0300, Alexander Shishkin wrote: >> For reasons unknown, the stm_source removal path uses device_destroy() >> to kill the underlying device object. Because device_destroy() uses >> devt to look for the device to

Re: [PATCH] PCI: Fixup the RTIT_BAR of Intel TH on Denverton

2017-09-19 Thread Alexander Shishkin
Bjorn Helgaas <helg...@kernel.org> writes: > Hi Alexander, Hi Bjorn, > On Tue, Sep 19, 2017 at 05:51:36PM +0300, Alexander Shishkin wrote: >> On some intergrations of the Intel TH the reported size of RTIT_BAR > > s/intergrations/integrations/ Oops. > What is &quo

Re: [PATCH] PCI: Fixup the RTIT_BAR of Intel TH on Denverton

2017-09-19 Thread Alexander Shishkin
Bjorn Helgaas writes: > Hi Alexander, Hi Bjorn, > On Tue, Sep 19, 2017 at 05:51:36PM +0300, Alexander Shishkin wrote: >> On some intergrations of the Intel TH the reported size of RTIT_BAR > > s/intergrations/integrations/ Oops. > What is "TH"? If there'

[GIT PULL 3/3] intel_th: pci: Add Lewisburg PCH support

2017-09-19 Thread Alexander Shishkin
This adds Intel(R) Trace Hub PCI ID for Lewisburg PCH. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> Cc: sta...@vger.kernel.org --- drivers/hwtracing/intel_th/pci.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/hwtracing/intel_th/pci.c b/d

[GIT PULL 3/3] intel_th: pci: Add Lewisburg PCH support

2017-09-19 Thread Alexander Shishkin
This adds Intel(R) Trace Hub PCI ID for Lewisburg PCH. Signed-off-by: Alexander Shishkin Cc: sta...@vger.kernel.org --- drivers/hwtracing/intel_th/pci.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/hwtracing/intel_th/pci.c b/drivers/hwtracing/intel_th/pci.c index 00ee60d978

[GIT PULL 2/3] intel_th: pci: Add Cedar Fork PCH support

2017-09-19 Thread Alexander Shishkin
This adds Intel(R) Trace Hub PCI ID for Cedar Fork PCH. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> Cc: sta...@vger.kernel.org --- drivers/hwtracing/intel_th/pci.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/hwtracing/intel_th/pci.c b/d

[GIT PULL 2/3] intel_th: pci: Add Cedar Fork PCH support

2017-09-19 Thread Alexander Shishkin
This adds Intel(R) Trace Hub PCI ID for Cedar Fork PCH. Signed-off-by: Alexander Shishkin Cc: sta...@vger.kernel.org --- drivers/hwtracing/intel_th/pci.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/hwtracing/intel_th/pci.c b/drivers/hwtracing/intel_th/pci.c index bc9cebc305

[GIT PULL 1/3] stm class: Fix a use-after-free

2017-09-19 Thread Alexander Shishkin
in the class, which may well be the wrong one. That is, loading stm_console and stm_heartbeat and then removing both will die in dereferencing a freed object. Since this should have been device_unregister() in the first place, use it instead of device_destroy(). Signed-off-by: Alexander Shishkin

[GIT PULL 1/3] stm class: Fix a use-after-free

2017-09-19 Thread Alexander Shishkin
in the class, which may well be the wrong one. That is, loading stm_console and stm_heartbeat and then removing both will die in dereferencing a freed object. Since this should have been device_unregister() in the first place, use it instead of device_destroy(). Signed-off-by: Alexander Shishkin Fixes

[GIT PULL 0/3] intel_th/stm class: Fixes for v4.14

2017-09-19 Thread Alexander Shishkin
) intel_th/stm class: Fixes for v4.14 These are: * 2 new PCI IDs * fix for stm_source device removal path Alexander Shishkin (3): stm class: Fix a use-after-free intel_th: pci: Add Cedar Fork PCH

[GIT PULL 0/3] intel_th/stm class: Fixes for v4.14

2017-09-19 Thread Alexander Shishkin
) intel_th/stm class: Fixes for v4.14 These are: * 2 new PCI IDs * fix for stm_source device removal path Alexander Shishkin (3): stm class: Fix a use-after-free intel_th: pci: Add Cedar Fork PCH

[PATCH] PCI: Fixup the RTIT_BAR of Intel TH on Denverton

2017-09-19 Thread Alexander Shishkin
On some intergrations of the Intel TH the reported size of RTIT_BAR doesn't match its actual size, which leads to overlaps with other devices' resources. For this reason, we need to resize the RTIT_BAR on Denverton where it would overlap with XHCI MMIO space. Signed-off-by: Alexander Shishkin

[PATCH] PCI: Fixup the RTIT_BAR of Intel TH on Denverton

2017-09-19 Thread Alexander Shishkin
On some intergrations of the Intel TH the reported size of RTIT_BAR doesn't match its actual size, which leads to overlaps with other devices' resources. For this reason, we need to resize the RTIT_BAR on Denverton where it would overlap with XHCI MMIO space. Signed-off-by: Alexander Shishkin

Re: [RFC PATCH 00/17] perf: Detached events

2017-09-13 Thread Alexander Shishkin
Borislav Petkov <b...@alien8.de> writes: > On Tue, Sep 05, 2017 at 04:30:09PM +0300, Alexander Shishkin wrote: >> Detached events: a new flag to the perf syscall makes a 'detached' event, >> which exists after its file descriptor is released. Not all detached events >>

Re: [RFC PATCH 00/17] perf: Detached events

2017-09-13 Thread Alexander Shishkin
Borislav Petkov writes: > On Tue, Sep 05, 2017 at 04:30:09PM +0300, Alexander Shishkin wrote: >> Detached events: a new flag to the perf syscall makes a 'detached' event, >> which exists after its file descriptor is released. Not all detached events >> are per-thread A

Re: [RFC PATCH 01/17] perf: Allow mmapping only user page

2017-09-13 Thread Alexander Shishkin
Borislav Petkov <b...@alien8.de> writes: > On Tue, Sep 05, 2017 at 04:30:10PM +0300, Alexander Shishkin wrote: >> The 'user page' contains offsets and sizes of data and aux areas of the >> ring buffer. If a user wants to mmap a pre-existing buffer, they need to >> k

Re: [RFC PATCH 01/17] perf: Allow mmapping only user page

2017-09-13 Thread Alexander Shishkin
Borislav Petkov writes: > On Tue, Sep 05, 2017 at 04:30:10PM +0300, Alexander Shishkin wrote: >> The 'user page' contains offsets and sizes of data and aux areas of the >> ring buffer. If a user wants to mmap a pre-existing buffer, they need to >> know these in o

Re: [PATCH] perf/aux: Only update aux_wakeup in non-overwrite mode

2017-09-13 Thread Alexander Shishkin
Will Deacon <will.dea...@arm.com> writes: > Hi Alexander, > > On Wed, Sep 06, 2017 at 07:08:11PM +0300, Alexander Shishkin wrote: >> Commit d9a50b0256 ("perf/aux: Ensure aux_wakeup represents most recent >> wakeup index") changed aux wakeup position calcul

Re: [PATCH] perf/aux: Only update aux_wakeup in non-overwrite mode

2017-09-13 Thread Alexander Shishkin
Will Deacon writes: > Hi Alexander, > > On Wed, Sep 06, 2017 at 07:08:11PM +0300, Alexander Shishkin wrote: >> Commit d9a50b0256 ("perf/aux: Ensure aux_wakeup represents most recent >> wakeup index") changed aux wakeup position calculation to rounddown(), >

[PATCH] perf/aux: Only update aux_wakeup in non-overwrite mode

2017-09-06 Thread Alexander Shishkin
we can simply forego the whole wakeup updates. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- kernel/events/ring_buffer.c | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/kernel/events/ring_buffer.c b/kernel/events/ring

[PATCH] perf/aux: Only update aux_wakeup in non-overwrite mode

2017-09-06 Thread Alexander Shishkin
we can simply forego the whole wakeup updates. Signed-off-by: Alexander Shishkin --- kernel/events/ring_buffer.c | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/kernel/events/ring_buffer.c b/kernel/events/ring_buffer.c index f4ebe42879..4dae1da4c1 100644

[RFC PATCH 17/17] perf/x86/intel/bts: Add PMU info

2017-09-05 Thread Alexander Shishkin
Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- arch/x86/events/intel/bts.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/arch/x86/events/intel/bts.c b/arch/x86/events/intel/bts.c index 16076eb346..ce1dac7115 100644 ---

[RFC PATCH 17/17] perf/x86/intel/bts: Add PMU info

2017-09-05 Thread Alexander Shishkin
Signed-off-by: Alexander Shishkin --- arch/x86/events/intel/bts.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/arch/x86/events/intel/bts.c b/arch/x86/events/intel/bts.c index 16076eb346..ce1dac7115 100644 --- a/arch/x86/events/intel/bts.c +++ b/arch

[RFC PATCH 15/17] perf: Allow controlled non-root access to detached events

2017-09-05 Thread Alexander Shishkin
The user who created the event should also be able to open its corresponding file in tracefs and/or remove it. Signed-off-by: Alexander Shishkin <alexander.shish...@linux.intel.com> --- kernel/events/core.c | 25 ++--- 1 file changed, 22 insertions(+), 3 deletions(-)

<    1   2   3   4   5   6   7   8   9   10   >