Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752181AbaJTRGn (ORCPT ); Mon, 20 Oct 2014 13:06:43 -0400 Received: from mail-wg0-f42.google.com ([74.125.82.42]:58085 "EHLO mail-wg0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750807AbaJTRGm (ORCPT ); Mon, 20 Oct 2014 13:06:42 -0400 From: Ebru Akagunduz To: arnd@arndb.de Cc: JBottomley@parallels.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, opw-kernel@googlegroups.com, Ebru Akagunduz Subject: [PATCH v2] scsi: ips.c: Use jiffies comparison instead of do_gettimeofday() Date: Mon, 20 Oct 2014 20:06:01 +0300 Message-Id: <1413824761-21426-1-git-send-email-ebru.akagunduz@gmail.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org do_gettimeofday() only can get 32-bit time types but the driver should be able to use dates that are after January 2038. Remove do_gettimeofday() and use jiffies comparison to supply 64-bit time types. Signed-off-by: Ebru Akagunduz --- drivers/scsi/ips.c | 25 +++++++++++++------------ drivers/scsi/ips.h | 2 +- 2 files changed, 14 insertions(+), 13 deletions(-) diff --git a/drivers/scsi/ips.c b/drivers/scsi/ips.c index e5afc38..6d27024 100644 --- a/drivers/scsi/ips.c +++ b/drivers/scsi/ips.c @@ -162,6 +162,7 @@ */ #include +#include #include #include #include @@ -297,7 +298,7 @@ static void ips_freescb(ips_ha_t *, ips_scb_t *); static void ips_setup_funclist(ips_ha_t *); static void ips_statinit(ips_ha_t *); static void ips_statinit_memio(ips_ha_t *); -static void ips_fix_ffdc_time(ips_ha_t *, ips_scb_t *, time_t); +static void ips_fix_ffdc_time(ips_ha_t *, ips_scb_t *, unsigned long); static void ips_ffdc_reset(ips_ha_t *, int); static void ips_ffdc_time(ips_ha_t *); static uint32_t ips_statupd_copperhead(ips_ha_t *); @@ -993,10 +994,10 @@ static int __ips_eh_reset(struct scsi_cmnd *SC) /* FFDC */ if (le32_to_cpu(ha->subsys->param[3]) & 0x300000) { - struct timeval tv; + unsigned long now; - do_gettimeofday(&tv); - ha->last_ffdc = tv.tv_sec; + now = jiffies; + ha->last_ffdc = jiffies; ha->reset_count++; ips_ffdc_reset(ha, IPS_INTR_IORL); } @@ -2404,7 +2405,7 @@ static int ips_hainit(ips_ha_t * ha) { int i; - struct timeval tv; + unsigned long now; METHOD_TRACE("ips_hainit", 1); @@ -2419,8 +2420,8 @@ ips_hainit(ips_ha_t * ha) /* Send FFDC */ ha->reset_count = 1; - do_gettimeofday(&tv); - ha->last_ffdc = tv.tv_sec; + now = jiffies; + ha->last_ffdc = jiffies; ips_ffdc_reset(ha, IPS_INTR_IORL); if (!ips_read_config(ha, IPS_INTR_IORL)) { @@ -2560,12 +2561,12 @@ ips_next(ips_ha_t * ha, int intr) if ((ha->subsys->param[3] & 0x300000) && (ha->scb_activelist.count == 0)) { - struct timeval tv; + unsigned long now; - do_gettimeofday(&tv); + now = jiffies; - if (tv.tv_sec - ha->last_ffdc > IPS_SECS_8HOURS) { - ha->last_ffdc = tv.tv_sec; + if (time_after(now, ha->last_ffdc + IPS_SECS_8HOURS * HZ)) { + ha->last_ffdc = now; ips_ffdc_time(ha); } } @@ -6000,7 +6001,7 @@ ips_ffdc_time(ips_ha_t * ha) /* */ /****************************************************************************/ static void -ips_fix_ffdc_time(ips_ha_t * ha, ips_scb_t * scb, time_t current_time) +ips_fix_ffdc_time(ips_ha_t * ha, ips_scb_t * scb, unsigned long current_time) { long days; long rem; diff --git a/drivers/scsi/ips.h b/drivers/scsi/ips.h index 45b9566..250beea 100644 --- a/drivers/scsi/ips.h +++ b/drivers/scsi/ips.h @@ -1054,7 +1054,7 @@ typedef struct ips_ha { uint8_t active; int ioctl_reset; /* IOCTL Requested Reset Flag */ uint16_t reset_count; /* number of resets */ - time_t last_ffdc; /* last time we sent ffdc info*/ + unsigned long last_ffdc; /* last time we sent ffdc info*/ uint8_t slot_num; /* PCI Slot Number */ int ioctl_len; /* size of ioctl buffer */ dma_addr_t ioctl_busaddr; /* dma address of ioctl buffer*/ -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/