On Thu, Oct 19, 2023 at 8:54 AM HAGIO KAZUHITO(萩尾 一仁) <k-hagio...@nec.com> wrote:
> > This depends on kernel version number, it might be more friendly for the > > distribution to output some debug info as below: > > > > if (CRASHDEBUG(1)) > > error(INFO, "ZRAM_FLAG_SHIFT=%lu\n", ZRAM_FLAG_SHIFT); > > > > Once the related kernel patches are backported to an old kernel, the > > similar checking may fail. But, at least we can know a little more with > the > > crash debug info. > > Sounds good, then I would like to show the shift value instead of mask > value, so changed a little. Is the attached patch OK? > > Looks better, thank you, Kazu. So: Ack Lianbo 6.1: > crash> rd 7fb60bdff000 > zram_flag_shift: 13 > ... > > RHEL8: > crash> rd 7ff668048000 > zram_flag_shift: 24 > > 4.18 and later kernels has ZRAM_LOCK, so I think usually the else-if > block isn't used and it's kind of a fail-safe for a future change. > > Anyway, thank you for the good comments. > > Thanks, > Kazu
-- Crash-utility mailing list Crash-utility@redhat.com https://listman.redhat.com/mailman/listinfo/crash-utility Contribution Guidelines: https://github.com/crash-utility/crash/wiki