From: Jan Stancek <jstancek@redhat.com>
To: ltp@lists.linux.it
Subject: [LTP] [v2] readahead02: fix read_ahead_kb file not found
Date: Mon, 5 Sep 2016 03:36:33 -0400 (EDT) [thread overview]
Message-ID: <747949240.1570236.1473060993124.JavaMail.zimbra@redhat.com> (raw)
In-Reply-To: <1472812953-30811-1-git-send-email-naresh.kamboju@linaro.org>
----- Original Message -----
> From: "naresh kamboju" <naresh.kamboju@linaro.org>
> To: chrubis@suse.cz, ltp@lists.linux.it
> Cc: akpm@linux-foundation.org, "fengguang wu" <fengguang.wu@intel.com>, "graeme gregory" <graeme.gregory@linaro.org>,
> "milosz wasilewski" <milosz.wasilewski@linaro.org>
> Sent: Friday, 2 September, 2016 12:42:33 PM
> Subject: [LTP] [v2] readahead02: fix read_ahead_kb file not found
>
> From: Naresh Kamboju <naresh.kamboju@linaro.org>
>
> Test case is looking for read_ahead_kb file on a partition device (8:2) and
> not on block device (8:0/8:16/8:32/8:48). The appropriate block device that
> holds the partition can be found at queue/read_ahead_kb entry.
>
> Before patch readahead02 test broken,
> readahead02 1 TBROK : safe_file_ops.c:144: Failed to open FILE
> '/sys/dev/block/8:2/queue/read_ahead_kb' for reading at
> readahead02.c:197: errno=ENOENT(2): No such file or directory
>
> After applying patch,
> readahead02 0 TINFO : creating test file of size: 67108864
> readahead02 0 TINFO : max readahead size is: 131072
> readahead02 0 TINFO : read_testfile(0)
> readahead02 0 TINFO : max readahead size is: 131072
> readahead02 0 TINFO : read_testfile(1)
> readahead02 0 TINFO : max readahead size is: 131072
> readahead02 1 TPASS : expected ret success - returned value = 0
> readahead02 2 TPASS : offset is still at 0 as expected
> readahead02 0 TINFO : read_testfile(0) took: 1437202 usec
> readahead02 0 TINFO : read_testfile(1) took: 1361915 usec
> readahead02 0 TINFO : read_testfile(0) read: 67108864 bytes
> readahead02 0 TINFO : read_testfile(1) read: 0 bytes
> readahead02 3 TPASS : readahead saved some I/O
> readahead02 0 TINFO : cache can hold at least: 157404 kB
> readahead02 0 TINFO : read_testfile(0) used cache: 65332 kB
> readahead02 0 TINFO : read_testfile(1) used cache: 65636 kB
> readahead02 4 TPASS : using cache as expected
This doesn't seem to be true for all entries under /sys/dev/block.
If I ran your patch on lvm, testcase fails:
# ./readahead02
readahead02 0 TINFO : creating test file of size: 67108864
readahead02 1 TBROK : safe_file_ops.c:144: Failed to open FILE '/sys/dev/block/253:1/../queue/read_ahead_kb' for reading at readahead02.c:202: errno=ENOENT(2): No such file or directory
readahead02 2 TBROK : safe_file_ops.c:144: Remaining cases broken
# ll /sys/dev/block/253:1
lrwxrwxrwx. 1 root root 0 Sep 5 09:21 /sys/dev/block/253:1 -> ../../devices/virtual/block/dm-1
# ll /sys/dev/block/253:1/
total 0
-r--r--r--. 1 root root 4096 Sep 5 09:25 alignment_offset
-rw-r--r--. 1 root root 4096 Sep 5 09:25 badblocks
lrwxrwxrwx. 1 root root 0 Sep 5 09:21 bdi -> ../../bdi/253:1
-r--r--r--. 1 root root 4096 Sep 5 09:25 capability
-r--r--r--. 1 root root 4096 Sep 5 09:25 dev
-r--r--r--. 1 root root 4096 Sep 5 09:25 discard_alignment
drwxr-xr-x. 2 root root 0 Sep 5 09:25 dm
-r--r--r--. 1 root root 4096 Sep 5 09:25 ext_range
drwxr-xr-x. 2 root root 0 Sep 5 09:25 holders
-r--r--r--. 1 root root 4096 Sep 5 09:25 inflight
drwxr-xr-x. 2 root root 0 Sep 5 09:25 integrity
drwxr-xr-x. 2 root root 0 Sep 5 09:25 power
drwxr-xr-x. 2 root root 0 Sep 5 09:25 queue
-r--r--r--. 1 root root 4096 Sep 5 09:25 range
-r--r--r--. 1 root root 4096 Sep 5 09:25 removable
-r--r--r--. 1 root root 4096 Sep 5 09:25 ro
-r--r--r--. 1 root root 4096 Sep 5 09:25 size
drwxr-xr-x. 2 root root 0 Sep 5 09:25 slaves
-r--r--r--. 1 root root 4096 Sep 5 09:25 stat
lrwxrwxrwx. 1 root root 0 Sep 5 09:21 subsystem -> ../../../../class/block
drwxr-xr-x. 2 root root 0 Sep 5 09:25 trace
-rw-r--r--. 1 root root 4096 Sep 5 09:25 uevent
# lsblk
NAME MAJ:MIN RM SIZE RO TYPE MOUNTPOINT
vda 252:0 0 35G 0 disk
├─vda1 252:1 0 500M 0 part /boot
└─vda2 252:2 0 34.5G 0 part
├─rhel_dhcp--27--118-swap 253:0 0 2G 0 lvm [SWAP]
└─rhel_dhcp--27--118-root 253:1 0 32.5G 0 lvm /
# uname -r
4.7.0-rc7+
Regards,
Jan
next prev parent reply other threads:[~2016-09-05 7:36 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-02 10:42 [LTP] [v2] readahead02: fix read_ahead_kb file not found naresh.kamboju
2016-09-05 7:36 ` Jan Stancek [this message]
2016-09-06 12:33 ` Naresh Kamboju
2016-09-06 14:22 ` Jan Stancek
2016-09-13 13:56 ` Jan Stancek
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=747949240.1570236.1473060993124.JavaMail.zimbra@redhat.com \
--to=jstancek@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox