Date: Sat, 28 Apr 2001 12:13:50 -0700 (PDT) From: Matthew Jacob <mjacob@feral.com> To: Joerg Wunsch <joerg_wunsch@uriah.heep.sax.de> Cc: freebsd-scsi@FreeBSD.ORG Subject: Re: sa(4) jamming Message-ID: <Pine.LNX.4.21.0104281212500.1249-100000@zeppo.feral.com> In-Reply-To: <20010428210359.Q50185@uriah.heep.sax.de>
next in thread | previous in thread | raw e-mail | index | archive | help
I should note that prior to CAM error handling changes, the sa driver lived in a state of a very uneasy truce with CAM. Now that the error handling has changed, this has to be rethought. I also have a wad of suggested changes from lots of people. I need to spend about a week or so on this. I'm trying schedule this for mid or late may. > As Justin T. Gibbs wrote: > > > >That depends on whether the sa driver relies on any settings (mode > > >page or otherwise) that are invalidated by a reset, but aren't > > >restored when the bus reset async event occurs. > > > > Just to be more clear here, it is the peripheral driver's responsibility > > to restore any of this state if it needs to. > > I don't doubt this. It only surprises me how many driver bugs have > been unobscured by the CAM error handling reorg. All those things (sa > driver ILI handling, pt driver invalidating the device after a > power-on UA, sa driver running into obscure problems after a bus reset > without reloading the tape) didn't happen with earlier versions of the > CAM subsystem. > > Please get me right, i don't want to grumble about CAM here, but i'd > like to see bugs fixed as they become obvious in -current, and the > first step is always to make sure where the bug actually is. > > Also Justin, you still didn't respond whether the patch for the `NO > SENSE' handling i posted i what you were referring to. Right now, > cam_periph_error() effectively returns ERESTART in that case. > > To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-scsi" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.LNX.4.21.0104281212500.1249-100000>