From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752046AbcBBBSG (ORCPT ); Mon, 1 Feb 2016 20:18:06 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34453 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751720AbcBBBSE (ORCPT ); Mon, 1 Feb 2016 20:18:04 -0500 From: Jessica Yu To: Josh Poimboeuf , Seth Jennings , Jiri Kosina , Vojtech Pavlik , Miroslav Benes , Rusty Russell , Steven Rostedt , Ingo Molnar Cc: live-patching@vger.kernel.org, linux-kernel@vger.kernel.org, Jessica Yu Subject: [PATCH v2 0/2] Fix ordering of ftrace/livepatch calls on module load and unload Date: Mon, 1 Feb 2016 20:17:34 -0500 Message-Id: <1454375856-27757-1-git-send-email-jeyu@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org As explained here [1], livepatch modules are failing to initialize properly because the ftrace coming module notifier (which calls ftrace_module_enable()) runs *after* the livepatch module notifier (which enables the patch(es)). Thus livepatch attempts to apply patches to modules before ftrace_module_enable() is even called for the corresponding module(s). As a result, patch modules break. Ftrace code must run before livepatch on module load, and the reverse is true on module unload. For ftrace and livepatch, order of initialization (plus exit/cleanup code) is important for loading and unloading modules, and using module notifiers to perform this work is not ideal since it is not always clear what gets called when. In this patchset, dependence on the module notifier call chain is removed in favor of hard coding the corresponding function calls in the module loader. This promotes better code visibility and ensures that ftrace and livepatch code get called in the correct order on patch module load and unload. Tested the changes with a test livepatch module that patches 9p and nilfs2, and verified that the issue described in [1] is fixed. Patches are based on linux-next. v1 can be found here - http://lkml.kernel.org/g/1454049827-3726-1-git-send-email-jeyu@redhat.com v2: - Instead of splitting the ftrace and livepatch notifiers into coming + going notifiers and adjusting their priorities, remove ftrace and livepatch notifiers completely and hard-code the necessary function calls in the module loader. [1] http://lkml.kernel.org/g/20160128204033.GA32131@packer-debian-8-amd64.digitalocean.com Jessica Yu (2): ftrace/module: remove ftrace module notifier livepatch/module: remove livepatch module notifier include/linux/ftrace.h | 6 +- include/linux/livepatch.h | 9 +++ kernel/livepatch/core.c | 144 ++++++++++++++++++++++------------------------ kernel/module.c | 12 ++++ kernel/trace/ftrace.c | 36 +----------- 5 files changed, 95 insertions(+), 112 deletions(-) -- 2.4.3