Differential D31892

iscsi: Abort data-out tasks queued on a terminating session.
ClosedPublic
Actions

Authored by jhb on Sep 9 2021, 6:27 PM.

Tags

None

Referenced Files

	F164811568: D31892.id95233.diff
	Tue, Aug 4, 4:10 AM

	F164809349: D31892.id94929.diff
	Tue, Aug 4, 3:54 AM

	F164720497: D31892.diff
	Mon, Aug 3, 8:32 AM

	Unknown Object (File)
	Wed, Jul 22, 8:00 PM

	Unknown Object (File)
	Mon, Jul 20, 10:37 PM

	Unknown Object (File)
	Tue, Jul 14, 3:37 AM

	Unknown Object (File)
	Fri, Jul 10, 10:08 PM

	Unknown Object (File)
	Jul 4 2026, 2:26 PM

Subscribers

Details

Reviewers

Group Reviewers

Commits

rGb2f1beeeff10: iscsi: Abort data-out tasks queued on a terminating session.
rG0cd6e85e242b: iscsi: Abort data-out tasks queued on a terminating session.

Summary

cfiscsi_datamove_out() can race with cfiscsi_session_terminate_tasks()
and enqueue a new task after the latter function has aborted existing
tasks. This could result in a deadlock as
cfiscsi_session_terminate_tasks() waited forever for this task to
complete.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Sep 9 2021, 6:27 PM

Herald added a reviewer: cam. · View Herald TranscriptSep 9 2021, 6:27 PM

Herald added a subscriber: imp. · View Herald Transcript

jhb requested review of this revision.Sep 9 2021, 6:27 PM

Harbormaster completed remote builds in B41450: Diff 94929.Sep 9 2021, 6:27 PM

jhb added a parent revision: D31891: iscsi: Add a helper routine to abort a data-out task..Sep 9 2021, 6:27 PM

mav accepted this revision.Sep 12 2021, 12:59 AM

This revision is now accepted and ready to land.Sep 12 2021, 12:59 AM

Closed by commit rG0cd6e85e242b: iscsi: Abort data-out tasks queued on a terminating session. (authored by jhb). · Explain WhySep 15 2021, 8:30 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG0cd6e85e242b: iscsi: Abort data-out tasks queued on a terminating session..

jhb added a commit: rGb2f1beeeff10: iscsi: Abort data-out tasks queued on a terminating session..Oct 30 2021, 12:02 AM

Revision Contents
Changeset List

Path

Size

sys/

cam/

ctl/

ctl_frontend_iscsi.c

5 lines

Diff 95233

sys/cam/ctl/ctl_frontend_iscsi.c

Loading...