Re: [PATCH v2] kdump: fix an error that can not parse the e820 reserved region

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



在 2018年10月02日 20:02, Simon Horman 写道:
> On Fri, Sep 07, 2018 at 09:05:50AM +0800, Dave Young wrote:
>> On 09/06/18 at 01:56pm, Lianbo Jiang wrote:
>>> When kexec-tools load the kernel and initramfs for kdump, kexec-tools will
>>> read /proc/iomem and recreate the e820 ranges for kdump kernel. But it fails
>>> to parse the e820 reserved region, because the memcmp() is case sensitive
>>> when comparing the string. In fact, it may be "Reserved" or "reserved" in
>>> the /proc/iomem, so we have to fix these cases.
>>>
>>> Signed-off-by: Lianbo Jiang <lijiang@xxxxxxxxxx>
>>> ---
>>>  kexec/arch/i386/crashdump-x86.c | 2 ++
>>>  1 file changed, 2 insertions(+)
>>>
>>> diff --git a/kexec/arch/i386/crashdump-x86.c b/kexec/arch/i386/crashdump-x86.c
>>> index 437e8a8..140f45b 100644
>>> --- a/kexec/arch/i386/crashdump-x86.c
>>> +++ b/kexec/arch/i386/crashdump-x86.c
>>> @@ -289,6 +289,8 @@ static int get_crash_memory_ranges(struct memory_range **range, int *ranges,
>>>  			type = RANGE_PMEM;
>>>  		} else if(memcmp(str,"reserved\n",9) == 0 ) {
>>>  			type = RANGE_RESERVED;
>>> +		} else if (memcmp(str, "Reserved\n", 9) == 0) {
>>> +			type = RANGE_RESERVED;
>>>  		} else if (memcmp(str, "GART\n", 5) == 0) {
>>>  			gart_start = start;
>>>  			gart_end = end;
>>> -- 
>>> 2.17.1
>>>
>>>
>>> _______________________________________________
>>> kexec mailing list
>>> kexec@xxxxxxxxxxxxxxxxxxx
>>> http://lists.infradead.org/mailman/listinfo/kexec
>>
>> Reviewed-by: Dave Young <dyoung@xxxxxxxxxx>
> 
> Sorry for letting this slip through the cracks.
> I've applied this now.
> 

Thank you, Simon and Dave.
> _______________________________________________
> kexec mailing list
> kexec@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/kexec
> 

_______________________________________________
kexec mailing list
kexec@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/kexec




[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux