Page MenuHomeFreeBSD
Differential D57293

usb: Add missing mtx lock and unlock in pushing dma queue
ClosedPublic
Actions

Authored by aokblast on May 28 2026, 9:08 AM.
Tags
Referenced Files
F160815989: D57293.id178824.diff
Sun, Jun 28, 4:24 AM
Unknown Object (File)
Sat, Jun 27, 10:44 AM
Unknown Object (File)
Tue, Jun 23, 9:48 AM
Unknown Object (File)
Tue, Jun 23, 2:43 AM
Unknown Object (File)
Sat, Jun 20, 6:13 PM
Unknown Object (File)
Sat, Jun 20, 1:20 PM
Unknown Object (File)
Mon, Jun 8, 11:31 AM
Unknown Object (File)
Mon, Jun 8, 11:21 AM
View All 26 Files
Subscribers
imp

Details

Reviewers
adrian
Group Reviewers
USB
Commits
rGe01b5034a627: usb: Add missing mtx lock and unlock in pushing dma queue
rG4f40c7c37fb4: usb: Add missing mtx lock and unlock in pushing dma queue
rGdf5e9e3da5b9: usb: Add missing mtx lock and unlock in pushing dma queue
Summary

Accessing usb_xfer_queue requires bus lock, we added this missing lock in here to prevent racing issue.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

aokblast created this revision.May 28 2026, 9:08 AM
Herald added a subscriber: imp. · View Herald TranscriptMay 28 2026, 9:08 AM
aokblast requested review of this revision.May 28 2026, 9:08 AM
Harbormaster completed remote builds in B73498: Diff 178797.May 28 2026, 9:08 AM
aokblast added reviewers: adrian, USB.May 28 2026, 9:09 AM
aokblast added a project: USB.
aokblast updated this revision to Diff 178818.May 28 2026, 12:43 PM

Refactor commit msg

Harbormaster completed remote builds in B73506: Diff 178818.May 28 2026, 12:43 PM
aokblast edited the summary of this revision. (Show Details)May 28 2026, 12:43 PM
adrian accepted this revision.May 28 2026, 2:29 PM

nice catch! Why's this not panicing though? I thought I saw a lock assertion in usb_command_wrapper() ?

This revision is now accepted and ready to land.May 28 2026, 2:29 PM
aokblast added a comment.May 28 2026, 2:31 PM
In D57293#1312969, @adrian wrote:

nice catch! Why's this not panicing though? I thought I saw a lock assertion in usb_command_wrapper() ?

We don't have that now. This is found when I am trying to do so (do lock assertion in usb_command_wrapper).

Closed by commit rGdf5e9e3da5b9: usb: Add missing mtx lock and unlock in pushing dma queue (authored by aokblast). · Explain WhyMay 28 2026, 2:38 PM
This revision was automatically updated to reflect the committed changes.
aokblast added a commit: rGdf5e9e3da5b9: usb: Add missing mtx lock and unlock in pushing dma queue.
aokblast added a commit: rG4f40c7c37fb4: usb: Add missing mtx lock and unlock in pushing dma queue.Tue, Jun 23, 8:53 PM
aokblast added a commit: rGe01b5034a627: usb: Add missing mtx lock and unlock in pushing dma queue.Tue, Jun 23, 8:57 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
usb/
2 lines

Diff 178824

View Options

sys/dev/usb/usb_transfer.c

Loading...