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