From nobody Tue Dec 06 10:57:07 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4NRHRz5Hckz4jLpr; Tue, 6 Dec 2022 10:57:19 +0000 (UTC) (envelope-from hps@selasky.org) Received: from mail.turbocat.net (turbocat.net [88.99.82.50]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4NRHRz311Cz3K31; Tue, 6 Dec 2022 10:57:19 +0000 (UTC) (envelope-from hps@selasky.org) Authentication-Results: mx1.freebsd.org; none Received: from [10.36.2.69] (unknown [84.210.222.10]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mail.turbocat.net (Postfix) with ESMTPSA id E234A260025; Tue, 6 Dec 2022 11:57:10 +0100 (CET) Message-ID: Date: Tue, 6 Dec 2022 11:57:07 +0100 List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Subject: Re: git: 3cf97e91fac5 - main - Revert "newbus: Change attach failure behavior" Content-Language: en-US To: Warner Losh , src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org References: <202212060209.2B629pnu053879@gitrepo.freebsd.org> From: Hans Petter Selasky In-Reply-To: <202212060209.2B629pnu053879@gitrepo.freebsd.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 4NRHRz311Cz3K31 X-Spamd-Bar: ---- X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; ASN(0.00)[asn:24940, ipnet:88.99.0.0/16, country:DE] X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-ThisMailContainsUnwantedMimeParts: N On 12/6/22 03:09, Warner Losh wrote: > The branch main has been updated by imp: > > URL: https://cgit.FreeBSD.org/src/commit/?id=3cf97e91fac5f53fc0375bc816cc541a8864ffc4 > > commit 3cf97e91fac5f53fc0375bc816cc541a8864ffc4 > Author: Warner Losh > AuthorDate: 2022-12-05 23:57:58 +0000 > Commit: Warner Losh > CommitDate: 2022-12-06 00:00:26 +0000 > > Revert "newbus: Change attach failure behavior" > > This reverts commit 68c3f0302106643207dcdfe3b414810e245228e5. There are > some weird crashes when KVMs switch caused by this, so revert this > commit until they are sorted out. > > Reported by: cy@ > Sponsored by: Netflix > --- > UPDATING | 2 ++ > sys/kern/subr_bus.c | 2 +- > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/UPDATING b/UPDATING > index 099066031b8e..001ec9f6de3a 100644 > --- a/UPDATING > +++ b/UPDATING > @@ -43,6 +43,8 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 14.x IS SLOW: > needs to use devctl to re-enable the device, and reprobe it (or set > the sysctl/tunable hw.bus.disable_failed_devices=false). > > + NOTE: This was reverted 20221205 due to unexpected compatibility issues > + > 20221122: > pf no longer accepts 'scrub fragment crop' or 'scrub fragment drop-ovl'. > These configurations are no longer automatically reinterpreted as > diff --git a/sys/kern/subr_bus.c b/sys/kern/subr_bus.c > index 6a5ec4efc38d..b9615b033007 100644 > --- a/sys/kern/subr_bus.c > +++ b/sys/kern/subr_bus.c > @@ -69,7 +69,7 @@ SYSCTL_NODE(_hw, OID_AUTO, bus, CTLFLAG_RW | CTLFLAG_MPSAFE, NULL, > SYSCTL_ROOT_NODE(OID_AUTO, dev, CTLFLAG_RW | CTLFLAG_MPSAFE, NULL, > NULL); > > -static bool disable_failed_devs = true; > +static bool disable_failed_devs = false; > SYSCTL_BOOL(_hw_bus, OID_AUTO, disable_failed_devices, CTLFLAG_RWTUN, &disable_failed_devs, > 0, "Do not retry attaching devices that return an error from DEVICE_ATTACH the first time"); > Thinking about it, this flag shouldn't be set for USB devices and HUBS and such. Probably only makes sense for PCI devices, though there is something called thunderbolt too, which may fail during probe/attach, because the user yanked the device. Regarding the assert in the USB stack, maybe the state was not correctly set on the device_t ? --HPS