On Tue, May 21, 2019 at 03:27:47PM -0400, Joe Lawrence wrote: > BTW, livepatching folks -- speaking of this window, does it make sense for > klp_init_object_loaded() to unconditionally frob the module section > permissions? Should it only bother iff it's going to apply > relocations/alternatives/paravirt?
Yeah, technically there shouldn't be a need to do the frobbing unless there are .klp.rela or .klp.arch sections for the given object. Though I'm not sure it really matters all that much since loading a livepatch is a pretty rare event. -- Josh
