Date: Mon, 18 Sep 2023 22:15:12 +0000 From: bugzilla-noreply@freebsd.org To: bugs@FreeBSD.org Subject: [Bug 273418] [panic] Repeating kernel panic on open(/dev/console) Message-ID: <bug-273418-227-QiLW4eX9XR@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-273418-227@https.bugs.freebsd.org/bugzilla/> References: <bug-273418-227@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=3D273418 --- Comment #18 from Jason A. Harmening <jah@FreeBSD.org> --- Shouldn't the asserts in my proposed patch already accomplish what we need here? It may not be truly necessary to add a new flag solely tracking active list membership (vs. CDP_ACTIVE), but it seemed worth doing given that a devnode= can be marked inactive while still on cdevp_list. Beyond that, the patch alrea= dy has: 1) An assert that the node isn't on cdevp_list when devfs_free() is called, which is probably the critical case here due to incorrect (un)referencing. 2) Asserts in the other places that add or remove from cdevp_list, to catch other incorrect behavior e.g. trying add a node to the tail of the list when it's already elsewhere on the list. --=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-273418-227-QiLW4eX9XR>