From: Richard Palethorpe <rpalethorpe@suse.de>
To: Ian Wienand <iwienand@redhat.com>
Cc: ltp@lists.linux.it
Subject: Re: [LTP] [PATCH v2] kernel/device-drivers/zram/zram01.sh : don't fill from /dev/zero
Date: Wed, 30 Aug 2023 09:20:44 +0100 [thread overview]
Message-ID: <87il8xhr05.fsf@suse.de> (raw)
In-Reply-To: <20230808035641.364676-2-iwienand@redhat.com>
Hello,
Ian Wienand <iwienand@redhat.com> writes:
> I have a system (virtualized aarch64, 4.18.0 kernel) that is
> consistently failing the zram01.sh test as it tries to divide the
> memory stats by zero. This has been reported before at [1] without
> resolution.
>
> After some investigation [2] my conclusion is that this zero value
> represents the pages allocated for compressed storage in the zram
> device, and due to same-page deduplication the extant method of
> filling with all-zeros can indeed lead us to not having any compressed
> data to measure.
>
> This is visible in the occasional divide-by-zero error, but in the
> bigger picture means this test is not exercising the compression path
> as desired.
Do zram{02,03} already do something similar?
In any case I'd prefer to see a zram04 written in C if some coverage is
missing.
--
Thank you,
Richard.
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next prev parent reply other threads:[~2023-08-30 8:30 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-03 1:51 [LTP] [PATCH] kernel/device-drivers/zram/zram01.sh : add a sync Ian Wienand
2023-08-03 10:52 ` Cyril Hrubis
2023-08-03 10:59 ` Martin Doucha
2023-08-03 11:02 ` Cyril Hrubis
2023-08-03 12:32 ` Ian Wienand
2023-08-08 3:56 ` [LTP] [PATCH v2] kernel/device-drivers/zram/zram01.sh : don't fill from /dev/zero Ian Wienand
2023-08-30 8:20 ` Richard Palethorpe [this message]
2023-09-07 6:46 ` Ian Wienand
2023-09-07 8:26 ` Richard Palethorpe
2023-09-08 1:58 ` Ian Wienand
2023-09-07 10:18 ` Martin Doucha
2023-09-07 22:29 ` Ian Wienand
2023-09-08 9:21 ` Martin Doucha
2023-09-12 1:03 ` Ian Wienand
2023-09-13 14:35 ` Richard Palethorpe
2023-09-13 22:21 ` Ian Wienand
2023-09-14 7:37 ` Richard Palethorpe
2023-09-14 11:04 ` Ian Wienand
2023-09-18 8:24 ` Richard Palethorpe
2023-09-21 1:17 ` Ian Wienand
2023-09-21 9:34 ` Richard Palethorpe
2023-09-21 1:12 ` [LTP] [PATCH v3] kernel/device-drivers/zram/zram01.sh : fill with compressible data Ian Wienand
2023-11-22 11:24 ` Richard Palethorpe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87il8xhr05.fsf@suse.de \
--to=rpalethorpe@suse.de \
--cc=iwienand@redhat.com \
--cc=ltp@lists.linux.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.