Add detailed comments to clarify the purpose of klp_add_nops() function. These comments are based on Petr's explanation[0]. Link: https://lore.kernel.org/all/Z6XUA7D0eU_YDMVp@xxxxxxxxxxxxxxx/ [0] Suggested-by: Josh Poimboeuf <jpoimboe@xxxxxxxxxx> Signed-off-by: Yafang Shao <laoar.shao@xxxxxxxxx> Cc: Petr Mladek <pmladek@xxxxxxxx> --- kernel/livepatch/core.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c index 0cd39954d5a1..5b2a52e7c2f6 100644 --- a/kernel/livepatch/core.c +++ b/kernel/livepatch/core.c @@ -604,6 +604,9 @@ static int klp_add_object_nops(struct klp_patch *patch, * Add 'nop' functions which simply return to the caller to run * the original function. The 'nop' functions are added to a * patch to facilitate a 'replace' mode. + * + * The 'nop' entries are added only for functions which are currently + * livepatched but are no longer included in the new livepatch. */ static int klp_add_nops(struct klp_patch *patch) { -- 2.43.5
