[PATCH] kexec, remove panic_on_warn kernel parameter from kdump situations

[Prarit Bhargava]

On 01/02/2015 07:54 AM, Vivek Goyal wrote:
> On Tue, Dec 30, 2014 at 09:57:51AM -0500, Prarit Bhargava wrote:
>> panic_on_warn kernel parameter will cause the kernel to panic when a
>> WARN() is hit in the kernel.  This is not a good situation for the kdump
>> kernel because then it would be possible for the kdump kernel to panic in
>> a non-fatal WARN().
>>
>> This patch removes panic_on_warn as a kernel parameter for the kdump
>> kernel.
>>
> 
> I think modifying kexec-tools is not best place for this. It probably is better to take care of this in distribution specific scripts.
> 
> In the past we have learnt that it is best that kexec-tools does least
> amount of manipulation with command line.

Well .. here's the question to think about: what does adding panic_on_warn to
the kdump kernel get you?  AFAICT, nothing.

If panic_on_warn is specified, the only thing that will happen is that kdump
will fail (which is always bad IMO).  There is no real difference in the stack
trace between the WARN() and panic situations so there is no information loss.

So I disagree -- we should never specify panic_on_warn on kdump kernel.

P.

> 
> Thanks
> Vivek
> 
>> Signed-off-by: Prarit Bhargava <prarit at redhat.com>
>> Cc: Dave Young <dyoung at redhat.com>
>> Cc: Vivek Goyal <vgoyal at redhat.com>
>> Cc: WANG Chao <chaowang at redhat.com>
>> ---
>>  kexec/kexec.c |    4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/kexec/kexec.c b/kexec/kexec.c
>> index b088916..323cafb 100644
>> --- a/kexec/kexec.c
>> +++ b/kexec/kexec.c
>> @@ -1048,8 +1048,10 @@ char *get_command_line(void)
>>  	line[strlen(line) - 1] = '\0';
>>  
>>  	remove_parameter(line, "BOOT_IMAGE");
>> -	if (kexec_flags & KEXEC_ON_CRASH)
>> +	if (kexec_flags & KEXEC_ON_CRASH) {
>>  		remove_parameter(line, "crashkernel");
>> +		remove_parameter(line, "panic_on_warn");
>> +	}
>>  
>>  	return line;
>>  }
>> -- 
>> 1.7.9.3