Hi Petr, On Thu, 2016-03-03 at 17:52 +0100, Petr Mladek wrote: > From: Balbir Singh <bsingharora@xxxxxxxxx> > > Changelog v4: > 1. Renamed klp_matchaddr() to klp_get_ftrace_location() > and used it just to convert the function address. > 2. Synced klp_write_module_reloc() with s390(); made it > inline, no error message, return -ENOSYS OK. > 3. Added an error message when including > powerpc/include/asm/livepatch.h without HAVE_LIVEPATCH I don't know why we want to do that, I don't see how it is helpful. It doesn't even do what it says: > +#ifdef CONFIG_LIVEPATCH ... > +#else /* CONFIG_LIVEPATCH */ > +#error Include linux/livepatch.h, not asm/livepatch.h > +#endif /* CONFIG_LIVEPATCH */ If I turn on CONFIG_LIVEPATCH then I can quite happily include asm/livepatch.h and not get an error. So the check doesn't do what the message suggests. If we *really* want to prevent people from including asm/livepatch.h then it needs to check for _LINUX_LIVEPATCH_H_. But there's no reason I can see why we *must* prevent people from including asm/livepatch.h. And on x86 & s390 it does: #else #error Live patching support is disabled; check CONFIG_LIVEPATCH #endif Which is also not helpful AFAICS, and just likely to break the build for no good reason. > This applies on top of the patches posted by Michael > https://patchwork.ozlabs.org/patch/589791/ > > It enables livepatching. This takes patch 6/8 and 7/8 of v8 as the base. > (See the reference [1] below) and adds logic for checking offset ranges > in livepatch with ftrace_location_range. > > I tested the sample in the livepatch Thanks. cheers -- To unsubscribe from this list: send the line "unsubscribe live-patching" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
