The patch titled Subject: kernel/module.c: do not inline do_init_module() has been added to the -mm tree. Its filename is module-do-not-inline-do_init_module.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/module-do-not-inline-do_init_module.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/module-do-not-inline-do_init_module.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Jan Kiszka <jan.kiszka@xxxxxxxxxxx> Subject: kernel/module.c: do not inline do_init_module() This provides a reliable breakpoint target, required for automatic symbol loading via the gdb helper command 'lx-symbols'. Signed-off-by: Jan Kiszka <jan.kiszka@xxxxxxxxxxx> Cc: Rusty Russell <rusty@xxxxxxxxxxxxxxx> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> Cc: Jason Wessel <jason.wessel@xxxxxxxxxxxxx> Cc: Andi Kleen <andi@xxxxxxxxxxxxxx> Cc: Ben Widawsky <ben@xxxxxxxxxxxx> Cc: Borislav Petkov <bp@xxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- kernel/module.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff -puN kernel/module.c~module-do-not-inline-do_init_module kernel/module.c --- a/kernel/module.c~module-do-not-inline-do_init_module +++ a/kernel/module.c @@ -3011,8 +3011,13 @@ static void do_free_init(struct rcu_head kfree(m); } -/* This is where the real work happens */ -static int do_init_module(struct module *mod) +/* + * This is where the real work happens. + * + * Keep it uninlined to provide a reliable breakpoint target, e.g. for the gdb + * helper command 'lx-symbols'. + */ +static noinline int do_init_module(struct module *mod) { int ret = 0; struct mod_initfree *freeinit; _ Patches currently in -mm which might be from jan.kiszka@xxxxxxxxxxx are scripts-gdb-add-infrastructure.patch scripts-gdb-add-cache-for-type-objects.patch scripts-gdb-add-container_of-helper-and-convenience-function.patch scripts-gdb-add-module-iteration-class.patch scripts-gdb-add-lx-symbols-command.patch module-do-not-inline-do_init_module.patch scripts-gdb-add-automatic-symbol-reloading-on-module-insertion.patch scripts-gdb-add-internal-helper-and-convenience-function-to-look-up-a-module.patch scripts-gdb-add-get_target_endianness-helper.patch scripts-gdb-add-read_u16-32-64-helpers.patch scripts-gdb-add-lx-dmesg-command.patch scripts-gdb-add-task-iteration-class.patch scripts-gdb-add-helper-and-convenience-function-to-look-up-tasks.patch scripts-gdb-add-is_target_arch-helper.patch scripts-gdb-add-internal-helper-and-convenience-function-to-retrieve-thread_info.patch scripts-gdb-add-get_gdbserver_type-helper.patch scripts-gdb-add-internal-helper-and-convenience-function-for-per-cpu-lookup.patch scripts-gdb-add-lx_current-convenience-function.patch scripts-gdb-add-class-to-iterate-over-cpu-masks.patch scripts-gdb-add-lx-lsmod-command.patch scripts-gdb-add-basic-documentation.patch scripts-gdb-port-to-python3-gdb77.patch scripts-gdb-ignore-byte-compiled-python-files.patch scripts-gdb-use-a-generator-instead-of-iterator-for-task-list.patch scripts-gdb-convert-modulelist-to-generator-function.patch scripts-gdb-convert-cpulist-to-generator-function.patch scripts-gdb-define-maintainer.patch scripts-gdb-disable-pagination-while-printing-from-breakpoint-handler.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html