Il 07/09/2012 20:33, Nicholas A. Bellinger ha scritto: >>> > > if (!cmd->data_length && >>> > > + cmd->t_task_cdb[0] != REQUEST_SENSE && >>> > > (cmd->se_dev->transport->transport_type != TRANSPORT_PLUGIN_PHBA_PDEV || >>> > > cmd->t_task_cdb[0] == REPORT_LUNS) { >>> > > spin_lock_irq(&cmd->t_state_lock); > Btw, this section does not compile. The idea was to add the missing parenthesis to cmd->t_task_cdb[0] == REPORT_LUNS)) but your fix is also correct after patch 3. Thanks! Paolo -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html