Opened 4 years ago

Closed 4 years ago

#16325 closed bug (duplicate)

[hda] no sound (regression)

Reported by: kim1963 Owned by: mmlr
Priority: normal Milestone: Unscheduled
Component: Drivers/Audio/HDA Version: R1/beta2
Keywords: Cc: korli
Blocked By: #16222 Blocking:
Platform: All

Description (last modified by diver)

hrev54298 broke sound for me.

device Multimedia controller (Audio device) [4|3|0]
  vendor 8086: Intel Corporation
  device a348: Cannon Lake PCH cAVS

Beta2 and hrev54280 - work fine.

Attachments (2)

syslog (176.6 KB ) - added by kim1963 4 years ago.
syslogbeta2 (173.4 KB ) - added by kim1963 4 years ago.

Download all attachments as: .zip

Change History (20)

comment:1 by diver, 4 years ago

Component: Audio & VideoDrivers/Audio/HDA
Description: modified (diff)
Summary: 64 bit HDA driver broken. No sound[hda] no sound (regression)

comment:2 by X512, 4 years ago

Can be related to hrev54298 (HDA: Fix mapping 64-bit PCI addresses.).

Last edited 4 years ago by X512 (previous) (diff)

comment:3 by waddlesplash, 4 years ago

Cc: korli added

The only functional difference in that commit for systems without a 64-bit address is the PCI_address_memory_32_mask, which most other things that use 64-bit PCI addresses conditionally use.

CC korli: is it possible that is actually incorrect and should not be done?

comment:4 by waddlesplash, 4 years ago

Actually I think the PCI bus may already do this masking when it reads the base registers: https://xref.landonf.org/source/xref/haiku/src/add-ons/kernel/bus_managers/pci/pci.cpp#1190

So that mask may indeed be wrong in the case of "IO space", and unnecessary otherwise.

comment:5 by kim1963, 4 years ago

Revert 54298 ?

comment:6 by waddlesplash, 4 years ago

Component: Drivers/Audio/HDADrivers
Owner: changed from nobody to mmlr
Status: newassigned

No, we indeed discovered multiple problems in the PCI bus that may be the actual culprit here. mmlr is working on them.

comment:7 by waddlesplash, 4 years ago

Please retest after hrev54417.

comment:8 by kim1963, 4 years ago

hrev54418 - no sound

comment:9 by waddlesplash, 4 years ago

Try waiting a few minutes before you play any audio and see if it then works.

comment:10 by kim1963, 4 years ago

Waiting 10 minutes. No sound!

comment:11 by waddlesplash, 4 years ago

Please attach a syslog.

comment:12 by mmlr, 4 years ago

Component: DriversDrivers/Audio/HDA

by kim1963, 4 years ago

Attachment: syslog added

comment:13 by ttcoder, 4 years ago

KERN: pci_reserve_device(0, 31, 3, hda)
KERN: HDA: Detected controller @ PCI:0:31:3, IRQ:16, type 8086/a348 (17aa/313d)
...
KERN: hda: Codec 0 Vendor: 10ec Product: 0274, Revision: 1.0.0.4 Quirks: 0700

I suppose it might help if you post a syslog from a working hrev (beta2 ?) and a dev would compare that old syslog side-by-side with the new non-working one.

(Though the differences in PCI or HDA behavior might not show up in the log, depending on the bug)

by kim1963, 4 years ago

Attachment: syslogbeta2 added

comment:14 by ttcoder, 4 years ago

I can see no significant difference in the hda part..

To my untrained eye it seems the nightly differs from beta2 mainly in regards to dealing with the CPUs, if even that:

diff syslogbeta2 syslog

< KERN: Haiku revision: hrev54154+116, debug level: 1
---
> KERN: Haiku revision: hrev54418, debug level: 2

> KERN: CPU 0: features: ............ xsaveopt xsavec 

< KERN: arch_altcodepatch_replace found 31 altcodepatches
< KERN: arch_altcodepatch_replace found 291 altcodepatches
---
> KERN: arch_altcodepatch_replace found 31 altcodepatches for tag 1
> KERN: arch_altcodepatch_replace found 291 altcodepatches for tag 2
> KERN: arch_altcodepatch_replace found 2 altcodepatches for tag 3
> KERN: arch_altcodepatch_replace found 4 altcodepatches for tag 4
> KERN: enable XSAVEC 0x7 832

Call in the pros! :-)

comment:15 by kim1963, 4 years ago

the sound turns on itself on revisions 54454 and 54459 after 10 -28 minutes .. the exact time could not be established

comment:16 by ttcoder, 4 years ago

Blocked By: 16222 added

Ah, that would make this ticket a dupe of #16222 I guess.

Though I see there's been lots of HDA changes in hrev54459.. So try that upgrade "just in case"~~

EDIT: sorry, I see in your message that you've already tried 54459

Last edited 4 years ago by ttcoder (previous) (diff)

comment:17 by kim1963, 4 years ago

hrev54459 - The sound turned on spontaneously 23 minutes 30 seconds after the start of the operating system.

comment:18 by mmlr, 4 years ago

Resolution: duplicate
Status: assignedclosed

This indeed is a duplicate then. I have a lead for a fix, the issue is with the times published to the TimeSource of the MultiAudioNode.

Note: See TracTickets for help on using tickets.