From: Karim Eshapa Date: Mon, 1 May 2017 13:58:08 +0000 (+0200) Subject: benet: Use time_before_eq for time comparison X-Git-Tag: v4.1.12-106.0.20170710_2300~3 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=48dd0c2454f28c58ef3f1ad08d32b9e5dcf7fa0a;p=users%2Fjedix%2Flinux-maple.git benet: Use time_before_eq for time comparison Use time_before_eq for time comparison more safe and dealing with timer wrapping to be future-proof. Orabug: 26403655 Signed-off-by: Karim Eshapa Signed-off-by: David S. Miller Signed-off-by: Kirtikar Kashyap Reviewed-by: Reviewed-by: Jack Vogel --- diff --git a/drivers/net/ethernet/emulex/benet/be_main.c b/drivers/net/ethernet/emulex/benet/be_main.c index 0258f4aef1ea..c3a2cf035203 100644 --- a/drivers/net/ethernet/emulex/benet/be_main.c +++ b/drivers/net/ethernet/emulex/benet/be_main.c @@ -5240,15 +5240,15 @@ static bool be_err_is_recoverable(struct be_adapter *adapter) dev_err(&adapter->pdev->dev, "Recoverable HW error code: 0x%x\n", ue_err_code); - if (jiffies - err_rec->probe_time <= initial_idle_time) { + if (time_before_eq(jiffies - err_rec->probe_time, initial_idle_time)) { dev_err(&adapter->pdev->dev, "Cannot recover within %lu sec from driver load\n", jiffies_to_msecs(initial_idle_time) / MSEC_PER_SEC); return false; } - if (err_rec->last_recovery_time && - (jiffies - err_rec->last_recovery_time <= recovery_interval)) { + if (err_rec->last_recovery_time && time_before_eq( + jiffies - err_rec->last_recovery_time, recovery_interval)) { dev_err(&adapter->pdev->dev, "Cannot recover within %lu sec from last recovery\n", jiffies_to_msecs(recovery_interval) / MSEC_PER_SEC);