Commit 22e98449 authored by Manish Chopra's avatar Manish Chopra Committed by David S. Miller

qlcnic: Fix CPU soft lockup while collecting firmware dump

Driver while collecting firmware dump takes longer time to
collect/process some of the firmware dump entries/memories.
Bigger capture masks makes it worse as it results in larger
amount of data being collected and results in CPU soft lockup.
Place cond_resched() in some of the driver flows that are
expectedly time consuming to relinquish the CPU to avoid CPU
soft lockup panic.
Signed-off-by: default avatarShahed Shaikh <shshaikh@marvell.com>
Tested-by: default avatarYonggen Xu <Yonggen.Xu@dell.com>
Signed-off-by: default avatarManish Chopra <manishc@marvell.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent bb48eb9b
...@@ -2043,6 +2043,7 @@ static void qlcnic_83xx_exec_template_cmd(struct qlcnic_adapter *p_dev, ...@@ -2043,6 +2043,7 @@ static void qlcnic_83xx_exec_template_cmd(struct qlcnic_adapter *p_dev,
break; break;
} }
entry += p_hdr->size; entry += p_hdr->size;
cond_resched();
} }
p_dev->ahw->reset.seq_index = index; p_dev->ahw->reset.seq_index = index;
} }
......
...@@ -703,6 +703,7 @@ static u32 qlcnic_read_memory_test_agent(struct qlcnic_adapter *adapter, ...@@ -703,6 +703,7 @@ static u32 qlcnic_read_memory_test_agent(struct qlcnic_adapter *adapter,
addr += 16; addr += 16;
reg_read -= 16; reg_read -= 16;
ret += 16; ret += 16;
cond_resched();
} }
out: out:
mutex_unlock(&adapter->ahw->mem_lock); mutex_unlock(&adapter->ahw->mem_lock);
...@@ -1383,6 +1384,7 @@ int qlcnic_dump_fw(struct qlcnic_adapter *adapter) ...@@ -1383,6 +1384,7 @@ int qlcnic_dump_fw(struct qlcnic_adapter *adapter)
buf_offset += entry->hdr.cap_size; buf_offset += entry->hdr.cap_size;
entry_offset += entry->hdr.offset; entry_offset += entry->hdr.offset;
buffer = fw_dump->data + buf_offset; buffer = fw_dump->data + buf_offset;
cond_resched();
} }
fw_dump->clr = 1; fw_dump->clr = 1;
......
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