Fix deadlock in IO pipeline
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 21, 2:48 PM

	Unknown Object (File)
	Tue, Apr 16, 10:33 AM

	Unknown Object (File)
	Fri, Apr 12, 5:18 AM

	Unknown Object (File)
	Fri, Apr 12, 5:15 AM

	Unknown Object (File)
	Fri, Apr 12, 3:25 AM

	Unknown Object (File)
	Wed, Apr 10, 4:04 AM

	Unknown Object (File)
	Dec 20 2023, 5:07 AM

	Unknown Object (File)
	Jun 30 2023, 1:17 PM

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