scsi: bfa: improve bfa_ioc_send_enable/disable data
authorArnd Bergmann <arnd@arndb.de>
Fri, 10 Nov 2017 15:37:11 +0000 (16:37 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 5 Dec 2017 01:32:52 +0000 (20:32 -0500)
In bfa_ioc_send_enable, we use the deprecated do_gettimeofday() function
to read the current time. This is not a problem, since the firmware
interface is already limited to 32-bit timestamps, but it's better to
use ktime_get_seconds() and document what the limitation is.

I noticed that I did the same change in commit a5af83925363 ("bna: avoid
writing uninitialized data into hw registers") for the ethernet
driver. That commit also changed the "disable" funtion to initialize the
data we pass to the firmware properly, so I'm doing the same thing here.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Anil Gurumurthy <Anil.Gurumurthy@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/bfa/bfa_ioc.c

index 256f4afaccf9633ebcd0b74c6efd04c87f4043be..117332537763a0f2556b588b53919a0251b0e186 100644 (file)
@@ -1809,13 +1809,12 @@ static void
 bfa_ioc_send_enable(struct bfa_ioc_s *ioc)
 {
        struct bfi_ioc_ctrl_req_s enable_req;
-       struct timeval tv;
 
        bfi_h2i_set(enable_req.mh, BFI_MC_IOC, BFI_IOC_H2I_ENABLE_REQ,
                    bfa_ioc_portid(ioc));
        enable_req.clscode = cpu_to_be16(ioc->clscode);
-       do_gettimeofday(&tv);
-       enable_req.tv_sec = be32_to_cpu(tv.tv_sec);
+       /* unsigned 32-bit time_t overflow in y2106 */
+       enable_req.tv_sec = be32_to_cpu(ktime_get_real_seconds());
        bfa_ioc_mbox_send(ioc, &enable_req, sizeof(struct bfi_ioc_ctrl_req_s));
 }
 
@@ -1826,6 +1825,9 @@ bfa_ioc_send_disable(struct bfa_ioc_s *ioc)
 
        bfi_h2i_set(disable_req.mh, BFI_MC_IOC, BFI_IOC_H2I_DISABLE_REQ,
                    bfa_ioc_portid(ioc));
+       disable_req.clscode = cpu_to_be16(ioc->clscode);
+       /* unsigned 32-bit time_t overflow in y2106 */
+       disable_req.tv_sec = be32_to_cpu(ktime_get_real_seconds());
        bfa_ioc_mbox_send(ioc, &disable_req, sizeof(struct bfi_ioc_ctrl_req_s));
 }