[PATCH 0/5] ARM: kprobes: Fixes and additions for ARM instruction emulation (part 6)

Tixy tixy at yxit.co.uk
Thu Apr 28 13:05:44 EDT 2011


On Thu, 2011-04-28 at 17:55 +0100, Russell King - ARM Linux wrote:
> On Thu, Apr 28, 2011 at 12:49:18PM -0400, Nicolas Pitre wrote:
> > On Thu, 28 Apr 2011, Tixy wrote:
> > 
> > > On Thu, 2011-04-28 at 08:51 +0100, Russell King - ARM Linux wrote:
> > > > On Tue, Apr 19, 2011 at 10:52:15AM +0100, Tixy wrote:
> > > > > (This is a sixth and final set of patches)
> > > > > 
> > > > > When kprobes are inserted into code an ARM instruction is replaced
> > > > > by a breakpoint. When this is hit, the original instruction must be
> > > > > emulated out-of-line. This patchset fixes some bugs in the instruction
> > > > > decoding and adds emulation for some newer ARMv7 instructions.
> > > > > 
> > > > > [PATCH 1/5] ARM: kprobes: Reject probing of undefined data processing instructions
> > > > > [PATCH 2/5] ARM: kprobes: Add emulation of MOVW and MOVT instructions
> > > > > [PATCH 3/5] ARM: kprobes: Add emulation of SBFX, UBFX, BFI and BFC instructions
> > > > > [PATCH 4/5] ARM: kprobes: Add emulation of hint instructions like NOP and WFI
> > > > > [PATCH 5/5] ARM: kprobes: Tidy-up kprobes-decode.c
> > > > 
> > > > I notice that Nicolas merged the 5th set rather than this set - and
> > > > Nicolas hasn't responded to these.  Please check with Nicolas whether
> > > > he's updated to the latest set.
> > > 
> > > Nicolas, I see you have part 6 in your personal Linaro kprobes branch,
> > > but Russell seems to imply that this is missing from a merge you did
> > > elsewhere. (I'm unsure where this merge might be.)
> > 
> > All the kprobes patches I've reviewed are in the kprobes branch of 
> > git://git.linaro.org/people/nico/linux.  So far I have 31 patches in 
> > there.  If something is missing please let me know.
> > 
> > I merged the lot in the Linaro kernel tree as well, that's all.
> 
> I'm only concerned because you apparantly replied to v5 of the patch
> series saying you merged that, and next day Tixy sent v6.  So I'm
> wondering whether you merged v5 without the v6 updates or whether you
> did merge v6.

I can confirm that Nicolas has merged all six of my patch sets (29
patches total) and the 2 patches from Viktor. Nothing is missing.

-- 
Tixy




More information about the linux-arm-kernel mailing list