> From: James Bottomley [mailto:[EMAIL PROTECTED]
> 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 [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to