* [PATCH] ARM: kprobes-decode: add support for MOVW instruction
@ 2010-09-22 15:42 Will Deacon
2010-09-27 13:16 ` Will Deacon
[not found] ` <000201cb5e46$3265b8c0$97312a40$%deacon@arm.com>
0 siblings, 2 replies; 4+ messages in thread
From: Will Deacon @ 2010-09-22 15:42 UTC (permalink / raw)
To: linux-arm-kernel
The MOVW instruction moves a 16-bit immediate into the bottom halfword
of the destination register.
This patch ensures that kprobes leaves the 16-bit immediate intact, rather
than assume a 12-bit immediate and mask out the upper 4 bits.
Cc: Nicolas Pitre <nico@fluxnic.net>
Signed-off-by: Will Deacon <will.deacon@arm.com>
---
arch/arm/kernel/kprobes-decode.c | 7 ++++---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/arch/arm/kernel/kprobes-decode.c b/arch/arm/kernel/kprobes-decode.c
index 8bccbfa..2c1f005 100644
--- a/arch/arm/kernel/kprobes-decode.c
+++ b/arch/arm/kernel/kprobes-decode.c
@@ -1162,11 +1162,12 @@ space_cccc_001x(kprobe_opcode_t insn, struct arch_specific_insn *asi)
{
/*
* MSR : cccc 0011 0x10 xxxx xxxx xxxx xxxx xxxx
- * Undef : cccc 0011 0x00 xxxx xxxx xxxx xxxx xxxx
+ * Undef : cccc 0011 0100 xxxx xxxx xxxx xxxx xxxx
* ALU op with S bit and Rd == 15 :
* cccc 001x xxx1 xxxx 1111 xxxx xxxx xxxx
*/
- if ((insn & 0x0f900000) == 0x03200000 || /* MSR & Undef */
+ if ((insn & 0x0fb00000) == 0x03200000 || /* MSR */
+ (insn & 0x0ff00000) == 0x03400000 || /* Undef */
(insn & 0x0e10f000) == 0x0210f000) /* ALU s-bit, R15 */
return INSN_REJECTED;
@@ -1177,7 +1178,7 @@ space_cccc_001x(kprobe_opcode_t insn, struct arch_specific_insn *asi)
* *S (bit 20) updates condition codes
* ADC/SBC/RSC reads the C flag
*/
- insn &= 0xfff00fff; /* Rn = r0, Rd = r0 */
+ insn &= 0xffff0fff; /* Rd = r0 */
asi->insn[0] = insn;
asi->insn_handler = (insn & (1 << 20)) ? /* S-bit */
emulate_alu_imm_rwflags : emulate_alu_imm_rflags;
--
1.7.0.4
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH] ARM: kprobes-decode: add support for MOVW instruction
2010-09-22 15:42 [PATCH] ARM: kprobes-decode: add support for MOVW instruction Will Deacon
@ 2010-09-27 13:16 ` Will Deacon
[not found] ` <000201cb5e46$3265b8c0$97312a40$%deacon@arm.com>
1 sibling, 0 replies; 4+ messages in thread
From: Will Deacon @ 2010-09-27 13:16 UTC (permalink / raw)
To: linux-arm-kernel
> The MOVW instruction moves a 16-bit immediate into the bottom halfword
> of the destination register.
>
> This patch ensures that kprobes leaves the 16-bit immediate intact, rather
> than assume a 12-bit immediate and mask out the upper 4 bits.
>
> Cc: Nicolas Pitre <nico@fluxnic.net>
> Signed-off-by: Will Deacon <will.deacon@arm.com>
> ---
> arch/arm/kernel/kprobes-decode.c | 7 ++++---
> 1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/kernel/kprobes-decode.c b/arch/arm/kernel/kprobes-decode.c
> index 8bccbfa..2c1f005 100644
> --- a/arch/arm/kernel/kprobes-decode.c
> +++ b/arch/arm/kernel/kprobes-decode.c
> @@ -1162,11 +1162,12 @@ space_cccc_001x(kprobe_opcode_t insn, struct arch_specific_insn *asi)
> {
> /*
> * MSR : cccc 0011 0x10 xxxx xxxx xxxx xxxx xxxx
> - * Undef : cccc 0011 0x00 xxxx xxxx xxxx xxxx xxxx
> + * Undef : cccc 0011 0100 xxxx xxxx xxxx xxxx xxxx
> * ALU op with S bit and Rd == 15 :
> * cccc 001x xxx1 xxxx 1111 xxxx xxxx xxxx
> */
> - if ((insn & 0x0f900000) == 0x03200000 || /* MSR & Undef */
> + if ((insn & 0x0fb00000) == 0x03200000 || /* MSR */
> + (insn & 0x0ff00000) == 0x03400000 || /* Undef */
> (insn & 0x0e10f000) == 0x0210f000) /* ALU s-bit, R15 */
> return INSN_REJECTED;
>
> @@ -1177,7 +1178,7 @@ space_cccc_001x(kprobe_opcode_t insn, struct arch_specific_insn *asi)
> * *S (bit 20) updates condition codes
> * ADC/SBC/RSC reads the C flag
> */
> - insn &= 0xfff00fff; /* Rn = r0, Rd = r0 */
> + insn &= 0xffff0fff; /* Rd = r0 */
> asi->insn[0] = insn;
> asi->insn_handler = (insn & (1 << 20)) ? /* S-bit */
> emulate_alu_imm_rwflags : emulate_alu_imm_rflags;
> --
> 1.7.0.4
I forgot to mention; without this patch the kprobes smoke tests
(CONFIG_KPROBES_SANITY_TEST) fail on ARM. If nobody objects, I'll
submit this to the patch system this week.
Thanks,
Will
^ permalink raw reply [flat|nested] 4+ messages in thread[parent not found: <000201cb5e46$3265b8c0$97312a40$%deacon@arm.com>]
* [PATCH] ARM: kprobes-decode: add support for MOVW instruction
[not found] ` <000201cb5e46$3265b8c0$97312a40$%deacon@arm.com>
@ 2010-09-27 17:01 ` Nicolas Pitre
2010-09-27 17:13 ` Will Deacon
0 siblings, 1 reply; 4+ messages in thread
From: Nicolas Pitre @ 2010-09-27 17:01 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 27 Sep 2010, Will Deacon wrote:
> > The MOVW instruction moves a 16-bit immediate into the bottom halfword
> > of the destination register.
> >
> > This patch ensures that kprobes leaves the 16-bit immediate intact, rather
> > than assume a 12-bit immediate and mask out the upper 4 bits.
> >
> > Cc: Nicolas Pitre <nico@fluxnic.net>
> > Signed-off-by: Will Deacon <will.deacon@arm.com>
Acked-by: Nicolas Pitre <nicolas.pitre@linaro.org>
> > ---
> > arch/arm/kernel/kprobes-decode.c | 7 ++++---
> > 1 files changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/kernel/kprobes-decode.c b/arch/arm/kernel/kprobes-decode.c
> > index 8bccbfa..2c1f005 100644
> > --- a/arch/arm/kernel/kprobes-decode.c
> > +++ b/arch/arm/kernel/kprobes-decode.c
> > @@ -1162,11 +1162,12 @@ space_cccc_001x(kprobe_opcode_t insn, struct arch_specific_insn *asi)
> > {
> > /*
> > * MSR : cccc 0011 0x10 xxxx xxxx xxxx xxxx xxxx
> > - * Undef : cccc 0011 0x00 xxxx xxxx xxxx xxxx xxxx
> > + * Undef : cccc 0011 0100 xxxx xxxx xxxx xxxx xxxx
> > * ALU op with S bit and Rd == 15 :
> > * cccc 001x xxx1 xxxx 1111 xxxx xxxx xxxx
> > */
> > - if ((insn & 0x0f900000) == 0x03200000 || /* MSR & Undef */
> > + if ((insn & 0x0fb00000) == 0x03200000 || /* MSR */
> > + (insn & 0x0ff00000) == 0x03400000 || /* Undef */
> > (insn & 0x0e10f000) == 0x0210f000) /* ALU s-bit, R15 */
> > return INSN_REJECTED;
> >
> > @@ -1177,7 +1178,7 @@ space_cccc_001x(kprobe_opcode_t insn, struct arch_specific_insn *asi)
> > * *S (bit 20) updates condition codes
> > * ADC/SBC/RSC reads the C flag
> > */
> > - insn &= 0xfff00fff; /* Rn = r0, Rd = r0 */
> > + insn &= 0xffff0fff; /* Rd = r0 */
> > asi->insn[0] = insn;
> > asi->insn_handler = (insn & (1 << 20)) ? /* S-bit */
> > emulate_alu_imm_rwflags : emulate_alu_imm_rflags;
> > --
> > 1.7.0.4
>
> I forgot to mention; without this patch the kprobes smoke tests
> (CONFIG_KPROBES_SANITY_TEST) fail on ARM. If nobody objects, I'll
> submit this to the patch system this week.
>
> Thanks,
>
> Will
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread* [PATCH] ARM: kprobes-decode: add support for MOVW instruction
2010-09-27 17:01 ` Nicolas Pitre
@ 2010-09-27 17:13 ` Will Deacon
0 siblings, 0 replies; 4+ messages in thread
From: Will Deacon @ 2010-09-27 17:13 UTC (permalink / raw)
To: linux-arm-kernel
> On Mon, 27 Sep 2010, Will Deacon wrote:
>
> > > The MOVW instruction moves a 16-bit immediate into the bottom halfword
> > > of the destination register.
> > >
> > > This patch ensures that kprobes leaves the 16-bit immediate intact, rather
> > > than assume a 12-bit immediate and mask out the upper 4 bits.
> > >
> > > Cc: Nicolas Pitre <nico@fluxnic.net>
> > > Signed-off-by: Will Deacon <will.deacon@arm.com>
>
> Acked-by: Nicolas Pitre <nicolas.pitre@linaro.org>
Cheers Nicolas,
Submitted as 6412/1.
Will
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-09-27 17:13 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-22 15:42 [PATCH] ARM: kprobes-decode: add support for MOVW instruction Will Deacon
2010-09-27 13:16 ` Will Deacon
[not found] ` <000201cb5e46$3265b8c0$97312a40$%deacon@arm.com>
2010-09-27 17:01 ` Nicolas Pitre
2010-09-27 17:13 ` Will Deacon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).