Differential D58669

ufshci: release the CCB after sending a start stop unit command
ClosedPublic
Actions

Authored by jaeyoon on Aug 5 2026, 8:12 AM.

Tags

None

Referenced Files

	F172852622: D58669.id183766.diff
	Mon, Sep 21, 2:35 PM

	F172837251: D58669.diff
	Mon, Sep 21, 11:00 AM

	Unknown Object (File)
	Sun, Sep 20, 4:42 PM

	Unknown Object (File)
	Mon, Sep 14, 11:20 PM

	Unknown Object (File)
	Sun, Sep 13, 3:38 PM

	Unknown Object (File)
	Fri, Sep 11, 8:48 AM

	Unknown Object (File)
	Tue, Sep 8, 8:38 PM

	Unknown Object (File)
	Tue, Sep 8, 5:37 PM

View All 44 Files

Subscribers

None

Details

Reviewers

Commits

rGaaf0e80e684d: ufshci: release the CCB after sending a start stop unit command

Summary

ufshci_sim_send_ssu() got a CCB from cam_periph_getccb() but never
returned it. Each call leaked the CCB and one slot of the device's
CCB allocation budget. When the budget runs out, the next
cam_periph_getccb() waits forever and the suspend path hangs.

Release the CCB while the periph lock is still held, as the other CAM
periph drivers do.

Sponsored by: Samsung Electronics

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jaeyoon requested review of this revision.Aug 5 2026, 8:12 AM

jaeyoon created this revision.

Harbormaster completed remote builds in B75387: Diff 183432.Aug 5 2026, 8:12 AM

jaeyoon added a parent revision: D58668: ufshci: check completions under the queue lock.Aug 5 2026, 8:13 AM

jaeyoon added a child revision: D58670: ufshci: free the taskqueue on detach.

imp accepted this revision.Aug 7 2026, 7:55 PM

This revision is now accepted and ready to land.Aug 7 2026, 7:55 PM

Closed by commit rGaaf0e80e684d: ufshci: release the CCB after sending a start stop unit command (authored by jaeyoon). · Explain WhyAug 10 2026, 2:32 AM

This revision was automatically updated to reflect the committed changes.

jaeyoon added a commit: rGaaf0e80e684d: ufshci: release the CCB after sending a start stop unit command.

Revision Contents
Changeset List

Path

Size

sys/

dev/

ufshci/

3 lines

Diff 183766

sys/dev/ufshci/ufshci_sim.c

Loading...