Page MenuHomeFreeBSD
Differential D22312

storvsc: port a Linux patch, properly set residual data length on errors
ClosedPublic
Actions

Authored by avg on Nov 11 2019, 1:23 PM.
Tags
None
Referenced Files
F153344670: D22312.id64169.diff
Mon, Apr 20, 2:47 PM
F153193654: D22312.diff
Sun, Apr 19, 5:41 PM
F153193601: D22312.diff
Sun, Apr 19, 5:40 PM
Unknown Object (File)
Wed, Apr 15, 1:47 PM
Unknown Object (File)
Wed, Apr 8, 8:08 PM
Unknown Object (File)
Mon, Apr 6, 2:50 PM
Unknown Object (File)
Mon, Apr 6, 3:37 AM
Unknown Object (File)
Mar 17 2026, 6:02 AM
View All Files
Subscribers
imp

Details

Reviewers
sepherosa_gmail.com
decui_microsoft.com
Commits
rS356730: storvsc: port a Linux patch, properly set residual data length on errors
Summary

This change is based on Linux commit 40630f462824ee.
csio.resid should account for transfer_len only for success and
SRB_STATUS_DATA_OVERRUN condition.

I am not sure how exactly this change works, but I have a report from a user
that they see lots of checksum errors when running a pool scrub concurrently
with iozone -l 1 -s 100G. After applying this patch the problem cannot be
reproduced.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
hyperv/
storvsc/
6 lines
3 lines

Diff 66739

View Options

head/sys/dev/hyperv/storvsc/hv_storvsc_drv_freebsd.c

Loading...
View Options

head/sys/dev/hyperv/storvsc/hv_vstorage.h

Loading...