On 2017/03/07 03:47PM, Steven Rostedt wrote:
>
> Please start a new thread. When sending patches as replies to other
> patch threads, especially this deep into the thread, they will most
> likely get ignored.
Sorry, got carried off. I will re-post in a new series.
- Naveen
Now that I've been carried back in (ugh!), please find the remaining
patches from the earlier series (*) here. Patches 1-4 are the same as in
v4. Patch 5 in the previous series was dropped and the previous patch 6
has been updated accordingly.
- Naveen
(*) https://www.mail-archive.com/linux-kerne
perf specifies an offset from _text and since this offset is fed
directly into the arch-specific helper, kprobes tracer rejects
installation of kretprobes through perf. Fix this by looking up the
actual offset from a function for the specified sym+offset.
Refactor and reuse existing routines to li
With ABIv2, we offset 8 bytes into a function to get at the local entry
point.
Acked-by: Ananth N Mavinakayanahalli
Acked-by: Michael Ellerman
Signed-off-by: Naveen N. Rao
---
arch/powerpc/kernel/kprobes.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/arch/powerpc/kernel/kprobes
Simplify and separate out the ftrace README scanning logic into a
separate helper. This is used subsequently to scan for all patterns of
interest and to cache the result.
Since we are only interested in availability of probe argument type x,
we will only scan for that.
Acked-by: Masami Hiramatsu
perf now uses an offset from _text/_stext for kretprobes if the kernel
supports it, rather than the actual function name. As such, let's choose
the LEP for powerpc ABIv2 so as to ensure the probe gets hit. Do it only
if the kernel supports specifying offsets with kretprobes.
Signed-off-by: Naveen
We indicate support for accepting sym+offset with kretprobes through a
line in ftrace README. Parse the same to identify support and choose the
appropriate format for kprobe_events.
As an example, without this perf patch, but with the ftrace changes:
naveen@ubuntu:~/linux/tools/perf$ sudo cat /
On Wed, 8 Mar 2017 02:09:29 +0530
"Naveen N. Rao" wrote:
> Along similar lines as commit 9326638cbee2 ("kprobes, x86: Use
> NOKPROBE_SYMBOL() instead of __kprobes annotation"), convert __kprobes
> annotation to either NOKPROBE_SYMBOL() or nokprobe_inline. The latter
> forces inlining, in which c
Daniel Axtens wrote:
> Turning on crypto self-tests on a POWER8 shows:
>
>alg: hash: Test 1 failed for crc32c-vpmsum
>: ff ff ff ff
>
> Comparing the code with the Intel CRC32c implementation on which
> ours is based shows that we are doing an init with 0, not ~0
> as CRC32c requ
On Mon, 13 Feb 2017, Josh Poimboeuf wrote:
> Here's v5 of the consistency model
I have ammended the patches with all the received ACKs and applied to
'for-4.12/klp-hybrid-consistency-model' branch of livepatching.git.
Thanks,
--
Jiri Kosina
SUSE Labs
On Tue, 2017-03-07 at 10:12 -0600, Josh Poimboeuf wrote:
> On Tue, Mar 07, 2017 at 05:50:55PM +1100, Balbir Singh wrote:
> > On Mon, 2017-02-13 at 19:42 -0600, Josh Poimboeuf wrote:
> > > For live patching and possibly other use cases, a stack trace is only
> > > useful if it can be assured that it
On 2017/03/08 09:42AM, Masami Hiramatsu wrote:
> On Wed, 8 Mar 2017 02:09:29 +0530
> "Naveen N. Rao" wrote:
>
> > Along similar lines as commit 9326638cbee2 ("kprobes, x86: Use
> > NOKPROBE_SYMBOL() instead of __kprobes annotation"), convert __kprobes
> > annotation to either NOKPROBE_SYMBOL() o
On Wed, 8 Mar 2017 13:56:10 +0530
"Naveen N. Rao" wrote:
> perf now uses an offset from _text/_stext for kretprobes if the kernel
> supports it, rather than the actual function name. As such, let's choose
> the LEP for powerpc ABIv2 so as to ensure the probe gets hit. Do it only
> if the kernel
"Naveen N. Rao" writes:
> With ABIv2, we offset 8 bytes into a function to get at the local entry
> point.
>
> Acked-by: Ananth N Mavinakayanahalli
> Acked-by: Michael Ellerman
> Signed-off-by: Naveen N. Rao
> ---
> arch/powerpc/kernel/kprobes.c | 9 +
> 1 file changed, 9 insertions(+
"Tobin C. Harding" writes:
> Sparse emits warning: symbol 'prepare_ftrace_return' was not
> declared. Should it be static? prepare_ftrace_return() is called
> from assembler and should not be static. Adding a header file
> declaring the function will fix the sparse warning while adding
> document
On 2017/03/08 11:31AM, Masami Hiramatsu wrote:
> On Wed, 8 Mar 2017 13:56:10 +0530
> "Naveen N. Rao" wrote:
>
> > perf now uses an offset from _text/_stext for kretprobes if the kernel
> > supports it, rather than the actual function name. As such, let's choose
> > the LEP for powerpc ABIv2 so a
The following warning is seen when a CPU is hot unplugged on a PowerKVM
guest:
refcount_t: underflow; use-after-free.
[ cut here ]
WARNING: CPU: 0 PID: 53 at lib/refcount.c:128 refcount_sub_and_test+0xd8/0xf0
Modules linked in:
CPU: 0 PID: 53 Comm: kworker/u510:1 Not tainte
Hi Gautham,
I'm just getting back to this. Sorry for the late reply, and
thanks for the reviews.
On Tue, 28 Feb 2017 22:45:46 +0530
Gautham R Shenoy wrote:
> Hi Nick,
>
> On Fri, Feb 17, 2017 at 12:09 AM, Nicholas Piggin wrote:
> > The ISA specifies power save wakeup can cause a machine check
The patch
ASoC: fsl: Remove unneeded init of static variable
has been applied to the asoc tree at
git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent
On Tue, Mar 07, 2017 at 10:01:04PM +0530, Abdul Haleem wrote:
>
> Hi,
>
> Today's mainline (4.11.0-rc1) booted with warnings on Power7 LPAR.
>
> Issue is not reproducible all the time.
>
> traces:
>
> Found device VDASD 5.
> Mounting /home...
> Reached target Swap.
> Found device VDASD
It is invalid to call del_gendisk() when disk->queue is NULL. Fix error
handling in axon_ram_probe() to avoid doing that.
Also del_gendisk() does not drop a reference to gendisk allocated by
alloc_disk(). That has to be done by put_disk(). Add that call where
needed.
Reported-by: Dan Carpenter
S
Hi Michael,
On 2017/03/08 09:43PM, Michael Ellerman wrote:
> "Naveen N. Rao" writes:
>
> > With ABIv2, we offset 8 bytes into a function to get at the local entry
> > point.
> >
> > Acked-by: Ananth N Mavinakayanahalli
> > Acked-by: Michael Ellerman
> > Signed-off-by: Naveen N. Rao
> > ---
>
Em Wed, Mar 08, 2017 at 07:54:12PM +0530, Naveen N. Rao escreveu:
> Hi Michael,
>
> On 2017/03/08 09:43PM, Michael Ellerman wrote:
> > "Naveen N. Rao" writes:
> >
> > > With ABIv2, we offset 8 bytes into a function to get at the local entry
> > > point.
> > >
> > > Acked-by: Ananth N Mavinakayan
On 2017/03/08 11:29AM, Arnaldo Carvalho de Melo wrote:
> Em Wed, Mar 08, 2017 at 07:54:12PM +0530, Naveen N. Rao escreveu:
> > Hi Michael,
> >
> > On 2017/03/08 09:43PM, Michael Ellerman wrote:
> > > "Naveen N. Rao" writes:
> > >
> > > > With ABIv2, we offset 8 bytes into a function to get at th
On 03/08/2017 06:56 AM, Jan Kara wrote:
> It is invalid to call del_gendisk() when disk->queue is NULL. Fix error
> handling in axon_ram_probe() to avoid doing that.
>
> Also del_gendisk() does not drop a reference to gendisk allocated by
> alloc_disk(). That has to be done by put_disk(). Add that
Recently the kernel version on the github
repository did not match up with the master branch of
https://kernel.googlesource.com/pub/scm/linux/kernel/git/powerpc/linux
Specifically, github was hosting v4.10-rc5 while master had
v4.10-rc8. All the while Linus' mainline was at v4.11-rc1.
Was this a
On POWER8 (ISA 2.07) lxvx and stxvx are defined to be extended mnemonics
of lxvd2x and stxvd2x. For POWER9 (ISA 3.0) the HW architects in their
infinite wisdom made lxvx and stxvx instructions in their own right.
POWER9 aware GCC will use the POWER9 instruction for lxvx and stxvx
causing these sel
On 03/04/2017 07:05 AM, Russell King - ARM Linux wrote:
> On Fri, Mar 03, 2017 at 01:27:10PM +0100, Jiri Slaby wrote:
>> diff --git a/kernel/futex.c b/kernel/futex.c
>> index b687cb22301c..c5ff9850952f 100644
>> --- a/kernel/futex.c
>> +++ b/kernel/futex.c
>> @@ -1457,6 +1457,42 @@ futex_wake(u32 _
On 08/03/17 14:00, Alexey Kardashevskiy wrote:
> This reworks helpers for checking TCE update parameters in way they
> can be used in KVM.
>
> This should cause no behavioral change.
>
> Signed-off-by: Alexey Kardashevskiy
(for patchwork) Forgot this from v6:
Reviewed-by: David Gibson
> --
The following warning is seen when a CPU is hot unplugged on a PowerKVM
guest:
refcount_t: underflow; use-after-free.
[ cut here ]
WARNING: CPU: 0 PID: 53 at lib/refcount.c:128 refcount_sub_and_test+0xd8/0xf0
Modules linked in:
CPU: 0 PID: 53 Comm: kworker/u510:1 Not tainte
,Thomas Gleixner ,Ingo Molnar
,Chris Zankel ,Max Filippov
,Arnd Bergmann
,x...@kernel.org,linux-al...@vger.kernel.org,linux-snps-...@lists.infradead.org,linux-arm-ker...@lists.infradead.org,linux-hexa...@vger.kernel.org,linux-i...@vger.kernel.org,linux-m...@linux-mips.org,openr...@lists.librecor
On Tue, 28 Feb 2017 12:37:23 +0530
Mahesh Jagannath Salgaonkar wrote:
> On 02/28/2017 07:30 AM, Nicholas Piggin wrote:
> > +long __machine_check_early_realmode_p9(struct pt_regs *regs)
> > +{
> > + uint64_t nip, addr;
> > + long handled;
> > + struct mce_error_info mce_error_info = { 0 };
.llong is an undocumented PPC specific directive. The generic
equivalent is .quad, but even better (because it's self describing) is
.8byte.
Convert directives .llong -> .8byte
Signed-off-by: Tobin C. Harding
---
Fixes: issue #33 (github)
Patch is not tested. Has been built on Power8
scripts/
From: Philippe Reynes
Date: Sun, 5 Mar 2017 23:21:06 +0100
> The ethtool api {get|set}_settings is deprecated.
> We move this driver to new api {get|set}_link_ksettings.
>
> As I don't have the hardware, I'd be very pleased if
> someone may test this patch.
>
> Signed-off-by: Philippe Reynes
> Fix this by ensuring that of_node_put() is called only from the
> error path in dlpar_cpu_remove_by_index(). In the normal path,
> of_node_put() happens as part of dlpar_detach_node().
>
> Signed-off-by: Bharata B Rao
> Cc: Nathan Fontenot
> ---
> Changes in v1:
> - Fixed the refcount problem
"Naveen N. Rao" writes:
> On 2017/03/08 11:29AM, Arnaldo Carvalho de Melo wrote:
>> > I wasn't sure if you were planning on picking up KPROBES_ON_FTRACE for
>> > v4.11. If so, it would be good to take this patch through the powerpc
>> > tree. Otherwise, this can go via Ingo's tree.
>>
>> If you
36 matches
Mail list logo
