RE: [Intel-wired-lan] [PATCH] ice: set the value of global config lock timeout longer

From: Brelinski, TonyX
Date: Wed May 12 2021 - 18:24:06 EST


> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of
> Liwei Song
> Sent: Monday, April 19, 2021 2:31 AM
> To: intel-wired-lan <intel-wired-lan@xxxxxxxxxxxxxxxx>; Brandeburg, Jesse
> <jesse.brandeburg@xxxxxxxxx>; Nguyen, Anthony L
> <anthony.l.nguyen@xxxxxxxxx>
> Cc: Jakub <kuba@xxxxxxxxxx>; David <davem@xxxxxxxxxxxxx>; linux-
> kernel@xxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx
> Subject: [Intel-wired-lan] [PATCH] ice: set the value of global config lock
> timeout longer
>
> It may need hold Global Config Lock a longer time when download DDP
> package file, extend the timeout value to 5000ms to ensure that download
> can be finished before other AQ command got time to run, this will fix the
> issue below when probe the device, 5000ms is a test value that work with
> both Backplane and BreakoutCable NVM image:
>
> ice 0000:f4:00.0: VSI 12 failed lan queue config, error ICE_ERR_CFG ice
> 0000:f4:00.0: Failed to delete VSI 12 in FW - error: ICE_ERR_AQ_TIMEOUT ice
> 0000:f4:00.0: probe failed due to setup PF switch: -12
> ice: probe of 0000:f4:00.0 failed with error -12
>
> Signed-off-by: Liwei Song <liwei.song@xxxxxxxxxxxxx>
> ---
> drivers/net/ethernet/intel/ice/ice_type.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Tested-by: Tony Brelinski <tonyx.brelinski@xxxxxxxxx> (A Contingent Worker at Intel)