Date: Fri, 31 Jul 2015 16:06:33 -0600 From: Warner Losh <imp@bsdimp.com> To: Glen Barber <gjb@FreeBSD.org> Cc: Larry Rosenman <ler@lerctr.org>, Benno Rice <benno@freebsd.org>, freebsd-current@freebsd.org Subject: Re: pmspcv panic on boot on this box Message-ID: <E7310E4C-44A7-4910-BD89-9A137EC4938A@bsdimp.com> In-Reply-To: <20150731213255.GC1304@FreeBSD.org> References: <20150731102721.GA824@oldtbh.lerctr.org> <20150731133214.GV90754@FreeBSD.org> <14ee48f81d8.2846.a6b027662ce9a8103f88b1e74d3c5524@lerctr.org> <9B59F252-D41D-4714-A983-356E525359FD@bsdimp.com> <14ee53e57a8.2846.a6b027662ce9a8103f88b1e74d3c5524@lerctr.org> <CBC7A977-EF25-44CF-B50B-6EAB62331866@bsdimp.com> <20150731204137.GA832@oldtbh.lerctr.org> <20150731204403.GB1304@FreeBSD.org> <d451b0f866624bad2d5f2f6a6aef517f@thebighonker.lerctr.org> <E859ED3F-404A-4996-8C60-609F16192012@bsdimp.com> <20150731213255.GC1304@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] > On Jul 31, 2015, at 3:32 PM, Glen Barber <gjb@FreeBSD.org> wrote: > > On Fri, Jul 31, 2015 at 03:11:15PM -0600, Warner Losh wrote: >> >>> On Jul 31, 2015, at 2:46 PM, Larry Rosenman <ler@lerctr.org> wrote: >>> >>> On 2015-07-31 15:44, Glen Barber wrote: >>>> On Fri, Jul 31, 2015 at 03:41:37PM -0500, Larry Rosenman wrote: >>>>> We have a winner -- rebuilt with just this patch, and GENERIC with pmspcv, >>>>> and we boot. >>>> Larry, thank you so much for your willingness to test this. It is >>>> greatly appreciated. >>>> Glen >>> Absolutely my pleasure. I appreciate ALL that the re@ and developers folks do for FreeBSD. >>> >>> Best OS ever :) >> >> Thanks Larry. I’ll cleanup the patch (I’d never knowing commit #if 0 w/o a good reason) >> and get it reviewed. The good news is that we can make a fairly low-risk patch for 10.2R >> I think, subject to the blessings of our benevolent re@ overlords... >> > > My gut instinct is to leave the driver in 10.2 as-is for now, as I'd > like to make sure this driver does not present more pain (and certainly > before being re-enabled in GENERIC on stable/10), since this situation > could have been much more of a disaster. > > Personally, I'd like to keep it out of GENERIC on stable/10 for at least > a month, while things get shaken out in head. > > That said, once the issues are flushed out, re@ will be happy to issue > an EN for 10.2 to resolve the existing (and any new) issues with this. You’re the boss. I can understand the skepticism given how badly the current code misunderstands newbus. This instance is quite easy to fix (I should have something in -current in a day or two). Fortunately, the series of errors would only cause a problem if you had two Adaptec devices in your system (and not just one). Since it is available as a module, perhaps a note explaining there was a problem with the auto-probing code so it was omitted from GENERIC, but if you have one of these devices put pms_load=YES in your loader.conf file. But only if you don’t have any other devices with the Adaptec vendor ID (since if you want to load pms, it is because you presumably have one of these cards). The path to doom is the second trip into probe after the first trip into probe failed. I’d be inclined to take a middle ground and fix the driver (and therefore the module) in 10.2R, but not put it in GENERIC until 10.3, but hey that’s just me. Interested parties should check out https://reviews.freebsd.org/D3263 for correctness or for testing. Warner [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQIcBAEBCgAGBQJVu/FqAAoJEGwc0Sh9sBEAprgP/jqe2P1yPHySRVIZSpR/tdZx QfQbTNkl54YcgF/j4j4B+k0anWUoHJ8m9fkCXueP5g/F42PJYVMqUq9tqf+biJgx b1mKHvbR45Ca8Ud9Rzm/vQno6nPEXfHnu9nsrux/ksCcbZqRYjD8CTLKgi9iA5Z8 hZ8sAe1cJE2oK5XnJtoClHhssY8YK3eDTdTIh9spL58u8lX18iGWocXG6O1ydgZa qmzPaWBwPf7CuU+JrjxxlEPq0gRPLRvHlEsfRzaTQ9CZaPTwgvjFjNmWZcQEZTau uozTehUrNFCaJAJb4ANA8Q40rpfGBzSU7MhIPPETrsp93QSV2S4WVgamABqmyItz /jiXPzg//d6URkjvcT1ciS+gxvFNtBcaaTtiqXiaHa8x3vo1rxbDXrJbZDDJA2Gv ogBend9CftwsNFx4qD367F7ME55j5Z4fQ8yD5kS9Sx6iLaWqqDYZv/WZaAueSbFG sDdx4hgSzRlZicDBJU211WO360RpC55mhqRoBP2s5JDLjl+YiQUfFtXaDXOz1mIX 0vWfjzs1wV/UHIwDc0LE9O2cu1qG6HBqUEDRHJcqpACpoEhffr8xRLX6G3WwzkIk 72+LkqwSsBfnQ5Xp+BPI1nDPUPL8fr82QlfgVZl+Cifq2gwytVaLMZrNsFCfAzCd 9DE4yrMbeMcRKfEFckjl =NzAp -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?E7310E4C-44A7-4910-BD89-9A137EC4938A>