[PATCH v5 0/9] Goodix touchscreen enhancements

2015-09-07 Thread Irina Tirdea
Add several enhancements to the Goodix touchscreen driver. This version adds runtime power management and includes some cleanup. Thanks, Irina Changes in v5: - add some more style cleanup (reorder includes, use error instead of ret for return values) - add runtime power management patch Change

[PATCH] tile: fix build failure

2015-09-07 Thread Sudip Mukherjee
When building with allmodconfig the build was failing with the error: arch/tile/kernel/usb.c:70:1: warning: data definition has no type or storage class [enabled by default] arch/tile/kernel/usb.c:70:1: error: type defaults to 'int' in declaration of 'arch_initcall' [-Werror=implicit-int] arch/t

[PATCH V2 05/14] perf tools: Fix perf_evlist__splice_list_tail not setting evlist

2015-09-07 Thread Adrian Hunter
Commit d49e46950772 ("perf evsel: Add a backpointer to the evlist a evsel is in") updated perf_evlist__add() but not perf_evlist__splice_list_tail(). This illustrates that it is better if perf_evlist__splice_list_tail() calls perf_evlist__add() instead of duplicating the logic, so do that. This wi

[PATCH V2 07/14] perf tools: Add evsel->own_cpus

2015-09-07 Thread Adrian Hunter
perf_evlist__propagate_maps() cannot easily tell if an evsel has its own cpu map. To make that simpler, keep a copy of the PMU cpu map and adjust the propagation logic accordingly. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 5 - tools/perf/util/evsel.c| 1 + t

Re: [RFC PATCH 0/3] Implement IRQ stack on ARM64

2015-09-07 Thread James Morse
On 04/09/15 15:23, Jungseok Lee wrote: > ARM64 kernel allocates 16KB kernel stack when creating a process. In case > of low memory platforms with tough workloads on userland, this order-2 > allocation request reaches to memory pressure and performance degradation > simultaenously since VM page allo

[PATCH V2 04/14] perf tools: Add evlist->has_user_cpus

2015-09-07 Thread Adrian Hunter
Subsequent patches will need to call perf_evlist__propagate_maps without reference to a "target". Add evlist->has_user_cpus to record whether the user has specified which cpus to target (and therefore whether that list of cpus should override the default settings for a selected event i.e. the cpu

[PATCH 2/4] hv: fcopy: use wrapper to propate state

2015-09-07 Thread Olaf Hering
The .state is used by several threads of execution. Propagate the state to make changes visible. Also propagate context change in hv_fcopy_onchannelcallback. Without this change fcopy may hang at random points. Signed-off-by: Olaf Hering --- drivers/hv/hv_fcopy.c | 36 ---

[PATCH V2 06/14] perf tools: Fix missing thread_map__put in perf_evlist__propagate_maps

2015-09-07 Thread Adrian Hunter
perf_evlist__propagate_maps() incorrectly assumes evsel->threads is NULL before reassigning it, but it won't be NULL when perf_evlist__set_maps() is used to set different (or NULL) maps. Thus thread_map__put must be used, which works even if evsel->threads is NULL. Signed-off-by: Adrian Hunter --

[PATCH V2 10/14] perf tools: Factor out a function to propagate maps for a single evsel

2015-09-07 Thread Adrian Hunter
Subsequent fixes will need a function that just propagates maps for a single evsel so factor it out. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 37 + 1 file changed, 21 insertions(+), 16 deletions(-) diff --git a/tools/perf/util/evlist.c b/to

[PATCH V2 13/14] perf tools: Fix task exit test setting maps

2015-09-07 Thread Adrian Hunter
The test titled "Test number of exit event of a simple workload" was setting cpu/thread maps directly. Make it use the proper function perf_evlist__set_maps() especially now that it also propagates the maps. Signed-off-by: Adrian Hunter --- tools/perf/tests/task-exit.c | 22 ++--

[PATCH V2 03/14] perf tools: Remove redundant validation from perf_evlist__propagate_maps

2015-09-07 Thread Adrian Hunter
The validation checks that the values that were just assigned, got assigned i.e. the error can't ever happen. Subsequent patches will call this code in places where errors are not being returned. Changing those code paths to return this non-existent error is counter-productive, so just remove it.

[PATCH V2 12/14] perf tools: Fix perf_evlist__create_syswide_maps() not propagating maps

2015-09-07 Thread Adrian Hunter
Fix it by making it call perf_evlist__set_maps() instead of setting the maps itself. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c index e0a

[PATCH V2 09/14] perf tools: Make perf_evlist__create_maps() use perf_evlist__set_maps()

2015-09-07 Thread Adrian Hunter
Since there is a function to set maps, perf_evlist__create_maps() should use it. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c index 606fc6e

[PATCH V2 08/14] perf tools: Make perf_evlist__set_maps() more resilient

2015-09-07 Thread Adrian Hunter
Make perf_evlist__set_maps() more resilient by allowing for the possibility that one or another of the maps isn't being changed and therefore should not be "put". Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-)

[PATCH V2 11/14] perf tools: Fix perf_evlist__add() not propagating maps

2015-09-07 Thread Adrian Hunter
If evsels are added after maps are created, then they won't have any maps propagated to them. Fix that. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 56 +--- 1 file changed, 29 insertions(+), 27 deletions(-) diff --git a/tools/perf/uti

[PATCH V2 02/14] perf tools: Simplify perf_evlist__set_maps logic

2015-09-07 Thread Adrian Hunter
Don't need to check for NULL when "putting" evlist->maps and evlist->threads because the "put" functions already do that. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/tools/perf/util/evlist.c b/tools/perf/ut

[PATCH V2 14/14] perf tools: Fix software clock events test setting maps

2015-09-07 Thread Adrian Hunter
The test titled "Test software clock events have valid period values" was setting cpu/thread maps directly. Make it use the proper function perf_evlist__set_maps() especially now that it also propagates the maps. Signed-off-by: Adrian Hunter --- tools/perf/tests/sw-clock.c | 22

[PATCH V2 01/14] perf tools: Simplify perf_evlist__propagate_maps logic

2015-09-07 Thread Adrian Hunter
If evsel->cpus is to be reassigned then the current value must be "put", which works even if it is NULL. Simplify the current logic by moving the "put" next to the assignment. Signed-off-by: Adrian Hunter --- tools/perf/util/evlist.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) d

[PATCH V2 00/14] perf tools: Fix gaps propagating maps

2015-09-07 Thread Adrian Hunter
Hi Here is V2 of "Fix gaps propagating maps" that fixes some problems revealed by to d988d5ee6478 ("perf evlist: Open event on evsel cpus and threads") Changes in V2: Split into multiple patches Add evsel->own_cpus to identify evsels with their own cpu map And consequentl

[PATCH 0/4] hv: utils: propagate state to interrupt thread

2015-09-07 Thread Olaf Hering
The Copy-VMFile cmdlet on the host may fail because the guest fcopy driver state machine gets out of sync. This happens because the ->state and ->context variables are accessed by the main thread and from interrupt context. If an interrupt happens between fcopy_respond_to_host and hv_poll_channel i

[PATCH 1/4] hv: add helpers to handle hv_util device state

2015-09-07 Thread Olaf Hering
The callbacks in kvp, vss and fcopy code are called the main thread and also from interrupt context. If a state change is done by the main thread it is not immediately seen by the interrupt. As a result the state machine gets out of sync. Force propagation of state changes via get/set helpers with

[PATCH 4/4] hv: vss: use wrapper to propate state

2015-09-07 Thread Olaf Hering
The .state is used by several threads of execution. Propagate the state to make changes visible. Also propagate context change in vss_on_msg. Signed-off-by: Olaf Hering --- drivers/hv/hv_snapshot.c | 37 - 1 file changed, 20 insertions(+), 17 deletions(-) dif

[PATCH 3/4] hv: kvp: use wrapper to propate state

2015-09-07 Thread Olaf Hering
The .state is used by several threads of execution. Propagate the state to make changes visible. Also propagate context change in kvp_on_msg. Signed-off-by: Olaf Hering --- drivers/hv/hv_kvp.c | 38 +- 1 file changed, 21 insertions(+), 17 deletions(-) diff --

[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> --- Rather than setting each member of ->delayed_refs by hand we should adhere to the practice of using memset()

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

2015-09-07 Thread Alexandru Moise
On Mon, Sep 07, 2015 at 04:01:22PM +0200, David Sterba wrote: > 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_delay

[PATCH] powerpc32: memcpy: only use dcbz once cache is enabled

2015-09-07 Thread Christophe Leroy
memcpy() uses instruction dcbz to speed up copy by not wasting time loading cache line with data that will be overwritten. Some platform like mpc52xx do no have cache active at startup and can therefore not use memcpy(). Allthough no part of the code explicitly uses memcpy(), GCC makes calls to it.

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

2015-09-07 Thread Jacek Anaszewski
Hi Takashi, Thanks for chasing this. Milo, could you express your opinion? On 09/07/2015 02:25 PM, Takashi Iwai wrote: 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_US

Re: [PATCH] [s390]: Use .rept directive to store/load fpu registers

2015-09-07 Thread Heiko Carstens
On Tue, Sep 01, 2015 at 09:51:32AM +0200, Christian Borntraeger wrote: > Am 28.08.2015 um 15:49 schrieb Alexander Kuleshov: > > arch/s390/kernel/swsusp.S | 49 > > --- > > 1 file changed, 17 insertions(+), 32 deletions(-) > > > > diff --git a/arch/s390

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

2015-09-07 Thread Luis Henriques
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 error. Fixes: beca3ec71fe5 ("zram: add multi stream functionality") Cc: sta...@vger.kernel.org Signed-off-by: Lu

[PATCH] char: remove unused variable res to clean up build warning

2015-09-07 Thread Colin King
From: Colin Ian King Commit f368ed6088ae9 ("char: make misc_deregister a void function") did not remove unused variable res and now we get a build warning: drivers/rtc/rtc-ds1374.c: In function 'ds1374_remove': drivers/rtc/rtc-ds1374.c:667:6: warning: unused variable 'res' [-Wunused-variable]

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

2015-09-07 Thread Luis Henriques
On Mon, Sep 07, 2015 at 10:33:32PM +0900, Sergey Senozhatsky wrote: > 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 potent

Re: [RFC PATCH 2/3] arm64: Introduce IRQ stack

2015-09-07 Thread Jungseok Lee
On Sep 5, 2015, at 2:12 AM, Alexnader Kuleshov wrote: > Hello Jungseok, Hello Alexnader, > On 09-04-15, Jungseok Lee wrote: >> +config IRQ_STACK >> +bool "Use separate kernel stack when handling interrupts" >> +depends on ARM64_4K_PAGES >> +help >> + Say Y here if you want to us

Re: [PATCH 4/5] [media] uvcvideo: create pad links after subdev registration

2015-09-07 Thread Javier Martinez Canillas
Hello, On 09/03/2015 06:00 PM, Javier Martinez Canillas wrote: > The uvc driver creates the pads links before the media entity is > registered with the media device. This doesn't work now that obj > IDs are used to create links so the media_device has to be set. > > Move entities registration log

[PATCH v2 2/2] leds: leds-ipaq-micro: Fix coding style issues

2015-09-07 Thread Muhammad Falak R Wani
Spaces at the starting of a line are removed, indentation using tab, instead of space. Also, warnings related to line width of more than 80 characters is also taken care of. Two warnings have been left alone to aid better readability. Signed-off-by: Muhammad Falak R Wani --- drivers/leds/leds-i

[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 deletions(

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 Ma

[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: leds-ipaq-micro

[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: Michael

[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 b/tools/testing/selftest

[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 inde

[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 d

[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: >>> On

[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 ps2dev->cmdc

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 addr

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 0

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 fil

[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 i

[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 changed

[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 a/fs/b

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 >> we'

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 error

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 > ‘virtio_timeline_value_st

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 Jan

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 series

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 Yingli

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 i

[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 Rui

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: Z

[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 n

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 > lay

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 +0200

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 qu

[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 impo

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 PERF_

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 > p

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 pr

[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 synchronization

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... Thi

[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 Microbla

[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 'xlnx

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 > > --- > > drivers/staging

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 directly,

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, s

[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 m

[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 com

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

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

[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 b/drivers/staging/gdm72xx/usb

[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 drivers/staging/gdm72xx/gd

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 use

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 > --- > --- > .../devicetree/bindings/hwmon/nct78

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 flag

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 t

[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. Signed-o

[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 playin

<    1   2   3   4   5   6   >