Re: [PATCH v2 4/7] diskname, zram: Notify necessity of loading zram module

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

 



-----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 as follows:
> 
>   crash> rd -u 0x7f520626e000
>   WARNING: Some pages are swapped out to zram. Please run mod -s zram.
>   rd: invalid user virtual address: 7f520626e000  type: "64-bit UVADDR"
> 
> Signed-off-by: HATAYAMA Daisuke <d.hatayama@xxxxxxxxxxx>
> ---
>  diskdump.c | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/diskdump.c b/diskdump.c
> index 587b45c..143ac3c 100644
> --- a/diskdump.c
> +++ b/diskdump.c
> @@ -2757,8 +2757,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 run mod -s zram.\n",

> +			      name);

This "name" is not needed, but if this patchset does not need another fix,
no need to resend.  We will fix it when merging.

Thanks,
Kazu

>  			return 0;
> +		}
>  	}
> 
>  	if (CRASHDEBUG(2))
> --
> 1.8.3.1
> 
> --
> Crash-utility mailing list
> Crash-utility@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/crash-utility


--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux