On Thu, Nov 13, 2014 at 11:16:00AM +0100, Miroslav Benes wrote: > > Hi, > > thank you for the first version of the united live patching core. > > The patch below implements some of our review objections. Changes are > described in the commit log. It simplifies the hierarchy of data > structures, removes data duplication (lp_ and lpc_ structures) and > simplifies sysfs directory. > > I did not try to repair other stuff (races, function names, function > prefix, api symmetry etc.). It should serve as a demonstration of our > point of view. > > There are some problems with this. try_module_get and module_put may be > called several times for each kernel module where some function is > patched in. This should be fixed with module going notifier as suggested > by Petr. > > The modified core was tested with modified testing live patch originally > from Seth's github. It worked as expected. > > Please take a look at these changes, so we can discuss them in more > detail. Hi Miroslav, Thanks for the code suggestions. This is a single patch with three major changes, which makes it hard to discuss each individual change on its own merits. Also, Seth has already made a lot of changes already based on previous comments, and is very close to having a v2 patch. Because this patch is so big, there are a lot of conflicts. Can you wait for the v2 patch set and then post your own patch set against it, with the patches split out so we can discuss them individually? -- Josh -- 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
