linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC] kprobe'ing conditionally executed instructions
Date: Fri, 11 Dec 2015 10:34:59 +0000	[thread overview]
Message-ID: <20151211103459.GS8644@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <1449829633.2815.27.camel@linaro.org>

On Fri, Dec 11, 2015 at 10:27:13AM +0000, Jon Medhurst (Tixy) wrote:
> On Fri, 2015-12-11 at 00:05 -0500, David Long wrote:
> > There is a moderate amount of code already in kprobes on ARM and the 
> > current ARMv8 patch to deal with conditional execution of instructions. 
> > One aspect of how this is handled is that instructions that fail their 
> > predicate and are not (technically) executed are also not treated as a 
> > hit kprobe. Steve Capper has suggested that the probe handling should 
> > still take place because we stepped through the instruction even if it 
> > was effectively a nop.  This would be a significant change in how it 
> > currently works on 32-bit ARM
> 
> 32-bit ARM uses undefined instructions for kprobe 'breakpoints' and the
> ARM ARM says it's implementation defined behaviour whether these
> generate exceptions or not, i.e. whether the kprobe handler will be
> called.

There are two classes of undefined instructions.  There are those which
fall into the above category, and there are those which are guaranteed
to raise an exception.  We should always be using the guaranteed ones,
not the other set.

-- 
RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

  reply	other threads:[~2015-12-11 10:34 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-11  5:05 [RFC] kprobe'ing conditionally executed instructions David Long
2015-12-11  9:34 ` Steve Capper
2015-12-11 10:27 ` Jon Medhurst (Tixy)
2015-12-11 10:34   ` Russell King - ARM Linux [this message]
2015-12-11 12:13     ` Jon Medhurst (Tixy)
2015-12-11 16:09 ` William Cohen
2015-12-12  5:56   ` David Long
2015-12-12 18:48     ` William Cohen
2015-12-12 20:13       ` David Long

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151211103459.GS8644@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).