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 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html