sound: Make CHN_REMOVE_SAFE() the default
AcceptedPublic
Actions

Authored by christos on Sun, Dec 29, 8:51 PM.

Details

Reviewers

markj
emaste
dev_submerge.ch

Summary

FILLME optimized CHN_REMOVE_SAFE() to perform almost equally to
CHN_REMOVE(), so we can turn CHN_REMOVE_SAFE() into CHN_REMOVE() without
fears of performance regressions, while also being more robust.

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 61399
Build 58283: arc lint + arc unit

Event Timeline

christos created this revision.Sun, Dec 29, 8:51 PM

Herald added a subscriber: imp. · View Herald TranscriptSun, Dec 29, 8:51 PM

christos requested review of this revision.Sun, Dec 29, 8:51 PM

Harbormaster completed remote builds in B61399: Diff 148539.Sun, Dec 29, 8:51 PM

christos added a parent revision: D48207: sound: Safely remove channel from list in one pass..Sun, Dec 29, 8:54 PM

christos mentioned this in D48185: sound: Do not fail from vchan_destroy() if children list is empty.Sun, Dec 29, 8:55 PM

christos added a child revision: D48185: sound: Do not fail from vchan_destroy() if children list is empty.Sun, Dec 29, 8:59 PM

dev_submerge.ch accepted this revision.Sun, Dec 29, 10:23 PM

This revision is now accepted and ready to land.Sun, Dec 29, 10:23 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

channel.h

5 lines

channel.c

2 lines

Diff 148539

View Options

sys/dev/sound/pcm/channel.h