snd_hda: Default to 256 when corb/rirb is invalid
ClosedPublic
Actions

Authored by christos on Dec 22 2025, 5:52 PM.

Details

Reviewers

markj
emaste

Commits

rGed61d50bd404: snd_hda: Default to 256 when corb/rirb is invalid
rG2455db44ef08: snd_hda: Default to 256 when corb/rirb is invalid

Summary

PR: 289284
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 69439
Build 66322: arc lint + arc unit

Event Timeline

christos created this revision.Dec 22 2025, 5:52 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 22 2025, 5:52 PM

christos requested review of this revision.Dec 22 2025, 5:52 PM

Harbormaster completed remote builds in B69439: Diff 168524.Dec 22 2025, 5:52 PM

From the PR submitted on Bugzilla:

Thank you for the fast fix! I can confirm that your patch works great.

emaste accepted this revision.Dec 23 2025, 2:23 PM

emaste added inline comments.

sys/dev/sound/pci/hda/hdac.c
548	Perhaps something like "Hardware reports invalid corb size"? Mainly I'm thinking about what a user who reads a kernel message will think or should do, and in this case there's no action for them to take.

This revision is now accepted and ready to land.Dec 23 2025, 2:23 PM

Closed by commit rG2455db44ef08: snd_hda: Default to 256 when corb/rirb is invalid (authored by christos). · Explain WhyJan 2 2026, 3:50 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rG2455db44ef08: snd_hda: Default to 256 when corb/rirb is invalid.

christos added a commit: rGed61d50bd404: snd_hda: Default to 256 when corb/rirb is invalid.Jan 9 2026, 11:27 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pci/

hda/

hdac.c

14 lines

Diff 168524

View Options

sys/dev/sound/pci/hda/hdac.c

snd_hda: Default to 256 when corb/rirb is invalidClosedPublicActions