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

	F132219052: D22312.id64169.diff
	Tue, Oct 14, 9:52 PM

	Unknown Object (File)
	Mon, Oct 13, 11:06 AM

	Unknown Object (File)
	Wed, Oct 8, 6:08 AM

	Unknown Object (File)
	Wed, Oct 8, 5:00 AM

	Unknown Object (File)
	Thu, Sep 25, 6:25 AM

	Unknown Object (File)
	Sep 1 2025, 11:02 PM

	Unknown Object (File)
	Aug 28 2025, 3:26 AM

	Unknown Object (File)
	Aug 20 2025, 11:57 AM

View All 99 Files

Subscribers

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

Event Timeline

avg created this revision.Nov 11 2019, 1:23 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 11 2019, 1:23 PM

Harbormaster completed remote builds in B27440: Diff 64169.Nov 11 2019, 1:23 PM

add a missing header change

Harbormaster completed remote builds in B27442: Diff 64171.Nov 11 2019, 1:34 PM

avg edited the summary of this revision. (Show Details)Nov 11 2019, 1:35 PM

Ping.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 14 2020, 1:20 PM

Closed by commit rS356730: storvsc: port a Linux patch, properly set residual data length on errors (authored by avg). · Explain Why

This revision was automatically updated to reflect the committed changes.

avg added a commit: rS356730: storvsc: port a Linux patch, properly set residual data length on errors.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

hyperv/

storvsc/

hv_storvsc_drv_freebsd.c

6 lines

3 lines

Diff 66739

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

Loading...

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

Loading...