i40e: make PF wait reset loop reliable
Use jiffies to limit max waiting time for PF reset to succeed. Previous wait loop was unreliable. It required unreasonably long time to wait for PF reset after reboot when NIC was about to enter recovery mode Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com> Signed-off-by: Piotr Kwapulinski <piotr.kwapulinski@intel.com> Tested-by: Andrew Bowers <andrewx.bowers@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
This commit is contained in:
Родитель
3c98f9ee6b
Коммит
91c534b5e3
|
@ -14606,25 +14606,23 @@ static bool i40e_check_recovery_mode(struct i40e_pf *pf)
|
|||
**/
|
||||
static i40e_status i40e_pf_loop_reset(struct i40e_pf *pf)
|
||||
{
|
||||
const unsigned short MAX_CNT = 1000;
|
||||
const unsigned short MSECS = 10;
|
||||
/* wait max 10 seconds for PF reset to succeed */
|
||||
const unsigned long time_end = jiffies + 10 * HZ;
|
||||
|
||||
struct i40e_hw *hw = &pf->hw;
|
||||
i40e_status ret;
|
||||
int cnt;
|
||||
|
||||
for (cnt = 0; cnt < MAX_CNT; ++cnt) {
|
||||
ret = i40e_pf_reset(hw);
|
||||
if (!ret)
|
||||
break;
|
||||
msleep(MSECS);
|
||||
}
|
||||
|
||||
if (cnt == MAX_CNT) {
|
||||
dev_info(&pf->pdev->dev, "PF reset failed: %d\n", ret);
|
||||
return ret;
|
||||
while (ret != I40E_SUCCESS && time_before(jiffies, time_end)) {
|
||||
usleep_range(10000, 20000);
|
||||
ret = i40e_pf_reset(hw);
|
||||
}
|
||||
|
||||
if (ret == I40E_SUCCESS)
|
||||
pf->pfr_count++;
|
||||
else
|
||||
dev_info(&pf->pdev->dev, "PF reset failed: %d\n", ret);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
|
Загрузка…
Ссылка в новой задаче