Linux 4.14: Reported regressions as of Sunday, 2017-10-08

2017-10-08 Thread Thorsten Leemhuis
Hi! Find below my second regression report for Linux 4.14. It lists 8 regressions I'm currently aware of. One regression was fixed since last weeks report. One was in there that shouldn't have been there. As always: Are you aware of any other regressions? Then please let me know by mail (a simple

Re: [v2, 1/6] powerpc/kprobes: Some cosmetic updates to try_to_emulate()

2017-10-08 Thread Michael Ellerman
On Fri, 2017-09-22 at 09:10:43 UTC, "Naveen N. Rao" wrote: > 1. This is only used in kprobes.c, so make it static. > 2. Remove the un-necessary (ret == 0) comparison in the else clause. > > Reviewed-by: Masami Hiramatsu > Reviewed-by: Kamalesh Babulal

Re: selftests/powerpc: Use snprintf to construct DSCR sysfs interface paths

2017-10-08 Thread Michael Ellerman
On Thu, 2017-09-28 at 13:34:26 UTC, Seth Forshee wrote: > Currently sprintf is used, and while paths should never exceed > the size of the buffer it is theoretically possible since > dirent.d_name is 256 bytes. As a result this trips > -Wformat-overflow, and since the test is built with -Wall

Re: powerpc: Always initialize input array when calling epapr_hypercall()

2017-10-08 Thread Michael Ellerman
On Thu, 2017-09-28 at 13:33:39 UTC, Seth Forshee wrote: > Several callers to epapr_hypercall() pass an uninitialized stack > allocated array for the input arguments, presumably because they > have no input arguments. However this can produce errors like > this one > >

Re: [1/1] powerpc: get_wchan(): solve possible race scenario due to parallel wakeup

2017-10-08 Thread Michael Ellerman
On Tue, 2016-04-19 at 10:18:21 UTC, Kautuk Consul wrote: > Add a check for p->state == TASK_RUNNING so that any wake-ups on > task_struct p in the interim lead to 0 being returned by get_wchan(). > > Signed-off-by: Kautuk Consul Applied to powerpc next, thanks.

Re: [V3] cxl: Add support for POWER9 DD2

2017-10-08 Thread Michael Ellerman
On Fri, 2017-09-08 at 13:52:11 UTC, Christophe Lombard wrote: > The PSL initialization sequence has been updated to DD2. > This patch adapts to the changes, retaining compatibility with DD1. > The patch includes some changes to DD1 fix-ups as well. > > Tests performed on some of the old/new

Re: powerpc: Add PPC_EMULATED_STATS to powernv_defconfig

2017-10-08 Thread Michael Ellerman
On Thu, 2017-09-21 at 05:24:49 UTC, Michael Neuling wrote: > This is useful, especially for developers. > > Signed-off-by: Michael Neuling Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/e366b921c7e32c9b92b4c524696fa8 cheers

Re: powerpc/xmon: Add option to show uptime information

2017-10-08 Thread Michael Ellerman
On Mon, 2017-09-18 at 14:16:58 UTC, "Guilherme G. Piccoli" wrote: > It might be useful to quickly get the uptime of a running > system on xmon, without needing to grab data from memory and > doing math on struct addresses. > > For example, it'd be useful to check for how long after a crash > a

Re: powerpc/powernv: Make opal_event_shutdown() callable from IRQ context

2017-10-08 Thread Michael Ellerman
On Fri, 2017-09-29 at 03:58:02 UTC, Michael Ellerman wrote: > In opal_event_shutdown() we free all the IRQs hanging off the > opal_event_irqchip. However it's not safe to do so if we're called > from IRQ context, because free_irq() wants to synchronise versus IRQ > context. This can lead to

Re: [PATCH 03/25] powerpc: track allocation status of all pkeys

2017-10-08 Thread Ram Pai
On Sat, Oct 07, 2017 at 09:02:55PM +1100, Michael Ellerman wrote: > Ram Pai writes: > > > Total 32 keys are available on power7 and above. However > > pkey 0,1 are reserved. So effectively we have 30 pkeys. > > > > On 4K kernels, we do not have 5 bits in the PTE to >

[PATCH] mm: remove unnecessary WARN_ONCE in page_vma_mapped_walk().

2017-10-08 Thread Zi Yan
From: Zi Yan A non present pmd entry can appear after pmd_lock is taken in page_vma_mapped_walk(), even if THP migration is not enabled. The WARN_ONCE is unnecessary. Fixes: 616b8371539a ("mm: thp: enable thp migration in generic path") Reported-and-tested-by: Abdul

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-08 Thread Andrea Parri
On Fri, Oct 6, 2017 at 12:02 AM, Andrea Parri wrote: > On Thu, Oct 05, 2017 at 04:02:06PM +, Mathieu Desnoyers wrote: >> - On Oct 5, 2017, at 8:12 AM, Peter Zijlstra pet...@infradead.org wrote: >> >> > On Wed, Oct 04, 2017 at 02:37:53PM -0700, Paul E. McKenney

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-08 Thread Andrea Parri
On Thu, Oct 05, 2017 at 04:02:06PM +, Mathieu Desnoyers wrote: > - On Oct 5, 2017, at 8:12 AM, Peter Zijlstra pet...@infradead.org wrote: > > > On Wed, Oct 04, 2017 at 02:37:53PM -0700, Paul E. McKenney wrote: > >> diff --git a/arch/powerpc/kernel/membarrier.c > >>

RE: [PATCH 07/13] timer: Remove last user of TIMER_INITIALIZER

2017-10-08 Thread Gross, Mark
Acked-by: mark gross --mark > -Original Message- > From: Kees Cook [mailto:keesc...@chromium.org] > Sent: Wednesday, October 4, 2017 4:27 PM > To: Thomas Gleixner > Cc: Kees Cook ; Arnd Bergmann ; > Greg

Re: [PATCH 10/13] timer: Remove expires and data arguments from DEFINE_TIMER

2017-10-08 Thread Kalle Valo
Kees Cook writes: > Drop the arguments from the macro and adjust all callers with the > following script: > > perl -pi -e 's/DEFINE_TIMER\((.*), 0, 0\);/DEFINE_TIMER($1);/g;' \ > $(git grep DEFINE_TIMER | cut -d: -f1 | sort -u | grep -v timer.h) > > Signed-off-by:

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-08 Thread Avi Kivity
On 10/05/2017 07:23 AM, Nicholas Piggin wrote: On Wed, 4 Oct 2017 14:37:53 -0700 "Paul E. McKenney" wrote: From: Mathieu Desnoyers Provide a new command allowing processes to register their intent to use the private expedited

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-08 Thread Andrea Parri
On Fri, Oct 06, 2017 at 10:32:19AM +0200, Peter Zijlstra wrote: > > AFAIU the scheduler rq->lock is held while preemption is disabled. > > synchronize_sched() is used here to ensure that all pre-existing > > preempt-off critical sections have completed. > > > > So saying that we use

Re: linux-4.14-rc3/arch/powerpc/perf/imc-pmu.c:599: pointless test ?

2017-10-08 Thread Michael Ellerman
David Binderman writes: > Hello there, > > linux-4.14-rc3/arch/powerpc/perf/imc-pmu.c:599]: (style) Unsigned variable > 'ncpu' can't be negative so it is unnecessary to test it. > > Source code is > > if (ncpu >= 0 && ncpu < nr_cpu_ids) { > > but > > unsigned int