[PATCH v2 1/2] leds: leds-ipaq-micro: Use devm_led_classdev_register

2015-09-07 Thread Muhammad Falak R Wani
Use of resource-managed function devm_led_classdev_register instead of led_classdev_register is preferred, consequently remove redundant function micro_leds_remove. Signed-off-by: Muhammad Falak R Wani --- drivers/leds/leds-ipaq-micro.c | 9 + 1 file changed, 1 insertion(+), 8

Re: [V9fs-developer] [PATCH] 9p: trans_fd, initialize recv fcall properly if not set

2015-09-07 Thread Eric Van Hensbergen
I thought the nature of trans_fd would have prevented any sort of true zero copy, but I suppose one less is always welcome :) -eric On Sun, Sep 6, 2015 at 1:55 AM, Dominique Martinet wrote: > Eric Van Hensbergen wrote on Sat, Sep 05, 2015: >> On Thu, Sep 3, 2015 at 4:38 AM, Dominique

[PATCH v2 0/2] leds: leds-ipaq-micro: Use resource managed functions and fix coding style.

2015-09-07 Thread Muhammad Falak R Wani
Hi, Based on the suggestion from Jaeck, the orginal patch is split into two, one patch fixes the coding style issues, and the other uses resource managed function, devm_led_classdev_register. Muhammad Falak R Wani (2): leds: leds-ipaq-micro: Use devm_led_classdev_register leds:

[PATCH v2 6/6] selftests: exec: simpification the Makefile

2015-09-07 Thread Bamvor Jian Zhang
With the previous patch, the installation method change from install to rsync. There is not no need to create subdir during test, the default RUN_TESTS is enough. This patch implicitly revert commit 84cbd9e4 ("selftests/exec: do not install subdir as it is already created"). Suggested-by:

[PATCH v2 5/6] selftests: change install command to rsync

2015-09-07 Thread Bamvor Jian Zhang
The command of install could not handle the special files in exec testcases, change the default rule to rsync to fix this. The result of installation is unchanged after this commit. Suggested-by: Michael Ellerman Signed-off-by: Bamvor Jian Zhang --- tools/testing/selftests/ftrace/Makefile | 2

[PATCH v2 1/6] selftests: rename jump label to static_keys

2015-09-07 Thread Bamvor Jian Zhang
commit "2bf9e0a locking/static_keys: Provide a selftest" rename jump_label directory to static_keys. Signed-off-by: Bamvor Jian Zhang --- tools/testing/selftests/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/Makefile

[PATCH v2 2/6] selftests: only compile userfaultfd for x86 and powperpc

2015-09-07 Thread Bamvor Jian Zhang
Check it before compiling to avoid the failure of building and installation. Signed-off-by: Bamvor Jian Zhang --- tools/testing/selftests/vm/Makefile | 12 1 file changed, 12 insertions(+) diff --git a/tools/testing/selftests/vm/Makefile b/tools/testing/selftests/vm/Makefile

[PATCH v2 3/6] selftests: mqueue: allow extra cflags

2015-09-07 Thread Bamvor Jian Zhang
change from = to += in order to pass the proper headers and librareis (popt.h and libpopt.so) in order to build successful in cross compiling. Suggested-by: Michael Ellermani Signed-off-by: Bamvor Jian Zhang --- tools/testing/selftests/mqueue/Makefile | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v2 4/6] selftests: mqueue: simpification the Makefile

2015-09-07 Thread Bamvor Jian Zhang
Suggested-by: Michael Ellermani Signed-off-by: Bamvor Jian Zhang --- tools/testing/selftests/mqueue/Makefile | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/mqueue/Makefile b/tools/testing/selftests/mqueue/Makefile index ca8327f..9f51aaa 100644

[PATCH v2 0/6] kselftest improvement and cleanup

2015-09-07 Thread Bamvor Jian Zhang
This is my second attempt for improving the kselftest for arm/arm64 architecture. Eventually, we hope we could build(in an cross compile environment) and run all the kselftest cases automatically(successful of courses). The first version is here[1]. In this series, I try to make all the testcases

RE: [PATCH] x86, arm64, acpi: Handle lapic/x2apic entries in MADT

2015-09-07 Thread Anaczkowski, Lukasz
From: Tomasz Nowicki [mailto:tomasz.nowi...@linaro.org] Sent: Tuesday, September 1, 2015 3:37 PM > On 01.09.2015 14:07, Anaczkowski, Lukasz wrote: >> From: Tomasz Nowicki [mailto:tomasz.nowi...@linaro.org] >> Sent: Tuesday, September 1, 2015 10:03 AM >>> To fix this, each LAPIC/X2APIC entry

Re: Potential data race in psmouse_interrupt

2015-09-07 Thread Dmitry Vyukov
I've mailed a separate patch that does serio_pause_rx before reading out data ("input: fix data race __ps2_command"). On Sat, Sep 5, 2015 at 3:21 PM, Dmitry Vyukov wrote: > On Fri, Sep 4, 2015 at 10:27 PM, Dmitry Torokhov > wrote: >> On Fri, Sep 4, 2015 at 12:32 PM, Dmitry Vyukov wrote: >>>

[PATCH] input: fix data race __ps2_command

2015-09-07 Thread Dmitry Vyukov
The data race happens on ps2dev->cmdcnt and ps2dev->cmdbuf contents. __ps2_command reads that data concurrently with the interrupt handler. As the result, for example, if a response arrives just after the timeout, __ps2_command can copy out garbage from ps2dev->cmdbuf but then see that

Re: [PATCH V2] arm64: add kc_offset_to_vaddr and kc_vaddr_to_offset macro

2015-09-07 Thread Jungseok Lee
On Sep 6, 2015, at 1:04 PM, yalin wang wrote: Hello Yalin, > This patch add kc_offset_to_vaddr() and kc_vaddr_to_offset(), > the default version doesn't work on arm64, because arm64 kernel address > is below the PAGE_OFFSET, like module address and vmemmap address are > all below PAGE_OFFSET

Re: [PATCH] sched: fix lose fair sleeper bonus in switch_to_fair()

2015-09-07 Thread Peter Zijlstra
Please always Cc at least the person who wrote the lines you modify. On Mon, Sep 07, 2015 at 05:45:20PM +0800, Wanpeng Li wrote: > The sleeper task will be normalized when moved from fair_sched_class, in > order that vruntime will be adjusted either the task is running or sleeping > when moved

Re: [PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-07 Thread David Sterba
So, updated patches should mention that in the subject eg: [PATCH v2] btrfs: memset cur_trans->delayed_refs to zero On Mon, Sep 07, 2015 at 04:45:02PM +0300, Alexandru Moise wrote: > Use memset() to null out the btrfs_delayed_ref_root of > btrfs_transaction instead of setting all the members to

Re: [PATCH v6 2/2] devicetree: Add documentation for UPISEMI us5182d ALS and Proximity sensor

2015-09-07 Thread Adriana Reus
Thanks for your feedback, some comments inline. On 31.08.2015 18:38, Rob Herring wrote: On Thu, Aug 27, 2015 at 3:23 PM, Jonathan Cameron wrote: On 20/08/15 11:12, Adriana Reus wrote: Added entries in i2c/vendor-prefixes for the us5182d als and proximity sensor. Also added a documentation

[PATCH? v2] fput: don't abuse task_work_add() too much

2015-09-07 Thread Oleg Nesterov
On 09/07, Oleg Nesterov wrote: > > Oh, I disagree. But I guess I can't convince you/Eric/Linus, so I have > to shut up. > > > Damn. But I can't relax ;) Al, Linus, could you comment the patch below? > > Not for inclusion, lacks the changelog/testing, fput() can be simplified. > But as you can see

[PATCH] ath6kl: remove redundant null pointer check on send_pkt

2015-09-07 Thread Colin King
From: Colin Ian King The check for send_pkt being NULL is redundant before the call to htc_reclaim_txctrl_buf, therefore it should be removed. This was detected by static analysis by cppcheck. Signed-off-by: Colin Ian King --- drivers/net/wireless/ath/ath6kl/htc_mbox.c | 4 +--- 1 file

[PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-07 Thread Alexandru Moise
Use memset() to null out the btrfs_delayed_ref_root of btrfs_transaction instead of setting all the members to 0 by hand. Signed-off-by: Alexandru Moise <00moses.alexande...@gmail.com> --- fs/btrfs/transaction.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

Re: [PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-07 Thread Alexandru Moise
On Mon, Sep 07, 2015 at 02:24:20PM +0200, David Sterba wrote: > On Sun, Sep 06, 2015 at 12:25:27PM +, Alexandru Moise wrote: > > Use memset() to null out the btrfs_delayed_ref_root of > > btrfs_transaction instead of setting all the members to 0 by hand. > > > > Signed-off-by: Alexandru Moise

Re: [PATCH] Remove #ifdef CONFIG_64BIT from all asm-generic/fcntl.h

2015-09-07 Thread Palmer Dabbelt
On Mon, 07 Sep 2015 06:16:37 PDT (-0700), a...@arndb.de wrote: > On Tuesday 01 September 2015 17:10:10 Palmer Dabbelt wrote: >> From: Palmer Dabbelt >> >> When working on the RISC-V port I noticed that F_SETLK64 was being >> defined on our 64-bit platform, despite our port being so new that >>

Re: [PATCH] zram: fix possible use after free in zcomp_create()

2015-09-07 Thread Sergey Senozhatsky
On (09/07/15 13:53), Luis Henriques wrote: > > On (09/07/15 11:33), Luis Henriques wrote: > > > zcomp_create() verifies the success of zcomp_strm_{multi,siggle}_create() > > > through comp->stream, which can potentially be pointing to memory that was > > > freed if these functions returned an

Re: [PATCH] virtio-gpu: fix compilation warnings

2015-09-07 Thread Gerd Hoffmann
On Mi, 2015-09-02 at 12:30 +0300, Mike Rapoport wrote: > Update snprintf format in virtgpu_fence.c and virtgpu_debugfs.c to fix the > following compilation warnings: > > C [M] drivers/gpu/drm/virtio/virtgpu_fence.o > drivers/gpu/drm/virtio/virtgpu_fence.c: In function >

Re: [PATCH] perf report: Fix invalid memory accessing

2015-09-07 Thread Wangnan (F)
On 2015/9/7 20:51, Wang Nan wrote: [SNIP] Although theoretically CPU_TOPOLOGY feature should always be selected by 'perf record', I did generate a perf.data without that feature. It has header like this: # perf report -i ./bad.perf.data --header-only # # captured on: Thu

Re: [PATCH 2/6] sched/fair: Convert arch_scale_cpu_capacity() from weak function to #define

2015-09-07 Thread Dietmar Eggemann
On 04/09/15 08:26, Vincent Guittot wrote: > On 3 September 2015 at 21:58, Dietmar Eggemann > wrote: [...] >>> So you change the way to declare arch_scale_cpu_capacity but i don't >>> see the update of the arm arch which declare a >>> arch_scale_cpu_capacity to reflect this change in your

Re: [RFC PATCH v1 2/4] irqchip: GICv3: set non-percpu irqs status with _IRQ_MOVE_PCNTXT

2015-09-07 Thread Thomas Gleixner
On Mon, 7 Sep 2015, Marc Zyngier wrote: > On 06/09/15 06:56, Jiang Liu wrote: > > On 2015/9/6 12:23, Yang Yingliang wrote: > >> Use irq_settings_set_move_pcntxt() helper irqs status with > >> _IRQ_MOVE_PCNTXT. So that it can do set affinity when calling > >> irq_set_affinity_locked(). > > Hi

Re: [PATCH 0/6] sched/fair: Compute capacity invariant load/utilization tracking

2015-09-07 Thread Peter Zijlstra
On Mon, Sep 07, 2015 at 02:42:20PM +0200, Peter Zijlstra wrote: > I'm of a mind to apply these patches; with two patches on top, which > I'll post shortly. --- Subject: sched: Optimize __update_load_avg() From: Peter Zijlstra Date: Mon Sep 7 15:09:15 CEST 2015 Prior to this patch; the line:

Re: [PATCH 0/6] sched/fair: Compute capacity invariant load/utilization tracking

2015-09-07 Thread Peter Zijlstra
On Mon, Sep 07, 2015 at 02:42:20PM +0200, Peter Zijlstra wrote: > I'm of a mind to apply these patches; with two patches on top, which > I'll post shortly. --- Subject: sched: Rename scale() From: Peter Zijlstra Date: Mon Sep 7 15:05:42 CEST 2015 Rename scale() to cap_scale() to better reflect

[PATCH v5 2/5] thermal: power_allocator: relax the requirement of a sustainable_power in tzp

2015-09-07 Thread Javi Merino
The power allocator governor currently requires that a sustainable power is passed as part of the thermal zone's thermal zone parameters. If that parameter is not provided, it doesn't register with the thermal zone. While this parameter is strongly recommended for optimal performance, it doesn't

[PATCH v5 5/5] thermal: power_allocator: exit early if there are no cooling devices

2015-09-07 Thread Javi Merino
Don't waste cycles in the power allocator governor's throttle function if there are no cooling devices and exit early. This commit doesn't change any functionality, but should provide better performance for the odd case of a thermal zone with trip points but without cooling devices. Cc: Zhang

Re: [PATCH 0/4] MIPS: ath79: Add USB support on the TL-WR1043ND

2015-09-07 Thread Arnd Bergmann
On Tuesday 01 September 2015 17:23:10 Alban Bedel wrote: > > this serie add a driver for the USB phy on the ATH79 SoCs and enable the > USB port on the TL-WR1043ND. The phy controller is really trivial as it > only use reset lines. > Is this a common thing to have? If other PHY devices are like

[PATCH v5 4/5] thermal: power_allocator: don't require tzp to be present for the thermal zone

2015-09-07 Thread Javi Merino
Thermal zones created using thermal_zone_device_create() may not have tzp. As the governor gets its parameters from there, allocate it while the governor is bound to the thermal zone so that it can operate in it. In this case, tzp is freed when the thermal zone switches to another governor. Cc:

[PATCH v5 3/5] thermal: power_allocator: relax the requirement of two passive trip points

2015-09-07 Thread Javi Merino
The power allocator governor currently requires that the thermal zone has at least two passive trip points. If there aren't, the governor refuses to bind to the thermal zone. This commit relaxes that requirement. Now the governor will bind to all thermal zones regardless of how many trip points

[PATCH v5 1/5] thermal: Add a function to get the minimum power

2015-09-07 Thread Javi Merino
The thermal core already has a function to get the maximum power of a cooling device: power_actor_get_max_power(). Add a function to get the minimum power of a cooling device. Cc: Zhang Rui Cc: Eduardo Valentin Signed-off-by: Javi Merino --- drivers/thermal/thermal_core.c | 28

[PATCH v5 0/5] Let the power allocator thermal governor run on any thermal zone

2015-09-07 Thread Javi Merino
Relax the thermal governor requirements of sustainable_power and at least two trip points so that it can be bound to any thermal zone. Its behavior won't be optimal, it would be the best possible with the data provided. Changes since v4: - Fix crash when a thermal zone with no trip points has

Re: [PATCH] Remove #ifdef CONFIG_64BIT from all asm-generic/fcntl.h

2015-09-07 Thread Arnd Bergmann
On Tuesday 01 September 2015 17:10:10 Palmer Dabbelt wrote: > From: Palmer Dabbelt > > When working on the RISC-V port I noticed that F_SETLK64 was being > defined on our 64-bit platform, despite our port being so new that > we've only ever had the 64-bit file ops. Since there's not compat >

Re: [PATCH 0/5] New Atmel PIO4 pinctrl/gpio driver

2015-09-07 Thread Ludovic Desroches
Hi Sascha, Any comments about this version? Maybe you missed it since you answered me on the RFC version. Of course I'll do the changes you requested for the next version (remove gpio_request_*() stuff and add macros for gpio muxing case). Thanks Ludovic On Tue, Aug 11, 2015 at 10:28:00AM

Re: [RFC v0 2/9] suspend: Add getter function to report if freezing is active

2015-09-07 Thread Rafael J. Wysocki
On Monday, September 07, 2015 10:55:43 AM Daniel Wagner wrote: > On 09/05/2015 04:11 AM, Rafael J. Wysocki wrote: > > On Friday, September 04, 2015 03:34:55 PM Daniel Wagner wrote: > >> Instead encode the FREEZE state via the CPU state we allow the > >> interesting subsystems (MCE, microcode) to

[PATCH] tty: don't leak cdev in tty_cdev_add()

2015-09-07 Thread Leon Yu
Commit a3a10ce3429e ("Avoid usb reset crashes by making tty_io cdevs truly dynamic") which mixes using cdev_alloc() and cdev_init() is problematic. Subsequent call to cdev_init() after cdev_alloc() sets kobj release method from cdev_dynamic_release() to cdev_default_release() and thus makes it

Re: [PATCH] perf report: Fix invalid memory accessing

2015-09-07 Thread Wangnan (F)
On 2015/9/7 21:03, Jiri Olsa wrote: On Mon, Sep 07, 2015 at 12:51:55PM +, Wang Nan wrote: Commit e1e499aba570a2ea84d29822b7ea637ac41d9a51 (perf tools: Add processor socket info to hist_entry and addr_location) reads env->cpu array for each sample using index al.cpu. However, al.cpu can be

Re: [PATCH] perf report: Fix invalid memory accessing

2015-09-07 Thread Jiri Olsa
On Mon, Sep 07, 2015 at 12:51:55PM +, Wang Nan wrote: > Commit e1e499aba570a2ea84d29822b7ea637ac41d9a51 (perf tools: Add > processor socket info to hist_entry and addr_location) reads env->cpu > array for each sample using index al.cpu. However, al.cpu can be -1 if > sample doesn't select

Re: [PATCH 0/3] perf tests: Add arch tests

2015-09-07 Thread Jiri Olsa
On Sat, Sep 05, 2015 at 08:02:19PM +0100, Matt Fleming wrote: > From: Matt Fleming > > To encourage developers to contribute arch-specific tests without > feeling guilty about messing up the generic perf tests, this patch > series introduces "arch tests" and then adds a test for the Intel CQM >

Re: [PATCH v6 1/2] dmaengine: Add Xilinx Central DMA DT Binding Documentation

2015-09-07 Thread Arnd Bergmann
On Monday 07 September 2015 18:03:08 Kedareswara rao Appana wrote: > +- dma-channel child node: Should have only one channel > What is the point of the child node if there is only one instance? Could you just fold this into the main node instead? (Sorry jumping in here without looking at the

[PATCH] perf report: Fix invalid memory accessing

2015-09-07 Thread Wang Nan
Commit e1e499aba570a2ea84d29822b7ea637ac41d9a51 (perf tools: Add processor socket info to hist_entry and addr_location) reads env->cpu array for each sample using index al.cpu. However, al.cpu can be -1 if sample doesn't select PERF_SAMPLE_CPU. Also, env->cpu can be invalid if feature CPU_TOPOLOGY

Re: [PATCH RFC tip/core/rcu 3/9] rcu_sync: Add CONFIG_PROVE_RCU checks

2015-09-07 Thread Oleg Nesterov
On 09/07, Daniel Wagner wrote: > > On 08/29/2015 05:35 AM, Paul E. McKenney wrote: > > +extern bool __rcu_sync_is_idle(struct rcu_sync *); > > + > > /** > > * rcu_sync_is_idle() - Are readers permitted to use their fastpaths? > > * @rsp: Pointer to rcu_sync structure to use for

Re: [PATCH] x86: Wire up 32-bit direct socket calls

2015-09-07 Thread Arnd Bergmann
On Wednesday 02 September 2015 13:16:19 H. Peter Anvin wrote: > On 09/02/2015 02:48 AM, Geert Uytterhoeven wrote: > > > > Should all other architectures follow suit? > > Or should we follow the s390 approach: > > > > It is up to the maintainer(s), largely dependent on how likely you are > going

Re: [PATCH] zram: fix possible use after free in zcomp_create()

2015-09-07 Thread Luis Henriques
On Mon, Sep 07, 2015 at 08:11:48PM +0900, Sergey Senozhatsky wrote: > On (09/07/15 11:33), Luis Henriques wrote: > > zcomp_create() verifies the success of zcomp_strm_{multi,siggle}_create() > > through comp->stream, which can potentially be pointing to memory that was > > freed if these functions

Re: [PATCH-v2 1/3] mmc: sdhci-pxav3: Fix tabbing issue

2015-09-07 Thread Vaibhav Hiremath
On Monday 07 September 2015 05:26 PM, Jisheng Zhang wrote: On Mon, 7 Sep 2015 17:01:09 +0530 Vaibhav Hiremath wrote: There were some coding style issues where spaces have been used instead of tabs, for example, in macro definitions, alignment of function declarations/definitions, etc...

[PATCH v6 2/2] dmaengine: Add Xilinx AXI Central Direct Memory Access Engine driver support

2015-09-07 Thread Kedareswara rao Appana
This is the driver for the AXI Central Direct Memory Access (AXI CDMA) core, which is a soft Xilinx IP core that provides high-bandwidth Direct Memory Access (DMA) between a memory-mapped source address and a memory-mapped destination address. This module works on Zynq (ARM Based SoC) and

[PATCH v6 1/2] dmaengine: Add Xilinx Central DMA DT Binding Documentation

2015-09-07 Thread Kedareswara rao Appana
Device-tree binding documentation of Xilinx Central DMA Engine. Signed-off-by: Kedareswara rao Appana --- Changes in v6: - None. Changes in v5: - None. Changes in v4: - None. Changes in v3: - Used proper alignment for all the properties. Changes in v2: - Change property 'xlnx,data-width' to

Re: [PATCH 3/5] staging: most: remove driver owner

2015-09-07 Thread Sudip Mukherjee
On Mon, Sep 07, 2015 at 02:13:38PM +0200, Andrey Shvetsov wrote: > On Fri, Sep 04, 2015 at 04:22:04PM +0530, Sudip Mukherjee wrote: > > The platform driver core will set the owner value, we do not need to do > > it in the module. > > > > Signed-off-by: Sudip Mukherjee > > --- > >

Re: [PATCH] mm/mmap.c: Remove redundent 'get_area' function pointer in get_unmapped_area()

2015-09-07 Thread Oleg Nesterov
On 09/05, Chen Gang wrote: > > From a1bf4726f71d6d0394b41309944646fc806a8a0c Mon Sep 17 00:00:00 2001 > From: Chen Gang > Date: Sat, 5 Sep 2015 21:51:08 +0800 > Subject: [PATCH] mm/mmap.c: Remove redundent 'get_area' function pointer in > get_unmapped_area() > > Call the function pointer

Re: [PATCH 0/6] sched/fair: Compute capacity invariant load/utilization tracking

2015-09-07 Thread Peter Zijlstra
On Mon, Aug 31, 2015 at 11:24:49AM +0200, Peter Zijlstra wrote: > A quick run here gives: > > IVB-EP (2*20*2): As noted by someone; that should be 2*10*2, for a total of 40 cpus in this machine. > > perf stat --null --repeat 10 -- perf bench sched messaging -g 50 -l 5000 > > Before:

Re: [PATCH] mm/mmap.c: Remove useless statement "vma = NULL" in find_vma()

2015-09-07 Thread Oleg Nesterov
On 09/05, Chen Gang wrote: > > From b12fa5a9263cf4c044988e59f0071f4bcc132215 Mon Sep 17 00:00:00 2001 > From: Chen Gang > Date: Sat, 5 Sep 2015 21:49:56 +0800 > Subject: [PATCH] mm/mmap.c: Remove useless statement "vma = NULL" in >  find_vma() > > Before the main looping, vma is already is NULL,

[PATCH v3 2/2] ring_buffer: Fix more races when terminating the producer in the benchmark

2015-09-07 Thread Petr Mladek
The commit b44754d8262d3aab8 ("ring_buffer: Allow to exit the ring buffer benchmark immediately") added a hack into ring_buffer_producer() that set @kill_test when kthread_should_stop() returned true. It improved the situation a lot. It stopped the kthread in most cases because the producer spent

[PATCH v3 0/2] ring_buffer: Make the benchmark slightly more safe

2015-09-07 Thread Petr Mladek
These two patches fix potential races in the ring buffer benchmark. The first two versions were reviewed as part of the patchset that tried to convert some kthreads into the kthread worker API, see http://thread.gmane.org/gmane.linux.kernel.api/13224/focus=13821 Changes in v3: + fixed several

[PATCH v3 1/2] ring_buffer: Do no not complete benchmark reader too early

2015-09-07 Thread Petr Mladek
It seems that complete(_done) might be called too early in some situations. 1st scenario: - CPU0CPU1 ring_buffer_producer_thread() wake_up_process(consumer); wait_for_completion(_start);

[PATCH 1/2] staging: gdm72xx: fix memory leak

2015-09-07 Thread Sudip Mukherjee
We were successfully requesting the firmware but on error it was not being released. Signed-off-by: Sudip Mukherjee --- drivers/staging/gdm72xx/usb_boot.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/staging/gdm72xx/usb_boot.c

[PATCH 2/2] staging: gdm72xx: NULL comparison style

2015-09-07 Thread Sudip Mukherjee
checkpatch complains if NULL comparison is done as if (var == NULL) Signed-off-by: Sudip Mukherjee --- drivers/staging/gdm72xx/gdm_qos.c | 2 +- drivers/staging/gdm72xx/gdm_sdio.c | 20 ++-- drivers/staging/gdm72xx/gdm_usb.c | 16

Re: [PATCH 1/3] perf tests: Introduce iterator function for tests

2015-09-07 Thread Jiri Olsa
On Sat, Sep 05, 2015 at 08:02:20PM +0100, Matt Fleming wrote: > From: Matt Fleming > > In preparation for introducing more arrays of tests, e.g. "arch tests" > (architecture-specific tests), abstract the code to iterate over the > list of tests into a helper function. > > This way, code that

Re: [PATCH v2] Documentation: add Device tree bindings for hwmon/nct7802

2015-09-07 Thread Mark Rutland
On Sun, Sep 06, 2015 at 10:59:49PM +0100, Constantine Shulyupin wrote: > Changed in v2: > - Removed nct7802,reg-init > - Added registers initialization by names > > Introduced in v1: > - nct7802,reg-init > > Signed-off-by: Constantine Shulyupin > --- > --- >

Re: [RFC PATCH v1 2/4] irqchip: GICv3: set non-percpu irqs status with _IRQ_MOVE_PCNTXT

2015-09-07 Thread Marc Zyngier
On 06/09/15 06:56, Jiang Liu wrote: > On 2015/9/6 12:23, Yang Yingliang wrote: >> Use irq_settings_set_move_pcntxt() helper irqs status with >> _IRQ_MOVE_PCNTXT. So that it can do set affinity when calling >> irq_set_affinity_locked(). > Hi Yingliang, > We could only set _IRQ_MOVE_PCNTCT

Re: [PATCH v3 02/18] of/platform: add of_platform_probe

2015-09-07 Thread Tomeu Vizoso
On 11 August 2015 at 11:37, Tomeu Vizoso wrote: > On 7 August 2015 at 14:19, Mark Brown wrote: >> On Thu, Aug 06, 2015 at 04:11:39PM +0200, Tomeu Vizoso wrote: >> >>> Walks the OF tree up and finds the closest ancestor that has a platform >>> device associated with it, probing it if isn't bound

[PATCH v4 11/22] i2c: core: Probe i2c adapters and devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up an i2c adapter or device through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed.

[PATCH v4 0/22] On-demand device probing

2015-09-07 Thread Tomeu Vizoso
Hello, I have a problem with the panel on my Tegra Chromebook taking longer than expected to be ready during boot (Stéphane Marchesin reported what is basically the same issue in [0]), and have looked into ordered probing as a better way of solving this than moving nodes around in the DT or

[PATCH v4 07/22] regulator: core: Reduce critical area in _regulator_get

2015-09-07 Thread Tomeu Vizoso
By moving the locking of regulator_list_mutex into regulator_dev_lookup, where it is iterated over. The reference count of the regulator's device is increased in case it's unregistered while in use. In _regulator_get() the regulator_list_mutex mutex was held for most of the function, but that is

Re: [PATCH] tty: fix data races on tty_buffer.commit

2015-09-07 Thread Dmitry Vyukov
Split this into 2 separate fixes (mailed separately). Also dropped some READ/WRITE_ONCE. We would like to commit them as well for the following reasons: https://github.com/google/ktsan/wiki/READ_ONCE-and-WRITE_ONCE What do you think about using READ/WRITE_ONCE for all concurrent accesses to

Re: [PATCH v3 0/5] KVM: optimize userspace exits with a new ioctl

2015-09-07 Thread Paolo Bonzini
On 02/09/2015 12:31, Christian Borntraeger wrote: > As far as I can see this should also work for s390 (when we implement > REQ_EXIT handling) > > To double check my understanding: these improvements come with a changed > userspace that does not use KVM_SET_SIGNAL_MASK and therefore this >

[PATCH v4 10/22] drm/tegra: Probe dpaux devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a dpaux device through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

Re: [PATCH 2/3] perf tests: Add arch tests

2015-09-07 Thread Jiri Olsa
On Sat, Sep 05, 2015 at 08:02:21PM +0100, Matt Fleming wrote: SNIP > > +static struct test *tests[] = { > + generic_tests, > + arch_tests, > +}; > + > static bool perf_test__matches(struct test *test, int curr, int argc, const > char *argv[]) > { > int i; > @@ -237,7 +229,11

[PATCH v4 12/22] pwm: Probe PWM chip devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a PWM chip through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by: Tomeu

[PATCH?] fput: don't abuse task_work_add() too much

2015-09-07 Thread Oleg Nesterov
On 09/05, Al Viro wrote: > > On Sat, Aug 29, 2015 at 02:49:21PM +0200, Oleg Nesterov wrote: > > > Let's look keyctl_session_to_parent(). It does task_work_cancel() > > but only because we can not trust user-space. Otherwise we could > > remove it and just do task_work_add(), but this needs fifo. >

[PATCH v4 14/22] usb: phy: Probe phy devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a phy through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by: Tomeu

[PATCH v4 01/22] driver core: Add pre_probe callback to bus_type

2015-09-07 Thread Tomeu Vizoso
Some buses (eg. AMBA) need access to some HW resources (it may need a clock to be enabled so a device ID can be read) before a device can be matched to a driver. The pre_probe callback allows the device-driver core to request the bus to perform this initialization and can defer the probe if any

[PATCH v4 02/22] ARM: amba: Move reading of periphid to pre_probe()

2015-09-07 Thread Tomeu Vizoso
Reading the periphid when the Primecell device is registered means that the apb pclk must be available by then or the device won't be registered at all. By moving this code to pre_probe (to be called by the device-driver core before the device is matched to a driver) we remove any order

[PATCH v4 18/22] dma: of: Probe DMA controllers on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a DMA controller through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

[PATCH v4 19/22] power-supply: Probe power supplies on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a power supply through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

Re: [PATCH 2/3] perf tests: Add arch tests

2015-09-07 Thread Jiri Olsa
On Sat, Sep 05, 2015 at 08:02:21PM +0100, Matt Fleming wrote: SNIP > }; > > +static struct test *tests[] = { > + generic_tests, > + arch_tests, > +}; > + > static bool perf_test__matches(struct test *test, int curr, int argc, const > char *argv[]) > { > int i; > @@ -237,7

[PATCH v4 15/22] clk: Probe clk providers on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a clock through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by: Tomeu

[PATCH v4 20/22] driver core: Allow deferring probes until late init

2015-09-07 Thread Tomeu Vizoso
Add a field to struct device that instructs the device-driver core to defer the probe of this device until the late_initcall level. By letting all built-in drivers to register before starting to probe, we can avoid any deferred probes by probing dependencies on demand. Signed-off-by: Tomeu

[PATCH v4 22/22] of/platform: Defer probes of registered devices

2015-09-07 Thread Tomeu Vizoso
Instead of trying to match and probe platform and AMBA devices right after each is registered, delay their probes until device_initcall_sync. This means that devices will start probing once all built-in drivers have registered, and after all platform and AMBA devices from the DT have been

[PATCH v4 04/22] of: add function to allow probing a device from a OF node

2015-09-07 Thread Tomeu Vizoso
Walks the OF tree up and finds the closest ancestor that has a struct device associated with it, probing it if isn't bound to a driver yet. The above should ensure that the dependency represented by the passed OF node is available, because probing a device should cause its descendants to be

Re: [PATCH] btrfs: memset cur_trans->delayed_refs to zero

2015-09-07 Thread David Sterba
On Sun, Sep 06, 2015 at 12:25:27PM +, Alexandru Moise wrote: > Use memset() to null out the btrfs_delayed_ref_root of > btrfs_transaction instead of setting all the members to 0 by hand. > > Signed-off-by: Alexandru Moise <00moses.alexande...@gmail.com> > --- > fs/btrfs/transaction.c | 10

[PATCH v4 06/22] gpio: Probe pinctrl devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a pin controller through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

[PATCH v4 08/22] regulator: core: Probe regulators on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a regulator through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by: Tomeu

[PATCH] tty: fix data race in tty_buffer_flush

2015-09-07 Thread Dmitry Vyukov
tty_buffer_flush frees not acquired buffers. As the result, for example, read of b->size in tty_buffer_free can return garbage value which will lead to a huge buffer hanging in the freelist. This is just the benignest manifestation of freeing of a not acquired object. If the object is passed to

[PATCH v4 09/22] drm: Probe panels on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a panel through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by: Tomeu

[PATCH v4 13/22] backlight: Probe backlight devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a backlight device through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed.

[PATCH v4 21/22] driver core: Start processing deferred probes earlier

2015-09-07 Thread Tomeu Vizoso
Some initcalls in the late level assume that some devices will have already probed without explicitly checking for that. After the recent move to defer most device probes when they are registered, pressure increased in the late initcall level. By starting the processing of the deferred queue in

[PATCH v4 17/22] phy: core: Probe phy providers on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a phy provider through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

[PATCH v4 16/22] pinctrl: Probe pinctrl devices on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a pin controller through its OF node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

[PATCH] leds:lp55xx: Correct Kconfig dependency for f/w user helper

2015-09-07 Thread Takashi Iwai
The commit [b67893206fc0: leds:lp55xx: fix firmware loading error] tries to address the firmware file handling with user helper, but it sets a wrong Kconfig CONFIG_FW_LOADER_USER_HELPER_FALLBACK. Since the wrong option was enabled, the system got a regression -- it suffers from the unexpected

[PATCH v4 05/22] gpio: Probe GPIO drivers on demand

2015-09-07 Thread Tomeu Vizoso
When looking up a gpiochip through its firmware node, probe it if it hasn't already. The goal is to reduce deferred probes to a minimum, as it makes it very cumbersome to find out why a device failed to probe, and can introduce very big delays in when a critical device is probed. Signed-off-by:

[PATCH v4 03/22] of/platform: Point to struct device from device node

2015-09-07 Thread Tomeu Vizoso
When adding a platform device, set the device node's device member to point to it. This speeds lookups considerably and is safe because we only create one platform device for any given device node. Signed-off-by: Tomeu Vizoso --- Changes in v4: None Changes in v3: None Changes in v2: None

Re: [PATCH 2/3] perf tests: Add arch tests

2015-09-07 Thread Jiri Olsa
On Sat, Sep 05, 2015 at 08:02:21PM +0100, Matt Fleming wrote: SNIP > diff --git a/tools/perf/tests/builtin-test.c b/tools/perf/tests/builtin-test.c > index 8cf0601d1662..a1b2265eaf55 100644 > --- a/tools/perf/tests/builtin-test.c > +++ b/tools/perf/tests/builtin-test.c > @@ -14,10 +14,17 @@ >

Re: [PATCH 3/5] staging: most: remove driver owner

2015-09-07 Thread Andrey Shvetsov
On Fri, Sep 04, 2015 at 04:22:04PM +0530, Sudip Mukherjee wrote: > The platform driver core will set the owner value, we do not need to do > it in the module. > > Signed-off-by: Sudip Mukherjee > --- > drivers/staging/most/hdm-dim2/dim2_hdm.c | 1 - > 1 file changed, 1 deletion(-) > > diff

Re: [PATCHv5 1/7] mm: drop page->slab_page

2015-09-07 Thread Kirill A. Shutemov
On Sun, Sep 06, 2015 at 10:00:57PM -0700, Alexander Duyck wrote: > On 09/03/2015 05:35 AM, Kirill A. Shutemov wrote: > >Since 8456a648cf44 ("slab: use struct page for slab management") nobody > >uses slab_page field in struct page. > > > >Let's drop it. > > > >Signed-off-by: Kirill A. Shutemov >

[GIT PULL] fbdev changes for 4.3

2015-09-07 Thread Tomi Valkeinen
Hi Linus, Please pull fbdev changes for 4.3. Not much this time. The following changes since commit 2c6625cd545bdd66acff14f3394865d43920a5c7: Linux 4.2-rc7 (2015-08-16 16:34:13 -0700) are available in the git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git

Re: [PATCH 2/6] perf, tools, stat: Abstract stat metrics printing

2015-09-07 Thread Jiri Olsa
On Fri, Sep 04, 2015 at 03:45:13PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Abstract the printing of shadow metrics. Instead of every > metric calling fprintf directly and taking care of indentation, > use two call backs: one to print metrics and another to > start a new line. > > This

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