Fix deadlock in IO pipeline
ClosedPublic
Actions

Authored by allanjude on Oct 10 2018, 5:25 AM.

Tags

None

Referenced Files

	F135155235: D17495.id48996.diff
	Fri, Nov 7, 12:01 AM

	F135138686: D17495.id.diff
	Thu, Nov 6, 8:16 PM

	Unknown Object (File)
	Thu, Nov 6, 10:21 AM

	Unknown Object (File)
	Thu, Nov 6, 9:21 AM

	Unknown Object (File)
	Mon, Oct 27, 9:27 PM

	Unknown Object (File)
	Mon, Oct 27, 4:33 AM

	Unknown Object (File)
	Fri, Oct 24, 8:44 AM

	Unknown Object (File)
	Sun, Oct 19, 4:23 AM

View All 64 Files

Subscribers

delphij

imp

Details

Reviewers

mav
mmacy
sef

Group Reviewers

ZFS

Commits

rS339299: Pull in a follow-on commit to resolve a deadlock in ZFS sequential

Summary

Merge https://github.com/zfsonlinux/zfs/commit/a76f3d0437e5e974f0f748f8735af3539443b388 from ZoL

This is a follow-on commit to the sequential scrub code that was imported earlier.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

allanjude created this revision.Oct 10 2018, 5:25 AM

Herald added subscribers: delphij, imp. · View Herald TranscriptOct 10 2018, 5:25 AM

Harbormaster completed remote builds in B20108: Diff 48960.Oct 10 2018, 5:25 AM

This missing code was identified during the ZFS call earlier today.

Dropping locks inside the function is a mess, but I don't see specific problems.

This revision is now accepted and ready to land.Oct 10 2018, 3:10 PM

Closed by commit rS339299: Pull in a follow-on commit to resolve a deadlock in ZFS sequential (authored by allanjude). · Explain WhyOct 10 2018, 10:59 PM

This revision was automatically updated to reflect the committed changes.

Looks ok to me.

Revision Contents
Changeset List

Path

Size

head/

sys/

cddl/

contrib/

opensolaris/

uts/

common/

fs/

zfs/

vdev_queue.c

12 lines

Diff 48996

View Options

head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_queue.c

Fix deadlock in IO pipelineClosedPublicActions