Re: kern/39297: mfi calls tsleep() from mfi_intr()

[Greg Oster <oster%cs.usask.ca@localhost>]

The following reply was made to PR kern/39297; it has been noted by GNATS.

From: Greg Oster <oster%cs.usask.ca@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: Mark Davies <mark%mcs.vuw.ac.nz@localhost>
Subject: Re: kern/39297: mfi calls tsleep() from mfi_intr() 
Date: Sat, 18 Oct 2008 15:31:24 -0600

 Manuel Bouyer writes:
 > The following reply was made to PR kern/39297; it has been noted by GNATS.
 > 
 > From: Manuel Bouyer <bouyer%antioche.eu.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: kern-bug-people%NetBSD.org@localhost, gnats-admin%NetBSD.org@localhost, 
 > netbsd-bugs%NetBSD.or@localhost
 > g
 > Subject: Re: kern/39297: mfi calls tsleep() from mfi_intr()
 > Date: Sat, 18 Oct 2008 22:27:32 +0200
 > 
 >  --Dxnq1zWXvFF0Q93v
 >  Content-Type: text/plain; charset=us-ascii
 >  Content-Disposition: inline
 >  
 >  On Tue, Aug 05, 2008 at 05:25:00PM +0000, oster%NetBSD.org@localhost wrote:
 >  >   Running 4.99.71 (and some revisions earlier) on a machine with
 >  > using the mfi will result in the machine eventually locking up.  Breaking
 >  > into ddb yields the following:
 >  > 
 >  > login: fatal breakpoint trap in supervisor mode
 >  > trap type 1 code 0 rip ffffffff804dba45 cs 8 rflags 202 cr2  ffff8000720a8
 > 000 cp
 >  > l 8 rsp ffff800062c4b7f8
 >  > Stopped in pid 0.2 (system) at  netbsd:breakpoint+0x5:  leave
 >  > db{0}> tr
 >  > breakpoint() at netbsd:breakpoint+0x5
 >  > comintr() at netbsd:comintr+0x53a
 >  > Xintr_ioapic_edge6() at netbsd:Xintr_ioapic_edge6+0xef
 >  > --- interrupt ---
 >  > mutex_spin_retry() at netbsd:mutex_spin_retry+0x5a
 >  > ltsleep() at netbsd:ltsleep+0xe5
 >  > mfi_mgmt() at netbsd:mfi_mgmt+0xe1
 >  > mfi_scsipi_request() at netbsd:mfi_scsipi_request+0x331
 >  > scsipi_run_queue() at netbsd:scsipi_run_queue+0x16e
 >  > mfi_intr() at netbsd:mfi_intr+0xc0
 >  > intr_biglock_wrapper() at netbsd:intr_biglock_wrapper+0x1d
 >  > Xintr_ioapic_level2() at netbsd:Xintr_ioapic_level2+0xf7
 >  
 >  Hi, can you try the attached patch ? It should avoid tsleep in the
 >  mfi_scsipi_request() path.
 
 Unfortunately I no longer have access to that hardware... :( 
 
 Mark Davis (CC'ed) has likely seen the same issue on a Dell 1950.
 Mark: You can find the patch here: http://gnats.netbsd.org/39297 
 If you are able to test it that would be great!
 
 Thanks (to both of you :) ).
 
 Later...
 
 Greg Oster