Page MenuHomeFreeBSD
Differential D45857

sound: Handle midistat_lock through function interface
ClosedPublic
Actions

Authored by christos on Jul 3 2024, 2:14 PM.
Tags
None
Referenced Files
F171423477: D45857.id.diff
Fri, Sep 11, 1:40 AM
Unknown Object (File)
Wed, Sep 9, 11:00 PM
Unknown Object (File)
Wed, Sep 9, 10:47 AM
Unknown Object (File)
Wed, Sep 9, 9:26 AM
Unknown Object (File)
Tue, Sep 8, 6:24 PM
Unknown Object (File)
Sun, Sep 6, 1:55 AM
Unknown Object (File)
Sat, Sep 5, 10:30 PM
Unknown Object (File)
Sat, Sep 5, 6:40 PM
View All Files
Subscribers
imp

Details

Reviewers
markj
emaste
dev_submerge.ch
Commits
rGff6b61a2c886: sound: Handle midistat_lock through function interface
rG2d6fc24673cc: sound: Handle midistat_lock through function interface
Summary

midistat_lock is used outside midi/midi.c as well, so implement lock,
unlock and lockassert functions in order to not expose the lock in the
header file.

Sponsored by: The FreeBSD Foundation
MFC after: 2 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58474
Build 55362: arc lint + arc unit

Event Timeline

christos created this revision.Jul 3 2024, 2:14 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 3 2024, 2:14 PM
christos requested review of this revision.Jul 3 2024, 2:14 PM
christos added a parent revision: D45770: sound: Fix lock order reversals in mseq_open().Jul 3 2024, 2:14 PM
christos mentioned this in D45770: sound: Fix lock order reversals in mseq_open().
Harbormaster completed remote builds in B58474: Diff 140507.Jul 3 2024, 2:15 PM
christos updated this revision to Diff 140508.Jul 3 2024, 2:16 PM

Move function declarations at the top to match the function definitions'
position.

Harbormaster completed remote builds in B58475: Diff 140508.Jul 3 2024, 2:16 PM
emaste added inline comments.Jul 6 2024, 3:43 PM
sys/dev/sound/midi/midi.c
180

I assume the rename here is to make sure there are no leftover (or external) consumers?

christos added inline comments.Jul 6 2024, 3:49 PM
sys/dev/sound/midi/midi.c
180

It's because midistat_lock() will have the same name as the lock and we'll get a compile error.

emaste accepted this revision.Jul 6 2024, 4:43 PM
This revision is now accepted and ready to land.Jul 6 2024, 4:43 PM
Closed by commit rG2d6fc24673cc: sound: Handle midistat_lock through function interface (authored by christos). · Explain WhyJul 6 2024, 6:24 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG2d6fc24673cc: sound: Handle midistat_lock through function interface.
christos added a commit: rGff6b61a2c886: sound: Handle midistat_lock through function interface.Jul 10 2024, 4:49 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
midi/
6 lines
76 lines
4 lines

Diff 140507

View Options

sys/dev/sound/midi/midi.h

Loading...
View Options

sys/dev/sound/midi/midi.c

Loading...
View Options

sys/dev/sound/midi/sequencer.c

Loading...