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
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
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
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
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
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
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
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
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 +
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
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,
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:
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
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
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
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
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
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
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
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
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
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
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
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.
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
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
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()
> > __
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
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 ]
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
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
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
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
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
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:
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
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
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
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
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
>
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
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
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));
>>
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
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_
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
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
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
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
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
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
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
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
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
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
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
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
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
58 matches
Mail list logo
