On Wed 2018-10-17 15:48:21, Josh Poimboeuf wrote: > On Mon, Oct 15, 2018 at 02:37:10PM +0200, Petr Mladek wrote: > > +void klp_discard_replaced_stuff(struct klp_patch *new_patch) > > +{ > > + klp_discard_replaced_patches(new_patch); > > + klp_discard_nops(new_patch); > > +} > > + > > +/* > > Stuff? Really? :-) How about klp_discard_replaced()? Anything looks confusing. I think that I'll just get rid of it and call the two functions directly. Best Regards, Petr