Commit bc6ac5e8 authored by Tomas Henzl's avatar Tomas Henzl Committed by Christoph Hellwig

megaraid_sas: fix a small problem when reading state value from hw

When the driver reads state values from the hw it might happen that
different values are read in subsequent reads and this can cause problems,
this may lead to a timeout in this function and a non working adapter.

Cc: Adam Radford <aradford@gmail.com>
Signed-off-by: default avatarTomas Henzl <thenzl@redhat.com>
Reviewed-by: default avatarShintaro Minemoto <fj3207hq@aa.jp.fujitsu.com>
Acked-by: default avatarSumit Saxena <sumit.saxena@lsi.com>
Signed-off-by: default avatarChristoph Hellwig <hch@lst.de>
parent d6d211db
...@@ -3061,7 +3061,8 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr) ...@@ -3061,7 +3061,8 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr)
u32 cur_state; u32 cur_state;
u32 abs_state, curr_abs_state; u32 abs_state, curr_abs_state;
fw_state = instance->instancet->read_fw_status_reg(instance->reg_set) & MFI_STATE_MASK; abs_state = instance->instancet->read_fw_status_reg(instance->reg_set);
fw_state = abs_state & MFI_STATE_MASK;
if (fw_state != MFI_STATE_READY) if (fw_state != MFI_STATE_READY)
printk(KERN_INFO "megasas: Waiting for FW to come to ready" printk(KERN_INFO "megasas: Waiting for FW to come to ready"
...@@ -3069,9 +3070,6 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr) ...@@ -3069,9 +3070,6 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr)
while (fw_state != MFI_STATE_READY) { while (fw_state != MFI_STATE_READY) {
abs_state =
instance->instancet->read_fw_status_reg(instance->reg_set);
switch (fw_state) { switch (fw_state) {
case MFI_STATE_FAULT: case MFI_STATE_FAULT:
...@@ -3223,10 +3221,8 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr) ...@@ -3223,10 +3221,8 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr)
* The cur_state should not last for more than max_wait secs * The cur_state should not last for more than max_wait secs
*/ */
for (i = 0; i < (max_wait * 1000); i++) { for (i = 0; i < (max_wait * 1000); i++) {
fw_state = instance->instancet->read_fw_status_reg(instance->reg_set) & curr_abs_state = instance->instancet->
MFI_STATE_MASK ; read_fw_status_reg(instance->reg_set);
curr_abs_state =
instance->instancet->read_fw_status_reg(instance->reg_set);
if (abs_state == curr_abs_state) { if (abs_state == curr_abs_state) {
msleep(1); msleep(1);
...@@ -3242,6 +3238,9 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr) ...@@ -3242,6 +3238,9 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr)
"in %d secs\n", fw_state, max_wait); "in %d secs\n", fw_state, max_wait);
return -ENODEV; return -ENODEV;
} }
abs_state = curr_abs_state;
fw_state = curr_abs_state & MFI_STATE_MASK;
} }
printk(KERN_INFO "megasas: FW now in Ready state\n"); printk(KERN_INFO "megasas: FW now in Ready state\n");
......
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