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
F167579055: D57293.id178797.diff
Sat, Aug 22, 8:24 PM
Unknown Object (File)
Fri, Aug 21, 3:46 PM
Unknown Object (File)
Thu, Aug 20, 7:40 PM
Unknown Object (File)
Mon, Aug 17, 2:25 AM
Unknown Object (File)
Sun, Aug 16, 6:48 PM
Unknown Object (File)
Fri, Aug 14, 9:50 AM
Unknown Object (File)
Thu, Aug 13, 12:47 AM
Unknown Object (File)
Wed, Aug 12, 11:52 PM
View All 53 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 Passed
Unit
No Test Coverage
Build Status
Buildable 73506
Build 70389: arc lint + arc unit

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.Jun 23 2026, 8:53 PM
aokblast added a commit: rGe01b5034a627: usb: Add missing mtx lock and unlock in pushing dma queue.Jun 23 2026, 8:57 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
usb/
2 lines

Diff 178818

View Options

sys/dev/usb/usb_transfer.c

Loading...