[Crash-utility] [PATCH 4/5] diskname, zram: Notify necessity of loading zram module

HAGIO KAZUHITO(萩尾 一仁) k-hagio-ab at nec.com
Mon Nov 2 03:03:02 UTC 2020


-----Original Message-----
> By the previous commit, trying to read swapped-out-into-zram-disk
> pages results in inaccessible memory error that is apparently
> irrelevant to zram and users cannot find it necessary to load zram
> module. Thus, let's add a warning message to indicate that.
> 
> Signed-off-by: HATAYAMA Daisuke <d.hatayama at fujitsu.com>
> ---
>  diskdump.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/diskdump.c b/diskdump.c
> index 304e9a1..cdbc2d5 100644
> --- a/diskdump.c
> +++ b/diskdump.c
> @@ -2751,8 +2751,13 @@ try_zram_decompress(ulonglong pte_val, unsigned char *buf, ulong len, ulonglong
> 
>  	if (INVALID_MEMBER(zram_compressor)) {
>  		zram_init();
> -		if (INVALID_MEMBER(zram_compressor))
> +		if (INVALID_MEMBER(zram_compressor)) {
> +			error(WARNING,
> +			      "Some pages are swapped out to zram. "
> +			      "Please load zram module.\n",
> +			      name);

Hi Hatayama-san,

Only a slight thing from me, this "name" looks unnecessary.
Otherwise, except for the compilation issue, looks good to me.

Thanks!
Kazu





More information about the Crash-utility mailing list