[PATCH v1 18/20] signal: make legacy_queue() return bool

2018-05-28 Thread Christian Brauner
legacy_queue() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/signal.c b/kernel/signal.c index

[PATCH v1 08/20] signal: simplify rt_sigaction()

2018-05-28 Thread Christian Brauner
The goto is not needed and does not add any clarity. Simply return -EINVAL on unexpected sigset_t struct size directly. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git

[PATCH v1 11/20] signal: make sig_task_ignored() return bool

2018-05-28 Thread Christian Brauner
sig_task_ignored() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c index

[PATCH v1 17/20] signal: make wants_signal() return bool

2018-05-28 Thread Christian Brauner
wants_signal() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

[PATCH v1 10/20] signal: make sig_handler_ignored() return bool

2018-05-28 Thread Christian Brauner
sig_handler_ignored() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

[PATCH v1 02/20] signal: make kill_as_cred_perm() return bool

2018-05-28 Thread Christian Brauner
kill_as_cred_perm() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * simplify to use a single return --- kernel/signal.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git

[PATCH v1 10/20] signal: make sig_handler_ignored() return bool

2018-05-28 Thread Christian Brauner
sig_handler_ignored() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

[PATCH v1 02/20] signal: make kill_as_cred_perm() return bool

2018-05-28 Thread Christian Brauner
kill_as_cred_perm() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * simplify to use a single return --- kernel/signal.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git

[PATCH v1 20/20] signal: make get_signal() return bool

2018-05-28 Thread Christian Brauner
make get_signal() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- include/linux/signal.h | 2 +- kernel/signal.c| 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH v1 20/20] signal: make get_signal() return bool

2018-05-28 Thread Christian Brauner
make get_signal() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- include/linux/signal.h | 2 +- kernel/signal.c| 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH v1 09/20] signal: make kill_ok_by_cred() return bool

2018-05-28 Thread Christian Brauner
kill_ok_by_cred() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/kernel/signal.c

[PATCH v1 09/20] signal: make kill_ok_by_cred() return bool

2018-05-28 Thread Christian Brauner
kill_ok_by_cred() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/kernel/signal.c

Re: [PATCH] powerpc/64: Fix build failure with GCC 8.1

2018-05-28 Thread Benjamin Herrenschmidt
On Mon, 2018-05-28 at 16:37 +, Christophe Leroy wrote: > CC arch/powerpc/kernel/nvram_64.o > arch/powerpc/kernel/nvram_64.c: In function 'nvram_create_partition': > arch/powerpc/kernel/nvram_64.c:1042:2: error: 'strncpy' specified bound 12 > equals destination size

Re: [PATCH] powerpc/64: Fix build failure with GCC 8.1

2018-05-28 Thread Benjamin Herrenschmidt
On Mon, 2018-05-28 at 16:37 +, Christophe Leroy wrote: > CC arch/powerpc/kernel/nvram_64.o > arch/powerpc/kernel/nvram_64.c: In function 'nvram_create_partition': > arch/powerpc/kernel/nvram_64.c:1042:2: error: 'strncpy' specified bound 12 > equals destination size

[PATCH v1 15/20] signal: make unhandled_signal() return bool

2018-05-28 Thread Christian Brauner
unhandled_signal() already behaves like a boolean function. Let's actually declare it as such too. All callers treat it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- include/linux/signal.h | 2 +- kernel/signal.c| 11 +++ 2 files changed, 8

[PATCH v1 14/20] signal: make recalc_sigpending_tsk() return bool

2018-05-28 Thread Christian Brauner
recalc_sigpending_tsk() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/kernel/signal.c

[PATCH v1 15/20] signal: make unhandled_signal() return bool

2018-05-28 Thread Christian Brauner
unhandled_signal() already behaves like a boolean function. Let's actually declare it as such too. All callers treat it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- include/linux/signal.h | 2 +- kernel/signal.c| 11 +++ 2 files changed, 8

[PATCH v1 14/20] signal: make recalc_sigpending_tsk() return bool

2018-05-28 Thread Christian Brauner
recalc_sigpending_tsk() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/kernel/signal.c

Re: [RFT v3 0/4] Perf script: Add python script for CoreSight trace disassembler

2018-05-28 Thread Mathieu Poirier
On 28 May 2018 at 14:03, Arnaldo Carvalho de Melo wrote: > Em Mon, May 28, 2018 at 04:44:59PM +0800, Leo Yan escreveu: >> This patch series is to support for using 'perf script' for CoreSight >> trace disassembler, for this purpose this patch series adds a new >> python script to parse CoreSight

Re: [RFT v3 0/4] Perf script: Add python script for CoreSight trace disassembler

2018-05-28 Thread Mathieu Poirier
On 28 May 2018 at 14:03, Arnaldo Carvalho de Melo wrote: > Em Mon, May 28, 2018 at 04:44:59PM +0800, Leo Yan escreveu: >> This patch series is to support for using 'perf script' for CoreSight >> trace disassembler, for this purpose this patch series adds a new >> python script to parse CoreSight

[PATCH V3 2/2] scsi: hpsa: drop shutdown callback

2018-05-28 Thread Sinan Kaya
'Commit cc27b735ad3a ("PCI/portdrv: Turn off PCIe services during shutdown")' has been added to kernel to shutdown pending PCIe port service interrupts during reboot so that a newly started kexec kernel wouldn't observe pending interrupts. pcie_port_device_remove() is disabling the root port and

Re: [PATCH V2] PCI/portdrv: do not disable device on reboot/shutdown

2018-05-28 Thread Sinan Kaya
On 5/24/2018 6:37 AM, Don Brace wrote: >> But we really need guidance from an hpsa expert. I don't know the >> filesystem/SCSI/hpsa details. >> >> Bjorn > It's most likely OCSD traffic that will stop when bus mastering is turned off. > So, I'll run some tests on my end before ACKing your patch.

[PATCH V3 2/2] scsi: hpsa: drop shutdown callback

2018-05-28 Thread Sinan Kaya
'Commit cc27b735ad3a ("PCI/portdrv: Turn off PCIe services during shutdown")' has been added to kernel to shutdown pending PCIe port service interrupts during reboot so that a newly started kexec kernel wouldn't observe pending interrupts. pcie_port_device_remove() is disabling the root port and

Re: [PATCH V2] PCI/portdrv: do not disable device on reboot/shutdown

2018-05-28 Thread Sinan Kaya
On 5/24/2018 6:37 AM, Don Brace wrote: >> But we really need guidance from an hpsa expert. I don't know the >> filesystem/SCSI/hpsa details. >> >> Bjorn > It's most likely OCSD traffic that will stop when bus mastering is turned off. > So, I'll run some tests on my end before ACKing your patch.

Re: [PATCH] HID: Add support for 146b:0902 Bigben Interactive Kids' gamepad

2018-05-28 Thread Roderick Colenbrander
Hi Hanno, A few suggestions for your patch based on a quick review (had limited time). In terms of code style, I noticed a lot of C++ style comments which are not part of the kernel code style instead use C comments. To check for potential other issues as well, run your patch through

Re: [PATCH] HID: Add support for 146b:0902 Bigben Interactive Kids' gamepad

2018-05-28 Thread Roderick Colenbrander
Hi Hanno, A few suggestions for your patch based on a quick review (had limited time). In terms of code style, I noticed a lot of C++ style comments which are not part of the kernel code style instead use C comments. To check for potential other issues as well, run your patch through

Re: [PATCH 4.16 000/272] 4.16.13-stable review

2018-05-28 Thread Davidlohr Bueso
On Tue, 29 May 2018, Naresh Kamboju wrote: On 28 May 2018 at 15:30, Greg Kroah-Hartman wrote: On all devices, ltp-syscalls-tests: * cve-2017-5669 ltp-cve-tests: * cve-2017-5669 Test log: cve-2017-5669.c:62: INFO: Attempting to attach shared memory to null page

Re: [PATCH 4.16 000/272] 4.16.13-stable review

2018-05-28 Thread Davidlohr Bueso
On Tue, 29 May 2018, Naresh Kamboju wrote: On 28 May 2018 at 15:30, Greg Kroah-Hartman wrote: On all devices, ltp-syscalls-tests: * cve-2017-5669 ltp-cve-tests: * cve-2017-5669 Test log: cve-2017-5669.c:62: INFO: Attempting to attach shared memory to null page

Re: Linux and autoconfig of kernel on iso-image boot.

2018-05-28 Thread Gabriel C
2018-05-28 22:40 GMT+02:00 Phillip Hugo : > Hi all, thanks for your work. > > Linux should have "make grab_config" that sets loaded modules to be > built in the kernel build and make modules. > make localmodconfig ?

Re: Linux and autoconfig of kernel on iso-image boot.

2018-05-28 Thread Gabriel C
2018-05-28 22:40 GMT+02:00 Phillip Hugo : > Hi all, thanks for your work. > > Linux should have "make grab_config" that sets loaded modules to be > built in the kernel build and make modules. > make localmodconfig ?

[PATCH v1 07/20] signal: make do_sigpending() void

2018-05-28 Thread Christian Brauner
do_sigpending() returned 0 unconditionally so it doesn't make sense to have it return at all. This allows us to simplify a bunch of syscall callers. Signed-off-by: Christian Brauner Acked-by: Al Viro --- v0->v1: * added Acked-By line from Al --- kernel/signal.c | 41

[PATCH v1 01/20] signal: make force_sigsegv() void

2018-05-28 Thread Christian Brauner
force_sigsegv() returned 0 unconditionally so it doesn't make sense to have it return at all. In addition, there are no callers that check force_sigsegv()'s return value. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- include/linux/sched/signal.h | 2 +- kernel/signal.c

[PATCH v1 07/20] signal: make do_sigpending() void

2018-05-28 Thread Christian Brauner
do_sigpending() returned 0 unconditionally so it doesn't make sense to have it return at all. This allows us to simplify a bunch of syscall callers. Signed-off-by: Christian Brauner Acked-by: Al Viro --- v0->v1: * added Acked-By line from Al --- kernel/signal.c | 41

[PATCH v1 01/20] signal: make force_sigsegv() void

2018-05-28 Thread Christian Brauner
force_sigsegv() returned 0 unconditionally so it doesn't make sense to have it return at all. In addition, there are no callers that check force_sigsegv()'s return value. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- include/linux/sched/signal.h | 2 +- kernel/signal.c

Re: [RFT v3 1/4] perf cs-etm: Generate branch sample for missed packets

2018-05-28 Thread Mathieu Poirier
Leo and/or Robert, On Mon, May 28, 2018 at 04:45:00PM +0800, Leo Yan wrote: > Commit e573e978fb12 ("perf cs-etm: Inject capabilitity for CoreSight > traces") reworks the samples generation flow from CoreSight trace to > match the correct format so Perf report tool can display the samples >

[PATCH v1 06/20] signal: drop else branch in do_signal_stop()

2018-05-28 Thread Christian Brauner
do_signal_stop() already returns in the if branch so there's no need to keep the else branch around. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/kernel/signal.c

Re: [RFT v3 1/4] perf cs-etm: Generate branch sample for missed packets

2018-05-28 Thread Mathieu Poirier
Leo and/or Robert, On Mon, May 28, 2018 at 04:45:00PM +0800, Leo Yan wrote: > Commit e573e978fb12 ("perf cs-etm: Inject capabilitity for CoreSight > traces") reworks the samples generation flow from CoreSight trace to > match the correct format so Perf report tool can display the samples >

[PATCH v1 06/20] signal: drop else branch in do_signal_stop()

2018-05-28 Thread Christian Brauner
do_signal_stop() already returns in the if branch so there's no need to keep the else branch around. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/kernel/signal.c

[PATCH v1 12/20] signal: make sig_ignored() return bool

2018-05-28 Thread Christian Brauner
sig_ignored() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c index

[PATCH v1 13/20] signal: make has_pending_signals() return bool

2018-05-28 Thread Christian Brauner
has_pending_signals() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git

[PATCH v1 12/20] signal: make sig_ignored() return bool

2018-05-28 Thread Christian Brauner
sig_ignored() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c index

[PATCH v1 13/20] signal: make has_pending_signals() return bool

2018-05-28 Thread Christian Brauner
has_pending_signals() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git

[PATCH v1 19/20] signal: make security_task_kill() return bool

2018-05-28 Thread Christian Brauner
security_task_kill() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

[PATCH v1 19/20] signal: make security_task_kill() return bool

2018-05-28 Thread Christian Brauner
security_task_kill() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch introduced --- kernel/signal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

LKMM litmus test for Roman Penyaev's rcu-rr

2018-05-28 Thread Paul E. McKenney
Hello! The litmus test below is a first attempt to model Roman's rcu-rr round-robin RCU-protected linked list. His test code, which includes the algorithm under test, may be found here: https://github.com/rouming/rcu-rr/blob/master/rcu-rr.c The P0() process below roughly corresponds to

LKMM litmus test for Roman Penyaev's rcu-rr

2018-05-28 Thread Paul E. McKenney
Hello! The litmus test below is a first attempt to model Roman's rcu-rr round-robin RCU-protected linked list. His test code, which includes the algorithm under test, may be found here: https://github.com/rouming/rcu-rr/blob/master/rcu-rr.c The P0() process below roughly corresponds to

Re: [PATCH] doc: document scope NOFS, NOIO APIs

2018-05-28 Thread Dave Chinner
On Mon, May 28, 2018 at 11:19:23AM +0200, Michal Hocko wrote: > On Mon 28-05-18 09:48:54, Dave Chinner wrote: > > On Fri, May 25, 2018 at 10:16:24AM +0200, Michal Hocko wrote: > > > On Fri 25-05-18 08:17:15, Dave Chinner wrote: > > > > On Thu, May 24, 2018 at 01:43:41PM +0200, Michal Hocko wrote:

Re: [PATCH] iptables-compat: homogenize error message

2018-05-28 Thread Pablo Neira Ayuso
On Sun, May 27, 2018 at 06:39:01PM +0530, Arushi Singhal wrote: > There is a difference between error messages in iptables and > iptables-compat: > > #sudo iptables-compat -D INPUT 4 > iptables: No chain/target/match by that name. > > #sudo iptables -D INPUT 4 > iptables: Index of deletion too

Re: [PATCH] doc: document scope NOFS, NOIO APIs

2018-05-28 Thread Dave Chinner
On Mon, May 28, 2018 at 11:19:23AM +0200, Michal Hocko wrote: > On Mon 28-05-18 09:48:54, Dave Chinner wrote: > > On Fri, May 25, 2018 at 10:16:24AM +0200, Michal Hocko wrote: > > > On Fri 25-05-18 08:17:15, Dave Chinner wrote: > > > > On Thu, May 24, 2018 at 01:43:41PM +0200, Michal Hocko wrote:

Re: [PATCH] iptables-compat: homogenize error message

2018-05-28 Thread Pablo Neira Ayuso
On Sun, May 27, 2018 at 06:39:01PM +0530, Arushi Singhal wrote: > There is a difference between error messages in iptables and > iptables-compat: > > #sudo iptables-compat -D INPUT 4 > iptables: No chain/target/match by that name. > > #sudo iptables -D INPUT 4 > iptables: Index of deletion too

[PATCH v1 05/20] signal: flatten do_send_sig_info()

2018-05-28 Thread Christian Brauner
Let's return early when lock_task_sighand() fails and move send_signal() and unlock_task_sighand() out of the if block. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH v1 03/20] signal: make may_ptrace_stop() return bool

2018-05-28 Thread Christian Brauner
may_ptrace_stop() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

[PATCH v1 00/20] signal: refactor some functions

2018-05-28 Thread Christian Brauner
Hey, This series refactors a bunch of functions in signal.c to simplify parts of the code. The greatest single change is declaring the static do_sigpending() helper as void which makes it possible to remove a bunch of unnecessary checks in the syscalls later on. Thanks! Christian Christian

[PATCH v1 05/20] signal: flatten do_send_sig_info()

2018-05-28 Thread Christian Brauner
Let's return early when lock_task_sighand() fails and move send_signal() and unlock_task_sighand() out of the if block. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[PATCH v1 03/20] signal: make may_ptrace_stop() return bool

2018-05-28 Thread Christian Brauner
may_ptrace_stop() already behaves like a boolean function. Let's actually declare it as such too. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/kernel/signal.c b/kernel/signal.c

[PATCH v1 00/20] signal: refactor some functions

2018-05-28 Thread Christian Brauner
Hey, This series refactors a bunch of functions in signal.c to simplify parts of the code. The greatest single change is declaring the static do_sigpending() helper as void which makes it possible to remove a bunch of unnecessary checks in the syscalls later on. Thanks! Christian Christian

[PATCH v1 04/20] signal: add copy_pending() helper

2018-05-28 Thread Christian Brauner
Instead of using a goto for this let's add a simple helper copy_pending() which can be called in both places. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 54 +++-- 1 file changed, 30 insertions(+), 24

[PATCH v1 04/20] signal: add copy_pending() helper

2018-05-28 Thread Christian Brauner
Instead of using a goto for this let's add a simple helper copy_pending() which can be called in both places. Signed-off-by: Christian Brauner --- v0->v1: * patch unchanged --- kernel/signal.c | 54 +++-- 1 file changed, 30 insertions(+), 24

Proposal

2018-05-28 Thread Miss Zeliha Omer Faruk
-- Hello I have been trying to contact you. Did you get my business proposal? Best Regards, Miss.Zeliha ömer faruk Esentepe Mahallesi Büyükdere Caddesi Kristal Kule Binasi No:215 Sisli - Istanbul, Turke

Proposal

2018-05-28 Thread Miss Zeliha Omer Faruk
-- Hello I have been trying to contact you. Did you get my business proposal? Best Regards, Miss.Zeliha ömer faruk Esentepe Mahallesi Büyükdere Caddesi Kristal Kule Binasi No:215 Sisli - Istanbul, Turke

Re: [PATCH v3] vfio: platform: Fix using devices in PM Domains

2018-05-28 Thread Geert Uytterhoeven
Hi Eric, On Mon, May 28, 2018 at 10:27 PM, Auger Eric wrote: > On 05/28/2018 05:26 PM, Geert Uytterhoeven wrote: >> If a device is part of a PM Domain (e.g. power and/or clock domain), its >> power state is managed using Runtime PM. Without Runtime PM, the device >> may not be powered up or

Re: [PATCH v3] vfio: platform: Fix using devices in PM Domains

2018-05-28 Thread Geert Uytterhoeven
Hi Eric, On Mon, May 28, 2018 at 10:27 PM, Auger Eric wrote: > On 05/28/2018 05:26 PM, Geert Uytterhoeven wrote: >> If a device is part of a PM Domain (e.g. power and/or clock domain), its >> power state is managed using Runtime PM. Without Runtime PM, the device >> may not be powered up or

Re: [PATCH] perf docu: Update section on cpu topology

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 09:44:33AM +0200, Thomas Richter escreveu: > Add an explanation of each cpu's core and socket > identifier to the documentation. Thanks, applying. I guess it is not that worth to mention that older files may have just the string lists, right? - Arnaldo > Signed-off-by:

Re: [PATCH] perf docu: Update section on cpu topology

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 09:44:33AM +0200, Thomas Richter escreveu: > Add an explanation of each cpu's core and socket > identifier to the documentation. Thanks, applying. I guess it is not that worth to mention that older files may have just the string lists, right? - Arnaldo > Signed-off-by:

Re: [PATCH] perf docu: Update section on cpu description

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 09:49:03AM +0200, Thomas Richter escreveu: > Fix the section HEADER_CPUDESC so the documentation > matches code. Swap the two members of struct nr_cpus. > > Signed-off-by: Thomas Richter I have this one already in my perf/core branch:

Re: [PATCH] perf docu: Update section on cpu description

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 09:49:03AM +0200, Thomas Richter escreveu: > Fix the section HEADER_CPUDESC so the documentation > matches code. Swap the two members of struct nr_cpus. > > Signed-off-by: Thomas Richter I have this one already in my perf/core branch:

Re: [PATCH v2] sched: Remove obscure comment from select_task_rq_fair

2018-05-28 Thread Joel Fernandes
On Mon, May 28, 2018 at 09:15:08AM +0200, Peter Zijlstra wrote: > On Sun, May 27, 2018 at 06:24:47PM -0700, Joel Fernandes (Google) wrote: > > This patch removes the obscure comment which was > > saying this path can be entered only for wake-balance. > > The comment was meant to ask if we want

Re: [PATCH v2] sched: Remove obscure comment from select_task_rq_fair

2018-05-28 Thread Joel Fernandes
On Mon, May 28, 2018 at 09:15:08AM +0200, Peter Zijlstra wrote: > On Sun, May 27, 2018 at 06:24:47PM -0700, Joel Fernandes (Google) wrote: > > This patch removes the obscure comment which was > > saying this path can be entered only for wake-balance. > > The comment was meant to ask if we want

Re: [RFT v3 0/4] Perf script: Add python script for CoreSight trace disassembler

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 04:44:59PM +0800, Leo Yan escreveu: > This patch series is to support for using 'perf script' for CoreSight > trace disassembler, for this purpose this patch series adds a new > python script to parse CoreSight tracing event and use command 'objdump' > for disassembled

Re: [RFT v3 0/4] Perf script: Add python script for CoreSight trace disassembler

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 04:44:59PM +0800, Leo Yan escreveu: > This patch series is to support for using 'perf script' for CoreSight > trace disassembler, for this purpose this patch series adds a new > python script to parse CoreSight tracing event and use command 'objdump' > for disassembled

Re: [PATCH v3 1/2] rtmutex: allow specifying a subclass for nested locking

2018-05-28 Thread Joel Fernandes
On Mon, May 28, 2018 at 09:17:51AM +0200, Peter Zijlstra wrote: > On Sun, May 27, 2018 at 10:19:36PM -0700, Joel Fernandes wrote: > > > > +static inline void __rt_mutex_lock(struct rt_mutex *lock, unsigned int > > > subclass) > > > +{ > > > + might_sleep(); > > > + > > > +

Re: [PATCH v3 1/2] rtmutex: allow specifying a subclass for nested locking

2018-05-28 Thread Joel Fernandes
On Mon, May 28, 2018 at 09:17:51AM +0200, Peter Zijlstra wrote: > On Sun, May 27, 2018 at 10:19:36PM -0700, Joel Fernandes wrote: > > > > +static inline void __rt_mutex_lock(struct rt_mutex *lock, unsigned int > > > subclass) > > > +{ > > > + might_sleep(); > > > + > > > +

Re: [PATCH 2/3] x86/mce: Fix incorrect "Machine check from unknown source" message

2018-05-28 Thread Borislav Petkov
On Fri, May 25, 2018 at 02:41:55PM -0700, Tony Luck wrote: > @@ -1287,12 +1292,17 @@ void do_machine_check(struct pt_regs *regs, long > error_code) > no_way_out = worst >= MCE_PANIC_SEVERITY; > } else { > /* > - * Local MCE skipped calling

Re: [PATCH 2/3] x86/mce: Fix incorrect "Machine check from unknown source" message

2018-05-28 Thread Borislav Petkov
On Fri, May 25, 2018 at 02:41:55PM -0700, Tony Luck wrote: > @@ -1287,12 +1292,17 @@ void do_machine_check(struct pt_regs *regs, long > error_code) > no_way_out = worst >= MCE_PANIC_SEVERITY; > } else { > /* > - * Local MCE skipped calling

Proposal

2018-05-28 Thread Miss Zeliha Omer Faruk
-- Hello I have been trying to contact you. Did you get my business proposal? Best Regards, Miss.Zeliha ömer faruk Esentepe Mahallesi Büyükdere Caddesi Kristal Kule Binasi No:215 Sisli - Istanbul, Turke

Proposal

2018-05-28 Thread Miss Zeliha Omer Faruk
-- Hello I have been trying to contact you. Did you get my business proposal? Best Regards, Miss.Zeliha ömer faruk Esentepe Mahallesi Büyükdere Caddesi Kristal Kule Binasi No:215 Sisli - Istanbul, Turke

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 11:13:59AM +0800, Leo Yan escreveu: > On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: > > The tail of a queue is supposed to be pointing to the next available slot > > in a queue. In this implementation the tail is incremented before it is > > used and as

Re: [PATCH] perf tools: Fix indexing for decoder packet queue

2018-05-28 Thread Arnaldo Carvalho de Melo
Em Mon, May 28, 2018 at 11:13:59AM +0800, Leo Yan escreveu: > On Fri, May 25, 2018 at 05:10:54PM -0600, Mathieu Poirier wrote: > > The tail of a queue is supposed to be pointing to the next available slot > > in a queue. In this implementation the tail is incremented before it is > > used and as

[PATCH v3] mailbox: arm_mhu: add support for mhuv2

2018-05-28 Thread Samarth Parikh
ARM has launched a next version of MHU i.e. MHUv2 with its latest subsystems. The main change is that the MHUv2 is now a distributed IP with different peripheral views (registers) for the sender and receiver. Also, the peripheral views (registers) for Secure & Non-Secure have been separated out

[PATCH v3] mailbox: arm_mhu: add support for mhuv2

2018-05-28 Thread Samarth Parikh
ARM has launched a next version of MHU i.e. MHUv2 with its latest subsystems. The main change is that the MHUv2 is now a distributed IP with different peripheral views (registers) for the sender and receiver. Also, the peripheral views (registers) for Secure & Non-Secure have been separated out

Linux and autoconfig of kernel on iso-image boot.

2018-05-28 Thread Phillip Hugo
Hi all, thanks for your work. Linux should have "make grab_config" that sets loaded modules to be built in the kernel build and make modules. Obviously this is so when you boot for the first time, a fully stocked kernel of modules, your first kernel build will have sensible settings for your

Linux and autoconfig of kernel on iso-image boot.

2018-05-28 Thread Phillip Hugo
Hi all, thanks for your work. Linux should have "make grab_config" that sets loaded modules to be built in the kernel build and make modules. Obviously this is so when you boot for the first time, a fully stocked kernel of modules, your first kernel build will have sensible settings for your

[PATCH 1/3] ubi: fastmap: Correctly handle interrupted erasures in EBA

2018-05-28 Thread Richard Weinberger
Fastmap cannot track the LEB unmap operation, therefore it can happen that after an interrupted erasure the mapping still looks good from Fastmap's point of view, while reading from the PEB will cause an ECC error and confuses the upper layer. Instead of teaching users of UBI how to deal with

Re: [patch v22 4/4] Documentation: jtag: Add ABI documentation

2018-05-28 Thread Randy Dunlap
On 05/28/2018 05:00 AM, Oleksandr Shamray wrote: > Added document that describe the ABI for JTAG class drivrer > > Signed-off-by: Oleksandr Shamray > Acked-by: Arnd Bergmann > --- > --- > Documentation/ABI/testing/jtag-dev | 27 + > Documentation/jtag/overview| 28

[PATCH 1/3] ubi: fastmap: Correctly handle interrupted erasures in EBA

2018-05-28 Thread Richard Weinberger
Fastmap cannot track the LEB unmap operation, therefore it can happen that after an interrupted erasure the mapping still looks good from Fastmap's point of view, while reading from the PEB will cause an ECC error and confuses the upper layer. Instead of teaching users of UBI how to deal with

Re: [patch v22 4/4] Documentation: jtag: Add ABI documentation

2018-05-28 Thread Randy Dunlap
On 05/28/2018 05:00 AM, Oleksandr Shamray wrote: > Added document that describe the ABI for JTAG class drivrer > > Signed-off-by: Oleksandr Shamray > Acked-by: Arnd Bergmann > --- > --- > Documentation/ABI/testing/jtag-dev | 27 + > Documentation/jtag/overview| 28

[PATCH 3/3] ubi: fastmap: Detect EBA mismatches on-the-fly

2018-05-28 Thread Richard Weinberger
Now we have the machinery to detect EBA mismatches on-the-fly by comparing the in-memory volume ID and LEB number with the found VID header. This helps to detect malfunction of Fastmap. Signed-off-by: Richard Weinberger --- drivers/mtd/ubi/eba.c | 17 + 1 file changed, 17

[PATCH 3/3] ubi: fastmap: Detect EBA mismatches on-the-fly

2018-05-28 Thread Richard Weinberger
Now we have the machinery to detect EBA mismatches on-the-fly by comparing the in-memory volume ID and LEB number with the found VID header. This helps to detect malfunction of Fastmap. Signed-off-by: Richard Weinberger --- drivers/mtd/ubi/eba.c | 17 + 1 file changed, 17

[PATCH 2/3] ubi: fastmap: Check each mapping only once

2018-05-28 Thread Richard Weinberger
Maintain a bitmap to keep track of which LEB->PEB mapping was checked already. That way we have to read back VID headers only once. Signed-off-by: Richard Weinberger --- drivers/mtd/ubi/build.c | 1 + drivers/mtd/ubi/eba.c | 4 drivers/mtd/ubi/fastmap.c | 20

[PATCH 2/3] ubi: fastmap: Check each mapping only once

2018-05-28 Thread Richard Weinberger
Maintain a bitmap to keep track of which LEB->PEB mapping was checked already. That way we have to read back VID headers only once. Signed-off-by: Richard Weinberger --- drivers/mtd/ubi/build.c | 1 + drivers/mtd/ubi/eba.c | 4 drivers/mtd/ubi/fastmap.c | 20

Re: [PATCH][RFC] [powerpc] arch_ptrace() uses of access_ok() are pointless

2018-05-28 Thread Al Viro
> Maybe this is just an RFC, but: > > CALL../arch/powerpc/kernel/systbl_chk.sh > ../arch/powerpc/kernel/ptrace.c: In function ‘arch_ptrace’: > ../arch/powerpc/kernel/ptrace.c:3086:4: error: expected ‘)’ before ‘return’ > return -EFAULT; > ^~ and the same a few lines later.

Re: [PATCH][RFC] [powerpc] arch_ptrace() uses of access_ok() are pointless

2018-05-28 Thread Al Viro
> Maybe this is just an RFC, but: > > CALL../arch/powerpc/kernel/systbl_chk.sh > ../arch/powerpc/kernel/ptrace.c: In function ‘arch_ptrace’: > ../arch/powerpc/kernel/ptrace.c:3086:4: error: expected ‘)’ before ‘return’ > return -EFAULT; > ^~ and the same a few lines later.

Re: [RFC PATCH] m68k: set dma and coherent masks for Macintosh SONIC based ethernet

2018-05-28 Thread Michael Schmitz
Hi Geert, my preference would be Finn's patch introducing a m68k arch_setup_pdev_archdata(). It nicely preserves what bus code sets up prior to registering a platform device (important for Zorro devices using platform or mfd devices), and allows overriding by drivers that need it. If ever a

Re: [PATCH 4.16 000/272] 4.16.13-stable review

2018-05-28 Thread Naresh Kamboju
On 28 May 2018 at 15:30, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.16.13 release. > There are 272 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > >

Re: WARNING and PANIC in irq_matrix_free

2018-05-28 Thread Thomas Gleixner
On Mon, 28 May 2018, Song Liu wrote: > This doesn't fix the issue with bnxt. Here is a trace with this patch: That's just the backtrace which is not really helpful. The real question is what leads to this scenaria. What I was asking for is to enable the irq_vector and irq_matrix trace points

Re: [PATCH v3] vfio: platform: Fix using devices in PM Domains

2018-05-28 Thread Auger Eric
Hi Geert, On 05/28/2018 05:26 PM, Geert Uytterhoeven wrote: > If a device is part of a PM Domain (e.g. power and/or clock domain), its > power state is managed using Runtime PM. Without Runtime PM, the device > may not be powered up or clocked, causing subtle failures, crashes, or > system

Re: [RFC PATCH] m68k: set dma and coherent masks for Macintosh SONIC based ethernet

2018-05-28 Thread Michael Schmitz
Hi Geert, my preference would be Finn's patch introducing a m68k arch_setup_pdev_archdata(). It nicely preserves what bus code sets up prior to registering a platform device (important for Zorro devices using platform or mfd devices), and allows overriding by drivers that need it. If ever a

Re: [PATCH 4.16 000/272] 4.16.13-stable review

2018-05-28 Thread Naresh Kamboju
On 28 May 2018 at 15:30, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.16.13 release. > There are 272 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > >

Re: WARNING and PANIC in irq_matrix_free

2018-05-28 Thread Thomas Gleixner
On Mon, 28 May 2018, Song Liu wrote: > This doesn't fix the issue with bnxt. Here is a trace with this patch: That's just the backtrace which is not really helpful. The real question is what leads to this scenaria. What I was asking for is to enable the irq_vector and irq_matrix trace points

Re: [PATCH v3] vfio: platform: Fix using devices in PM Domains

2018-05-28 Thread Auger Eric
Hi Geert, On 05/28/2018 05:26 PM, Geert Uytterhoeven wrote: > If a device is part of a PM Domain (e.g. power and/or clock domain), its > power state is managed using Runtime PM. Without Runtime PM, the device > may not be powered up or clocked, causing subtle failures, crashes, or > system

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