scsi: smartpqi: enhance BMIC cache flush

 - distinguish between shutdown and non-shutdown.

Reviewed-by: Scott Benesh <scott.benesh@microsemi.com>
Signed-off-by: Kevin Barnett <kevin.barnett@microsemi.com>
Signed-off-by: Don Brace <don.brace@microsemi.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
(cherry picked from commit 58322fe0069a2ae2a19cf29023cc0b82c7245762)

Orabug: 26943380

Signed-off-by: Kirtikar Kashyap <kirtikar.kashyap@oracle.com>
Reviewed-by: Jack Vogel <jack.vogel@oracle.com>

diff --git a/drivers/scsi/smartpqi/smartpqi.h b/drivers/scsi/smartpqi/smartpqi.h
index 58e17e1d4b043c65f7d3cdb92aa279cd8072cb37..b73ebfcd2d459f1d2f9d8542503e4da0bae2532c 100644 (file)
--- a/drivers/scsi/smartpqi/smartpqi.h
+++ b/drivers/scsi/smartpqi/smartpqi.h
@@ -1080,9 +1080,9 @@ enum pqi_ctrl_mode {
 #define BMIC_SENSE_CONTROLLER_PARAMETERS       0x64
 #define BMIC_SENSE_SUBSYSTEM_INFORMATION       0x66
 #define BMIC_WRITE_HOST_WELLNESS               0xa5
-#define BMIC_CACHE_FLUSH                       0xc2
+#define BMIC_FLUSH_CACHE                       0xc2
 
-#define SA_CACHE_FLUSH                         0x1
+#define SA_FLUSH_CACHE                         0x1
 
 #define MASKED_DEVICE(lunid)                   ((lunid)[3] & 0xc0)
 #define CISS_GET_LEVEL_2_BUS(lunid)            ((lunid)[7] & 0x3f)
@@ -1188,6 +1188,23 @@ struct bmic_identify_physical_device {
        u8      padding_to_multiple_of_512[9];
 };
 
+struct bmic_flush_cache {
+       u8      disable_flag;
+       u8      system_power_action;
+       u8      ndu_flush;
+       u8      shutdown_event;
+       u8      reserved[28];
+};
+
+/* for shutdown_event member of struct bmic_flush_cache */
+enum bmic_flush_cache_shutdown_event {
+       NONE_CACHE_FLUSH_ONLY = 0,
+       SHUTDOWN = 1,
+       HIBERNATE = 2,
+       SUSPEND = 3,
+       RESTART = 4
+};
+
 #pragma pack()
 
 int pqi_add_sas_host(struct Scsi_Host *shost, struct pqi_ctrl_info *ctrl_info);

diff --git a/drivers/scsi/smartpqi/smartpqi_init.c b/drivers/scsi/smartpqi/smartpqi_init.c
index 9e96c9cbc4174e8d51206967b6d4b93ba1b2a168..34f27b220d3284607432287d6914990645d2dc7f 100644 (file)
--- a/drivers/scsi/smartpqi/smartpqi_init.c
+++ b/drivers/scsi/smartpqi/smartpqi_init.c
@@ -431,10 +431,10 @@ static int pqi_build_raid_path_request(struct pqi_ctrl_info *ctrl_info,
                cdb[1] = CISS_GET_RAID_MAP;
                put_unaligned_be32(buffer_length, &cdb[6]);
                break;
-       case SA_CACHE_FLUSH:
+       case SA_FLUSH_CACHE:
                request->data_direction = SOP_WRITE_FLAG;
                cdb[0] = BMIC_WRITE;
-               cdb[6] = BMIC_CACHE_FLUSH;
+               cdb[6] = BMIC_FLUSH_CACHE;
                put_unaligned_be16(buffer_length, &cdb[7]);
                break;
        case BMIC_IDENTIFY_CONTROLLER:
@@ -585,14 +585,13 @@ static int pqi_identify_physical_device(struct pqi_ctrl_info *ctrl_info,
        return rc;
 }
 
-#define SA_CACHE_FLUSH_BUFFER_LENGTH   4
-
-static int pqi_flush_cache(struct pqi_ctrl_info *ctrl_info)
+static int pqi_flush_cache(struct pqi_ctrl_info *ctrl_info,
+       enum bmic_flush_cache_shutdown_event shutdown_event)
 {
        int rc;
        struct pqi_raid_path_request request;
        int pci_direction;
-       u8 *buffer;
+       struct bmic_flush_cache *flush_cache;
 
        /*
         * Don't bother trying to flush the cache if the controller is
@@ -601,13 +600,15 @@ static int pqi_flush_cache(struct pqi_ctrl_info *ctrl_info)
        if (pqi_ctrl_offline(ctrl_info))
                return -ENXIO;
 
-       buffer = kzalloc(SA_CACHE_FLUSH_BUFFER_LENGTH, GFP_KERNEL);
-       if (!buffer)
+       flush_cache = kzalloc(sizeof(*flush_cache), GFP_KERNEL);
+       if (!flush_cache)
                return -ENOMEM;
 
+       flush_cache->shutdown_event = shutdown_event;
+
        rc = pqi_build_raid_path_request(ctrl_info, &request,
-               SA_CACHE_FLUSH, RAID_CTLR_LUNID, buffer,
-               SA_CACHE_FLUSH_BUFFER_LENGTH, 0, &pci_direction);
+               SA_FLUSH_CACHE, RAID_CTLR_LUNID, flush_cache,
+               sizeof(*flush_cache), 0, &pci_direction);
        if (rc)
                goto out;
 
@@ -618,7 +619,7 @@ static int pqi_flush_cache(struct pqi_ctrl_info *ctrl_info)
                pci_direction);
 
 out:
-       kfree(buffer);
+       kfree(flush_cache);
 
        return rc;
 }
@@ -6731,7 +6732,7 @@ static void pqi_shutdown(struct pci_dev *pci_dev)
         * Write all data in the controller's battery-backed cache to
         * storage.
         */
-       rc = pqi_flush_cache(ctrl_info);
+       rc = pqi_flush_cache(ctrl_info, SHUTDOWN);
        if (rc == 0)
                return;
 
@@ -6775,7 +6776,7 @@ static __maybe_unused int pqi_suspend(struct pci_dev *pci_dev, pm_message_t stat
        pqi_cancel_rescan_worker(ctrl_info);
        pqi_wait_until_scan_finished(ctrl_info);
        pqi_wait_until_lun_reset_finished(ctrl_info);
-       pqi_flush_cache(ctrl_info);
+       pqi_flush_cache(ctrl_info, SUSPEND);
        pqi_ctrl_block_requests(ctrl_info);
        pqi_ctrl_wait_until_quiesced(ctrl_info);
        pqi_wait_until_inbound_queues_empty(ctrl_info);