Commit e8fb00e0 authored by Manish Rangankar's avatar Manish Rangankar Committed by James Bottomley

[SCSI] qla4xxx: Modified debug log messages for boot info.

In some configurations user may not have boot targets configured.
In such cases the debug messages printed out by driver look like
some kind of failure happening. However this could be a valid
case, so modified the messages to appear as warning messages
versus failure messages.
Signed-off-by: default avatarManish Rangankar <manish.rangankar@qlogic.com>
Signed-off-by: default avatarVikas Chaudhary <vikas.chaudhary@qlogic.com>
Reviewed-by: default avatarMike Christie <michaelc@cs.wisc.edu>
Signed-off-by: default avatarJames Bottomley <JBottomley@Parallels.com>
parent 20e835b4
...@@ -3393,9 +3393,8 @@ static int get_fw_boot_info(struct scsi_qla_host *ha, uint16_t ddb_index[]) ...@@ -3393,9 +3393,8 @@ static int get_fw_boot_info(struct scsi_qla_host *ha, uint16_t ddb_index[])
/* Check Boot Mode */ /* Check Boot Mode */
val = rd_nvram_byte(ha, addr); val = rd_nvram_byte(ha, addr);
if (!(val & 0x07)) { if (!(val & 0x07)) {
DEBUG2(ql4_printk(KERN_ERR, ha, DEBUG2(ql4_printk(KERN_INFO, ha, "%s: Adapter boot "
"%s: Failed Boot options : 0x%x\n", "options : 0x%x\n", __func__, val));
__func__, val));
ret = QLA_ERROR; ret = QLA_ERROR;
goto exit_boot_info; goto exit_boot_info;
} }
...@@ -3440,9 +3439,8 @@ static int get_fw_boot_info(struct scsi_qla_host *ha, uint16_t ddb_index[]) ...@@ -3440,9 +3439,8 @@ static int get_fw_boot_info(struct scsi_qla_host *ha, uint16_t ddb_index[])
} }
/* Check Boot Mode */ /* Check Boot Mode */
if (!(buf[1] & 0x07)) { if (!(buf[1] & 0x07)) {
DEBUG2(ql4_printk(KERN_INFO, ha, DEBUG2(ql4_printk(KERN_INFO, ha, "Firmware boot options"
"Failed: Boot options : 0x%x\n", " : 0x%x\n", buf[1]));
buf[1]));
ret = QLA_ERROR; ret = QLA_ERROR;
goto exit_boot_info_free; goto exit_boot_info_free;
} }
...@@ -3548,8 +3546,8 @@ static int qla4xxx_get_boot_target(struct scsi_qla_host *ha, ...@@ -3548,8 +3546,8 @@ static int qla4xxx_get_boot_target(struct scsi_qla_host *ha,
if (qla4xxx_bootdb_by_index(ha, fw_ddb_entry, if (qla4xxx_bootdb_by_index(ha, fw_ddb_entry,
fw_ddb_entry_dma, ddb_index)) { fw_ddb_entry_dma, ddb_index)) {
DEBUG2(ql4_printk(KERN_ERR, ha, DEBUG2(ql4_printk(KERN_INFO, ha, "%s: No Flash DDB found at "
"%s: Flash DDB read Failed\n", __func__)); "index [%d]\n", __func__, ddb_index));
ret = QLA_ERROR; ret = QLA_ERROR;
goto exit_boot_target; goto exit_boot_target;
} }
...@@ -3627,8 +3625,8 @@ static int qla4xxx_get_boot_info(struct scsi_qla_host *ha) ...@@ -3627,8 +3625,8 @@ static int qla4xxx_get_boot_info(struct scsi_qla_host *ha)
ddb_index[1] = 0xffff; ddb_index[1] = 0xffff;
ret = get_fw_boot_info(ha, ddb_index); ret = get_fw_boot_info(ha, ddb_index);
if (ret != QLA_SUCCESS) { if (ret != QLA_SUCCESS) {
DEBUG2(ql4_printk(KERN_ERR, ha, DEBUG2(ql4_printk(KERN_INFO, ha,
"%s: Failed to set boot info.\n", __func__)); "%s: No boot target configured.\n", __func__));
return ret; return ret;
} }
...@@ -3641,8 +3639,8 @@ static int qla4xxx_get_boot_info(struct scsi_qla_host *ha) ...@@ -3641,8 +3639,8 @@ static int qla4xxx_get_boot_info(struct scsi_qla_host *ha)
rval = qla4xxx_get_boot_target(ha, &(ha->boot_tgt.boot_pri_sess), rval = qla4xxx_get_boot_target(ha, &(ha->boot_tgt.boot_pri_sess),
ddb_index[0]); ddb_index[0]);
if (rval != QLA_SUCCESS) { if (rval != QLA_SUCCESS) {
DEBUG2(ql4_printk(KERN_ERR, ha, "%s: Failed to get " DEBUG2(ql4_printk(KERN_INFO, ha, "%s: Primary boot target not "
"primary target\n", __func__)); "configured\n", __func__));
} else } else
ret = QLA_SUCCESS; ret = QLA_SUCCESS;
...@@ -3653,8 +3651,8 @@ static int qla4xxx_get_boot_info(struct scsi_qla_host *ha) ...@@ -3653,8 +3651,8 @@ static int qla4xxx_get_boot_info(struct scsi_qla_host *ha)
rval = qla4xxx_get_boot_target(ha, &(ha->boot_tgt.boot_sec_sess), rval = qla4xxx_get_boot_target(ha, &(ha->boot_tgt.boot_sec_sess),
ddb_index[1]); ddb_index[1]);
if (rval != QLA_SUCCESS) { if (rval != QLA_SUCCESS) {
DEBUG2(ql4_printk(KERN_ERR, ha, "%s: Failed to get " DEBUG2(ql4_printk(KERN_INFO, ha, "%s: Secondary boot target not"
"secondary target\n", __func__)); " configured\n", __func__));
} else } else
ret = QLA_SUCCESS; ret = QLA_SUCCESS;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment