Re: [PATCH] powerpc: Fix handling of alignment interrupt on dcbz instruction

2017-09-13 Thread Michal Sojka
On Wed, Sep 13 2017, Michael Ellerman wrote: > Michal, Christian, can you please confirm this fixes the problems you > were seeing. Yes, it fixes my problem. Thanks. -Michal

Re: [PATCH] powerpc: Fix handling of alignment interrupt on dcbz instruction

2017-09-13 Thread Michael Ellerman
Michal, Christian, can you please confirm this fixes the problems you were seeing. cheers On 13 September 2017 2:51:24 pm AEST, Paul Mackerras wrote: >This fixes the emulation of the dcbz instruction in the alignment >interrupt handler. The error was that we were comparing just the >instructio

Re: [PATCH 1/7] powerpc: introduce pte_set_hash_slot() helper

2017-09-13 Thread Balbir Singh
On Sat, Sep 9, 2017 at 8:44 AM, Ram Pai wrote: > Introduce pte_set_hash_slot().It sets the (H_PAGE_F_SECOND|H_PAGE_F_GIX) > bits at the appropriate location in the PTE of 4K PTE. For > 64K PTE, it sets the bits in the second part of the PTE. Though > the implementation f

Re: [PATCH] ASoC: fsl_ssi: Override bit clock rate based on slot number

2017-09-13 Thread Arnaud Mouiche
On 12/09/2017 23:32, Nicolin Chen wrote: On Tue, Sep 12, 2017 at 04:35:13PM +0200, Arnaud Mouiche wrote: - * freq: Output BCLK frequency = samplerate * 32 (fixed) * channels - * dir: SND_SOC_CLOCK_OUT -> TxBCLK, SND_SOC_CLOCK_IN -> RxBCLK. + * freq: Output BCLK frequency = samplerate * 32 (fi

Re: [PATCH v3 1/2] powerpc/mm: Export flush_all_mm()

2017-09-13 Thread Frederic Barrat
Le 13/09/2017 à 06:04, Alistair Popple a écrit : +static inline void hash__local_flush_all_mm(struct mm_struct *mm) +{ + /* +* There's no Page Walk Cache for hash, so what is needed is +* the same as flush_tlb_mm(), which doesn't really make sense +* with hash. So

Re: [PATCH v2 3/5] powerpc/mce: Hookup derror (load/store) UE errors

2017-09-13 Thread Nicholas Piggin
On Wed, 13 Sep 2017 16:26:59 +1000 Balbir Singh wrote: > On Wed, Sep 13, 2017 at 4:21 PM, Nicholas Piggin wrote: > > On Wed, 13 Sep 2017 16:10:47 +1000 > > Balbir Singh wrote: > > > >> Extract physical_address for UE errors by walking the page > >> tables for the mm and address at the NIP, to

Re: [PATCH 2/7] powerpc: introduce pte_get_hash_gslot() helper

2017-09-13 Thread Balbir Singh
On Sat, Sep 9, 2017 at 8:44 AM, Ram Pai wrote: > Introduce pte_get_hash_gslot()() which returns the slot number of the > HPTE in the global hash table. > > This function will come in handy as we work towards re-arranging the > PTE bits in the later patches. > > Reviewed-by: Aneesh Kumar K.V > Sig

[PATCH] crypto: talitos - fix sha224

2017-09-13 Thread Christophe Leroy
Kernel crypto tests report the following error at startup [2.752626] alg: hash: Test 4 failed for sha224-talitos [2.757907] : 30 e2 86 e2 e7 8a dd 0d d7 eb 9f d5 83 fe f1 b0 0010: 2d 5a 6c a5 f9 55 ea fd 0e 72 05 22 This patch fixes it Signed-off-by: Christophe Leroy --- dr

[PATCH] crypto: talitos - fix hashing

2017-09-13 Thread Christophe Leroy
md5sum on some files gives wrong result Exemple: With the md5sum from libkcapi: c15115c05bad51113f81bdaee735dd09 test With the original md5sum: bbdf41d80ba7e8b2b7be3a0772be76cb test This patch fixes this issue Signed-off-by: Christophe Leroy --- drivers/crypto/talitos.c | 2 +- 1 file chan

Re: [PATCH v3 04/20] mm: VMA sequence count

2017-09-13 Thread Sergey Senozhatsky
Hi, On (09/08/17 20:06), Laurent Dufour wrote: [..] > @@ -903,6 +910,7 @@ int __vma_adjust(struct vm_area_struct *vma, unsigned > long start, > mm->map_count--; > mpol_put(vma_policy(next)); > kmem_cache_free(vm_area_cachep, next); > + write_s

Re: [RFC PATCH 2/2] powerpc/powernv: implement NMI IPIs with OPAL_SIGNAL_SYSTEM_RESET

2017-09-13 Thread Nicholas Piggin
On Wed, 13 Sep 2017 02:05:53 +1000 Nicholas Piggin wrote: > There are two complications. The first is that sreset from stop states > come in with SRR1 set to do a powersave wakeup, with an sreset reason > encoded. > > The second is that threads on the same core can't be signalled directly > so w

Re: [RFC PATCH 1/2] core: implement OPAL_SIGNAL_SYSTEM_RESET with POWER9 scoms

2017-09-13 Thread Nicholas Piggin
On Wed, 13 Sep 2017 09:18:34 +1000 Benjamin Herrenschmidt wrote: > On Wed, 2017-09-13 at 02:05 +1000, Nicholas Piggin wrote: > > This implements a way to raise system reset interrupts on other > > cores. This has not yet been tested on DD2 or with deeper sleep > > states. > > Reminds me, we ne

[PATCH] powerpc: Fix handling of alignment interrupt on dcbz instruction

2017-09-13 Thread Christian Zigotzky
On 13 September 2017 at 09:15AM, Michal Sojka wrote: On Wed, Sep 13 2017, Michael Ellerman wrote: Michal, Christian, can you please confirm this fixes the problems you were seeing. Yes, it fixes my problem. Thanks. -Michal It also fixes my problem. Many thanks! -- Christian

[PATCH] powerpc/tm: Flush TM only if CPU has TM feature

2017-09-13 Thread Gustavo Romero
Commit cd63f3cf1d59 ("powerpc/tm: Fix saving of TM SPRs in core dump") added code to access TM SPRs in flush_tmregs_to_thread(). However flush_tmregs_to_thread() does not check if TM feature is available on CPU before trying to access TM SPRs in order to copy live state to thread structures. flush_

Re: POWER9 PMU stops after idle workaround

2017-09-13 Thread Michal Suchánek
On Thu, 3 Aug 2017 20:19:38 +1000 (AEST) Michael Ellerman wrote: > On Thu, 2017-07-20 at 01:53:22 UTC, Nicholas Piggin wrote: > > POWER9 DD2 PMU can stop after a state-loss idle in some conditions. > > > > A solution is to set then clear MMCRA[60] after wake from state-loss > > idle. > > > > S

Re: [PATCH v3 04/20] mm: VMA sequence count

2017-09-13 Thread Laurent Dufour
Hi Sergey, On 13/09/2017 13:53, Sergey Senozhatsky wrote: > Hi, > > On (09/08/17 20:06), Laurent Dufour wrote: > [..] >> @@ -903,6 +910,7 @@ int __vma_adjust(struct vm_area_struct *vma, unsigned >> long start, >> mm->map_count--; >> mpol_put(vma_policy(next)); >>

Re: [PATCH] uprobe: Warn if unable to install breakpoint

2017-09-13 Thread Naveen N. Rao
On 2017/09/01 03:09PM, Michael Ellerman wrote: > "Naveen N. Rao" writes: > > > When we try to install a uprobe breakpoint in uprobe_mmap(), we ignore > > all errors encountered in the process per this comment at the top of > > the function: > > /* > > * Called from mmap_region/vma_adjust

[RFC PATCH] powerpc/uprobes: Fixup si_addr if we took an exception while single stepping

2017-09-13 Thread Naveen N. Rao
If the single-stepped instruction causes an exception, we may end up setting siginfo.si_addr to the address of the uprobe xol area. This is not desirable since the address won't make sense for the process if it wants to handle the exception. Fixup the si_addr field in such cases. Reported-by: Anto

Re: [PATCH 2/7] powerpc: introduce pte_get_hash_gslot() helper

2017-09-13 Thread Ram Pai
On Wed, Sep 13, 2017 at 07:32:57PM +1000, Balbir Singh wrote: > On Sat, Sep 9, 2017 at 8:44 AM, Ram Pai wrote: > > Introduce pte_get_hash_gslot()() which returns the slot number of the > > HPTE in the global hash table. > > > > This function will come in handy as we work towards re-arranging the >

[PATCH 1/5] powerpc/kprobes: Some cosmetic updates to try_to_emulate()

2017-09-13 Thread Naveen N. Rao
1. This is only used in kprobes.c, so make it static. 2. Remove the un-necessary (ret == 0) comparison in the else clause. Signed-off-by: Naveen N. Rao --- arch/powerpc/kernel/kprobes.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/kprobes.c b/arch/p

[PATCH 2/5] powerpc/kprobes: Do not suppress instruction emulation if a single run failed

2017-09-13 Thread Naveen N. Rao
Currently, we disable instruction emulation if emulate_step() fails for any reason. However, such failures could be transient and specific to a particular run. Instead, only disable instruction emulation if we have never been able to emulate this. If we had emulated this instruction successfully at

[PATCH 3/5] powerpc/kprobes: Fix warnings from __this_cpu_read() on preempt kernels

2017-09-13 Thread Naveen N. Rao
Kamalesh pointed out that we are getting the below call traces with livepatched functions when we enable CONFIG_PREEMPT: [ 495.470721] BUG: using __this_cpu_read() in preemptible [] code: cat/8394 [ 495.471167] caller is is_current_kprobe_addr+0x30/0x90 [ 495.471171] CPU: 4 PID: 8394 C

[PATCH 4/5] powerpc/jprobes: Disable preemption when triggered through ftrace

2017-09-13 Thread Naveen N. Rao
KPROBES_SANITY_TEST throws the below splat when CONFIG_PREEMPT is enabled: [3.140410] Kprobe smoke test: started [3.149680] DEBUG_LOCKS_WARN_ON(val > preempt_count()) [3.149684] [ cut here ] [3.149695] WARNING: CPU: 19 PID: 1 at kernel/sched/core.c:3094 pre

[PATCH 5/5] powerpc/jprobes: Validate break handler invocation as being due to a jprobe_return()

2017-09-13 Thread Naveen N. Rao
Fix a circa 2005 FIXME by implementing a check to ensure that we actually got into the jprobe break handler() due to the trap in jprobe_return(). Signed-off-by: Naveen N. Rao --- This is v2 of the patch posted previously (*) to change the WARN() to a pr_debug() as suggested by Masami. (*) https:

Re: POWER9 PMU stops after idle workaround

2017-09-13 Thread Michael Ellerman
Yeah I messed up the subject when comitting, from memory, so I had to rebase and force push. I think this is it: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?id=09539f9b123652e969894d6299ae0df2fe12cb5d cheers On 14 September 2017 2:11:16 am AEST, "Michal Suchánek

Re: [PATCH] ASoC: fsl_ssi: Override bit clock rate based on slot number

2017-09-13 Thread Nicolin Chen
On Wed, Sep 13, 2017 at 10:02:20AM +0200, Arnaud Mouiche wrote: > >Could you please give me a few set of examples of how you set > >set_sysclk(), set_tdm_slot() with the current driver? The idea > >here is to figure out a way to calculate the bclk in hw_params > >without getting set_sysclk() invol

Re: [PATCH 1/5] powerpc/kprobes: Some cosmetic updates to try_to_emulate()

2017-09-13 Thread Masami Hiramatsu
On Thu, 14 Sep 2017 02:50:32 +0530 "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. > > Signed-off-by: Naveen N. Rao Reviewed-by: Masami Hiramatsu Thanks! > --- > arch/powerpc/kernel/kprobe

[PATCH v3] Make initramfs honor CONFIG_DEVTMPFS_MOUNT

2017-09-13 Thread Rob Landley
From: Rob Landley Make initramfs honor CONFIG_DEVTMPFS_MOUNT, and move /dev/console open after devtmpfs mount. Add workaround for Debian bug that was copied by Ubuntu. Signed-off-by: Rob Landley --- v2 discussion: http://lkml.iu.edu/hypermail/linux/kernel/1705.2/05611.html drivers/base/Kcon

Re: [PATCH 2/5] powerpc/kprobes: Do not suppress instruction emulation if a single run failed

2017-09-13 Thread Masami Hiramatsu
On Thu, 14 Sep 2017 02:50:33 +0530 "Naveen N. Rao" wrote: > Currently, we disable instruction emulation if emulate_step() fails for > any reason. However, such failures could be transient and specific to a > particular run. Instead, only disable instruction emulation if we have > never been able

Re: [PATCH 4/5] powerpc/jprobes: Disable preemption when triggered through ftrace

2017-09-13 Thread Masami Hiramatsu
On Thu, 14 Sep 2017 02:50:35 +0530 "Naveen N. Rao" wrote: > KPROBES_SANITY_TEST throws the below splat when CONFIG_PREEMPT is > enabled: > > [3.140410] Kprobe smoke test: started > [3.149680] DEBUG_LOCKS_WARN_ON(val > preempt_count()) > [3.149684] [ cut here ]

Regression for Power PC in 4.14-rc0 - bisected to commit 31bfdb036f12

2017-09-13 Thread Larry Finger
When booting my PowerBook Aluminum G4, I get a pop-up screen that says "The system is running in low-graphics mode. Your screen, graphics card, and input device settings could not be detected correctly. You will need to configure these yourself." This is a big-endian 74xx CPU. The lscpu command

Re: [PATCH v3 04/20] mm: VMA sequence count

2017-09-13 Thread Sergey Senozhatsky
Hi, On (09/13/17 18:56), Laurent Dufour wrote: > Hi Sergey, > > On 13/09/2017 13:53, Sergey Senozhatsky wrote: > > Hi, > > > > On (09/08/17 20:06), Laurent Dufour wrote: [..] > > ok, so what I got on my box is: > > > > vm_munmap() -> down_write_killable(&mm->mmap_sem) > > do_munmap() > > __

Re: [PATCH 3/5] powerpc/kprobes: Fix warnings from __this_cpu_read() on preempt kernels

2017-09-13 Thread Masami Hiramatsu
On Thu, 14 Sep 2017 02:50:34 +0530 "Naveen N. Rao" wrote: > Kamalesh pointed out that we are getting the below call traces with > livepatched functions when we enable CONFIG_PREEMPT: > > [ 495.470721] BUG: using __this_cpu_read() in preemptible [] code: > cat/8394 > [ 495.471167] call

Re: Regression for Power PC in 4.14-rc0 - bisected to commit 31bfdb036f12

2017-09-13 Thread Andrew Donnellan
On 14/09/17 10:07, Larry Finger wrote: When booting my PowerBook Aluminum G4, I get a pop-up screen that says "The system is running in low-graphics  mode. Your screen, graphics card, and input device settings could not be detected correctly. You will need to configure these yourself." This is

Re: [PATCH 5/5] powerpc/jprobes: Validate break handler invocation as being due to a jprobe_return()

2017-09-13 Thread Masami Hiramatsu
On Thu, 14 Sep 2017 02:50:36 +0530 "Naveen N. Rao" wrote: > Fix a circa 2005 FIXME by implementing a check to ensure that we > actually got into the jprobe break handler() due to the trap in > jprobe_return(). > Looks good to me. Acked-by: Masami Hiramatsu Thanks! > Signed-off-by: Naveen N.

Re: Regression for Power PC in 4.14-rc0 - bisected to commit 31bfdb036f12

2017-09-13 Thread Larry Finger
On 09/13/2017 07:37 PM, Andrew Donnellan wrote: On 14/09/17 10:07, Larry Finger wrote: When booting my PowerBook Aluminum G4, I get a pop-up screen that says "The system is running in low-graphics  mode. Your screen, graphics card, and input device settings could not be detected correctly. You

Re: [PATCH 2/2] powerpc/powernv: Rework EEH initialization on powernv

2017-09-13 Thread Russell Currey
On Thu, 2017-09-07 at 16:35 +1000, Benjamin Herrenschmidt wrote: > Remove the post_init callback which is only used > by powernv, we can just call it explicitly from > the powernv code. > > This partially kills the ability to "disable" eeh at > runtime via debugfs as this was calling that same > c

Re: [PATCH 1/2] powerpc/eeh: Create PHB PEs after EEH is initialized

2017-09-13 Thread Russell Currey
On Thu, 2017-09-07 at 16:35 +1000, Benjamin Herrenschmidt wrote: > Otherwise we end up not yet having computed the right > diag data size on powernv where EEH initialization > is delayed, thus causing memory corruption later on > when calling OPAL. > > Signed-off-by: Benjamin Herrenschmidt Acked

Re: [PATCH 3/7] powerpc: Free up four 64K PTE bits in 4K backed HPTE pages

2017-09-13 Thread Balbir Singh
On Fri, 8 Sep 2017 15:44:43 -0700 Ram Pai wrote: > Rearrange 64K PTE bits to free up bits 3, 4, 5 and 6, > in the 4K backed HPTE pages.These bits continue to be used > for 64K backed HPTE pages in this patch, but will be freed > up in the next patch. The bit numbers are big-endian as > d

Re: [PATCH 4/7] powerpc: Free up four 64K PTE bits in 64K backed HPTE pages

2017-09-13 Thread Balbir Singh
On Fri, 8 Sep 2017 15:44:44 -0700 Ram Pai wrote: > Rearrange 64K PTE bits to free up bits 3, 4, 5 and 6 > in the 64K backed HPTE pages. This along with the earlier > patch will entirely free up the four bits from 64K PTE. > The bit numbers are big-endian as defined in the ISA3.0 > > Th

Re: [PATCH 5/7] powerpc: Swizzle around 4K PTE bits to free up bit 5 and bit 6

2017-09-13 Thread Balbir Singh
On Fri, 8 Sep 2017 15:44:45 -0700 Ram Pai wrote: > We need PTE bits 3 ,4, 5, 6 and 57 to support protection-keys, > because these are the bits we want to consolidate on across all > configuration to support protection keys. > > Bit 3,4,5 and 6 are currently used on 4K-pte kernels. But bit

[PATCH v2] powerpc/tm: Flush TM only if CPU has TM feature

2017-09-13 Thread Gustavo Romero
Commit cd63f3c ("powerpc/tm: Fix saving of TM SPRs in core dump") added code to access TM SPRs in flush_tmregs_to_thread(). However flush_tmregs_to_thread() does not check if TM feature is available on CPU before trying to access TM SPRs in order to copy live state to thread structures. flush_tmreg

Re: [RFC PATCH 2/2] powerpc/powernv: implement NMI IPIs with OPAL_SIGNAL_SYSTEM_RESET

2017-09-13 Thread Benjamin Herrenschmidt
On Wed, 2017-09-13 at 23:13 +1000, Nicholas Piggin wrote: > On Wed, 13 Sep 2017 02:05:53 +1000 > Nicholas Piggin wrote: > > > There are two complications. The first is that sreset from stop states > > come in with SRR1 set to do a powersave wakeup, with an sreset reason > > encoded. > > > > The

Re: [RFC PATCH 1/2] core: implement OPAL_SIGNAL_SYSTEM_RESET with POWER9 scoms

2017-09-13 Thread Benjamin Herrenschmidt
On Wed, 2017-09-13 at 23:27 +1000, Nicholas Piggin wrote: > On Wed, 13 Sep 2017 09:18:34 +1000 > Benjamin Herrenschmidt wrote: > > > On Wed, 2017-09-13 at 02:05 +1000, Nicholas Piggin wrote: > > > This implements a way to raise system reset interrupts on other > > > cores. This has not yet been t

Re: [PATCH kernel] powerpc/powernv: Update comment about shifting IOV BAR

2017-09-13 Thread Alexey Kardashevskiy
On 31/08/17 13:34, Alexey Kardashevskiy wrote: > From: Benjamin Herrenschmidt Oops, this was not right :) Anyway, Ben, please comment. Thanks. > > From: Alexey Kardashevskiy > > This updates the comment about creating a hole in /proc/iomem which > should not be normally happening but it doe

[PATCH v2] ASoC: fsl_ssi: Caculate bit clock rate using slot number and width

2017-09-13 Thread Nicolin Chen
The set_sysclk() now is used to override the output bit clock rate. But this is not a common way to implement a set_dai_sysclk(). And this creates a problem when a general machine driver (simple-card for example) tries to do set_dai_sysclk() by passing an input clock rate for the baud clock instead

Re: [PATCH kernel] powerpc/powernv: Update comment about shifting IOV BAR

2017-09-13 Thread Benjamin Herrenschmidt
On Thu, 2017-09-14 at 12:45 +1000, Alexey Kardashevskiy wrote: > On 31/08/17 13:34, Alexey Kardashevskiy wrote: > > From: Benjamin Herrenschmidt > > Oops, this was not right :) > > Anyway, Ben, please comment. Thanks. This is incorrect, we can do hotplug behind switches afaik. > > > > > From:

Re: [PATCH kernel] powerpc/powernv: Update comment about shifting IOV BAR

2017-09-13 Thread Alexey Kardashevskiy
On 14/09/17 13:07, Benjamin Herrenschmidt wrote: > On Thu, 2017-09-14 at 12:45 +1000, Alexey Kardashevskiy wrote: >> On 31/08/17 13:34, Alexey Kardashevskiy wrote: >>> From: Benjamin Herrenschmidt >> >> Oops, this was not right :) >> >> Anyway, Ben, please comment. Thanks. > > This is incorrect,

Re: [PATCH 7/7] powerpc: capture the PTE format changes in the dump pte report

2017-09-13 Thread Balbir Singh
On Fri, 8 Sep 2017 15:44:47 -0700 Ram Pai wrote: > The H_PAGE_F_SECOND,H_PAGE_F_GIX are not in the 64K main-PTE. > capture these changes in the dump pte report. > > Reviewed-by: Aneesh Kumar K.V > Signed-off-by: Ram Pai > --- So we lose slot and secondary information for 64K PTE's with this

Re: [PATCH 01/25] powerpc: initial pkey plumbing

2017-09-13 Thread Balbir Singh
On Fri, 8 Sep 2017 15:44:49 -0700 Ram Pai wrote: > Basic plumbing to initialize the pkey system. > Nothing is enabled yet. A later patch will enable it > ones all the infrastructure is in place. > > Signed-off-by: Ram Pai > --- > arch/powerpc/Kconfig | 16 +

Re: [PATCH kernel] powerpc/powernv: Update comment about shifting IOV BAR

2017-09-13 Thread Benjamin Herrenschmidt
On Thu, 2017-09-14 at 13:18 +1000, Alexey Kardashevskiy wrote: > On 14/09/17 13:07, Benjamin Herrenschmidt wrote: > > On Thu, 2017-09-14 at 12:45 +1000, Alexey Kardashevskiy wrote: > > > On 31/08/17 13:34, Alexey Kardashevskiy wrote: > > > > From: Benjamin Herrenschmidt > > > > > > Oops, this was

Re: [PATCH 02/25] powerpc: define an additional vma bit for protection keys.

2017-09-13 Thread Balbir Singh
On Fri, 8 Sep 2017 15:44:50 -0700 Ram Pai wrote: > powerpc needs an additional vma bit to support 32 keys. > Till the additional vma bit lands in include/linux/mm.h > we have to define it in powerpc specific header file. > This is needed to get pkeys working on power. > > Signed-off-by: Ram

Possible bug in commit f3b3f28493d9

2017-09-13 Thread Paul Mackerras
Commit f3b3f28493d9 ("powerpc/powernv/idle: Don't override default/deepest directly in kernel", 2017-03-22) made the following change in pnv_cpu_offline() in arch/powerpc/platforms/powernv/idle.c: - if (cpu_has_feature(CPU_FTR_ARCH_300)) { + if (cpu_has_feature(CPU_FTR_ARCH_300) && dee

Re: [PATCH 1/5] powerpc/kprobes: Some cosmetic updates to try_to_emulate()

2017-09-13 Thread Kamalesh Babulal
On Thursday 14 September 2017 02:50 AM, 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. Signed-off-by: Naveen N. Rao Reviewed-by: Kamalesh Babulal --- arch/powerpc/kernel/kprobes.c | 4 ++-- 1

Re: [RFC PATCH 2/2] powerpc/powernv: implement NMI IPIs with OPAL_SIGNAL_SYSTEM_RESET

2017-09-13 Thread Nicholas Piggin
On Thu, 14 Sep 2017 12:24:49 +1000 Benjamin Herrenschmidt wrote: > On Wed, 2017-09-13 at 23:13 +1000, Nicholas Piggin wrote: > > On Wed, 13 Sep 2017 02:05:53 +1000 > > Nicholas Piggin wrote: > > > > > There are two complications. The first is that sreset from stop states > > > come in with SR

Re: [PATCH 2/5] powerpc/kprobes: Do not suppress instruction emulation if a single run failed

2017-09-13 Thread Naveen N. Rao
On 2017/09/13 04:53PM, Masami Hiramatsu wrote: > On Thu, 14 Sep 2017 02:50:33 +0530 > "Naveen N. Rao" wrote: > > > Currently, we disable instruction emulation if emulate_step() fails for > > any reason. However, such failures could be transient and specific to a > > particular run. Instead, only

Re: [RFC PATCH 2/2] powerpc/powernv: implement NMI IPIs with OPAL_SIGNAL_SYSTEM_RESET

2017-09-13 Thread Alistair Popple
On Thu, 14 Sep 2017 04:32:28 PM Nicholas Piggin wrote: > On Thu, 14 Sep 2017 12:24:49 +1000 > Benjamin Herrenschmidt wrote: > > > On Wed, 2017-09-13 at 23:13 +1000, Nicholas Piggin wrote: > > > On Wed, 13 Sep 2017 02:05:53 +1000 > > > Nicholas Piggin wrote: > > > > > > > There are two complic

Re: [PATCH 3/5] powerpc/kprobes: Fix warnings from __this_cpu_read() on preempt kernels

2017-09-13 Thread Naveen N. Rao
On 2017/09/13 05:36PM, Masami Hiramatsu wrote: > On Thu, 14 Sep 2017 02:50:34 +0530 > "Naveen N. Rao" wrote: > > > Kamalesh pointed out that we are getting the below call traces with > > livepatched functions when we enable CONFIG_PREEMPT: > > > > [ 495.470721] BUG: using __this_cpu_read() in p