Date: Tue, 13 Oct 2020 20:44:24 +0000 From: bugzilla-noreply@freebsd.org To: multimedia@FreeBSD.org Subject: [Bug 250248] 12.2-RC2 kernel reboots with hw.acpi.debug=1 after a warm reboot. Message-ID: <bug-250248-12827-EDDvtKgs3r@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-250248-12827@https.bugs.freebsd.org/bugzilla/> References: <bug-250248-12827@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D250248 --- Comment #12 from Alexander Motin <mav@FreeBSD.org> --- About the first patch adding shutdown handling I think it is an overkill for workaround and wrong approach in general, since there is no reason to detach the device on shutdown. It is designed to flush some caches and do alike things, and I know only one kind of devices detaching on shutdown -- USB, though I don't know why Hans done it that way. The second patch moving enabling UNSOL looks better to me, since I don't see what good for us could those message be before we even probe the codecs, not mentioning attaching them. Theoretically, if we ever implement codec hot-p= lug (for which I've never had a hardware), it would be required to better handle UNSOL events during the CODEC probe, since it may happen during run time.=20 Meanwhile if this patch works -- great, but generally it looks like a parti= al case to me, since hdac_unsolq_flush() already checks for sc->codecs[cad].de= v to be NULL, that is assigned the last by the probe loop inside hdac_attach2().= So it may just affect some timings, not really fix the issue. --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-250248-12827-EDDvtKgs3r>