From: Masami Hiramatsu <mhiramat@kernel.org>
To: Steven Rostedt <rostedt@goodmis.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
Shuah Khan <shuah@kernel.org>,
Arnaldo Carvalho de Melo <acme@kernel.org>,
Peter Zijlstra <peterz@infradead.org>
Cc: mhiramat@kernel.org, linux-kernel@vger.kernel.org,
Andy Lutomirski <luto@amacapital.net>,
Ingo Molnar <mingo@kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
Changbin Du <changbin.du@gmail.com>, Jann Horn <jannh@google.com>,
Kees Cook <keescook@chromium.org>,
Andy Lutomirski <luto@kernel.org>,
Alexei Starovoitov <alexei.starovoitov@gmail.com>,
Nadav Amit <namit@vmware.com>,
Joel Fernandes <joel@joelfernandes.org>,
yhs@fb.com
Subject: [PATCH -tip v9 0/6] tracing/probes: uaccess: Add support user-space access
Date: Wed, 15 May 2019 14:37:44 +0900 [thread overview]
Message-ID: <155789866428.26965.8344923934342528416.stgit@devnote2> (raw)
Hi,
Here is the v9 series of probe-event to support user-space access.
Previous version is here.
https://lkml.kernel.org/r/155741476971.28419.15837024173365724167.stgit@devnote2
In this version, I fixed more typos/style issues.
Changes in v9:
[3/6]
- Fix other style & coding issues (Thanks Ingo!)
- Update fetch_store_string() for style consistency.
[4/6]
- Remove an unneeded line break.
- Move || and && in if-condition at the end of line.
In summary, strncpy_from_user() should work as below
- strncpy_from_user() can access user memory with set_fs(USER_DS)
in task context
- strncpy_from_user() can access kernel memory with set_fs(KERNEL_DS)
in task context (e.g. devtmpfsd and init)
- strncpy_from_user() can access user/kernel memory (depends on DS)
in IRQ context if pagefault is disabled. (both verified)
Note that this changes the warning behavior when
CONFIG_DEBUG_ATOMIC_SLEEP=y, it still warns when
__copy_from_user_inatomic() is called in IRQ context, but don't
warn if pagefault is disabled because it will not sleep in
atomic.
====
Kprobe event user-space memory access features:
For user-space access extension, this series adds 2 features,
"ustring" type and user-space dereference syntax. "ustring" is
used for recording a null-terminated string in user-space from
kprobe events.
"ustring" type is easy, it is able to use instead of "string"
type, so if you want to record a user-space string via
"__user char *", you can use ustring type instead of string.
For example,
echo 'p do_sys_open path=+0($arg2):ustring' >> kprobe_events
will record the path string from user-space.
The user-space dereference syntax is also simple. Thi just
adds 'u' prefix before an offset value.
+|-u<OFFSET>(<FETCHARG>)
e.g. +u8(%ax), +u0(+0(%si))
This is more generic. If you want to refer the variable in user-
space from its address or access a field in data structure in
user-space, you need to use this.
For example, if you probe do_sched_setscheduler(pid, policy,
param) and record param->sched_priority, you can add new
probe as below;
p do_sched_setscheduler priority=+u0($arg3)
Actually, with this feature, "ustring" type is not absolutely
necessary, because these are same meanings.
+0($arg2):ustring == +u0($arg2):string
Note that kprobe event provides these methods, but it doesn't
change it from kernel to user automatically because we do not
know whether the given address is in userspace or kernel on
some arch.
Thank you,
---
Masami Hiramatsu (6):
x86/uaccess: Allow access_ok() in irq context if pagefault_disabled
uaccess: Add non-pagefault user-space read functions
tracing/probe: Add ustring type for user-space string
tracing/probe: Support user-space dereference
selftests/ftrace: Add user-memory access syntax testcase
perf-probe: Add user memory access attribute support
Documentation/trace/kprobetrace.rst | 28 ++++-
Documentation/trace/uprobetracer.rst | 10 +-
arch/x86/include/asm/uaccess.h | 4 -
include/linux/uaccess.h | 19 +++
kernel/trace/trace.c | 7 +
kernel/trace/trace_kprobe.c | 48 +++++++-
kernel/trace/trace_probe.c | 37 +++++-
kernel/trace/trace_probe.h | 3
kernel/trace/trace_probe_tmpl.h | 36 +++++-
kernel/trace/trace_uprobe.c | 19 +++
mm/maccess.c | 122 +++++++++++++++++++-
tools/perf/Documentation/perf-probe.txt | 3
tools/perf/util/probe-event.c | 11 ++
tools/perf/util/probe-event.h | 2
tools/perf/util/probe-file.c | 7 +
tools/perf/util/probe-file.h | 1
tools/perf/util/probe-finder.c | 19 ++-
.../ftrace/test.d/kprobe/kprobe_args_user.tc | 32 +++++
18 files changed, 364 insertions(+), 44 deletions(-)
create mode 100644 tools/testing/selftests/ftrace/test.d/kprobe/kprobe_args_user.tc
--
Masami Hiramatsu (Linaro) <mhiramat@kernel.org>
next reply other threads:[~2019-05-15 5:38 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-15 5:37 Masami Hiramatsu [this message]
2019-05-15 5:38 ` [PATCH -tip v9 1/6] x86/uaccess: Allow access_ok() in irq context if pagefault_disabled Masami Hiramatsu
2019-05-15 5:38 ` [PATCH -tip v9 2/6] uaccess: Add non-pagefault user-space read functions Masami Hiramatsu
2019-05-15 5:38 ` [PATCH -tip v9 3/6] tracing/probe: Add ustring type for user-space string Masami Hiramatsu
2019-05-15 5:38 ` [PATCH -tip v9 4/6] tracing/probe: Support user-space dereference Masami Hiramatsu
2019-05-15 5:38 ` [PATCH -tip v9 5/6] selftests/ftrace: Add user-memory access syntax testcase Masami Hiramatsu
2019-05-15 5:39 ` [PATCH -tip v9 6/6] perf-probe: Add user memory access attribute support Masami Hiramatsu
2019-05-15 5:55 ` [PATCH -tip v9 0/6] tracing/probes: uaccess: Add support user-space access Ingo Molnar
2019-05-15 8:49 ` Masami Hiramatsu
2019-05-20 15:26 ` Steven Rostedt
2019-05-21 8:28 ` Ingo Molnar
2019-05-21 12:57 ` Steven Rostedt
2019-05-21 15:57 ` Masami Hiramatsu
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=155789866428.26965.8344923934342528416.stgit@devnote2 \
--to=mhiramat@kernel.org \
--cc=acme@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=alexei.starovoitov@gmail.com \
--cc=changbin.du@gmail.com \
--cc=jannh@google.com \
--cc=joel@joelfernandes.org \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=luto@amacapital.net \
--cc=luto@kernel.org \
--cc=mingo@kernel.org \
--cc=namit@vmware.com \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
--cc=shuah@kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=yhs@fb.com \
/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).