This conditional is missing a bang, with the intent being to break when the retry count reaches zero.
Unified · Split
| @@ -184,7 +184,7 @@ static int ibmvnic_wait_for_completion(struct ibmvnic_adapter *adapter, · 183 unmodified lines | ||
| 184 | 184 | netdev_err(netdev, "Device down!\n"); |
| 185 | 185 | return -ENODEV; |
| 186 | 186 | } |
| 187 | − if (retry--) | |
| 187 | + if (!retry--) | |
| 188 | 188 | break; |
| 189 | 189 | if (wait_for_completion_timeout(comp_done, div_timeout)) |
| 190 | 190 | return 0; |