All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] x86/ftrace: make ftrace_int3_handler() not to skip fops invocation
@ 2019-04-27 10:06 ` Nicolai Stange
  0 siblings, 0 replies; 192+ messages in thread
From: nstange @ 2019-04-27 10:06 UTC (permalink / raw)


Hi,

this series is the result of the discussion to the RFC patch found at [1].

The goal is to make x86' ftrace_int3_handler() not to simply skip over
the trapping instruction as this is problematic in the context of
the live patching consistency model. For details, c.f. the commit message
of [3/4] ("x86/ftrace: make ftrace_int3_handler() not to skip fops
invocation").

Everything is based on v5.1-rc6, please let me know in case you want me to
rebase on somehing else.

For x86_64, the live patching selftest added in [4/4] succeeds with this
series applied and fails without it. On 32 bits I only compile-tested.

checkpatch reports warnings about
- an overlong line in assembly -- I chose to ignore that
- MAINTAINERS perhaps needing updates due to the new files
  arch/x86/kernel/ftrace_int3_stubs.S and
  tools/testing/selftests/livepatch/test-livepatch-vs-ftrace.sh.
  As the existing arch/x86/kernel/ftrace_{32,64}.S haven't got an
  explicit entry either, this one is probably Ok? The selftest
  definitely is.

Changes to the RFC patch:
- s/trampoline/stub/ to avoid confusion with the ftrace_ops' trampolines,
- use a fixed size stack kept in struct thread_info for passing the
  (adjusted) ->ip values from ftrace_int3_handler() to the stubs,
- provide one stub for each of the two possible jump targets and hardcode
  those,
- add the live patching selftest.

Thanks,

Nicolai

Nicolai Stange (4):
  x86/thread_info: introduce ->ftrace_int3_stack member
  ftrace: drop 'static' qualifier from ftrace_ops_list_func()
  x86/ftrace: make ftrace_int3_handler() not to skip fops invocation
  selftests/livepatch: add "ftrace a live patched function" test

 arch/x86/include/asm/thread_info.h                 | 11 +++
 arch/x86/kernel/Makefile                           |  1 +
 arch/x86/kernel/asm-offsets.c                      |  8 +++
 arch/x86/kernel/ftrace.c                           | 79 +++++++++++++++++++---
 arch/x86/kernel/ftrace_int3_stubs.S                | 61 +++++++++++++++++
 kernel/trace/ftrace.c                              |  8 +--
 tools/testing/selftests/livepatch/Makefile         |  3 +-
 .../livepatch/test-livepatch-vs-ftrace.sh          | 44 ++++++++++++
 8 files changed, 199 insertions(+), 16 deletions(-)
 create mode 100644 arch/x86/kernel/ftrace_int3_stubs.S
 create mode 100755 tools/testing/selftests/livepatch/test-livepatch-vs-ftrace.sh

-- 
2.13.7

^ permalink raw reply	[flat|nested] 192+ messages in thread

end of thread, other threads:[~2019-05-01 19:42 UTC | newest]

Thread overview: 192+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-04-27 10:06 [PATCH 0/4] x86/ftrace: make ftrace_int3_handler() not to skip fops invocation nstange
2019-04-27 10:06 ` Nicolai Stange
2019-04-27 10:06 ` Nicolai Stange
2019-04-27 10:06 ` [PATCH 1/4] x86/thread_info: introduce ->ftrace_int3_stack member nstange
2019-04-27 10:06   ` Nicolai Stange
2019-04-27 10:06   ` Nicolai Stange
2019-04-28 17:41   ` luto
2019-04-28 17:41     ` Andy Lutomirski
2019-04-28 17:41     ` Andy Lutomirski
2019-04-28 17:51     ` rostedt
2019-04-28 17:51       ` Steven Rostedt
2019-04-28 17:51       ` Steven Rostedt
2019-04-28 18:08       ` luto
2019-04-28 18:08         ` Andy Lutomirski
2019-04-28 18:08         ` Andy Lutomirski
2019-04-28 19:43         ` rostedt
2019-04-28 19:43           ` Steven Rostedt
2019-04-28 19:43           ` Steven Rostedt
2019-04-28 20:56           ` luto
2019-04-28 20:56             ` Andy Lutomirski
2019-04-28 20:56             ` Andy Lutomirski
2019-04-28 21:22       ` nstange
2019-04-28 21:22         ` Nicolai Stange
2019-04-28 21:22         ` Nicolai Stange
2019-04-28 23:27         ` luto
2019-04-28 23:27           ` Andy Lutomirski
2019-04-28 23:27           ` Andy Lutomirski
2019-04-27 10:06 ` [PATCH 2/4] ftrace: drop 'static' qualifier from ftrace_ops_list_func() nstange
2019-04-27 10:06   ` Nicolai Stange
2019-04-27 10:06   ` Nicolai Stange
2019-04-27 10:06 ` [PATCH 3/4] x86/ftrace: make ftrace_int3_handler() not to skip fops invocation nstange
2019-04-27 10:06   ` Nicolai Stange
2019-04-27 10:06   ` Nicolai Stange
2019-04-27 10:26   ` peterz
2019-04-27 10:26     ` Peter Zijlstra
2019-04-27 10:26     ` Peter Zijlstra
2019-04-28 17:38     ` rostedt
2019-04-28 17:38       ` Steven Rostedt
2019-04-28 17:38       ` Steven Rostedt
2019-04-29 18:06       ` torvalds
2019-04-29 18:06         ` Linus Torvalds
2019-04-29 18:06         ` Linus Torvalds
2019-04-29 18:22         ` torvalds
2019-04-29 18:22           ` Linus Torvalds
2019-04-29 18:22           ` Linus Torvalds
2019-04-29 18:42           ` luto
2019-04-29 18:42             ` Andy Lutomirski
2019-04-29 18:42             ` Andy Lutomirski
     [not found]             ` <CAHk-=whtt4K2f0KPtG-4Pykh3FK8UBOjD8jhXCUKB5nWDj_YRA@mail.gmail.com>
2019-04-29 18:56               ` luto
2019-04-29 18:56                 ` Andy Lutomirski
2019-04-29 18:56                 ` Andy Lutomirski
     [not found]                 ` <CAHk-=wgewK4eFhF3=0RNtk1KQjMANFH6oDE=8m=84RExn2gxhw@mail.gmail.com>
     [not found]                   ` <CAHk-=whay7eN6+2gZjY-ybRbkbcqAmgrLwwszzHx8ws3c=S-MA@mail.gmail.com>
2019-04-29 19:24                     ` luto
2019-04-29 19:24                       ` Andy Lutomirski
2019-04-29 19:24                       ` Andy Lutomirski
2019-04-29 20:07                       ` torvalds
2019-04-29 20:07                         ` Linus Torvalds
2019-04-29 20:07                         ` Linus Torvalds
2019-04-30 13:56                         ` peterz
2019-04-30 13:56                           ` Peter Zijlstra
2019-04-30 13:56                           ` Peter Zijlstra
2019-04-30 16:06                           ` torvalds
2019-04-30 16:06                             ` Linus Torvalds
2019-04-30 16:06                             ` Linus Torvalds
2019-04-30 16:33                             ` luto
2019-04-30 16:33                               ` Andy Lutomirski
2019-04-30 16:33                               ` Andy Lutomirski
2019-04-30 17:03                               ` rostedt
2019-04-30 17:03                                 ` Steven Rostedt
2019-04-30 17:03                                 ` Steven Rostedt
2019-04-30 17:20                                 ` rostedt
2019-04-30 17:20                                   ` Steven Rostedt
2019-04-30 17:20                                   ` Steven Rostedt
2019-04-30 17:49                                   ` [RFC][PATCH] ftrace/x86: Emulate call function while updating in breakpoint handler rostedt
2019-04-30 17:49                                     ` Steven Rostedt
2019-04-30 17:49                                     ` Steven Rostedt
2019-04-30 18:33                                     ` torvalds
2019-04-30 18:33                                       ` Linus Torvalds
2019-04-30 18:33                                       ` Linus Torvalds
2019-04-30 19:00                                       ` rostedt
2019-04-30 19:00                                         ` Steven Rostedt
2019-04-30 19:00                                         ` Steven Rostedt
2019-04-30 21:08                                       ` rostedt
2019-04-30 21:08                                         ` Steven Rostedt
2019-04-30 21:08                                         ` Steven Rostedt
2019-05-01 13:11                                       ` peterz
2019-05-01 13:11                                         ` Peter Zijlstra
2019-05-01 13:11                                         ` Peter Zijlstra
2019-05-01 18:58                                         ` rostedt
2019-05-01 18:58                                           ` Steven Rostedt
2019-05-01 18:58                                           ` Steven Rostedt
2019-05-01 19:03                                           ` peterz
2019-05-01 19:03                                             ` Peter Zijlstra
2019-05-01 19:03                                             ` Peter Zijlstra
2019-05-01 19:03                                         ` torvalds
2019-05-01 19:03                                           ` Linus Torvalds
2019-05-01 19:03                                           ` Linus Torvalds
2019-05-01 19:13                                           ` peterz
2019-05-01 19:13                                             ` Peter Zijlstra
2019-05-01 19:13                                             ` Peter Zijlstra
2019-05-01 19:13                                           ` rostedt
2019-05-01 19:13                                             ` Steven Rostedt
2019-05-01 19:13                                             ` Steven Rostedt
2019-05-01 19:33                                             ` jikos
2019-05-01 19:33                                               ` Jiri Kosina
2019-05-01 19:33                                               ` Jiri Kosina
2019-05-01 19:41                                               ` peterz
2019-05-01 19:41                                                 ` Peter Zijlstra
2019-05-01 19:41                                                 ` Peter Zijlstra
2019-04-30 21:53                                     ` [RFC][PATCH v2] " rostedt
2019-04-30 21:53                                       ` Steven Rostedt
2019-04-30 21:53                                       ` Steven Rostedt
2019-05-01  1:35                                       ` rostedt
2019-05-01  1:35                                         ` Steven Rostedt
2019-05-01  1:35                                         ` Steven Rostedt
2019-05-01  1:58                                         ` torvalds
2019-05-01  1:58                                           ` Linus Torvalds
2019-05-01  1:58                                           ` Linus Torvalds
2019-05-01  8:26                                       ` nstange
2019-05-01  8:26                                         ` Nicolai Stange
2019-05-01  8:26                                         ` Nicolai Stange
2019-05-01 13:22                                         ` rostedt
2019-05-01 13:22                                           ` Steven Rostedt
2019-05-01 13:22                                           ` Steven Rostedt
2019-04-29 20:16                   ` [PATCH 3/4] x86/ftrace: make ftrace_int3_handler() not to skip fops invocation torvalds
2019-04-29 20:16                     ` Linus Torvalds
2019-04-29 20:16                     ` Linus Torvalds
2019-04-29 22:08                     ` sean.j.christopherson
2019-04-29 22:08                       ` Sean Christopherson
2019-04-29 22:08                       ` Sean Christopherson
2019-04-29 22:22                       ` torvalds
2019-04-29 22:22                         ` Linus Torvalds
2019-04-29 22:22                         ` Linus Torvalds
2019-04-30  0:08                         ` sean.j.christopherson
2019-04-30  0:08                           ` Sean Christopherson
2019-04-30  0:08                           ` Sean Christopherson
2019-04-30  0:45                           ` sean.j.christopherson
2019-04-30  0:45                             ` Sean Christopherson
2019-04-30  0:45                             ` Sean Christopherson
2019-04-30  2:26                             ` torvalds
2019-04-30  2:26                               ` Linus Torvalds
2019-04-30  2:26                               ` Linus Torvalds
2019-04-30 10:40                               ` peterz
2019-04-30 10:40                                 ` Peter Zijlstra
2019-04-30 10:40                                 ` Peter Zijlstra
2019-04-30 11:17                               ` jikos
2019-04-30 11:17                                 ` Jiri Kosina
2019-04-30 11:17                                 ` Jiri Kosina
2019-04-29 22:06                 ` torvalds
2019-04-29 22:06                   ` Linus Torvalds
2019-04-29 22:06                   ` Linus Torvalds
2019-04-30 11:18                   ` peterz
2019-04-30 11:18                     ` Peter Zijlstra
2019-04-30 11:18                     ` Peter Zijlstra
2019-04-29 18:52         ` rostedt
2019-04-29 18:52           ` Steven Rostedt
2019-04-29 18:52           ` Steven Rostedt
     [not found]           ` <CAHk-=wjm93jLtVxTX4HZs6K4k1Wqh3ujjmapqaYtcibVk_YnzQ@mail.gmail.com>
2019-04-29 19:07             ` rostedt
2019-04-29 19:07               ` Steven Rostedt
2019-04-29 19:07               ` Steven Rostedt
2019-04-29 20:06               ` torvalds
2019-04-29 20:06                 ` Linus Torvalds
2019-04-29 20:06                 ` Linus Torvalds
2019-04-29 20:20                 ` torvalds
2019-04-29 20:20                   ` Linus Torvalds
2019-04-29 20:20                   ` Linus Torvalds
2019-04-29 20:30                 ` rostedt
2019-04-29 20:30                   ` Steven Rostedt
2019-04-29 20:30                   ` Steven Rostedt
2019-04-29 21:38                   ` torvalds
2019-04-29 21:38                     ` Linus Torvalds
2019-04-29 21:38                     ` Linus Torvalds
2019-04-29 22:07                     ` rostedt
2019-04-29 22:07                       ` Steven Rostedt
2019-04-29 22:07                       ` Steven Rostedt
2019-04-30  9:24                       ` nstange
2019-04-30  9:24                         ` Nicolai Stange
2019-04-30  9:24                         ` Nicolai Stange
2019-04-30 10:46           ` peterz
2019-04-30 10:46             ` Peter Zijlstra
2019-04-30 10:46             ` Peter Zijlstra
2019-04-30 13:44             ` rostedt
2019-04-30 13:44               ` Steven Rostedt
2019-04-30 13:44               ` Steven Rostedt
2019-04-30 14:20               ` peterz
2019-04-30 14:20                 ` Peter Zijlstra
2019-04-30 14:20                 ` Peter Zijlstra
2019-04-30 14:36                 ` rostedt
2019-04-30 14:36                   ` Steven Rostedt
2019-04-30 14:36                   ` Steven Rostedt
2019-04-27 10:06 ` [PATCH 4/4] selftests/livepatch: add "ftrace a live patched function" test nstange
2019-04-27 10:06   ` Nicolai Stange
2019-04-27 10:06   ` Nicolai Stange

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.