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

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

 



Hi Mikhail,

> -----Original Message-----
> Hello Kazu,
> 
> I've refreshed my patch on top of 'origin/add-get_kaslr_offset_general'. Please find it attached below.
> Hope there is still a chance to include it into the release.

Thank you. Your patch is queued for makedumpfile v1.6.7.

Kazu

> 
> Thanks,
> Mikhail Zaslonko
> 
> 
> On 06.01.2020 22:42, HAGIO KAZUHITO(萩尾 一仁) wrote:
> >
> >> -----Original Message-----
> >> Hi Kazu,
> >>
> >> Sorry, I did't have a chance to try your patch yet.
> >> I will update mine next week.
> >
> > OK.
> > FYI, I'm planning to relase the next version of makedumpfile by the end
> > of next week.  It needs some regression testing with old vmcores, etc., so
> > I can include patches merged by the beginning of next week in the release.
> >
> > Thanks,
> > Kazu
> >
> >>
> >> 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