Re: [PATCH 1/2 v2] sched: fix find_idlest_group for fork

2016-11-29 Thread Vincent Guittot
On 29 November 2016 at 15:50, Morten Rasmussen wrote: > On Tue, Nov 29, 2016 at 02:04:27PM +0100, Vincent Guittot wrote: >> On 29 November 2016 at 11:57, Morten Rasmussen >> wrote: >> > On Fri, Nov 25, 2016 at 04:34:32PM +0100, Vincent Guittot wrote: >> >> @@ -5708,13 +5708,6 @@ static int selec

[PATCH 09/22 v2] mm/vmstat: Convert to hotplug state machine

2016-11-29 Thread Sebastian Andrzej Siewior
Install the callbacks via the state machine, but do not invoke them as we can initialize the node state without calling the callbacks on all online CPUs. start_shepherd_timer() is now called outside the get_online_cpus() block which is safe as it only operates on cpu possible mask. Cc: Andrew Mor

Re: [PATCH 1/2] mailbox: mailbox-test: Fix module autoload

2016-11-29 Thread Javier Martinez Canillas
Hello Jessi, On 10/24/2016 06:10 AM, Lee Jones wrote: > On Thu, 20 Oct 2016, Javier Martinez Canillas wrote: > >> If the driver is built as a module, autoload won't work because the module >> alias information is not filled. So user-space can't match the registered >> device with the correspondin

Re: [PATCH V2] leds: pca963x: Add ACPI support

2016-11-29 Thread Tin Huynh
On Tue, Nov 29, 2016 at 6:04 PM, Mika Westerberg wrote: > On Tue, Nov 29, 2016 at 11:55:47AM +0100, Jacek Anaszewski wrote: >> On 11/29/2016 11:21 AM, Tin Huynh wrote: >> > This patch enables ACPI support for leds-pca963x driver. >> > >> > Signed-off-by: Tin Huynh >> > --- >> > drivers/leds/leds

Re: [PATCH] mmc: pwrseq: add support for Marvell SD8787 chip

2016-11-29 Thread Rob Herring
On Mon, Nov 28, 2016 at 9:54 AM, Ulf Hansson wrote: > [...] > >>> + >>> +Example: >>> + >>> + wifi_pwrseq: wifi_pwrseq { >>> + compatible = "mmc-pwrseq-sd8787"; >>> + pwrdn-gpio = <&twl_gpio 0 GPIO_ACTIVE_LOW>; >>> + reset-gpio = <&twl_gpio 1 GPIO_ACTIVE_LOW

[PATCH v4 net-next 3/7] net: mvneta: Use cacheable memory to store the rx buffer virtual address

2016-11-29 Thread Gregory CLEMENT
Until now the virtual address of the received buffer were stored in the cookie field of the rx descriptor. However, this field is 32-bits only which prevents to use the driver on a 64-bits architecture. With this patch the virtual address is stored in an array not shared with the hardware (no more

[PATCH v4 net-next 1/7] net: mvneta: Optimize rx path for small frame

2016-11-29 Thread Gregory CLEMENT
For small frame reuse the phys_addr variable instead of accessing the uncacheable value in the rx descriptor. Signed-off-by: Gregory CLEMENT --- drivers/net/ethernet/marvell/mvneta.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/marvell/mvneta.c b/driv

[PATCH v4 net-next 0/7] Support Armada 37xx SoC (ARMv8 64-bits) in mvneta driver

2016-11-29 Thread Gregory CLEMENT
Hi, The Armada 37xx is a new ARMv8 SoC from Marvell using same network controller as the older Armada 370/38x/XP SoCs. This series adapts the driver in order to be able to use it on this new SoC. The main changes are: - 64-bits support: the first patches allow using the driver on a 64-bit archi

[PATCH v4 net-next 2/7] net: mvneta: Do not allocate buffer in rxq init with HWBM

2016-11-29 Thread Gregory CLEMENT
For HWBM all buffers are allocated in mvneta_bm_construct() and in runtime they are put into descriptors by hardware. There is no need to fill them at this point. Suggested-by: Marcin Wojtas Signed-off-by: Gregory CLEMENT --- drivers/net/ethernet/marvell/mvneta.c | 4 ++-- 1 file changed, 2 ins

[PATCH v4 net-next 7/7] ARM64: dts: marvell: Add network support for Armada 3700

2016-11-29 Thread Gregory CLEMENT
Add neta nodes for network support both in device tree for the SoC and the board. Signed-off-by: Gregory CLEMENT --- arch/arm64/boot/dts/marvell/armada-3720-db.dts | 23 +++- arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 23 +++- 2 files changed, 46 insertions(

[PATCH 0/4] tracing: Have tracepoint reg functions allowed to fail

2016-11-29 Thread Steven Rostedt
I added a 'trace_event=*:*' to my command line and it crashed the system. The reason was because it enabled the tracepoint benchmark which has a reg function that creates a thread. But the kernel command line tracepoint enabling is done before the scheduler is set up, and thus the creation of a thr

[PATCH 2/4] tracing: Do not start benchmark on boot up

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" Trace events are enabled every early on boot up via the boot command line parameter. The benchmark tool creates a new thread to perform the trace event benchmarking. But at start up, it is called before scheduling is set up and this crashes the kerenl. Have the b

Re: [PATCH] leds: netxbig: fix module autoload for OF registration

2016-11-29 Thread Javier Martinez Canillas
Hello Jacek, On 10/21/2016 12:52 PM, Javier Martinez Canillas wrote: > If the driver is built as a module, autoload won't work because the module > alias information is not filled. So user-space can't match the registered > device with the corresponding module. > > Export the module alias informa

[PATCH 5/5] net: ethernet: ti: cpsw: split tx budget according between channels

2016-11-29 Thread Ivan Khoronzhuk
Split device budget between channels according to channel rate. Signed-off-by: Ivan Khoronzhuk --- drivers/net/ethernet/ti/cpsw.c | 159 + 1 file changed, 130 insertions(+), 29 deletions(-) diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/etherne

[PATCH 3/4] tracing: Have system enable return error if one of the events fail

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" If one of the events of within a system fails to enable when "1" is written to the system "enable" file, it should return an error. Note, some events may still be enabled, but the user should know that something did go wrong. Signed-off-by: Steven Rostedt --- k

[PATCH 4/5] net: ethernet: ti: cpsw: optimize end of poll cycle

2016-11-29 Thread Ivan Khoronzhuk
Check budget fullness only after it's updated and update channel mask only once to keep budget balance between channels. It's also needed for farther changes. Signed-off-by: Ivan Khoronzhuk --- drivers/net/ethernet/ti/cpsw.c | 24 ++-- 1 file changed, 6 insertions(+), 18 dele

[PATCH 2/5] net: ethernet: ti: davinci_cpdma: add set rate for a channel

2016-11-29 Thread Ivan Khoronzhuk
The cpdma has 8 rate limited tx channels. This patch adds ability for cpdma driver to use 8 tx h/w shapers. If at least one channel is not rate limited then it must have higher number, this is because the rate limited channels have to have higher priority then not rate limited channels. The channel

[PATCH 4/4] tracing: Allow benchmark to be enabled at early_initcall()

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The trace event start up selftests fails when the trace benchmark is enabled, because it is disabled during boot. It really only needs to be disabled before scheduling is set up, as it creates a thread. Signed-off-by: Steven Rostedt --- kernel/trace/trace_bench

Re: [PATCH v3 08/13] clocksource/drivers/rockchip_timer: drop unused rk_base() and rk_ctrl()

2016-11-29 Thread Heiko Stübner
Am Dienstag, 29. November 2016, 16:45:13 schrieb Alexander Kochetkov: > Use of functions has been ceased by previous commit. Then why do you need another patch to remove them and don't do that in the patch removing their respective usage? > > Signed-off-by: Alexander Kochetkov > --- > drivers

Re: [PATCH] clocksource/arm_global_timer: reconfigure clockevents after cpufreq change

2016-11-29 Thread Alexander Kochetkov
> 29 нояб. 2016 г., в 17:51, Marc Zyngier написал(а): > > That'd be my preferred course of action. I've located some documentation > over there [1], and page 1126 seems to indicate a profusion of > additional timers, some of which are in an always-on domain. Seems like > a much better use of som

[PATCH 0/5] cpsw: add per channel shaper configuration

2016-11-29 Thread Ivan Khoronzhuk
This series is intended to allow user to set rate for per channel shapers at cpdma level. This patchset doesn't have impact on performance. The rate can be set with: echo 100 > /sys/class/net/ethX/queues/tx-0/tx_maxrate Tested on am572xx Based on net-next/master Ivan Khoronzhuk (5): net: ether

Re: [PATCH v3 13/13] clocksource/drivers/rockchip_timer: Prevent ftrace recursion

2016-11-29 Thread Heiko Stübner
Am Dienstag, 29. November 2016, 16:45:18 schrieb Alexander Kochetkov: > Currently rockchip_timer can be used as a scheduler clock. We properly > marked rk_timer_sched_clock_read() as notrace but we then call another > function rk_timer_counter_read() that _wasn't_ notrace. > > Having a traceable f

[PATCH 3/5] net: ethernet: ti: cpsw: add .ndo to set per-queue rate

2016-11-29 Thread Ivan Khoronzhuk
This patch allows to rate limit queues tx queues for cpsw interface. The rate is set in absolute Mb/s units and cannot be more a speed an interface is connected with. The rate for a tx queue can be tested with: ethtool -L eth0 rx 4 tx 4 echo 100 > /sys/class/net/eth0/queues/tx-0/tx_maxrate echo

[PATCH 1/4] tracing: Have the reg function allow to fail

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" Some tracepoints have a registration function that gets enabled when the tracepoint is enabled. There may be cases that the registraction function must fail (for example, can't allocate enough memory). In this case, the tracepoint should also fail to register, oth

[PATCH 1/5] net: ethernet: ti: davinci_cpdma: add weight function for channels

2016-11-29 Thread Ivan Khoronzhuk
The weight of a channel is needed to split descriptors between channels. The weight can depend on maximum rate of channels, maximum rate of an interface or other reasons. The channel weight is in percentage and is independent for rx and tx channels. Signed-off-by: Ivan Khoronzhuk --- drivers/net

[PATCH v4 net-next 6/7] net: mvneta: Add network support for Armada 3700 SoC

2016-11-29 Thread Gregory CLEMENT
From: Marcin Wojtas Armada 3700 is a new ARMv8 SoC from Marvell using same network controller as older Armada 370/38x/XP. There are however some differences that needed taking into account when adding support for it: * open default MBUS window to 4GB of DRAM - Armada 3700 SoC's Mbus configurat

Re: perf: fuzzer BUG: KASAN: stack-out-of-bounds in __unwind_start

2016-11-29 Thread Paul E. McKenney
On Tue, Nov 29, 2016 at 01:43:23PM +0100, Peter Zijlstra wrote: > On Mon, Nov 28, 2016 at 11:52:41PM -0600, Josh Poimboeuf wrote: > > > Did a little digging on git blame and found the following commit (which > > seems to be the cause of the KASAN warning and missing stack dump): > > > > bc1dce5

Re: [PATCH] doc: Explain light-handed markup preference a bit better

2016-11-29 Thread Jani Nikula
On Tue, 29 Nov 2016, Mauro Carvalho Chehab wrote: > Sorry, but I agree with Daniel here: we should provide a guide > for those people that will be helping with the document conversion. That goal is not mutually exclusive with keeping this document concise. That is all. Jani. -- Jani Nikula,

[for-next][PATCH 4/8] tracing: Make tracepoint_printk a static_key

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" Currently, when tracepoint_printk is set (enabled by the "tp_printk" kernel command line), it causes trace events to print via printk(). This is a very dangerous operation, but is useful for debugging. The issue is, it's seldom used, but it is always checked even

Re: perf: fuzzer BUG: KASAN: stack-out-of-bounds in __unwind_start

2016-11-29 Thread Josh Poimboeuf
On Tue, Nov 29, 2016 at 06:07:34AM -0800, Paul E. McKenney wrote: > On Tue, Nov 29, 2016 at 10:16:50AM +0100, Peter Zijlstra wrote: > > On Mon, Nov 28, 2016 at 11:52:41PM -0600, Josh Poimboeuf wrote: > > > > We used to do that, but the resulting NMIs were problematic on some > > > > platforms. Per

Re: [PATCH V5] i2c: mux: pca954x: Add ACPI support for pca954x

2016-11-29 Thread Tin Huynh
On Tue, Nov 29, 2016 at 5:15 PM, Peter Rosin wrote: > On 2016-11-29 11:12, Tin Huynh wrote: >> This patch enables ACPI support for mux-pca954x driver. > > Looks good, thanks! > >> Signed-off-by: Tin Huynh > > Acked-by: Peter Rosin > > Cheers, > Peter > Added Mika However , in http://git.kernel.

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-29 Thread Rob Herring
On Mon, Nov 28, 2016 at 9:30 AM, Frank Rowand wrote: > On 11/26/16 13:39, Frank Rowand wrote: >> On 11/23/16 13:58, Rob Herring wrote: >>> On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer >>> wrote: On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand wrote: > On 11/17/16 15:40, Frank Row

Re: [PATCH V7 0/3] irqchip: qcom: Add IRQ combiner driver

2016-11-29 Thread Agustin Vega-Frias
Hi Hanjun, On 2016-11-29 06:31, Hanjun Guo wrote: Hi Agustin, On 2016/11/14 5:59, Agustin Vega-Frias wrote: Add support for IRQ combiners in the Top-level Control and Status Registers (TCSR) hardware block in Qualcomm Technologies chips. The first patch fixes IRQ probe deferral by allowing pl

[for-next][PATCH 0/8] tracing: Some micro optimizations

2016-11-29 Thread Steven Rostedt
Andi Kleen pointed out a trace that showed lots of functions being called in the trace event hot path. These functions would help to be inlined. More needs to be done, but this is just a start. git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next Head SHA1: 38e11df134

[for-next][PATCH 3/8] ring-buffer: Always inline rb_event_data()

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The rb_event_data() is the fast path of getting the ring buffer data from an event. Externally, ring_buffer_event_data() is used to access this function. But unfortunately, rb_event_data() is not inlined, and calling ring_buffer_event_data() causes that function t

[for-next][PATCH 2/8] ring-buffer: Make rb_reserve_next_event() always inlined

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The function rb_reserved_next_event() is called by two functions: ring_buffer_lock_reserve() and ring_buffer_write(). This is in a very hot path of the tracing code, and it is best that they are not functions. The two callers are basically wrapers for rb_reserver_

[for-next][PATCH 5/8] tracing: Make __buffer_unlock_commit() always_inline

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The function __buffer_unlock_commit() is called in a few places outside of trace.c. But for the most part, it should really be inlined, as it is in the hot path of the trace_events. For the callers outside of trace.c, create a new function trace_buffer_unlock_comm

[for-next][PATCH 6/8] ring-buffer: Force inline of hotpath helper functions

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" There's several small helper functions in ring_buffer.c that are used in the hot path. For some reason, even though they are marked inline, gcc tends not to enforce it. Make sure these functions are always inlined. Link: http://lkml.kernel.org/r/20161121183700.gw

Re: [PATCH] HID: asus: Add i2c touchpad support

2016-11-29 Thread Jiri Kosina
On Tue, 29 Nov 2016, Benjamin Tissoires wrote: > On Nov 29 2016 or thereabouts, Brendan McGrath wrote: > > Update the hid-asus module to add multitouch support for the Asus i2c > > touchpad. > > > > Signed-off-by: Brendan McGrath > > Signed-off-by: Victor Vlasenko > > Signed-off-by: Frederik W

Re: [PATCH] clocksource/arm_global_timer: reconfigure clockevents after cpufreq change

2016-11-29 Thread Robin Murphy
On 29/11/16 14:51, Alexander Kochetkov wrote: > >> 29 нояб. 2016 г., в 17:32, Thomas Gleixner написал(а): >> >> Can we just disable that global timer on affected SoCs and use something >> else instead? > > I’ve sent patch series for fixing that on rockchip SoC. > http://lists.infradead.org/piper

[for-next][PATCH 7/8] ring-buffer: Froce rb_update_write_stamp() to be inlined

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" The function rb_update_write_stamp() is in the hotpath of the ring buffer recording. Make sure that it is inlined as well. There's not many places that call it. Link: http://lkml.kernel.org/r/20161121183700.gw26...@two.firstfloor.org Reported-by: Andi Kleen Sig

[for-next][PATCH 1/8] tracing: Create a always_inlined __trace_buffer_lock_reserve()

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" As Andi Kleen pointed out in the Link below, the trace events has quite a bit of code execution. A lot of that happens to be calling functions, where some of them should simply be inlined. One of these functions happens to be trace_buffer_lock_reserve() which is a

Re: [PATCH] proc: mm: export PTE sizes directly in smaps (v2)

2016-11-29 Thread Catalin Marinas
On Mon, Nov 28, 2016 at 01:39:49PM -0800, Dave Hansen wrote: > On 11/28/2016 01:07 PM, Vlastimil Babka wrote: > > On 11/28/2016 05:52 PM, Dave Hansen wrote: > >> On 11/24/2016 06:22 AM, Vlastimil Babka wrote: > >>> On 11/17/2016 01:28 AM, Dave Hansen wrote: > @@ -702,11 +707,13 @@ static int s

[PATCH] iio: misc: add a generic regulator driver

2016-11-29 Thread Bartosz Golaszewski
Some iio devices are powered externally by a regulator which, for example, can be used to power-cycle an adc. This patch proposes to add a simple driver representing a regulator to the iio framework which exports attributes allowing to manipulate the underlying hardware. The reason for connecting

Re: [PATCH v2] Fix objtool with clang

2016-11-29 Thread Jiri Olsa
On Tue, Nov 29, 2016 at 12:49:04PM -0200, Arnaldo Carvalho de Melo wrote: > Em Tue, Nov 29, 2016 at 03:39:35PM +0100, Jiri Olsa escreveu: > > On Tue, Nov 29, 2016 at 12:33:15PM -0200, Arnaldo Carvalho de Melo wrote: > > > Em Tue, Nov 29, 2016 at 03:28:08PM +0100, Jiri Olsa escreveu: > > > > On Mon,

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Agustin Vega-Frias
Hi Lorenzo, On 2016-11-29 07:11, Lorenzo Pieralisi wrote: Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great if you can comment on the reason ACPI does things in a certai

[PATCH] printk.c: removed unnecessary code

2016-11-29 Thread Sebastian Duda
snprintf((char *) ?, 0, ...); always returns Zero and doesn't change the data. Thus the execution of snprintf(NULL, 0, "[%5lu.00] ", (unsigned long)ts); has no effect on program. The substitution with 0 increases the readability of the code. Signed-off-by: Sebastian Duda Signed-off-by

Re: [PATCH] ipv6:ip6_xmit remove unnecessary np NULL check

2016-11-29 Thread Eric Dumazet
On Tue, 2016-11-29 at 12:02 +0530, Manjeet Pawar wrote: > From: Rohit Thapliyal > > np NULL check doesn't seem required here as it shall never > be NULL anyways in inet6_sk(sk). > > Signed-off-by: Rohit Thapliyal > Signed-off-by: Manjeet Pawar > Signed-off-by: David Miller > Reviewed-by: Akhi

[PATCH] tun: Use netif_receive_skb instead of netif_rx

2016-11-29 Thread Andrey Konovalov
This patch changes tun.c to call netif_receive_skb instead of netif_rx when a packet is received. The difference between the two is that netif_rx queues the packet into the backlog, and netif_receive_skb proccesses the packet in the current context. This patch is required for syzkaller [1] to coll

Re: [PATCH 08/22 v2] mm/vmstat: Avoid on each online CPU loops

2016-11-29 Thread Michal Hocko
On Tue 29-11-16 15:51:14, Sebastian Andrzej Siewior wrote: > Both iterations over online cpus can be replaced by the proper node > specific functions. > > Cc: Andrew Morton > Cc: Vlastimil Babka > Cc: Mel Gorman > Cc: Johannes Weiner > Cc: linux...@kvack.org > Signed-off-by: Sebastian Andrzej

[RFC] ARC: mm: Restrict definition of pfn_valid() macro for CONFIG_FLATMEM

2016-11-29 Thread Yuriy Kolerov
Despite the fact that subtraction of unsigned integers is a defined behaviour however such operations can lead to unexpected results. Thus it is better to check both left and right boundaries to avoid potential bugs as it done in the generic page.h. Signed-off-by: Yuriy Kolerov --- arch/arc/incl

Re: mwifiex: pcie: implement timeout loop for FW programming doorbell

2016-11-29 Thread Kalle Valo
Brian Norris wrote: > Marvell Wifi PCIe modules don't always behave nicely for PCIe power > management when their firmware hasn't been loaded, particularly after > suspending the PCIe link one or more times. When this happens, we might > end up spinning forever in this status-polling tight loop. L

Re: [PATCH] iio: misc: add a generic regulator driver

2016-11-29 Thread Lars-Peter Clausen
On 11/29/2016 04:22 PM, Bartosz Golaszewski wrote: [...] > diff --git a/Documentation/devicetree/bindings/iio/misc/iio-regulator.txt > b/Documentation/devicetree/bindings/iio/misc/iio-regulator.txt > new file mode 100644 > index 000..147458f > --- /dev/null > +++ b/Documentation/devicetree/bin

Re: net: GPF in eth_header

2016-11-29 Thread Andrey Konovalov
On Tue, Nov 29, 2016 at 3:58 PM, Eric Dumazet wrote: > On Tue, 2016-11-29 at 11:26 +0100, Andrey Konovalov wrote: >> On Sat, Nov 26, 2016 at 9:05 PM, Eric Dumazet wrote: >> >> I actually see multiple places where skb_network_offset() is used as >> >> an argument to skb_pull(). >> >> So I guess ev

Re: [PATCH 1/1] usb: abort on exception path

2016-11-29 Thread Johan Hovold
On Tue, Nov 29, 2016 at 08:53:35PM +0800, Pan Bian wrote: > Function klsi_105_open() calls usb_control_msg() and checks its return > value. When the return value is unexpected, it only assigns the error > code to the return variable retval, but does not terminate the exception > path. This patch fi

Re: [PATCH] tun: Use netif_receive_skb instead of netif_rx

2016-11-29 Thread Eric Dumazet
On Tue, 2016-11-29 at 16:25 +0100, Andrey Konovalov wrote: > This patch changes tun.c to call netif_receive_skb instead of netif_rx > when a packet is received. The difference between the two is that netif_rx > queues the packet into the backlog, and netif_receive_skb proccesses the > packet in the

Re: [PATCH V2] leds: pca963x: Add ACPI support

2016-11-29 Thread Jacek Anaszewski
Hi Tin, On 11/29/2016 03:47 PM, Tin Huynh wrote: On Tue, Nov 29, 2016 at 6:04 PM, Mika Westerberg wrote: On Tue, Nov 29, 2016 at 11:55:47AM +0100, Jacek Anaszewski wrote: On 11/29/2016 11:21 AM, Tin Huynh wrote: This patch enables ACPI support for leds-pca963x driver. Signed-off-by: Tin Huy

Re: [PATCH] iio: misc: add a generic regulator driver

2016-11-29 Thread Bartosz Golaszewski
2016-11-29 16:30 GMT+01:00 Lars-Peter Clausen : > On 11/29/2016 04:22 PM, Bartosz Golaszewski wrote: > [...] >> diff --git a/Documentation/devicetree/bindings/iio/misc/iio-regulator.txt >> b/Documentation/devicetree/bindings/iio/misc/iio-regulator.txt >> new file mode 100644 >> index 000..1474

Re: [PATCH] leds: netxbig: fix module autoload for OF registration

2016-11-29 Thread Jacek Anaszewski
Hi Javier, On 11/29/2016 04:00 PM, Javier Martinez Canillas wrote: Hello Jacek, On 10/21/2016 12:52 PM, Javier Martinez Canillas wrote: If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device wi

[PATCH] scsi: hisi_sas: fix free'ing in probe and remove

2016-11-29 Thread John Garry
From: Xiaofei Tan This patch addresses 4 problems in the module probe/remove: - When hisi_sas_shost_alloc() fails after we alloc shost memory, we should free shost memory before the function returns. - When hisi_sas_probe() fails after we alloc the HBA memories, we should also free the HBA memori

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Agustin Vega-Frias
Hi Rafael, On 2016-11-29 07:43, Rafael J. Wysocki wrote: On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi wrote: Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great if

Re: [PATCH] mfd: cpcap: Add minimal support

2016-11-29 Thread Rob Herring
On Thu, Nov 24, 2016 at 2:59 AM, Lee Jones wrote: > On Wed, 23 Nov 2016, Tony Lindgren wrote: > >> * Lee Jones [161121 03:43]: >> > On Fri, 18 Nov 2016, Tony Lindgren wrote: >> > > --- a/drivers/mfd/Makefile >> > > +++ b/drivers/mfd/Makefile >> > > @@ -97,6 +97,7 @@ obj-$(CONFIG_MFD_MC13XXX_I2C)

Re: bcma: add Dell Inspiron 3148

2016-11-29 Thread Kalle Valo
Jiri Slaby wrote: > This is what is in the laptop: > 01:00.0 Network controller [0280]: Broadcom Limited BCM43142 802.11b/g/n > [14e4:4365] (rev 01) > Subsystem: Dell Device [1028:0018] > Flags: bus master, fast devsel, latency 0, IRQ 18 > Memory at b040 (64-bit, non-p

Re: [PATCH V5 03/10] efi: parse ARMv8 processor error

2016-11-29 Thread Baicar, Tyler
Hello James, On 11/25/2016 11:23 AM, James Morse wrote: Hi Tyler, On 21/11/16 22:35, Tyler Baicar wrote: Add support for ARMv8 Common Platform Error Record (CPER). UEFI 2.6 specification adds support for ARMv8 specific processor error information to be reported as part of the CPER records. Thi

Re: [PATCH v3 13/13] clocksource/drivers/rockchip_timer: Prevent ftrace recursion

2016-11-29 Thread Alexander Kochetkov
Hello Heiko! Thank you for patch review! > 29 нояб. 2016 г., в 18:01, Heiko Stübner написал(а): > > you introduced the issue yourself in patch 11/13. In general any patch should > never leave the kernel in a worse state than it was before, so no patch > should > ever introduce known issues it

Re: [PATCH] leds: netxbig: fix module autoload for OF registration

2016-11-29 Thread Javier Martinez Canillas
Hello Jacek, On 11/29/2016 12:34 PM, Jacek Anaszewski wrote: > Hi Javier, > > On 11/29/2016 04:00 PM, Javier Martinez Canillas wrote: >> Hello Jacek, >> >> On 10/21/2016 12:52 PM, Javier Martinez Canillas wrote: >>> If the driver is built as a module, autoload won't work because the module >>> al

[PATCH V2 2/5] ASoC: arizona: Use component pin control functions

2016-11-29 Thread Richard Fitzgerald
We need to modify the state of some of our own pins and are currently not taking account that the pin name may have a name_prefix applied to it. Replace the snd_soc_dapm_x_pin functions with the equivalent snd_soc_component_x_pin functions so that any name_prefix will be handled automatically. Si

[PATCH V2 1/5] ASoC: core: Add component pin control functions

2016-11-29 Thread Richard Fitzgerald
It's often the case that a codec driver will need to control its own pins. However, if a name_prefix has been applied to this codec it must be included in the name passed to any of the snd_soc_dapm_x_pin() functions. The behaviour of the existing pin control functions is reasonable, since you may

[PATCH V2 3/5] regulator: arizona-micsupp: Use SoC component pin control functions

2016-11-29 Thread Richard Fitzgerald
The name of a codec pin can have an optional prefix string, which is defined by the SoC machine driver. The snd_soc_dapm_x_pin functions take the fully-specified name including the prefix and so the existing code would fail to find the pin if the audio machine driver had added a prefix. Switch to

[PATCH V2 4/5] extcon: arizona: Use SoC component pin control functions

2016-11-29 Thread Richard Fitzgerald
The name of a codec pin can have an optional prefix string, which is defined by the SoC machine driver. The snd_soc_dapm_x_pin functions take the fully-specified name including the prefix and so the existing code would fail to find the pin if the audio machine driver had added a prefix. Switch to

[PATCH V2 0/5] ASoC/arizona: Ensure pin searches use widget name prefix

2016-11-29 Thread Richard Fitzgerald
The name of a codec pin can have an optional prefix string, which is defined by the audio machine driver. The snd_soc_dapm_x_pin functions take the fully-specified name including the prefix and so the existing code would fail to find the pin if the audio machine driver had added a prefix. This pat

[PATCH V2 5/5] Input: arizona-haptics - Use SoC component pin control functions

2016-11-29 Thread Richard Fitzgerald
The name of a codec pin can have an optional prefix string, which is defined by the SoC machine driver. The snd_soc_dapm_x_pin functions take the fully-specified name including the prefix and so the existing code would fail to find the pin if the audio machine driver had added a prefix. Switch to

Re: [PATCH] pwm: fix device reference leak

2016-11-29 Thread Thierry Reding
On Tue, Nov 01, 2016 at 11:46:39AM +0100, Johan Hovold wrote: > Make sure to drop the reference to the parent device taken by > class_find_device() after "unexporting" any children when deregistering > a pwm chip. > > Fixes: 0733424c9ba9 ("pwm: Unexport children before chip removal") > Signed-off-

Re: [PATCH] mfd: cpcap: Add minimal support

2016-11-29 Thread Tony Lindgren
* Rob Herring [161129 07:20]: > On Thu, Nov 24, 2016 at 2:59 AM, Lee Jones wrote: > > On Wed, 23 Nov 2016, Tony Lindgren wrote: > > > >> * Lee Jones [161121 03:43]: > >> > On Fri, 18 Nov 2016, Tony Lindgren wrote: > >> > > --- a/drivers/mfd/Makefile > >> > > +++ b/drivers/mfd/Makefile > >> > > @

[for-next][PATCH 8/8] ring-buffer: Force rb_end_commit() and rb_set_commit_to_write() inline

2016-11-29 Thread Steven Rostedt
From: "Steven Rostedt (Red Hat)" Both rb_end_commit() and rb_set_commit_to_write() are in the fast path of the ring buffer recording. Make sure they are always inlined. Link: http://lkml.kernel.org/r/20161121183700.gw26...@two.firstfloor.org Reported-by: Andi Kleen Signed-off-by: Steven Rosted

Re: [PATCH v2 07/13] net: ethernet: ti: cpts: rework initialization/deinitialization

2016-11-29 Thread Grygorii Strashko
Hi Richard, On 11/29/2016 04:07 AM, Richard Cochran wrote: > On Mon, Nov 28, 2016 at 05:03:31PM -0600, Grygorii Strashko wrote: >> +int cpts_register(struct cpts *cpts) >> { >> int err, i; >> >> -cpts->info = cpts_info; >> -spin_lock_init(&cpts->lock); >> - >> -cpts->cc.read =

[PATCH] iio: 104-quad-8: Fix active level mismatch for the preset enable option

2016-11-29 Thread William Breathitt Gray
The set_to_preset_on_index sysfs attribute provides a boolean configuration option to enable a preset operation on the respective channel's counter when Index occurs. However, the corresponding configuration bit on the Input/Output Control register must be set low to enable the preset operation. T

Re: 4.9-rc: only 3 CPUs out of 4 used

2016-11-29 Thread Meelis Roos
> I have a HP ProLiant DL380 G3 server. It has two > dual-core 32-bit Xeon CPUs. CONFIG_NR_CPUS=4 and has always been. Same happens on DL360 G3 - 1U server of the same era. -- Meelis Roos (mr...@linux.ee)

[PATCH 2/2] USB: serial: kl5kusb105: abort on open exception path

2016-11-29 Thread Johan Hovold
From: Pan Bian Function klsi_105_open() calls usb_control_msg() (to "enable read") and checks its return value. When the return value is unexpected, it only assigns the error code to the return variable retval, but does not terminate the exception path. This patch fixes the bug by inserting "goto

[PATCH 1/2] USB: serial: kl5kusb105: fix open error path

2016-11-29 Thread Johan Hovold
Kill urbs and disable read before returning from open on failure to retrieve the line state. Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") Cc: stable Signed-off-by: Johan Hovold --- drivers/usb/serial/kl5kusb105.c | 33 - 1 file changed, 24 insertions(+), 9 deletions(

Re: 4.8.8 kernel trigger OOM killer repeatedly when I have lots of RAM that should be free

2016-11-29 Thread Marc MERLIN
On Mon, Nov 28, 2016 at 08:23:15AM +0100, Michal Hocko wrote: > Marc, could you try this patch please? I think it should be pretty clear > it should help you but running it through your use case would be more > than welcome before I ask Greg to take this to the 4.8 stable tree. I ran it overnight

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-29 Thread Moritz Fischer
Hi Rob, On Tue, Nov 29, 2016 at 09:06:08AM -0600, Rob Herring wrote: > On Mon, Nov 28, 2016 at 9:30 AM, Frank Rowand wrote: > > On 11/26/16 13:39, Frank Rowand wrote: > >> On 11/23/16 13:58, Rob Herring wrote: > >>> On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer > >>> wrote: > On Thu, Nov

[PATCH 0/2] USB: serial: kl5kusb105: fix open error paths

2016-11-29 Thread Johan Hovold
Pan Bian found an issue with the kl5kusb105 open error handling, which would not abort an open attempt when a vendor command to "enable read" failed. Turns out there were more issues with this function, specifically any urbs submitted would not be killed before returning on failures to retrieve t

Re: [PATCH v5 4/9] dt-bindings: iio: iio-mux: document iio-mux bindings

2016-11-29 Thread Peter Rosin
On 2016-11-29 11:10, Peter Rosin wrote: > +Example: > + mux: mux-controller { > + compatible = "mux-gpio"; > + #mux-control-cells = <0>; > + > + mux-gpios = <&pioA 0 GPIO_ACTIVE_HIGH>, > + <&pioA 1 GPIO_ACTIVE_HIGH>; > + }; > + > +

Re: [fuse-devel] fuse: feasible to distinguish between umount and abort?

2016-11-29 Thread Nikolaus Rath
On Nov 29 2016, Miklos Szeredi wrote: > On Fri, Nov 25, 2016 at 1:33 AM, Nikolaus Rath wrote: >> On Nov 24 2016, Miklos Szeredi wrote: >>> On Thu, Nov 24, 2016 at 12:11 AM, Nikolaus Rath wrote: Hello, Currently, both a call to umount(2) and writing "1" to /sys/fs/fuse/connec

Re: [PATCH 1/6] perf tool: Add time-based utility functions

2016-11-29 Thread Arnaldo Carvalho de Melo
Em Mon, Nov 28, 2016 at 10:27:08AM -0700, David Ahern escreveu: > On 11/28/16 6:58 AM, Jiri Olsa wrote: > > On Fri, Nov 25, 2016 at 02:39:54PM -0700, David Ahern wrote: > > > > SNIP > > > >> diff --git a/tools/perf/util/time-utils.h b/tools/perf/util/time-utils.h > >> new file mode 100644 > >> in

Re: [PATCH v2 08/13] net: ethernet: ti: cpts: move dt props parsing to cpts driver

2016-11-29 Thread Grygorii Strashko
On 11/29/2016 04:11 AM, Richard Cochran wrote: > On Mon, Nov 28, 2016 at 05:03:32PM -0600, Grygorii Strashko wrote: >> +static int cpts_of_parse(struct cpts *cpts, struct device_node *node) >> +{ >> +int ret = -EINVAL; >> +u32 prop; >> + >> +if (of_property_read_u32(node, "cpts_clock_

Re: [PATCH 1/6] perf tool: Add time-based utility functions

2016-11-29 Thread David Ahern
On 11/29/16 9:02 AM, Arnaldo Carvalho de Melo wrote: > Humm, I'd prefer 'time_interval' or 'perf_time_interval', plain > 'interval' doesn't convey what kind if interval is this, we could be > talking about counter values intervals, etc. agreed, perf_time_interval makes more sense.

Re: [PATCH v3 0/7] mux controller abstraction and iio/i2c muxes

2016-11-29 Thread Peter Rosin
On 2016-11-27 13:00, Jonathan Cameron wrote: > On 23/11/16 11:47, Peter Rosin wrote: >> On 2016-11-22 21:58, Lars-Peter Clausen wrote: >>> On 11/21/2016 02:17 PM, Peter Rosin wrote: >>> [...] I have a piece of hardware that is using the same 3 GPIO pins to control four 8-way muxes. Three

Re: [PATCH] x86/kbuild: enable modversions for symbols exported from asm

2016-11-29 Thread Michal Marek
Dne 29.11.2016 v 16:27 Linus Torvalds napsal(a): > On Nov 29, 2016 5:51 AM, "Adam Borowski" > wrote: >> > >> > >> > (a) tested >> >> By many people. > > No. > > I've tested the build *without* this, and it works fine. > >> > (b) explains it >> >> The actual logic

Re: [PATCH v9 1/7] perf config: Introduce default_config_section and default_config_item for default config key-value pairs

2016-11-29 Thread Arnaldo Carvalho de Melo
Em Mon, Nov 28, 2016 at 05:51:12PM +0900, Taeung Song escreveu: > When initializing default perf config values, > we currently use values of actual type(int, bool, char *, etc.). > > For example, > If there isn't a user config value for 'annotate.use_offset' > config variable at ~/.perfconfig, > d

Re: [PATCH v7 04/16] drivers: iommu: make of_iommu_set/get_ops() DT agnostic

2016-11-29 Thread Joerg Roedel
On Wed, Nov 16, 2016 at 09:56:15AM +, Lorenzo Pieralisi wrote: > I can easily make the changes Robin suggests above, I need to know > what to do with this patch it is the last blocking point for this > series and time is running out I can revert to using dev->bus to > retrieve iommu_ops (even t

Re: 4.8.8 kernel trigger OOM killer repeatedly when I have lots of RAM that should be free

2016-11-29 Thread Michal Hocko
On Tue 29-11-16 07:55:37, Marc MERLIN wrote: > On Mon, Nov 28, 2016 at 08:23:15AM +0100, Michal Hocko wrote: > > Marc, could you try this patch please? I think it should be pretty clear > > it should help you but running it through your use case would be more > > than welcome before I ask Greg to t

Re: [PATCH 7/7] trace: Update documentation for mono, mono_raw and boot clock

2016-11-29 Thread Joel Fernandes
On Mon, Nov 28, 2016 at 11:26 PM, Ingo Molnar wrote: > > * John Stultz wrote: > >> + boot: This is the boot clock (CLOCK_BOOTTIME) and is based on the >> + fast monotonic clock, but also accounts for time spent in >> + suspend. Since the clock access is designed for

Re: [RFC v3 04/10] iommu: iommu_alloc_resv_region

2016-11-29 Thread Joerg Roedel
On Tue, Nov 15, 2016 at 01:09:17PM +, Eric Auger wrote: > +static inline struct iommu_resv_region * > +iommu_alloc_resv_region(phys_addr_t start, size_t length, unsigned int prot) > +{ > + return NULL; > +} > + Will this function be called outside of iommu code? Joerg

Re: perf: fuzzer BUG: KASAN: stack-out-of-bounds in __unwind_start

2016-11-29 Thread Petr Mladek
On Tue 2016-11-29 09:09:17, Josh Poimboeuf wrote: > On Tue, Nov 29, 2016 at 06:07:34AM -0800, Paul E. McKenney wrote: > > On Tue, Nov 29, 2016 at 10:16:50AM +0100, Peter Zijlstra wrote: > > > On Mon, Nov 28, 2016 at 11:52:41PM -0600, Josh Poimboeuf wrote: > > > > > We used to do that, but the resul

Re: [PATCH] audit: remove the audit freelist

2016-11-29 Thread Richard Guy Briggs
On 2016-11-15 14:16, Florian Westphal wrote: > allows better debugging as freeing audit buffers now always honors slub > debug hooks (e.g. object poisoning) and leak checker can detect the > free operation. > > Removal also results in a small speedup (using > single rule 'iptables -A INPUT -i lo -

Re: [PATCH 7/7] trace: Update documentation for mono, mono_raw and boot clock

2016-11-29 Thread Thomas Gleixner
On Tue, 29 Nov 2016, Joel Fernandes wrote: > On Mon, Nov 28, 2016 at 11:26 PM, Ingo Molnar wrote: > > > > * John Stultz wrote: > > > >> + boot: This is the boot clock (CLOCK_BOOTTIME) and is based on the > >> + fast monotonic clock, but also accounts for time spent in > >> +

Re: [PATCH v2] scsi: aic94xx: Add a missing call to kfree

2016-11-29 Thread Martin K. Petersen
> "Quentin" == Quentin Lambert writes: Quentin> Most error branches following the call to kzalloc contain a Quentin> call to kfree. This patch add these calls where they are Quentin> missing and set the relevant pointers to NULL. Applied to 4.10/scsi-queue. -- Martin K. Petersen Oracl

[PATCH v4 0/9] Implement clocksource for rockchip SoC using rockchip timer

2016-11-29 Thread Alexander Kochetkov
Hello, This patch series contain: - devicetree bindings clarification for rockchip timers - dts files fixes for rk3228-evb, rk3229-evb and rk3188 - implementation of clocksource and sched clock for rockchip SoC The clock supplying the arm-global-timer on the rk3188 is coming from the the cpu cloc

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