On Thu, 3 Dec 2015 11:48:24 +0000 Will Deacon <[email protected]> wrote:
> Hmm, so this should all be fine if we exclusively use the probe_kernel_* > functions and handle the -EFAULT gracefully. Now, that leaves an > interesting scenario with the flush_icache_range call in > aarch64_insn_patch_text_nosync, since that's not run with > KERNEL_DS/pagefault_disable() and so we'll panic if the text disappears > underneath us. Nothing should remove the text from underneath you if everything matches up fine before that. Module unloading will block on the ftrace_lock if it has any functions that can be traced. -- Steve > > So we probably need to add that code and call __flush_cache_user_range > instead. > > What do you think? > > Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
