int *used_number, int sector_size,
obd_dif_csum_fn *fn)
{
- unsigned int i;
+ unsigned int i = offset;
+ unsigned int end = offset + length;
char *data_buf;
__u16 *guard_buf = guard_start;
unsigned int data_size;
int used = 0;
data_buf = kmap(page) + offset;
- for (i = 0; i < length; i += sector_size) {
+ while (i < end) {
if (used >= guard_number) {
CERROR("%s: unexpected used guard number of DIF %u/%u, "
"data length %u, sector size %u: rc = %d\n",
sector_size, -E2BIG);
return -E2BIG;
}
- data_size = length - i;
- if (data_size > sector_size)
- data_size = sector_size;
+ data_size = min(round_up(i + 1, sector_size), end) - i;
*guard_buf = fn(data_buf, data_size);
guard_buf++;
data_buf += data_size;
+ i += data_size;
used++;
}
kunmap(page);
if [[ $(uname -m) = aarch64 ]]; then
# bug number: LU-11596
ALWAYS_EXCEPT+=" $GRANT_CHECK_LIST"
- # bug number: LU-11671 LU-11667 LU-11729 LU-4398
- ALWAYS_EXCEPT+=" 45 317 810 817"
+ # bug number: LU-11671 LU-11667 LU-4398
+ ALWAYS_EXCEPT+=" 45 317 817"
fi
# 5 12 (min)"
run_test 809 "Verify no SOM xattr store for DoM-only files"
test_810() {
- local ORIG
- local CSUM
-
- # t10 seem to dislike partial pages
- lctl set_param osc.*.checksum_type=adler
- lctl set_param fail_loc=0x411
- dd if=/dev/urandom of=$DIR/$tfile bs=10240 count=2
- ORIG=$(md5sum $DIR/$tfile)
- lctl set_param ldlm.namespaces.*osc*.lru_size=clear
- CSUM=$(md5sum $DIR/$tfile)
- set_checksum_type adler
- if [ "$ORIG" != "$CSUM" ]; then
- error "$ORIG != $CSUM"
- fi
+ [ $PARALLEL == "yes" ] && skip "skip parallel run"
+ $GSS && skip_env "could not run with gss"
+
+ set_checksums 1
+ stack_trap "set_checksums $ORIG_CSUM" EXIT
+ stack_trap "set_checksum_type $ORIG_CSUM_TYPE" EXIT
+
+ local csum
+ local before
+ local after
+ for csum in $CKSUM_TYPES; do
+ #define OBD_FAIL_OSC_NO_GRANT 0x411
+ $LCTL set_param osc.*.checksum_type=$csum fail_loc=0x411
+ for i in "10240 0" "10000 0" "4000 1" "500 1"; do
+ eval set -- $i
+ dd if=/dev/urandom of=$DIR/$tfile bs=$1 count=2 seek=$2
+ before=$(md5sum $DIR/$tfile)
+ $LCTL set_param ldlm.namespaces.*osc*.lru_size=clear
+ after=$(md5sum $DIR/$tfile)
+ [ "$before" == "$after" ] ||
+ error "$csum: $before != $after bs=$1 seek=$2"
+ done
+ done
}
run_test 810 "partial page writes on ZFS (LU-11663)"