Fix deadlock in IO pipeline
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F169586683: D17495.diff
	Wed, Sep 2, 12:03 AM

	Unknown Object (File)
	Fri, Aug 28, 10:39 PM

	Unknown Object (File)
	Fri, Aug 28, 10:38 PM

	Unknown Object (File)
	Fri, Aug 28, 10:37 PM

	Unknown Object (File)
	Fri, Aug 28, 10:08 PM

	Unknown Object (File)
	Fri, Aug 28, 1:54 AM

	Unknown Object (File)
	Sun, Aug 16, 6:43 PM

	Unknown Object (File)
	Sun, Aug 16, 9:23 AM

View All 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