Date: Mon, 26 Aug 2019 15:00:28 -0700 From: Conrad Meyer <cem@freebsd.org> To: Alexander Motin <mav@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r351514 - head/sys/dev/pci Message-ID: <CAG6CVpU4ROha2a1CWVt5Hev2Cd1uSNXLO-VXfe_JvsiucWVkoQ@mail.gmail.com> In-Reply-To: <201908261727.x7QHRbsc020978@repo.freebsd.org> References: <201908261727.x7QHRbsc020978@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Alexander, On Mon, Aug 26, 2019 at 10:27 AM Alexander Motin <mav@freebsd.org> wrote: > > Author: mav > Date: Mon Aug 26 17:27:36 2019 > New Revision: 351514 > ... > - device_printf(dev, > - "HotPlug Command Timed Out - forcing detach\n"); > - sc->flags &= ~(PCIB_HOTPLUG_CMD_PENDING | PCIB_DETACH_PENDING); > - sc->flags |= PCIB_DETACHING; > - pcib_pcie_hotplug_update(sc, 0, 0, true); > + device_printf(dev, "HotPlug Command Timed Out\n"); > + sc->flags &= ~PCIB_HOTPLUG_CMD_PENDING; If this case is expected (and it seems like it is, at least in relatively common hardware), and the kernel isn't taking an action based on it anymore (formerly, it would then cause detach), is there any reason to print a message at all? What information should a user glean from the message, or what action should be taken? It seems like Nothing and None, respectively. So perhaps it is better to not print anything in this case? Or maybe mask it behind bootverbose. Best, Conrad
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpU4ROha2a1CWVt5Hev2Cd1uSNXLO-VXfe_JvsiucWVkoQ>