sound: Busy the device in sysctl_dev_pcm_mode()
Needs ReviewPublic
Actions

Authored by christos on Fri, Oct 9, 9:12 AM.

Tags

None

Referenced Files

	F175491192: D60509.diff
	Sun, Oct 11, 5:48 AM

	F175470960: D60509.id189176.diff
	Sun, Oct 11, 2:37 AM

	F175435500: D60509.id.diff
	Sat, Oct 10, 8:23 PM

	F175415625: D60509.id189176.diff
	Sat, Oct 10, 5:11 PM

	F175410177: D60509.diff
	Sat, Oct 10, 4:16 PM

	Unknown Object (File)
	Sat, Oct 10, 10:09 AM

	Unknown Object (File)
	Sat, Oct 10, 3:03 AM

	Unknown Object (File)
	Fri, Oct 9, 4:08 PM

View All 13 Files

Subscribers

imp

Details

Reviewers

markj
emaste
kib

Summary

PCM_LOCK does not protect anything here, since the counts are plain
ints, and the mixer pointer is set once in mixer_init() and never
cleared. It also does not protect against the device being freed.

Busy the device instead, as the other handlers do.

MFC after: 2 weeks
Sponsored by: The FreeBSD Foundation

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 77896
Build 74779: arc lint + arc unit

Event Timeline

christos created this revision.Fri, Oct 9, 9:12 AM

Herald added a subscriber: imp. · View Herald TranscriptFri, Oct 9, 9:12 AM

christos requested review of this revision.Fri, Oct 9, 9:12 AM

Harbormaster completed remote builds in B77896: Diff 189176.Fri, Oct 9, 9:12 AM

christos added a parent revision: D60507: sound: Busy devices before giving up the topology lock.Fri, Oct 9, 9:30 AM

christos added a child revision: D60512: sound: Stop taking PCM_LOCK to create and destroy channels.Fri, Oct 9, 9:33 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

sound.c

4 lines

Diff 189176

View Options

sound: Busy the device in sysctl_dev_pcm_mode()Needs ReviewPublicActions