[LU-11665] sanity test 101c: FAIL: Small 4k read IO 240 !

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.13.0, Lustre 2.12.7
• Affects Version/s: Lustre 2.12.0, Lustre 2.12.6
• Labels: arm ppc
• Environment: Arch: aarch64 (client)
• Severity: 3
• Assignee: Andreas Dilger
• Reporter: Jian Yu

Description

sanity test 101c failed as follows:

== sanity test 101c: check stripe_size aligned read-ahead ============================================ 02:36:43 (1540521403)
CMD: trevis-66vm1 /usr/sbin/lctl set_param -n obdfilter.lustre-OST*.read_cache_enable=0 		osd-*.lustre-OST*.read_cache_enable=0 2>&1
CMD: trevis-66vm1 /usr/sbin/lctl set_param -n obdfilter.lustre-OST*.writethrough_cache_enable=0 		osd-*.lustre-OST*.writethrough_cache_enable=0 2>&1
osc.lustre-OST0000-osc-ffff80002ec4d800.rpc_stats=0
osc.lustre-OST0001-osc-ffff80002ec4d800.rpc_stats=0
osc.lustre-OST0002-osc-ffff80002ec4d800.rpc_stats=0
osc.lustre-OST0003-osc-ffff80002ec4d800.rpc_stats=0
osc.lustre-OST0004-osc-ffff80002ec4d800.rpc_stats=0
osc.lustre-OST0005-osc-ffff80002ec4d800.rpc_stats=0
osc.lustre-OST0006-osc-ffff80002ec4d800.rpc_stats=0

10.977066s, 59.7027MB/s
 sanity test_101c: @@@@@@ FAIL: Small 4k read IO 240 ! 

Maloo report: https://testing.whamcloud.com/test_sets/88bbf5c2-d9d0-11e8-b46b-52540065bddc

Comments

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/40702/
Subject: LU-11665 tests: check number of pages correctly
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 223df6c86b79a88f7d94777e53a28ca4a43b35b5

[Gerrit Updater]

James Nunez ([e-mail]) uploaded a new patch: https://review.whamcloud.com/40702
Subject: LU-11665 tests: check number of pages correctly
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: 7d464eda3aa50c7fab3c213bb9af25a9cb7aa45d

[Peter Jones]

Landed for 2.13

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/33772/
Subject: LU-11665 tests: check number of pages correctly
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: df0bcc96ee578d6661fcc63e82b94e1e569f9efe

[Andreas Dilger]

It looks like this test is easily fixed, it just needs to adjust the test based on the number of 64KB pages being read, instead of 4KB pages. 64KB = 1 page on ARM, so something like the following should be correct:

                for size in 1 2 4 8; do
                        local rpc=$(echo "$stats" |
                                    awk '($1 == "'$size':") {print $2; exit; }')
-                       [ $rpc != 0 ] &&
+                       [ $rpc != 0 ] && ((size * PAGE_SIZE < 65536)) &&
                                error "Small $((size*4))k read IO $rpc !"
                done

[Gerrit Updater]

Andreas Dilger ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33772
Subject: LU-11665 tests: check number of pages correctly
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 079dddccc8ae200f7df72bce9ac0c58614e083b7

[Gerrit Updater]

Jian Yu ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33653
Subject: LU-11665 tests: disable sanity test 101c for ARM
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: f0193cc0542e2cfae5bbdfa2ba69c3e802c4ff63