]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
nvme.h: untangle AEN notice definitions
authorChristoph Hellwig <hch@lst.de>
Tue, 22 May 2018 09:09:54 +0000 (11:09 +0200)
committerChristoph Hellwig <hch@lst.de>
Thu, 31 May 2018 16:46:48 +0000 (18:46 +0200)
Stop including the event type in the definitions for the notice type.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
drivers/nvme/host/core.c
include/linux/nvme.h

index 2c4cf65641a6b12d7706ae1214f64d1f6c3bc15f..86cb78653155cb1ddba4d3d680be354deb0b0af8 100644 (file)
@@ -3321,6 +3321,21 @@ static void nvme_fw_act_work(struct work_struct *work)
        nvme_get_fw_slot_info(ctrl);
 }
 
+static void nvme_handle_aen_notice(struct nvme_ctrl *ctrl, u32 result)
+{
+       switch ((result & 0xff00) >> 8) {
+       case NVME_AER_NOTICE_NS_CHANGED:
+               dev_info(ctrl->device, "rescanning\n");
+               nvme_queue_scan(ctrl);
+               break;
+       case NVME_AER_NOTICE_FW_ACT_STARTING:
+               queue_work(nvme_wq, &ctrl->fw_act_work);
+               break;
+       default:
+               dev_warn(ctrl->device, "async event result %08x\n", result);
+       }
+}
+
 void nvme_complete_async_event(struct nvme_ctrl *ctrl, __le16 status,
                volatile union nvme_result *res)
 {
@@ -3330,6 +3345,9 @@ void nvme_complete_async_event(struct nvme_ctrl *ctrl, __le16 status,
                return;
 
        switch (result & 0x7) {
+       case NVME_AER_NOTICE:
+               nvme_handle_aen_notice(ctrl, result);
+               break;
        case NVME_AER_ERROR:
        case NVME_AER_SMART:
        case NVME_AER_CSS:
@@ -3339,18 +3357,6 @@ void nvme_complete_async_event(struct nvme_ctrl *ctrl, __le16 status,
        default:
                break;
        }
-
-       switch (result & 0xff07) {
-       case NVME_AER_NOTICE_NS_CHANGED:
-               dev_info(ctrl->device, "rescanning\n");
-               nvme_queue_scan(ctrl);
-               break;
-       case NVME_AER_NOTICE_FW_ACT_STARTING:
-               queue_work(nvme_wq, &ctrl->fw_act_work);
-               break;
-       default:
-               dev_warn(ctrl->device, "async event result %08x\n", result);
-       }
        queue_work(nvme_wq, &ctrl->async_event_work);
 }
 EXPORT_SYMBOL_GPL(nvme_complete_async_event);
index 4112e2bd747f5828f031e5ac638888d4f38384cd..c37103a4ad380d9ea516becfea1d38e8060521fc 100644 (file)
@@ -436,10 +436,14 @@ enum {
 enum {
        NVME_AER_ERROR                  = 0,
        NVME_AER_SMART                  = 1,
+       NVME_AER_NOTICE                 = 2,
        NVME_AER_CSS                    = 6,
        NVME_AER_VS                     = 7,
-       NVME_AER_NOTICE_NS_CHANGED      = 0x0002,
-       NVME_AER_NOTICE_FW_ACT_STARTING = 0x0102,
+};
+
+enum {
+       NVME_AER_NOTICE_NS_CHANGED      = 0x00,
+       NVME_AER_NOTICE_FW_ACT_STARTING = 0x01,
 };
 
 struct nvme_lba_range_type {