Re: [PATCH] makedumpfile/s390: Add get_kaslr_offset() for s390x

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

 



Hi Kazu,

Sorry, I did't have a chance to try your patch yet. 
I will update mine next week.

Thanks,
Mikhail


On 03.01.2020 21:55, HAGIO KAZUHITO(萩尾 一仁) wrote:
> Hi Lianbo, Mikhail,
> 
>> -----Original Message-----
>>>>
>>>> In addition, the above code confused me, it will always return 0 on s390(please refer to my logs).
>>>
>>> The aim of get_kaslr_offset() here is only setting info->kaslr_offset to the value
>>> from vmcoreinfo for the SYMBOL_INIT() macro.
>>> (get_kaslr_offset() returns the kaslr offset in the resolve_config_entry().)
>>>
>> Thanks for your explanation, Kazu.
>>
>>> But yeah, the get_kaslr_offset(SYMBOL(_stext)) is confusing and not good.
>>> Passing 0 might be a bit better..?
>>>
>> Yes, looks good to me.
> 
> OK, I pushed an additional patch fixing it to the test branch:
> https://github.com/k-hagio/makedumpfile/tree/add-get_kaslr_offset_general
> Thanks Lianbo for pointing it out.
> 
> Mikhail, if you update your patch on top of the tree above,
> I'll merge it upstream.
> 
> Thanks,
> Kazu
> 
> P.S. My email address has been changed to k-hagio-ab@xxxxxxx.
> Please send email to this address in the future. Thanks.
> (Ugh, it seems I cannot remove my kanji name in the From: field..)
> 

_______________________________________________
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