sound: Remove unncessary locking in sysctl_hw_snd_hwvol_mixer()
Needs ReviewPublic
Actions

Authored by christos on Sat, Aug 22, 1:43 PM.

Tags

None

Referenced Files

	F169934255: D59109.diff
	Thu, Sep 3, 12:42 AM

	F169781617: D59109.id184737.diff
	Wed, Sep 2, 1:29 PM

	Unknown Object (File)
	Wed, Sep 2, 12:56 AM

	Unknown Object (File)
	Tue, Sep 1, 8:57 PM

	Unknown Object (File)
	Tue, Sep 1, 3:06 PM

	Unknown Object (File)
	Mon, Aug 31, 1:44 PM

	Unknown Object (File)
	Sun, Aug 30, 7:07 PM

	Unknown Object (File)
	Sun, Aug 30, 3:14 PM

View All 27 Files

Subscribers

imp

Details

Reviewers

markj
emaste
kib

Summary

The locking around strlcpy() was because of m->hwvol_mixer, but this is
just an int, so we don't need to lock in this case. Instead lock only
when m->hwvol_mixer is written.

While here, add parentheses around the returns.

Sponsored by: The FreeBSD Foundation
MFC after: 1 month

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 75964
Build 72847: arc lint + arc unit

Event Timeline

christos created this revision.Sat, Aug 22, 1:43 PM

Herald added a subscriber: imp. · View Herald TranscriptSat, Aug 22, 1:43 PM

christos requested review of this revision.Sat, Aug 22, 1:43 PM

Harbormaster completed remote builds in B75964: Diff 184737.Sat, Aug 22, 1:43 PM

christos added a child revision: D59110: sound: Lock around mixer_set*() in mixer_init() for consistency.Sat, Aug 22, 1:43 PM

christos added a parent revision: D59078: sound: Improve some mixer return values and their handling.Sat, Aug 22, 1:43 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

mixer.c

18 lines

Diff 184737

View Options

sound: Remove unncessary locking in sysctl_hw_snd_hwvol_mixer()Needs ReviewPublicActions