[PATCH 16/27] tools include: Grab copies of arm64 dependent unistd.h files

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Kim Phillips Will be used for generating the syscall id/string translation table. The arm64 unistd.h file simply #includes the asm-generic/unistd.h, so, since we will want to know whether either change, we grab both: arch/arm64/include/uapi/asm/unistd.h and

[PATCH 16/27] tools include: Grab copies of arm64 dependent unistd.h files

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Kim Phillips Will be used for generating the syscall id/string translation table. The arm64 unistd.h file simply #includes the asm-generic/unistd.h, so, since we will want to know whether either change, we grab both: arch/arm64/include/uapi/asm/unistd.h and

[PATCH 23/27] perf machine: Use last_match threads cache only in single thread mode

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa There's an issue with using threads::last_match in multithread mode which is enabled during the perf top synthesize. It might crash with following assertion: perf: ...include/linux/refcount.h:109: refcount_inc: Assertion `!(!refcount_inc_not_zero(r))' failed. The gdb

[PATCH 27/27] perf test: Fix subtest number when showing results

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter Perf test 40 for example has several subtests numbered 1-4 when displaying the start of the subtest. When the subtest results are displayed the subtests are numbered 0-3. Use this command to generate trace output: [root@s35lp76 perf]# ./perf test -Fv 40 2>/tmp/bpf1 Fix

[PATCH 25/27] perf tools: Use perf_evsel__match instead of open coded equivalent

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Use perf_evsel__match() helper in perf_evsel__is_bpf_output(). Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: David Ahern Cc: Namhyung Kim Cc: Peter Zijlstra Link: http://lkml.kernel.org/r/20180720110036.32251-1-jo...@kernel.org Signed-off-by: Arnaldo Carvalho de Melo

[PATCH 26/27] perf stat: Get rid of extra clock display function

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa There's no reason to have separate function to display clock events. It's only purpose was to convert the nanosecond value into microseconds. We do that now in generic code, if the unit and scale values are properly set, which this patch do for clock events. The output differs

[PATCH 24/27] perf tools: Fix struct comm_str removal crash

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa We occasionaly hit following assert failure in 'perf top', when processing the /proc info in multiple threads. perf: ...include/linux/refcount.h:109: refcount_inc: Assertion `!(!refcount_inc_not_zero(r))' failed. The gdb backtrace looks like this: [Switching to

[PATCH 23/27] perf machine: Use last_match threads cache only in single thread mode

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa There's an issue with using threads::last_match in multithread mode which is enabled during the perf top synthesize. It might crash with following assertion: perf: ...include/linux/refcount.h:109: refcount_inc: Assertion `!(!refcount_inc_not_zero(r))' failed. The gdb

[PATCH 27/27] perf test: Fix subtest number when showing results

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter Perf test 40 for example has several subtests numbered 1-4 when displaying the start of the subtest. When the subtest results are displayed the subtests are numbered 0-3. Use this command to generate trace output: [root@s35lp76 perf]# ./perf test -Fv 40 2>/tmp/bpf1 Fix

[PATCH 25/27] perf tools: Use perf_evsel__match instead of open coded equivalent

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Use perf_evsel__match() helper in perf_evsel__is_bpf_output(). Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: David Ahern Cc: Namhyung Kim Cc: Peter Zijlstra Link: http://lkml.kernel.org/r/20180720110036.32251-1-jo...@kernel.org Signed-off-by: Arnaldo Carvalho de Melo

[PATCH 26/27] perf stat: Get rid of extra clock display function

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa There's no reason to have separate function to display clock events. It's only purpose was to convert the nanosecond value into microseconds. We do that now in generic code, if the unit and scale values are properly set, which this patch do for clock events. The output differs

[PATCH 24/27] perf tools: Fix struct comm_str removal crash

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa We occasionaly hit following assert failure in 'perf top', when processing the /proc info in multiple threads. perf: ...include/linux/refcount.h:109: refcount_inc: Assertion `!(!refcount_inc_not_zero(r))' failed. The gdb backtrace looks like this: [Switching to

[PATCH 22/27] perf machine: Add threads__set_last_match function

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Separating threads::last_match cache set into separate threads__set_last_match function. This will be useful in following patch. Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: Andi Kleen Cc: David Ahern Cc: Kan Liang Cc: Lukasz Odzioba Cc: Namhyung Kim Cc: Peter

[PATCH 21/27] perf machine: Add threads__get_last_match function

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Separating threads::last_match cache read/check into separate threads__get_last_match function. This will be useful in following patch. Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: Andi Kleen Cc: David Ahern Cc: Kan Liang Cc: Lukasz Odzioba Cc: Namhyung Kim Cc:

[PATCH 22/27] perf machine: Add threads__set_last_match function

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Separating threads::last_match cache set into separate threads__set_last_match function. This will be useful in following patch. Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: Andi Kleen Cc: David Ahern Cc: Kan Liang Cc: Lukasz Odzioba Cc: Namhyung Kim Cc: Peter

[PATCH 21/27] perf machine: Add threads__get_last_match function

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Separating threads::last_match cache read/check into separate threads__get_last_match function. This will be useful in following patch. Signed-off-by: Jiri Olsa Cc: Alexander Shishkin Cc: Andi Kleen Cc: David Ahern Cc: Kan Liang Cc: Lukasz Odzioba Cc: Namhyung Kim Cc:

[PATCH 13/27] perf tests: Fix record+probe_libc_inet_pton.sh for powerpc64

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das For powerpc64, this test currently fails due to a mismatch in the expected output. This can be observed on a powerpc64le system running Fedora 27 as shown below. # perf test -v "probe libc's inet_pton & backtrace it with ping" Before: 62: probe libc's inet_pton &

[PATCH 18/27] perf trace arm64: Use generated syscall table

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Kim Phillips This should speed up accessing new system calls introduced with the kernel rather than waiting for libaudit updates to include them. It also enables users to specify wildcards, for example, perf trace -e 'open*', just like was already possible on x86, s390, and powerpc, which

[PATCH 13/27] perf tests: Fix record+probe_libc_inet_pton.sh for powerpc64

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das For powerpc64, this test currently fails due to a mismatch in the expected output. This can be observed on a powerpc64le system running Fedora 27 as shown below. # perf test -v "probe libc's inet_pton & backtrace it with ping" Before: 62: probe libc's inet_pton &

[PATCH 18/27] perf trace arm64: Use generated syscall table

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Kim Phillips This should speed up accessing new system calls introduced with the kernel rather than waiting for libaudit updates to include them. It also enables users to specify wildcards, for example, perf trace -e 'open*', just like was already possible on x86, s390, and powerpc, which

[PATCH 20/27] perf tools: Synthesize GROUP_DESC feature in pipe mode

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Stephan reported, that pipe mode does not carry the group information and thus the piped report won't display the grouped output for following command: # perf record -e '{cycles,instructions,branches}' -a sleep 4 | perf report It has no idea about the group setup, so it will

[PATCH 19/27] perf script: Show correct offsets for DWARF-based unwinding

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das When perf/data is recorded with the dwarf call-graph option, the callchain shown by 'perf script' still shows the binary offsets of the userspace symbols instead of their virtual addresses. Since the symbol offset calculation is based on using virtual address as the ip, we see

[PATCH 20/27] perf tools: Synthesize GROUP_DESC feature in pipe mode

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Jiri Olsa Stephan reported, that pipe mode does not carry the group information and thus the piped report won't display the grouped output for following command: # perf record -e '{cycles,instructions,branches}' -a sleep 4 | perf report It has no idea about the group setup, so it will

[PATCH 19/27] perf script: Show correct offsets for DWARF-based unwinding

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das When perf/data is recorded with the dwarf call-graph option, the callchain shown by 'perf script' still shows the binary offsets of the userspace symbols instead of their virtual addresses. Since the symbol offset calculation is based on using virtual address as the ip, we see

Re: [PATCH v6 10/18] x86/power/64: Remove VLA usage

2018-07-25 Thread Kees Cook
On Wed, Jul 25, 2018 at 4:32 AM, Rafael J. Wysocki wrote: > On Tue, Jul 24, 2018 at 6:49 PM, Kees Cook wrote: >> In the quest to remove all stack VLA usage from the kernel[1], this >> removes the discouraged use of AHASH_REQUEST_ON_STACK by switching to >> shash directly and allocating the

[PATCH 17/27] perf arm64: Generate system call table from asm/unistd.h

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Kim Phillips This should speed up accessing new system calls introduced with the kernel rather than waiting for libaudit updates to include them. Using the existing other arch scripts resulted in this error: tools/perf/arch/arm64/entry/syscalls//mksyscalltbl: 25: printf:

[PATCH 14/27] perf tests: Fix record+probe_libc_inet_pton.sh to ensure cleanups

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das If there is a mismatch in the perf script output, this test fails and exits before the event and temporary files created during its execution are cleaned up. This can be observed on a powerpc64 system running Fedora 27 as shown below. # perf test -v "probe libc's inet_pton

[PATCH 15/27] perf tests: Fix record+probe_libc_inet_pton.sh when event exists

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das If the event 'probe_libc:inet_pton' already exists, this test fails and deletes the existing event before exiting. This will then pass for any subsequent executions. Instead of skipping to deleting the existing event because of failing to add a new event, a duplicate event is

Re: [PATCH v6 10/18] x86/power/64: Remove VLA usage

2018-07-25 Thread Kees Cook
On Wed, Jul 25, 2018 at 4:32 AM, Rafael J. Wysocki wrote: > On Tue, Jul 24, 2018 at 6:49 PM, Kees Cook wrote: >> In the quest to remove all stack VLA usage from the kernel[1], this >> removes the discouraged use of AHASH_REQUEST_ON_STACK by switching to >> shash directly and allocating the

[PATCH 17/27] perf arm64: Generate system call table from asm/unistd.h

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Kim Phillips This should speed up accessing new system calls introduced with the kernel rather than waiting for libaudit updates to include them. Using the existing other arch scripts resulted in this error: tools/perf/arch/arm64/entry/syscalls//mksyscalltbl: 25: printf:

[PATCH 14/27] perf tests: Fix record+probe_libc_inet_pton.sh to ensure cleanups

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das If there is a mismatch in the perf script output, this test fails and exits before the event and temporary files created during its execution are cleaned up. This can be observed on a powerpc64 system running Fedora 27 as shown below. # perf test -v "probe libc's inet_pton

[PATCH 15/27] perf tests: Fix record+probe_libc_inet_pton.sh when event exists

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das If the event 'probe_libc:inet_pton' already exists, this test fails and deletes the existing event before exiting. This will then pass for any subsequent executions. Instead of skipping to deleting the existing event because of failing to add a new event, a duplicate event is

[PATCH 12/27] perf powerpc: Fix callchain ip filtering when return address is in a register

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das For powerpc64, perf will filter out the second entry in the callchain, i.e. the LR value, if the return address of the function corresponding to the probed location has already been saved on its caller's stack. The state of the return address is determined using debug

[PATCH 03/27] perf cs-etm: Introduce invalid address macro

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Leo Yan This patch introduces invalid address macro and uses it to replace dummy value '0xdeadbeefdeadbeefUL'. Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Kim Phillips Cc: Mike Leach Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Robert

[PATCH 06/27] perf list: Add s390 support for detailed PMU event description

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter Correct the support of detailed/verbose PMU event description by using the "Unit": keyword in the json files to address event names refering to the /sys/devices/cpum_[cs]f devices. Signed-off-by: Thomas Richter Reviewed-by: Hendrik Brueckner Cc: Heiko Carstens Cc: Martin

[PATCH 08/27] perf stat: Add transaction flag (-T) support for s390

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter The 'perf stat' command line flag -T to display transaction counters is currently supported for x86 only. Add support for s390. It is based on the metrics flag -M transaction using the architecture dependent JSON files. This requires a metric named "transaction" in the JSON

[PATCH 12/27] perf powerpc: Fix callchain ip filtering when return address is in a register

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das For powerpc64, perf will filter out the second entry in the callchain, i.e. the LR value, if the return address of the function corresponding to the probed location has already been saved on its caller's stack. The state of the return address is determined using debug

[PATCH 03/27] perf cs-etm: Introduce invalid address macro

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Leo Yan This patch introduces invalid address macro and uses it to replace dummy value '0xdeadbeefdeadbeefUL'. Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier Cc: Alexander Shishkin Cc: Jiri Olsa Cc: Kim Phillips Cc: Mike Leach Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Robert

[PATCH 06/27] perf list: Add s390 support for detailed PMU event description

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter Correct the support of detailed/verbose PMU event description by using the "Unit": keyword in the json files to address event names refering to the /sys/devices/cpum_[cs]f devices. Signed-off-by: Thomas Richter Reviewed-by: Hendrik Brueckner Cc: Heiko Carstens Cc: Martin

[PATCH 08/27] perf stat: Add transaction flag (-T) support for s390

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter The 'perf stat' command line flag -T to display transaction counters is currently supported for x86 only. Add support for s390. It is based on the metrics flag -M transaction using the architecture dependent JSON files. This requires a metric named "transaction" in the JSON

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

2018-07-25 Thread Arnaldo Carvalho de Melo
/git/acme/linux.git tags/perf-core-for-mingo-4.19-20180725 for you to fetch changes up to 9ef0112442bdddef5fb55adf20b3a5464b33de75: perf test: Fix subtest number when showing results (2018-07-24 14:55:51 -0300) perf/cores fixes

[PATCH 11/27] perf powerpc: Fix callchain ip filtering

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das For powerpc64, redundant entries in the callchain are filtered out by determining the state of the return address and the stack frame using DWARF debug information. For making these filtering decisions we must analyze the debug information for the location corresponding to

[PATCH 07/27] perf json: Add s390 transaction counter definition

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter 'perf stat' displays transactional counters using flag -T on x86. On s390 use a JSON file defined metric named transaction to achieve the same result. Output before: none Output after: [root@s35lp76 perf]# ./perf stat -M transaction -- \

[PATCH 10/27] perf list: Add missing documentation for --desc and --debug options

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sangwon Hong Add missing documentation for --desc and --debug options to the 'perf list' man page. Signed-off-by: Sangwon Hong Cc: Andi Kleen Cc: Jiri Olsa Cc: Namhyung Kim Link: http://lkml.kernel.org/r/20180717110738.10779-1-qpa...@gmail.com [ Clarify that --desc is by default

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

2018-07-25 Thread Arnaldo Carvalho de Melo
/git/acme/linux.git tags/perf-core-for-mingo-4.19-20180725 for you to fetch changes up to 9ef0112442bdddef5fb55adf20b3a5464b33de75: perf test: Fix subtest number when showing results (2018-07-24 14:55:51 -0300) perf/cores fixes

[PATCH 11/27] perf powerpc: Fix callchain ip filtering

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sandipan Das For powerpc64, redundant entries in the callchain are filtered out by determining the state of the return address and the stack frame using DWARF debug information. For making these filtering decisions we must analyze the debug information for the location corresponding to

[PATCH 07/27] perf json: Add s390 transaction counter definition

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter 'perf stat' displays transactional counters using flag -T on x86. On s390 use a JSON file defined metric named transaction to achieve the same result. Output before: none Output after: [root@s35lp76 perf]# ./perf stat -M transaction -- \

[PATCH 10/27] perf list: Add missing documentation for --desc and --debug options

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Sangwon Hong Add missing documentation for --desc and --debug options to the 'perf list' man page. Signed-off-by: Sangwon Hong Cc: Andi Kleen Cc: Jiri Olsa Cc: Namhyung Kim Link: http://lkml.kernel.org/r/20180717110738.10779-1-qpa...@gmail.com [ Clarify that --desc is by default

[PATCH 09/27] perf kvm: Fix subcommands on s390

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter With commit eca0fa28cd0d ("perf record: Provide detailed information on s390 CPU") s390 platform provides detailed type/model/capacity information in the CPU identifier string instead of just "IBM/S390". This breaks 'perf kvm' support which uses hard coded string IBM/S390

[PATCH 09/27] perf kvm: Fix subcommands on s390

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter With commit eca0fa28cd0d ("perf record: Provide detailed information on s390 CPU") s390 platform provides detailed type/model/capacity information in the CPU identifier string instead of just "IBM/S390". This breaks 'perf kvm' support which uses hard coded string IBM/S390

Re: [PATCH v2] tracefs: Annotate tracefs_ops with __ro_after_init

2018-07-25 Thread Kees Cook
On Wed, Jul 25, 2018 at 10:19 AM, Zubin Mithra wrote: > tracefs_ops is initialized inside tracefs_create_instance_dir and not > modified after. tracefs_create_instance_dir allows for initialization > only once, and is called from create_trace_instances(marked __init), > which is called from

[PATCH 05/27] Revert "perf list: Add s390 support for detailed/verbose PMU event description"

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter This reverts commit 038586c34301578e538f6c5aa79ca82bce1b9152. Fix the support of detailed/verbose PMU event description by using the "Unit": keyword in the json files to address event names refering to the /sys/devices/cpum_[cs]f devices. Signed-off-by: Thomas Richter

Re: [PATCH v2] tracefs: Annotate tracefs_ops with __ro_after_init

2018-07-25 Thread Kees Cook
On Wed, Jul 25, 2018 at 10:19 AM, Zubin Mithra wrote: > tracefs_ops is initialized inside tracefs_create_instance_dir and not > modified after. tracefs_create_instance_dir allows for initialization > only once, and is called from create_trace_instances(marked __init), > which is called from

[PATCH 05/27] Revert "perf list: Add s390 support for detailed/verbose PMU event description"

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Thomas Richter This reverts commit 038586c34301578e538f6c5aa79ca82bce1b9152. Fix the support of detailed/verbose PMU event description by using the "Unit": keyword in the json files to address event names refering to the /sys/devices/cpum_[cs]f devices. Signed-off-by: Thomas Richter

[PATCH 01/27] perf tests: Check that complex event name is parsed correctly

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Alexey Budankov Extend regression testing to cover case of complex event names enabled by the cset f92da71280fb ("perf record: Enable arbitrary event names thru name= modifier"). Testing it: # perf test 1: vmlinux symtab matches kallsyms : Skip 2: Detect

[PATCH 04/27] perf cs-etm: Bail out immediately for instruction sample failure

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Leo Yan If the instruction sample failure has happened, it isn't necessary to execute to the end of the function cs_etm__flush(). This commit is to bail out immediately and return the error code. Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier Cc: Alexander Shishkin Cc: Jiri Olsa

[PATCH 02/27] perf hists: Clarify callchain disabling when available

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We want to allow having mixed events with/without callchains, not using a global flag to show callchains, but allowing supressing callchains when they are present. So invert the logic of the last parameter to hists__fprint() to that effect. Cc: Adrian Hunter Cc:

[PATCH 01/27] perf tests: Check that complex event name is parsed correctly

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Alexey Budankov Extend regression testing to cover case of complex event names enabled by the cset f92da71280fb ("perf record: Enable arbitrary event names thru name= modifier"). Testing it: # perf test 1: vmlinux symtab matches kallsyms : Skip 2: Detect

[PATCH 04/27] perf cs-etm: Bail out immediately for instruction sample failure

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Leo Yan If the instruction sample failure has happened, it isn't necessary to execute to the end of the function cs_etm__flush(). This commit is to bail out immediately and return the error code. Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier Cc: Alexander Shishkin Cc: Jiri Olsa

[PATCH 02/27] perf hists: Clarify callchain disabling when available

2018-07-25 Thread Arnaldo Carvalho de Melo
From: Arnaldo Carvalho de Melo We want to allow having mixed events with/without callchains, not using a global flag to show callchains, but allowing supressing callchains when they are present. So invert the logic of the last parameter to hists__fprint() to that effect. Cc: Adrian Hunter Cc:

Re: [PATCH 4/5] memstick: rtsx_usb_ms: Support runtime power management

2018-07-25 Thread Alan Stern
On Wed, 25 Jul 2018, Kai-Heng Feng wrote: > In order to let host's parent device, rtsx_usb, to use USB remote wake > up signaling to do card detection, it needs to be suspended. Hence it's > necessary to add runtime PM support for the memstick host. > > To keep memstick host stays suspended when

[PATCH] media: omap2: omapfb: fix bugon.cocci warnings

2018-07-25 Thread kbuild test robot
From: kbuild test robot drivers/video/fbdev/omap2/omapfb/dss/dss_features.c:895:2-5: WARNING: Use BUG_ON instead of if condition followed by BUG. Please make sure the condition has no side effects (see conditional BUG_ON definition in include/asm-generic/bug.h) Use BUG_ON instead of a if

Re: [PATCH 4/5] memstick: rtsx_usb_ms: Support runtime power management

2018-07-25 Thread Alan Stern
On Wed, 25 Jul 2018, Kai-Heng Feng wrote: > In order to let host's parent device, rtsx_usb, to use USB remote wake > up signaling to do card detection, it needs to be suspended. Hence it's > necessary to add runtime PM support for the memstick host. > > To keep memstick host stays suspended when

[PATCH] media: omap2: omapfb: fix bugon.cocci warnings

2018-07-25 Thread kbuild test robot
From: kbuild test robot drivers/video/fbdev/omap2/omapfb/dss/dss_features.c:895:2-5: WARNING: Use BUG_ON instead of if condition followed by BUG. Please make sure the condition has no side effects (see conditional BUG_ON definition in include/asm-generic/bug.h) Use BUG_ON instead of a if

[PATCH] media: omap2: omapfb: fix boolreturn.cocci warnings

2018-07-25 Thread kbuild test robot
From: kbuild test robot drivers/video/fbdev/omap2/omapfb/omapfb-main.c:290:9-10: WARNING: return of 0/1 in function 'cmp_var_to_colormode' with return type bool Return statements in functions returning bool should use true/false instead of 1/0. Generated by:

[PATCH] media: omap2: omapfb: fix boolreturn.cocci warnings

2018-07-25 Thread kbuild test robot
From: kbuild test robot drivers/video/fbdev/omap2/omapfb/omapfb-main.c:290:9-10: WARNING: return of 0/1 in function 'cmp_var_to_colormode' with return type bool Return statements in functions returning bool should use true/false instead of 1/0. Generated by:

[PATCH] media: omap2: omapfb: fix ifnullfree.cocci warnings

2018-07-25 Thread kbuild test robot
From: kbuild test robot drivers/video/fbdev/omap2/omapfb/dss/core.c:141:2-26: WARNING: NULL check before some freeing functions is not needed. NULL check before some freeing functions is not needed. Based on checkpatch warning "kfree(NULL) is safe this check is probably not required" and

[PATCH] media: omap2: omapfb: fix ifnullfree.cocci warnings

2018-07-25 Thread kbuild test robot
From: kbuild test robot drivers/video/fbdev/omap2/omapfb/dss/core.c:141:2-26: WARNING: NULL check before some freeing functions is not needed. NULL check before some freeing functions is not needed. Based on checkpatch warning "kfree(NULL) is safe this check is probably not required" and

drivers/video/fbdev/omap2/omapfb/omapfb-main.c:290:9-10: WARNING: return of 0/1 in function 'cmp_var_to_colormode' with return type bool

2018-07-25 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 9981b4fb8684883dcc0daf088891ff32260b9794 commit: 7378f1149884b183631c6c16c0f1c62bcd7d759d media: omap2: omapfb: allow building it with COMPILE_TEST date: 3 months ago coccinelle warnings: (new ones

drivers/video/fbdev/omap2/omapfb/omapfb-main.c:290:9-10: WARNING: return of 0/1 in function 'cmp_var_to_colormode' with return type bool

2018-07-25 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 9981b4fb8684883dcc0daf088891ff32260b9794 commit: 7378f1149884b183631c6c16c0f1c62bcd7d759d media: omap2: omapfb: allow building it with COMPILE_TEST date: 3 months ago coccinelle warnings: (new ones

Re: [PATCH 2/5] memstick: Prevent memstick host from getting runtime suspended during card detection

2018-07-25 Thread Alan Stern
On Wed, 25 Jul 2018, Kai-Heng Feng wrote: > We can use MEMSTICK_POWER_{ON,OFF} along with pm_runtime_{get,put} > helpers to let memstick host support runtime pm. > > There's a small window between memstick_detect_change() and its queued > work, memstick_check(). In this window the rpm count may

Re: [PATCH 2/5] memstick: Prevent memstick host from getting runtime suspended during card detection

2018-07-25 Thread Alan Stern
On Wed, 25 Jul 2018, Kai-Heng Feng wrote: > We can use MEMSTICK_POWER_{ON,OFF} along with pm_runtime_{get,put} > helpers to let memstick host support runtime pm. > > There's a small window between memstick_detect_change() and its queued > work, memstick_check(). In this window the rpm count may

Re: [PATCH v2] RFC: clear 1G pages with streaming stores on x86

2018-07-25 Thread Cannon Matthews
On Wed, Jul 25, 2018 at 5:57 AM Michal Hocko wrote: > > [Cc Huang] > On Tue 24-07-18 19:37:28, Cannon Matthews wrote: > > Reimplement clear_gigantic_page() to clear gigabytes pages using the > > non-temporal streaming store instructions that bypass the cache > > (movnti), since an entire 1GiB

Re: [PATCH v2] RFC: clear 1G pages with streaming stores on x86

2018-07-25 Thread Cannon Matthews
On Wed, Jul 25, 2018 at 5:57 AM Michal Hocko wrote: > > [Cc Huang] > On Tue 24-07-18 19:37:28, Cannon Matthews wrote: > > Reimplement clear_gigantic_page() to clear gigabytes pages using the > > non-temporal streaming store instructions that bypass the cache > > (movnti), since an entire 1GiB

Re: [PATCH V1 2/3] Documentation: sdhci-msm: Add entries for passing load values

2018-07-25 Thread Rob Herring
On Fri, Jul 20, 2018 at 04:16:05PM +0530, Vijay Viswanath wrote: > The load a particular sdhc controller should request from a regulator > is device specific and hence each device should individually vote for > the required load. "dt-bindings: mmc: ..." for the subject. > > Signed-off-by: Vijay

Re: [PATCH V1 2/3] Documentation: sdhci-msm: Add entries for passing load values

2018-07-25 Thread Rob Herring
On Fri, Jul 20, 2018 at 04:16:05PM +0530, Vijay Viswanath wrote: > The load a particular sdhc controller should request from a regulator > is device specific and hence each device should individually vote for > the required load. "dt-bindings: mmc: ..." for the subject. > > Signed-off-by: Vijay

Re: [PATCH 3/6] irqchip: RISC-V Local Interrupt Controller Driver

2018-07-25 Thread Atish Patra
On 7/25/18 4:37 AM, Marc Zyngier wrote: On 25/07/18 12:24, Christoph Hellwig wrote: On Wed, Jul 25, 2018 at 12:18:39PM +0100, Marc Zyngier wrote: This feels odd. It means that you cannot have the following sequence: local_irq_disable(); enable_irq(x); // where x is owned by a

Re: [PATCH 3/6] irqchip: RISC-V Local Interrupt Controller Driver

2018-07-25 Thread Atish Patra
On 7/25/18 4:37 AM, Marc Zyngier wrote: On 25/07/18 12:24, Christoph Hellwig wrote: On Wed, Jul 25, 2018 at 12:18:39PM +0100, Marc Zyngier wrote: This feels odd. It means that you cannot have the following sequence: local_irq_disable(); enable_irq(x); // where x is owned by a

Re: [PATCH v2 2/3] dt-bindings: iio: adc: add bindings for mcp3911

2018-07-25 Thread Rob Herring
On Tue, Jul 24, 2018 at 08:30:03PM +0200, Marcus Folkesson wrote: > MCP3911 is a dual channel Analog Front End (AFE) containing two > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). > > Signed-off-by: Marcus Folkesson > Signed-off-by: Kent Gustavsson > --- > > Notes: >

Re: [PATCH v2 2/3] dt-bindings: iio: adc: add bindings for mcp3911

2018-07-25 Thread Rob Herring
On Tue, Jul 24, 2018 at 08:30:03PM +0200, Marcus Folkesson wrote: > MCP3911 is a dual channel Analog Front End (AFE) containing two > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). > > Signed-off-by: Marcus Folkesson > Signed-off-by: Kent Gustavsson > --- > > Notes: >

[PATCH v0 2/4] drivers: soc: Support to add cache erp driver for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
Cache error reporting controller is to detect and report single and double bit errors on Last Level Cache Controller (LLCC) cache. Add required support to register cache erp driver as platform driver, from LLCC driver. Signed-off-by: Channagoud Kadabi Signed-off-by: Venkata Narendra Kumar Gutta

[PATCH v0 2/4] drivers: soc: Support to add cache erp driver for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
Cache error reporting controller is to detect and report single and double bit errors on Last Level Cache Controller (LLCC) cache. Add required support to register cache erp driver as platform driver, from LLCC driver. Signed-off-by: Channagoud Kadabi Signed-off-by: Venkata Narendra Kumar Gutta

[PATCH v0 4/4] dt-bindigs: Update documentation of qcom,llcc

2018-07-25 Thread Venkata Narendra Kumar Gutta
Add reg-names and interrupts for LLCC documentation and the usage examples. llcc broadcast base is added in addition to llcc base, which is used for llcc broadcast writes. Signed-off-by: Venkata Narendra Kumar Gutta --- Documentation/devicetree/bindings/arm/msm/qcom,llcc.txt | 15

[PATCH v0 3/4] drivers: edac: Add cache erp driver for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
Add cache error reporting driver for single and double bit errors on Last Level Cache Controller (LLCC) cache. This driver takes care of dumping registers and add config options to enable and disable panic when these errors happen. Signed-off-by: Channagoud Kadabi Signed-off-by: Venkata Narendra

[PATCH v0 4/4] dt-bindigs: Update documentation of qcom,llcc

2018-07-25 Thread Venkata Narendra Kumar Gutta
Add reg-names and interrupts for LLCC documentation and the usage examples. llcc broadcast base is added in addition to llcc base, which is used for llcc broadcast writes. Signed-off-by: Venkata Narendra Kumar Gutta --- Documentation/devicetree/bindings/arm/msm/qcom,llcc.txt | 15

[PATCH v0 3/4] drivers: edac: Add cache erp driver for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
Add cache error reporting driver for single and double bit errors on Last Level Cache Controller (LLCC) cache. This driver takes care of dumping registers and add config options to enable and disable panic when these errors happen. Signed-off-by: Channagoud Kadabi Signed-off-by: Venkata Narendra

[PATCH v0 1/4] drivers: soc: Add broadcast base for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
Currently, boradcast base is set to end of the LLCC banks, which may not be correct always. As the number of banks may vary for each chipset and the broadcast base could be at a different address as well. This info depends on the chipset, so get the broadcast base info from the device tree (DT).

[PATCH v0 1/4] drivers: soc: Add broadcast base for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
Currently, boradcast base is set to end of the LLCC banks, which may not be correct always. As the number of banks may vary for each chipset and the broadcast base could be at a different address as well. This info depends on the chipset, so get the broadcast base info from the device tree (DT).

[PATCH v0 0/4] Add cache erp driver for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
This series implements cache erp driver for Last Level Cache Controller (LLCC). Cache erp driver is to detect and report single and double bit errors on Last Level Cache Controller (LLCC) cache. This driver also takes care of dumping registers and have config options to enable and disable panic

[PATCH v0 0/4] Add cache erp driver for Last Level Cache Controller (LLCC)

2018-07-25 Thread Venkata Narendra Kumar Gutta
This series implements cache erp driver for Last Level Cache Controller (LLCC). Cache erp driver is to detect and report single and double bit errors on Last Level Cache Controller (LLCC) cache. This driver also takes care of dumping registers and have config options to enable and disable panic

Re: no public email address excluding Gmail

2018-07-25 Thread Theodore Y. Ts'o
On Wed, Jul 25, 2018 at 03:25:07PM +0800, 张宁 wrote: > Hi, everyone > > I notice each developer in this email list has a company email address > or Gmail, or maybe selfhosted email. > > last night I have tried my outlook.com, yahoo.com, QQ.com email, all > of them are rejected by mechine. There

Re: no public email address excluding Gmail

2018-07-25 Thread Theodore Y. Ts'o
On Wed, Jul 25, 2018 at 03:25:07PM +0800, 张宁 wrote: > Hi, everyone > > I notice each developer in this email list has a company email address > or Gmail, or maybe selfhosted email. > > last night I have tried my outlook.com, yahoo.com, QQ.com email, all > of them are rejected by mechine. There

Re: [PATCH v2 4/5] dt-bindings: drm/bridge Document Cadence MHDP DPI/DP bridge bindings

2018-07-25 Thread Rob Herring
On Tue, Jul 24, 2018 at 12:13:35PM +0100, Damian Kos wrote: > From: Quentin Schulz > > Signed-off-by: Damian Kos > --- > .../bindings/display/bridge/cdns,mhdp.txt | 43 +++ > 1 file changed, 43 insertions(+) > create mode 100644 >

Re: [PATCH v2 4/5] dt-bindings: drm/bridge Document Cadence MHDP DPI/DP bridge bindings

2018-07-25 Thread Rob Herring
On Tue, Jul 24, 2018 at 12:13:35PM +0100, Damian Kos wrote: > From: Quentin Schulz > > Signed-off-by: Damian Kos > --- > .../bindings/display/bridge/cdns,mhdp.txt | 43 +++ > 1 file changed, 43 insertions(+) > create mode 100644 >

Re: [RFT v2 04/10] pinctrl: samsung: Add dedicated compatible for S5Pv210 wakeup interrupts

2018-07-25 Thread Rob Herring
On Mon, Jul 23, 2018 at 07:52:56PM +0200, Krzysztof Kozlowski wrote: > The S5Pv210 external wakeup interrupts differ from Exynos therefore > separate compatible is needed. Duplicate existing flavor specific data > from exynos4210_wkup_irq_chip and add new compatible for S5Pv210. > At this point

Re: [RFT v2 04/10] pinctrl: samsung: Add dedicated compatible for S5Pv210 wakeup interrupts

2018-07-25 Thread Rob Herring
On Mon, Jul 23, 2018 at 07:52:56PM +0200, Krzysztof Kozlowski wrote: > The S5Pv210 external wakeup interrupts differ from Exynos therefore > separate compatible is needed. Duplicate existing flavor specific data > from exynos4210_wkup_irq_chip and add new compatible for S5Pv210. > At this point

Re: [RFT v2 03/10] pinctrl: samsung: Document hidden requirement about one external wakeup

2018-07-25 Thread Rob Herring
On Mon, Jul 23, 2018 at 07:52:55PM +0200, Krzysztof Kozlowski wrote: > Hardware (S5Pv210 and all Exynos SoCs) provides only 32 external > interrupts which can wakeup device from deep sleep modes. On S5Pv210 > these are gph0-gph3. On all Exynos designs these are gpx0-gpx3. > There is only one

Re: [RFT v2 03/10] pinctrl: samsung: Document hidden requirement about one external wakeup

2018-07-25 Thread Rob Herring
On Mon, Jul 23, 2018 at 07:52:55PM +0200, Krzysztof Kozlowski wrote: > Hardware (S5Pv210 and all Exynos SoCs) provides only 32 external > interrupts which can wakeup device from deep sleep modes. On S5Pv210 > these are gph0-gph3. On all Exynos designs these are gpx0-gpx3. > There is only one

Re: [PATCH RFC/RFT net-next 00/17] net: Convert neighbor tables to per-namespace

2018-07-25 Thread Eric W. Biederman
David Ahern writes: > On 7/25/18 6:33 AM, Eric W. Biederman wrote: >> Cong Wang writes: >> >>> On Tue, Jul 24, 2018 at 8:14 AM David Ahern wrote: On 7/19/18 11:12 AM, Cong Wang wrote: > On Thu, Jul 19, 2018 at 9:16 AM David Ahern wrote: >> >> Chatting with Nikolay about

Re: [PATCH RFC/RFT net-next 00/17] net: Convert neighbor tables to per-namespace

2018-07-25 Thread Eric W. Biederman
David Ahern writes: > On 7/25/18 6:33 AM, Eric W. Biederman wrote: >> Cong Wang writes: >> >>> On Tue, Jul 24, 2018 at 8:14 AM David Ahern wrote: On 7/19/18 11:12 AM, Cong Wang wrote: > On Thu, Jul 19, 2018 at 9:16 AM David Ahern wrote: >> >> Chatting with Nikolay about

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