Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE
From: Karl Denninger (karl_at_denninger.net)
Date: 03/30/05
- Previous message: J. Buck Caldwell: "Re: Problem with IBM xSeries 226 with ServeRAID 6i+ using FreeBSD 5.3"
- In reply to: Jim Trigg: "Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE"
- Next in thread: Drew Tomlinson: "Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE"
- Reply: Drew Tomlinson: "Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Wed, 30 Mar 2005 11:39:31 -0600 To: Jim Trigg <jtrigg@spamcop.net>
On Wed, Mar 30, 2005 at 12:07:22PM -0500, Jim Trigg wrote:
> On Wed, March 30, 2005 12:43 am, Karl Denninger said:
> > Here's the diff and some thoughts....
>
> [snip, including first diff]
>
> > 241,243c244,249
> > <
> > < /* if reinit succeeded and retries still permit, reinject request
> > */
> > < if (ata_reinit(ch) && request->retries-- > 0 &&
> > request->device->param){
> > ---
> >> /*
> >> * if reinit succeeds, retries still permit and device didn't
> >> * get removed by the reinit, reinject request
> >> */
> >> if (!ata_reinit(ch) && request->retries-- > 0
> >> && request->device->param){
> [snip third diff]
>
> > The second diff is really just a formatting and comment change.. you're
> > certainly correct that the changes are small! :-)
>
> No, it is not -- it reverses the sense of the first condition. At first
> glance that is what I would expect to be the core of the problem, but I
> don't have appropriate hardware to test on. (It also adds a third
> condition, but that is presumably the intent of the change and should give
> the desired results once the first condition is corrected.)
>
> Jim
> --
> Jim Trigg, Lord High Everything Else O- /"\
> Hostmaster, Huie Kin family website \ / ASCII RIBBON CAMPAIGN
> Verger and System Administrator, X HELP CURE HTML MAIL
> All Saints Church - Sharon Chapel / \
You're correct of course - I missed the "!". Too darn late at night...
I've got my sandbox up and the world rebuilt so its consistent with the
machine that's having the problem - will add a SATA disk and see if I can
duplicate this and then figure out what's going on here this afternoon -
and hopefully how to fix it.
-- -- Karl Denninger (karl@denninger.net) Internet Consultant & Kids Rights Activist http://www.denninger.net My home on the net - links to everything I do! http://scubaforum.org Your UNCENSORED place to talk about DIVING! http://www.spamcuda.net SPAM FREE mailboxes - FREE FOR A LIMITED TIME! http://genesis3.blogspot.com Musings Of A Sentient Mind _______________________________________________ freebsd-stable@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-stable To unsubscribe, send any mail to "freebsd-stable-unsubscribe@freebsd.org"
- Previous message: J. Buck Caldwell: "Re: Problem with IBM xSeries 226 with ServeRAID 6i+ using FreeBSD 5.3"
- In reply to: Jim Trigg: "Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE"
- Next in thread: Drew Tomlinson: "Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE"
- Reply: Drew Tomlinson: "Re: DANGER WILL ROBINSON! SERIOUS problem with current 5.4-PRERELEASE"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]