Re: [PATCH] kernel/crash_core.c : Remove redundant checks for ck_cmdline is NULL

2022-05-05 Thread Baoquan He
On 05/04/22 at 05:36am, lizhe wrote: > When ck_cmdline is NULL. The last three lines of > this function(get_last_crashkernel()) are equivalent to : > if (!NULL) > return NULL; > > return NULL; > This is obviously a redundant check Now the patch log correctly reflects

[PATCH] kernel/crash_core.c : Remove redundant checks for ck_cmdline is NULL

2022-05-04 Thread lizhe
When ck_cmdline is NULL. The last three lines of this function(get_last_crashkernel()) are equivalent to : if (!NULL) return NULL; return NULL; This is obviously a redundant check Signed-off-by: lizhe --- kernel/crash_core.c | 3 --- 1 file changed, 3

Re: [PATCH] kernel/crash_core.c : Remove redundant checks for ck_cmdline is NULL

2022-05-03 Thread Philipp Rudo
Hi lizhe, On Mon, 2 May 2022 18:11:20 +0800 (CST) lizhe wrote: > HI Philipp Rudo. > > > When ck_cmdline is NULL. The last three lines of this function are > equivalent to : > if ( ! NULL) > return NULL; > return NULL; > This is obviously a

Re: [PATCH] kernel/crash_core.c : Remove redundant checks for ck_cmdline is NULL

2022-04-26 Thread Philipp Rudo
Hi lizhe, On Mon, 25 Apr 2022 08:38:57 -0700 lizhe wrote: > When ck_cmdline is NULL, the only caller of get_last_crashkernel() > has already done non-NULL check(see __parse_crashkernel()), > so it doesn't make any sense to make a check here sorry, but I still don't like the description.

[PATCH] kernel/crash_core.c : Remove redundant checks for ck_cmdline is NULL

2022-04-25 Thread lizhe
When ck_cmdline is NULL, the only caller of get_last_crashkernel() has already done non-NULL check(see __parse_crashkernel()), so it doesn't make any sense to make a check here Signed-off-by: lizhe --- kernel/crash_core.c | 3 --- 1 file changed, 3 deletions(-) diff --git