Subject: Re: dev/scsipi/ch.c:chattach() fix ? (and approval for commit?)
To: Bill Studenmund <wrstuden@NetBSD.org>
From: Charles M. Hannum <abuse@spamalicious.com>
List: tech-kern
Date: 06/17/2004 00:41:03
On Wednesday 16 June 2004 23:33, Bill Studenmund wrote:
> On Thu, Jun 17, 2004 at 01:10:57AM +0200, Stoned Elipot wrote:
> > On Thu, Jun 17, 2004 at 12:07:38AM +0200, Manuel Bouyer wrote:
> > > On Wed, Jun 16, 2004 at 11:20:36PM +0200, Manuel Bouyer wrote:
> > > > Hum. Can you apply the attached patch and see if there's any message
> > > > for ch0 ? This is to make sure that this changer doesn't return any
> > > > sense at all.
> >
> > Here is what it's saying:
> >
> > ch0 at scsibus1 target 4 lun 0: <QUALSTAR, RLS-4221, 0047> changer
> > removable ch0(esiop1:0:4:0): requesting sense
> > ch0(esiop1:0:4:0):  sense debug information:
> >         code 0x70 valid 0x0
>
>                           ^^^ Looks like that's the bug. The changer isn't
> 			      setting 'valid'

Nope.  The problem is that ch_interpret_sense() is eating the UNIT ATTENTION.