Fix deadlock in IO pipeline
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F133635375: D17495.diff
	Mon, Oct 27, 4:33 AM

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

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

	Unknown Object (File)
	Fri, Oct 10, 3:46 AM

	Unknown Object (File)
	Thu, Oct 9, 5:38 AM

	Unknown Object (File)
	Wed, Oct 8, 12:52 AM

	Unknown Object (File)
	Tue, Oct 7, 1:06 AM

	Unknown Object (File)
	Sep 9 2025, 2:44 AM

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

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 20108
Build 19603: arc lint + arc unit

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

sys/

cddl/

contrib/

opensolaris/

uts/

common/

fs/

zfs/

vdev_queue.c

12 lines

Diff 48960

View Options

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

Fix deadlock in IO pipelineClosedPublicActions