[PATCH 21/27] perf intel-pt: Enable decoder to handle TIP.PGD with missing IP

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter When address filters are used, the decoder must detect the end of a filter region (or a branch into a tracestop region) by matching Packet Generation Disabled (TIP.PGD) packets against the object code using the IP given in the packet. However, due to errata SKL014 "Intel PT

[PATCH 25/27] perf probe: Fix to cut off incompatible chars from group name

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Masami Hiramatsu Cut off the characters which can not use for group name of uprobes when making it based on executable filename. For example, if the exec name is libstdc++.so, without this fix perf probe generates "probe_libstdc++" as the group name, but it is failed to set because '+'

[PATCH 22/27] perf intel-pt: Fix decoding when there are address filters

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Due to errata SKL014 "Intel PT TIP.PGD May Not Have Target IP Payload", the Intel PT decoder needs to match address filters against TIP.PGD packets. Parse the address filters and implement the decoder's 'pgd_ip()' callback to match the IP against the

[PATCH 22/27] perf intel-pt: Fix decoding when there are address filters

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Due to errata SKL014 "Intel PT TIP.PGD May Not Have Target IP Payload", the Intel PT decoder needs to match address filters against TIP.PGD packets. Parse the address filters and implement the decoder's 'pgd_ip()' callback to match the IP against the filter regions.

[PATCH 13/27] perf record: Add support for using symbols in address filters

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Symbols come from either the DSO or /proc/kallsyms for the kernel. Details of the functionality can be found in Documentation/perf-record.txt. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami

[PATCH 19/27] perf intel-pt: Record address filter in AUXTRACE_INFO event

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter The address filter is needed to help decode the trace, so store it in the AUXTRACE_INFO event. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier

[PATCH 18/27] perf intel-pt: Add a helper function for processing AUXTRACE_INFO

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Add a helper function 'intel_pt_has()' to make it easier to determine which members the AUXTRACE_INFO event contains. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu

[PATCH 12/27] perf symbols: Add dso__last_symbol()

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Add a function to find the last symbol in a DSO. This will be used when parsing address filters to calculate a region that includes the entire DSO. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami

[PATCH 15/27] perf intel-pt: Fix snapshot overlap detection decoder errors

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Fix occasional decoder errors decoding trace data collected in snapshot mode. Snapshot mode can take successive snapshots of trace which might overlap. The decoder checks whether there is an overlap but only looks at the current and previous buffer.

[PATCH 16/27] perf intel-pt: Add support for recording the max non-turbo ratio

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Previously the maximum non-turbo ratio was calculated from TSC assuming a 100 MHz multiplier which is correct for current hardware supporting Intel PT. However more recent kernels also now export the value, so use that in preference to the calculated

[PATCH 19/27] perf intel-pt: Record address filter in AUXTRACE_INFO event

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter The address filter is needed to help decode the trace, so store it in the AUXTRACE_INFO event. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link: http://lkml.kernel.org/r/1474641528-18776-14-git-send-email-adrian.hun...@intel.com

[PATCH 18/27] perf intel-pt: Add a helper function for processing AUXTRACE_INFO

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Add a helper function 'intel_pt_has()' to make it easier to determine which members the AUXTRACE_INFO event contains. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link:

[PATCH 12/27] perf symbols: Add dso__last_symbol()

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Add a function to find the last symbol in a DSO. This will be used when parsing address filters to calculate a region that includes the entire DSO. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link:

[PATCH 15/27] perf intel-pt: Fix snapshot overlap detection decoder errors

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Fix occasional decoder errors decoding trace data collected in snapshot mode. Snapshot mode can take successive snapshots of trace which might overlap. The decoder checks whether there is an overlap but only looks at the current and previous buffer. However buffers that do

[PATCH 16/27] perf intel-pt: Add support for recording the max non-turbo ratio

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Previously the maximum non-turbo ratio was calculated from TSC assuming a 100 MHz multiplier which is correct for current hardware supporting Intel PT. However more recent kernels also now export the value, so use that in preference to the calculated value. Signed-off-by:

[PATCH 13/27] perf record: Add support for using symbols in address filters

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Symbols come from either the DSO or /proc/kallsyms for the kernel. Details of the functionality can be found in Documentation/perf-record.txt. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link:

[PATCH 11/27] perf record: Fix error paths

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Some error paths do not tidy-up. Fix that. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link:

[PATCH 08/27] perf evsel: Add support for address filters

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Mathieu Poirier This patch makes it possible to use the current filter framework with address filters. That way address filters for HW tracers such as CoreSight and Intel PT can be communicated to the kernel drivers. Signed-off-by: Mathieu Poirier

[PATCH 26/27] perf probe: Match linkage name with mangled name

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Masami Hiramatsu Match linkage name with mangled name if exists. The linkage_name is used for storing mangled name of the object. Thus, this allows 'perf probe' to find appropriate probe point from mangled symbol as below. E.g. without this fix: $ perf probe

[PATCH 07/27] perf evsel: New tracepoint specific function

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Mathieu Poirier Making function perf_evsel__append_filter() static and introducing a new tracepoint specific function to append filters. That way we eliminate redundant code and avoid formatting mistake. Signed-off-by: Mathieu Poirier

[PATCH 11/27] perf record: Fix error paths

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Some error paths do not tidy-up. Fix that. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link: http://lkml.kernel.org/r/1474641528-18776-6-git-send-email-adrian.hun...@intel.com Signed-off-by: Arnaldo Carvalho de Melo ---

[PATCH 08/27] perf evsel: Add support for address filters

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Mathieu Poirier This patch makes it possible to use the current filter framework with address filters. That way address filters for HW tracers such as CoreSight and Intel PT can be communicated to the kernel drivers. Signed-off-by: Mathieu Poirier Acked-by: Adrian Hunter Cc: Alexander

[PATCH 26/27] perf probe: Match linkage name with mangled name

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Masami Hiramatsu Match linkage name with mangled name if exists. The linkage_name is used for storing mangled name of the object. Thus, this allows 'perf probe' to find appropriate probe point from mangled symbol as below. E.g. without this fix: $ perf probe -x

[PATCH 07/27] perf evsel: New tracepoint specific function

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Mathieu Poirier Making function perf_evsel__append_filter() static and introducing a new tracepoint specific function to append filters. That way we eliminate redundant code and avoid formatting mistake. Signed-off-by: Mathieu Poirier Acked-by: Adrian Hunter Cc: Alexander Shishkin Cc:

[PATCH 03/27] perf data: Fix building in 32 bit platform with libbabeltrace

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Wang Nan On ARM32 building it report following error when we build with libbabeltrace: util/data-convert-bt.c: In function 'add_bpf_output_values': util/data-convert-bt.c:440:3: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has

[PATCH 04/27] perf trace: Beautify sched_[gs]et_attr return value

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Both return errno, show the string associated then. More work needed to capture the sched_attr arg to beautify it in turn, probably using BPF. Before: 0.210 ( 0.001 ms): sched_setattr(uattr: 0x7ffc684f02b0) = -22 After the patch, for this

[PATCH 05/27] perf tools: Update documentation info about quipper

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Simon Que The existing link is outdated. The most recent quipper code can be found at the new URL. Committer notes: Quipper is a C++ parser that can be used to convert from a perf.data file to and from a protobuf, a Chromium OS facility. Signed-off-by: Simon Que

[PATCH 03/27] perf data: Fix building in 32 bit platform with libbabeltrace

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Wang Nan On ARM32 building it report following error when we build with libbabeltrace: util/data-convert-bt.c: In function 'add_bpf_output_values': util/data-convert-bt.c:440:3: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'unsigned int'

[PATCH 04/27] perf trace: Beautify sched_[gs]et_attr return value

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo Both return errno, show the string associated then. More work needed to capture the sched_attr arg to beautify it in turn, probably using BPF. Before: 0.210 ( 0.001 ms): sched_setattr(uattr: 0x7ffc684f02b0) = -22 After the patch, for this sched_attr, all

[PATCH 05/27] perf tools: Update documentation info about quipper

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Simon Que The existing link is outdated. The most recent quipper code can be found at the new URL. Committer notes: Quipper is a C++ parser that can be used to convert from a perf.data file to and from a protobuf, a Chromium OS facility. Signed-off-by: Simon Que Acked-by: Andi Kleen

[PATCH 09/27] perf script: Fix vanished idle symbols

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Commit 608c34de0b3d ("perf symbols: Mark if a symbol is idle in the library") causes idle symbols to vanish from perf script output. That is because print functions suppress symbols marked as 'idle'. However, suppression of 'idle' functions is only

[PATCH 10/27] perf record: Rename label 'out_symbol_exit'

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter In preparation for fixing the error paths, rename label 'out_symbol_exit' to be 'out' because that error path can be used irrespective of whether symbols (or anything else) has been initialized. Signed-off-by: Adrian Hunter

[PATCH 02/27] perf tools: Fix MMAP event synthesis broken by MAP_HUGETLB change

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Patch "perf record: Mark MAP_HUGETLB when synthesizing mmap events") breaks MMAP event synthesis. The executable name comparison will match any name if the length is zero, resulting in all the user space maps becoming anonymous. This is particularly

[PATCH 06/27] perf tools: Make perf_evsel__append_filter() generic

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Mathieu Poirier By making function perf_evsel__append_filter() take a format rather than an operator it is possible to reuse the code for other purposes (ex. Intel PT and CoreSight) than tracepoints. Signed-off-by: Mathieu Poirier

[PATCH 09/27] perf script: Fix vanished idle symbols

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Commit 608c34de0b3d ("perf symbols: Mark if a symbol is idle in the library") causes idle symbols to vanish from perf script output. That is because print functions suppress symbols marked as 'idle'. However, suppression of 'idle' functions is only used by 'perf top' and

[PATCH 10/27] perf record: Rename label 'out_symbol_exit'

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter In preparation for fixing the error paths, rename label 'out_symbol_exit' to be 'out' because that error path can be used irrespective of whether symbols (or anything else) has been initialized. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu

[PATCH 02/27] perf tools: Fix MMAP event synthesis broken by MAP_HUGETLB change

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Patch "perf record: Mark MAP_HUGETLB when synthesizing mmap events") breaks MMAP event synthesis. The executable name comparison will match any name if the length is zero, resulting in all the user space maps becoming anonymous. This is particularly noticeable with

[PATCH 06/27] perf tools: Make perf_evsel__append_filter() generic

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Mathieu Poirier By making function perf_evsel__append_filter() take a format rather than an operator it is possible to reuse the code for other purposes (ex. Intel PT and CoreSight) than tracepoints. Signed-off-by: Mathieu Poirier Acked-by: Adrian Hunter Cc: Alexander Shishkin Cc: Jiri

[GIT PULL 00/27] perf/core improvements and fixes

2016-09-29 Thread Arnaldo Carvalho de Melo
/acme/linux into perf/core (2016-09-23 07:21:38 +0200) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo-20160929 for you to fetch changes up to d18019a53a07e009899ff6b8dc5ec30f249360d9: perf tests: Add dwarf unwind

[GIT PULL 00/27] perf/core improvements and fixes

2016-09-29 Thread Arnaldo Carvalho de Melo
/acme/linux into perf/core (2016-09-23 07:21:38 +0200) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo-20160929 for you to fetch changes up to d18019a53a07e009899ff6b8dc5ec30f249360d9: perf tests: Add dwarf unwind

[PATCH 27/27] perf tests: Add dwarf unwind test for powerpc

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Ravi Bangoria The user stack dump feature was recently added for powerpc. But there was no test case available to test it. This test works same as on other architectures by preparing a stack frame on the perf test thread and comparing each frame by

[PATCH 27/27] perf tests: Add dwarf unwind test for powerpc

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Ravi Bangoria The user stack dump feature was recently added for powerpc. But there was no test case available to test it. This test works same as on other architectures by preparing a stack frame on the perf test thread and comparing each frame by unwinding it. $ ./perf test 50

[PATCH 01/27] perf record: Fix documentation 'event_sources' -> 'event_source'

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Change '/sys/bus/event_sources' to the correct path which is '/sys/bus/event_source'. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier

[PATCH 01/27] perf record: Fix documentation 'event_sources' -> 'event_source'

2016-09-29 Thread Arnaldo Carvalho de Melo
From: Adrian Hunter Change '/sys/bus/event_sources' to the correct path which is '/sys/bus/event_source'. Signed-off-by: Adrian Hunter Cc: Jiri Olsa Cc: Masami Hiramatsu Cc: Mathieu Poirier Link: http://lkml.kernel.org/r/1474641528-18776-2-git-send-email-adrian.hun...@intel.com

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Sinan Kaya
On 9/29/2016 10:18 AM, Wim Osterholt wrote: > On Wed, Sep 28, 2016 at 07:38:41PM -0400, Sinan Kaya wrote: >> >> Can you try these patches on your machines please? > > I applied the included patches on vanilla 4.8-rc8 and my machine booted > fine. (I saw a remark about SCSI interrupts, but I have

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Sinan Kaya
On 9/29/2016 10:18 AM, Wim Osterholt wrote: > On Wed, Sep 28, 2016 at 07:38:41PM -0400, Sinan Kaya wrote: >> >> Can you try these patches on your machines please? > > I applied the included patches on vanilla 4.8-rc8 and my machine booted > fine. (I saw a remark about SCSI interrupts, but I have

Re: [PATCH 3.12 000/119] 3.12.64-stable review

2016-09-29 Thread Guenter Roeck
On 09/29/2016 03:25 AM, Jiri Slaby wrote: This is the start of the stable review cycle for the 3.12.64 release. There are 119 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by

Re: [PATCH 3.12 000/119] 3.12.64-stable review

2016-09-29 Thread Guenter Roeck
On 09/29/2016 03:25 AM, Jiri Slaby wrote: This is the start of the stable review cycle for the 3.12.64 release. There are 119 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know. Responses should be made by

[PATCH] clk: mvebu: armada-37xx-periph: Fix the clock provider registration

2016-09-29 Thread Gregory CLEMENT
While trying using a peripheral clock on a driver, I saw that the clock pointer returned by the provider was NULL. The problem was a missing indirection. It was the pointer stored in the hws array which needed to be updated not the value it contains. Signed-off-by: Gregory CLEMENT

[PATCH] clk: mvebu: armada-37xx-periph: Fix the clock provider registration

2016-09-29 Thread Gregory CLEMENT
While trying using a peripheral clock on a driver, I saw that the clock pointer returned by the provider was NULL. The problem was a missing indirection. It was the pointer stored in the hws array which needed to be updated not the value it contains. Signed-off-by: Gregory CLEMENT ---

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Sinan Kaya
On 9/29/2016 9:49 AM, Ondrej Zary wrote: >> Ok, since I have not seen the full boot log I am guessing that isa api >> > gets called before the link objects are initialized. > Netconsole did not work (probably because it crashes too early?) and I don't > have a null-modem cable. > Probably, this

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Sinan Kaya
On 9/29/2016 9:49 AM, Ondrej Zary wrote: >> Ok, since I have not seen the full boot log I am guessing that isa api >> > gets called before the link objects are initialized. > Netconsole did not work (probably because it crashes too early?) and I don't > have a null-modem cable. > Probably, this

Re: lspci not showing motherboard ethernet controller after PCIe card firmware update change from 32-bit to 64-bit BAR

2016-09-29 Thread Lennart Sorensen
On Wed, Sep 28, 2016 at 09:05:44AM -0500, Steve Kenton wrote: > I decided to experiment with /sys/bus/pci and the BIOS settings to try > and understand things better. > > The BIOS has a settings to enable/disable the on-board LAN. When the > Blackmagic card firmware is upgraded > bus[03] and

Re: lspci not showing motherboard ethernet controller after PCIe card firmware update change from 32-bit to 64-bit BAR

2016-09-29 Thread Lennart Sorensen
On Wed, Sep 28, 2016 at 09:05:44AM -0500, Steve Kenton wrote: > I decided to experiment with /sys/bus/pci and the BIOS settings to try > and understand things better. > > The BIOS has a settings to enable/disable the on-board LAN. When the > Blackmagic card firmware is upgraded > bus[03] and

Re: Regression in 4.8 - CPU speed set very low

2016-09-29 Thread Lennart Sorensen
On Wed, Sep 28, 2016 at 09:26:42PM -0500, Larry Finger wrote: > By the time it gets slow, the CPU's cool, and one cannot see the temp just > before that event happened. Hmm, I would not expect the CPU to drop from 80 to 40 degrees in a few seconds if the fan is not spinning. I wouldn't even

Re: Regression in 4.8 - CPU speed set very low

2016-09-29 Thread Lennart Sorensen
On Wed, Sep 28, 2016 at 09:26:42PM -0500, Larry Finger wrote: > By the time it gets slow, the CPU's cool, and one cannot see the temp just > before that event happened. Hmm, I would not expect the CPU to drop from 80 to 40 degrees in a few seconds if the fan is not spinning. I wouldn't even

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Wim Osterholt
On Wed, Sep 28, 2016 at 07:38:41PM -0400, Sinan Kaya wrote: > > Can you try these patches on your machines please? I applied the included patches on vanilla 4.8-rc8 and my machine booted fine. (I saw a remark about SCSI interrupts, but I have no SCSI.) Regards, Wim.

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Wim Osterholt
On Wed, Sep 28, 2016 at 07:38:41PM -0400, Sinan Kaya wrote: > > Can you try these patches on your machines please? I applied the included patches on vanilla 4.8-rc8 and my machine booted fine. (I saw a remark about SCSI interrupts, but I have no SCSI.) Regards, Wim.

Re: [PATCH v5 01/14] drivers: iommu: add FWNODE_IOMMU fwnode type

2016-09-29 Thread Lorenzo Pieralisi
Hi Rafael, On Fri, Sep 09, 2016 at 03:23:30PM +0100, Lorenzo Pieralisi wrote: > On systems booting with a device tree, every struct device is > associated with a struct device_node, that represents its DT > representation. The device node can be used in generic kernel > contexts (eg IRQ

Re: [PATCH v5 01/14] drivers: iommu: add FWNODE_IOMMU fwnode type

2016-09-29 Thread Lorenzo Pieralisi
Hi Rafael, On Fri, Sep 09, 2016 at 03:23:30PM +0100, Lorenzo Pieralisi wrote: > On systems booting with a device tree, every struct device is > associated with a struct device_node, that represents its DT > representation. The device node can be used in generic kernel > contexts (eg IRQ

Re: [Nbd] [PATCH][V3] nbd: add multi-connection support

2016-09-29 Thread Josef Bacik
On 09/29/2016 05:52 AM, Wouter Verhelst wrote: Hi Josef, On Wed, Sep 28, 2016 at 04:01:32PM -0400, Josef Bacik wrote: NBD can become contended on its single connection. We have to serialize all writes and we can only process one read response at a time. Fix this by allowing userspace to

Re: [Nbd] [PATCH][V3] nbd: add multi-connection support

2016-09-29 Thread Josef Bacik
On 09/29/2016 05:52 AM, Wouter Verhelst wrote: Hi Josef, On Wed, Sep 28, 2016 at 04:01:32PM -0400, Josef Bacik wrote: NBD can become contended on its single connection. We have to serialize all writes and we can only process one read response at a time. Fix this by allowing userspace to

Re: [PATCH 2/6] ipr: use pci_irq_allocate_vectors

2016-09-29 Thread Brian King
Thanks Christoph. Very nice. As I was reviewing the patch, I noticed the additional PCI_IRQ_AFFINITY flag, which is currently not being set in this patch. Is the intention to set that globally by default, or should I follow up with a one liner to add that to the ipr driver in the next patch set I

Re: [PATCH 2/6] ipr: use pci_irq_allocate_vectors

2016-09-29 Thread Brian King
Thanks Christoph. Very nice. As I was reviewing the patch, I noticed the additional PCI_IRQ_AFFINITY flag, which is currently not being set in this patch. Is the intention to set that globally by default, or should I follow up with a one liner to add that to the ipr driver in the next patch set I

Re: [PATCH v3 2/2] lockdep: Limit static allocations if PROVE_LOCKING_SMALL is defined

2016-09-29 Thread Babu Moger
CCing Dave. Dave, Please ack it if it looks good. Thanks. On 9/27/2016 2:33 PM, Babu Moger wrote: Reduce the size of data structure for lockdep entries by half if PROVE_LOCKING_SMALL if defined. This is used only for sparc. Signed-off-by: Babu Moger ---

Re: [PATCH v3 2/2] lockdep: Limit static allocations if PROVE_LOCKING_SMALL is defined

2016-09-29 Thread Babu Moger
CCing Dave. Dave, Please ack it if it looks good. Thanks. On 9/27/2016 2:33 PM, Babu Moger wrote: Reduce the size of data structure for lockdep entries by half if PROVE_LOCKING_SMALL if defined. This is used only for sparc. Signed-off-by: Babu Moger --- kernel/locking/lockdep_internals.h

Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc

2016-09-29 Thread Babu Moger
On 9/28/2016 3:39 AM, Peter Zijlstra wrote: On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: This new config parameter limits the space used for "Lock debugging: prove locking correctness" by about 4MB. The current sparc systems have the limitation of 32MB size for kernel size

[PATCH cgroup/for-4.6] cgroup: fix error handling regressions in proc_cgroup_show() and cgroup_release_agent()

2016-09-29 Thread Tejun Heo
4c737b41de7f ("cgroup: make cgroup_path() and friends behave in the style of strlcpy()") broke error handling in proc_cgroup_show() and cgroup_release_agent() by not handling negative return values from cgroup_path_ns_locked(). Fix it. Reported-by: Dan Carpenter

Re: [PATCH v3 1/2] config: Adding the new config parameter CONFIG_PROVE_LOCKING_SMALL for sparc

2016-09-29 Thread Babu Moger
On 9/28/2016 3:39 AM, Peter Zijlstra wrote: On Tue, Sep 27, 2016 at 12:33:27PM -0700, Babu Moger wrote: This new config parameter limits the space used for "Lock debugging: prove locking correctness" by about 4MB. The current sparc systems have the limitation of 32MB size for kernel size

[PATCH cgroup/for-4.6] cgroup: fix error handling regressions in proc_cgroup_show() and cgroup_release_agent()

2016-09-29 Thread Tejun Heo
4c737b41de7f ("cgroup: make cgroup_path() and friends behave in the style of strlcpy()") broke error handling in proc_cgroup_show() and cgroup_release_agent() by not handling negative return values from cgroup_path_ns_locked(). Fix it. Reported-by: Dan Carpenter Signed-off-by: Tejun Heo ---

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Ondrej Zary
On Thursday 29 September 2016, ok...@codeaurora.org wrote: > On 2016-09-29 05:10, Ondrej Zary wrote: > > On Thursday 29 September 2016, Sinan Kaya wrote: > >> On 9/28/2016 3:23 PM, Ondrej Zary wrote: > >> > On Wednesday 28 September 2016 20:22:40 Sinan Kaya wrote: > >> >> On 9/28/2016 1:02 PM,

Re: 4.7 regression: ACPI: No IRQ available for PCI Interrupt Link [LNKD]. Try pci=noacpi or acpi=off

2016-09-29 Thread Ondrej Zary
On Thursday 29 September 2016, ok...@codeaurora.org wrote: > On 2016-09-29 05:10, Ondrej Zary wrote: > > On Thursday 29 September 2016, Sinan Kaya wrote: > >> On 9/28/2016 3:23 PM, Ondrej Zary wrote: > >> > On Wednesday 28 September 2016 20:22:40 Sinan Kaya wrote: > >> >> On 9/28/2016 1:02 PM,

Re: Race condition between iget_locked() and evict_inodes()

2016-09-29 Thread Al Viro
On Thu, Sep 29, 2016 at 12:56:49PM +, Anton Altaparmakov wrote: > Hi Al, > > > On 29 Sep 2016, at 13:17, Al Viro wrote: > > > > On Thu, Sep 29, 2016 at 11:53:21AM +, Anton Altaparmakov wrote: > >> Thus if the events happen in this order: > >> > >>

Re: Race condition between iget_locked() and evict_inodes()

2016-09-29 Thread Al Viro
On Thu, Sep 29, 2016 at 12:56:49PM +, Anton Altaparmakov wrote: > Hi Al, > > > On 29 Sep 2016, at 13:17, Al Viro wrote: > > > > On Thu, Sep 29, 2016 at 11:53:21AM +, Anton Altaparmakov wrote: > >> Thus if the events happen in this order: > >> > >> evict_inodes()

Re: [PATCH v3 1/2] mm, proc: Fix region lost in /proc/self/smaps

2016-09-29 Thread Michal Hocko
On Thu 29-09-16 21:14:40, Robert Hu wrote: > On Mon, 2016-09-26 at 10:46 +0200, Michal Hocko wrote: > > On Fri 23-09-16 17:53:51, Oleg Nesterov wrote: > > > On 09/23, Michal Hocko wrote: > > > > > > > > On Fri 23-09-16 15:56:36, Oleg Nesterov wrote: > > > > > > > > > > I think we can simplify

Re: [PATCH v3 1/2] mm, proc: Fix region lost in /proc/self/smaps

2016-09-29 Thread Michal Hocko
On Thu 29-09-16 21:14:40, Robert Hu wrote: > On Mon, 2016-09-26 at 10:46 +0200, Michal Hocko wrote: > > On Fri 23-09-16 17:53:51, Oleg Nesterov wrote: > > > On 09/23, Michal Hocko wrote: > > > > > > > > On Fri 23-09-16 15:56:36, Oleg Nesterov wrote: > > > > > > > > > > I think we can simplify

[PATCH v2 1/4] ACPI / property: Allow holes in reference properties

2016-09-29 Thread Mika Westerberg
DT allows holes or empty phandles for references. This is used for example in SPI subsystem where some chip selects are native and others are regular GPIOs. In ACPI _DSD we currently do not support this but instead the preceding reference consumes all following integer arguments. For example we

[PATCH v2 3/4] ACPI / gpio: Add hogging support

2016-09-29 Thread Mika Westerberg
GPIO hogging means that the GPIO controller can "hog" and configure certain GPIOs without need for a driver or userspace to do that. This is useful in open-connected boards where BIOS cannot possibly know beforehand which devices will be connected to the board. This adds GPIO hogging mechanism to

[PATCH v2 2/4] ACPI / gpio: Add support for naming GPIOs

2016-09-29 Thread Mika Westerberg
DT has property 'gpio-line-names' to name GPIO lines the controller has if present. Use this very same property in ACPI as well to provide nice names for the GPIOS. Signed-off-by: Mika Westerberg --- Documentation/acpi/gpio-properties.txt | 21 +

[PATCH v2 1/4] ACPI / property: Allow holes in reference properties

2016-09-29 Thread Mika Westerberg
DT allows holes or empty phandles for references. This is used for example in SPI subsystem where some chip selects are native and others are regular GPIOs. In ACPI _DSD we currently do not support this but instead the preceding reference consumes all following integer arguments. For example we

[PATCH v2 3/4] ACPI / gpio: Add hogging support

2016-09-29 Thread Mika Westerberg
GPIO hogging means that the GPIO controller can "hog" and configure certain GPIOs without need for a driver or userspace to do that. This is useful in open-connected boards where BIOS cannot possibly know beforehand which devices will be connected to the board. This adds GPIO hogging mechanism to

[PATCH v2 2/4] ACPI / gpio: Add support for naming GPIOs

2016-09-29 Thread Mika Westerberg
DT has property 'gpio-line-names' to name GPIO lines the controller has if present. Use this very same property in ACPI as well to provide nice names for the GPIOS. Signed-off-by: Mika Westerberg --- Documentation/acpi/gpio-properties.txt | 21 + drivers/gpio/gpiolib-acpi.c

[PATCH v2 4/4] ACPI / gpio: Allow holes in list of GPIOs for a device

2016-09-29 Thread Mika Westerberg
Make it possible to have an empty GPIOs in a GPIO list for device. For example a SPI master may use both GPIOs and native pins as chip selects and we need to be able to distinguish between the two. This makes it mandatory to have exactly 3 arguments for GPIOs and then converts gpiolib to use of

[PATCH v2 4/4] ACPI / gpio: Allow holes in list of GPIOs for a device

2016-09-29 Thread Mika Westerberg
Make it possible to have an empty GPIOs in a GPIO list for device. For example a SPI master may use both GPIOs and native pins as chip selects and we need to be able to distinguish between the two. This makes it mandatory to have exactly 3 arguments for GPIOs and then converts gpiolib to use of

[PATCH v2 0/4] ACPI / gpio: Updates to properties

2016-09-29 Thread Mika Westerberg
Hi, This series brings couple of useful GPIO related properties from Device Tree to ACPI _DSD device properties: - Names for GPIO lines - GPIO hogging - Holes in GPIO property lists We are using these to get Intel Galileo better supported in the mainline kernel (but these may be useful

[PATCH v2 0/4] ACPI / gpio: Updates to properties

2016-09-29 Thread Mika Westerberg
Hi, This series brings couple of useful GPIO related properties from Device Tree to ACPI _DSD device properties: - Names for GPIO lines - GPIO hogging - Holes in GPIO property lists We are using these to get Intel Galileo better supported in the mainline kernel (but these may be useful

Re: [PATCH 2/2] drm: Simplify drm_printk to reduce object size quite a bit

2016-09-29 Thread Sean Paul
On Tue, Sep 27, 2016 at 11:54 AM, Sean Paul wrote: > On Mon, Sep 26, 2016 at 4:36 AM, Chris Wilson > wrote: >> On Sun, Sep 25, 2016 at 07:18:34PM -0700, Joe Perches wrote: >>> Remove function name and special " *ERROR*" from argument list >>> >>>

Re: [PATCH 2/2] drm: Simplify drm_printk to reduce object size quite a bit

2016-09-29 Thread Sean Paul
On Tue, Sep 27, 2016 at 11:54 AM, Sean Paul wrote: > On Mon, Sep 26, 2016 at 4:36 AM, Chris Wilson > wrote: >> On Sun, Sep 25, 2016 at 07:18:34PM -0700, Joe Perches wrote: >>> Remove function name and special " *ERROR*" from argument list >>> >>> $ size drivers/gpu/drm/built-in.o* (x86-32

Re: [PATCH] uapi: add missing install of sync_file.h

2016-09-29 Thread Sean Paul
On Tue, Sep 27, 2016 at 12:06 PM, Gustavo Padovan wrote: > 2016-09-27 Emilio López : > >> As part of the sync framework destaging, the sync_file.h header >> was moved, but an entry was not added on Kbuild to install it. >> This patch

Re: [PATCH] uapi: add missing install of sync_file.h

2016-09-29 Thread Sean Paul
On Tue, Sep 27, 2016 at 12:06 PM, Gustavo Padovan wrote: > 2016-09-27 Emilio López : > >> As part of the sync framework destaging, the sync_file.h header >> was moved, but an entry was not added on Kbuild to install it. >> This patch resolves this omission so that "make headers_install" >>

Re: [PATCH v3 3/3] mfd: lpc_ich: Add support for Intel Apollo Lake SoC

2016-09-29 Thread Mika Westerberg
On Thu, Sep 29, 2016 at 01:19:30PM +, Kraemer, Matthias (Ferchau; ADITG/SW1) wrote: > Hello Mika, > > In line 1239 you specify "struct pci_bus *bus = dev->bus;" > }, @@ -1122,6 +1131,36 @@ static int lpc_ich_init_spi(struct pci_dev *dev) > } > break; > > +

Re: [PATCH v3 3/3] mfd: lpc_ich: Add support for Intel Apollo Lake SoC

2016-09-29 Thread Mika Westerberg
On Thu, Sep 29, 2016 at 01:19:30PM +, Kraemer, Matthias (Ferchau; ADITG/SW1) wrote: > Hello Mika, > > In line 1239 you specify "struct pci_bus *bus = dev->bus;" > }, @@ -1122,6 +1131,36 @@ static int lpc_ich_init_spi(struct pci_dev *dev) > } > break; > > +

Re: [tip:x86/platform] x86/platform/mellanox: Introduce support for Mellanox systems platform

2016-09-29 Thread Thomas Gleixner
On Wed, 28 Sep 2016, Darren Hart wrote: > This through me as I was trying to reconcile this series with another mellanox > platform driver from Vadim to the drivers/platform/x86 tree as this one also > came to the platform-driver-x86 mailing list. > > There are no other entries in MAINTAINERS

Re: [tip:x86/platform] x86/platform/mellanox: Introduce support for Mellanox systems platform

2016-09-29 Thread Thomas Gleixner
On Wed, 28 Sep 2016, Darren Hart wrote: > This through me as I was trying to reconcile this series with another mellanox > platform driver from Vadim to the drivers/platform/x86 tree as this one also > came to the platform-driver-x86 mailing list. > > There are no other entries in MAINTAINERS

RE: [PATCH v3 3/3] mfd: lpc_ich: Add support for Intel Apollo Lake SoC

2016-09-29 Thread Kraemer, Matthias (Ferchau; ADITG/SW1)
Hello Mika, In line 1239 you specify "struct pci_bus *bus = dev->bus;" }, @@ -1122,6 +1131,36 @@ static int lpc_ich_init_spi(struct pci_dev *dev) } break; + case INTEL_SPI_BXT: { + unsigned int p2sb = PCI_DEVFN(13, 0); + unsigned

Re: [PATCH cgroup/for-4.9] cpuset: fix error error handling regression in proc_cpuset_show()

2016-09-29 Thread Dan Carpenter
That works but there are two other related warnings: kernel/cgroup.c:5811 proc_cgroup_show() warn: signed compare with PATH_MAX 'retval' kernel/cgroup.c:6099 cgroup_release_agent() warn: signed compare with PATH_MAX 'ret' regards, dan carpenter

RE: [PATCH v3 3/3] mfd: lpc_ich: Add support for Intel Apollo Lake SoC

2016-09-29 Thread Kraemer, Matthias (Ferchau; ADITG/SW1)
Hello Mika, In line 1239 you specify "struct pci_bus *bus = dev->bus;" }, @@ -1122,6 +1131,36 @@ static int lpc_ich_init_spi(struct pci_dev *dev) } break; + case INTEL_SPI_BXT: { + unsigned int p2sb = PCI_DEVFN(13, 0); + unsigned

Re: [PATCH cgroup/for-4.9] cpuset: fix error error handling regression in proc_cpuset_show()

2016-09-29 Thread Dan Carpenter
That works but there are two other related warnings: kernel/cgroup.c:5811 proc_cgroup_show() warn: signed compare with PATH_MAX 'retval' kernel/cgroup.c:6099 cgroup_release_agent() warn: signed compare with PATH_MAX 'ret' regards, dan carpenter

Re: [RFC][PATCH 7/7] printk: new printk() recursion detection

2016-09-29 Thread Petr Mladek
On Tue 2016-09-27 23:22:37, Sergey Senozhatsky wrote: > We can't use alt_printk when we are flushing alt_printk buffers to logbuf, > because otherwise we would just flush messages back to alt_printk buffers. > At the same time, this opens a door for potential deadlock, should flushing > endup in

Re: [RFC][PATCH 7/7] printk: new printk() recursion detection

2016-09-29 Thread Petr Mladek
On Tue 2016-09-27 23:22:37, Sergey Senozhatsky wrote: > We can't use alt_printk when we are flushing alt_printk buffers to logbuf, > because otherwise we would just flush messages back to alt_printk buffers. > At the same time, this opens a door for potential deadlock, should flushing > endup in

<    4   5   6   7   8   9   10   11   12   13   >