> From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxx] > Sent: Saturday, January 27, 2007 8:08 AM > On Mon, 2007-01-22 at 12:26 -0800, David C Somayajulu wrote: > > The included patch fixes the following issues: > > 1. qla3xxx/qla4xxx co-existence issue which can result in a lockup when qla3xxx driver is unloaded, > or when ifdown; ifup is performed on one of the interfaces correponding to qla3xxx. This is because > qla4xxx HBA supports one ethernet and iscsi interfaces per port. Both iscsi and ethernet interfaces > share the same state machine. The problem has to do with synchronizing access to the state machine in > the event of a reset > > 2. mutex_lock() is sometimes not followed by mutex_unlock() prior to invoking a msleep() in > qla4xxx_mailbox_command() > > Git applymbox picked up this nasty (which I've corrected): > > Applying 'qla4xxx: bug fixes' > > Space in indent is followed by a tab. > .dotest/patch:122: set_bit(AF_MBOX_COMMAND, > &ha->flags); > Space in indent is followed by a tab. > .dotest/patch:123: mutex_unlock(&ha->mbox_sem); > Space in indent is followed by a tab. > .dotest/patch:124: break; > Space in indent is followed by a tab. > .dotest/patch:125: } > Space in indent is followed by a tab. > .dotest/patch:126: mutex_unlock(&ha->mbox_sem); > warning: squelched 7 whitespace errors > warning: 12 lines add trailing whitespaces. > > James > Sorry about that. Really appreciate your correcting them. Thanks David S. - 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
