Re: + aacraid-fix-unchecked-down_interruptible.patch added to -mm tree

[James Bottomley <James.Bottomley@HansenPartnership.com>]

On Thu, 2008-04-10 at 17:44 -0400, Mark Salyzyn wrote:
> 	if (down_interruptible(&fibptr->event_wait) == 0) {
> 		fibptr->done = 2;
>                 up(&fibptr->event_wait);
> 	}
> 
> would make me feel much more comfortable ... The fib can never be  
> 'freed'. The fib is owned by the adapter until it completes it on it's  
> own (if ever, the assumption here is that the adapter is misbehaving  
> on this 'one' fib). Setting the value of 2 allows us to report back to  
> the ioctl caller a failed operation without actually closing out the  
> fib.
> 
> I am having a problem with my bandwidth right now to do a unit test to  
> see if this gets the job done.

That's OK ... send the patch when you're ready; I'll make sure it gets
upstream.  We'll do the standard put it into scsi-misc and backport if
it seems stable, so we'll have time to shake out any problems.

James