On 2023/11/21 9:07, HAGIO KAZUHITO(萩尾 一仁) wrote:
> On 2023/11/20 17:43, lijiang wrote:
>
>>> Lianbo, zram related offsets have some typos, irregular names and lack
>>> of "help -o" output. I made the patch 2/2 in this opportunity, could
>>> you review this too?
>>>
>>>
>> Sorry for the late response, Kazu and Chengen.
>>
>> These two patches in attachment are fine to me, only two comments:
>> For patch [1], it has two related kernel commits, another one
>> is: 84b33bf78889 ("zram: introduce recompress sysfs knob")
>
> Thanks for the info, I'll add this.
sorry for the delay, applied.
https://github.com/crash-utility/crash/commit/38acd02c7fc09843ffb10fc2d695cccdd10cc7f6
Thanks,
Kazu
>
>> For patch[2], I saw the crash extensions include the defs.h, could you
>> please check if the relevant variables are referenced directly in the crash
>> extensions? If no, it does not break compatibility.
>
> As far as I've looked for, no extension module uses them. Even if any,
> the patch does not change the offset of the existing members, so it will
> not break extension modules previously compiled with old definitions.
>
> Thanks,
> Kazu
>
>>
>> Otherwise, for the patch[1] and [2]: Ack.
>>
>> Thanks.
>> Lianbo
>
> --
> Crash-utility mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxxxxxx
> To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxxxxxx
> %(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
> Contribution Guidelines: https://github.com/crash-utility/crash/wiki
--
Crash-utility mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxxxxxx
https://${domain_name}/admin/lists/devel.lists.crash-utility.osci.io/
Contribution Guidelines: https://github.com/crash-utility/crash/wiki
