[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <YxDLTbbsqER7yrvj@alley>
Date: Thu, 1 Sep 2022 17:10:05 +0200
From: Petr Mladek <pmladek@...e.com>
To: Zhen Lei <thunder.leizhen@...wei.com>
Cc: Josh Poimboeuf <jpoimboe@...nel.org>,
Jiri Kosina <jikos@...nel.org>,
Miroslav Benes <mbenes@...e.cz>,
Joe Lawrence <joe.lawrence@...hat.com>,
live-patching@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] livepatch: Add a missing newline character in
klp_module_coming()
On Tue 2022-08-30 19:28:55, Zhen Lei wrote:
> The error message is not printed immediately because it does not end with
> a newline character.
>
> Before:
> root@...alhost:~# insmod vmlinux.ko
> insmod: ERROR: could not insert module vmlinux.ko: Invalid parameters
>
> After:
> root@...alhost:~# insmod vmlinux.ko
> [ 43.982558] livepatch: vmlinux.ko: invalid module name
> insmod: ERROR: could not insert module vmlinux.ko: Invalid parameters
>
> Fixes: dcf550e52f56 ("livepatch: Disallow vmlinux.ko")
> Signed-off-by: Zhen Lei <thunder.leizhen@...wei.com>
Great catch! The message is not pushed to the console because
the line is not finished and might be eventually extended by pr_cont().
Reviewed-by: Petr Mladek <pmladek@...e.com>
The fix is trivial. I have pushed it into livepatching/livepatching.git,
branch for-6.1/fixes.
Best Regards,
Petr
Powered by blists - more mailing lists