Differential D55523

nda: Assume all cases in ndaasync can sleep
ClosedPublic
Actions

Authored by imp on Feb 25 2026, 8:58 PM.

Tags

None

Referenced Files

	F153585866: D55523.id.diff
	Wed, Apr 22, 3:27 AM

	Unknown Object (File)
	Sat, Apr 18, 11:51 AM

	Unknown Object (File)
	Mon, Apr 13, 10:31 PM

	Unknown Object (File)
	Sun, Apr 12, 9:05 PM

	Unknown Object (File)
	Sat, Apr 11, 11:10 AM

	Unknown Object (File)
	Thu, Apr 9, 1:59 PM

	Unknown Object (File)
	Mon, Apr 6, 6:33 PM

	Unknown Object (File)
	Sun, Apr 5, 3:32 PM

View All 25 Files

Subscribers

Details

Reviewers

mav
cperciva
chuck
chs

Group Reviewers

Commits

rG824530a5c4fd: nda: Assume all cases in ndaasync can sleep

Summary

The error recovery is nicer if we can wait for the tiny memory we need
to send the messages when the physpath changes. Since we've moved the
async handler into a sleepable context, we can wait for the allocation
to complete since async events are rare enough and it's not an
indefinite wait.

Sponsored by: Netflix

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 71006
Build 67889: arc lint + arc unit

Event Timeline

imp created this revision.Feb 25 2026, 8:58 PM

Herald added a reviewer: cam. · View Herald TranscriptFeb 25 2026, 8:58 PM

Herald added a subscriber: dab. · View Herald Transcript

imp requested review of this revision.Feb 25 2026, 8:58 PM

Harbormaster completed remote builds in B71006: Diff 172706.Feb 25 2026, 8:58 PM

imp added a parent revision: D55522: nvme_sim: Fix a cut and paste error.Feb 25 2026, 8:58 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 3 2026, 6:01 PM

Closed by commit rG824530a5c4fd: nda: Assume all cases in ndaasync can sleep (authored by imp). · Explain Why

This revision was automatically updated to reflect the committed changes.

imp added a commit: rG824530a5c4fd: nda: Assume all cases in ndaasync can sleep.

Revision Contents
Changeset List

Path

Size

sys/

cam/

nvme/

10 lines

Diff 172706

sys/cam/nvme/nvme_da.c

Loading...