net: hns3: remove redundant param mbx_event_pending
authorPeng Li <lipeng321@huawei.com>
Fri, 27 Aug 2021 09:28:19 +0000 (17:28 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 27 Aug 2021 10:20:21 +0000 (11:20 +0100)
This patch removes the redundant param mbx_event_pending.

Signed-off-by: Peng Li <lipeng321@huawei.com>
Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_main.h
drivers/net/ethernet/hisilicon/hns3/hns3vf/hclgevf_mbx.c

index 73e8bb5..1de8e2d 100644 (file)
@@ -317,7 +317,6 @@ struct hclgevf_dev {
 
        struct hclgevf_mac_table_cfg mac_table;
 
-       bool mbx_event_pending;
        struct hclgevf_mbx_resp_status mbx_resp; /* mailbox response */
        struct hclgevf_mbx_arq_ring arq; /* mailbox async rx queue */
 
index b339b9b..5030950 100644 (file)
@@ -236,13 +236,6 @@ void hclgevf_mbx_handler(struct hclgevf_dev *hdev)
                case HCLGE_MBX_LINK_STAT_MODE:
                case HCLGE_MBX_PUSH_VLAN_INFO:
                case HCLGE_MBX_PUSH_PROMISC_INFO:
-                       /* set this mbx event as pending. This is required as we
-                        * might loose interrupt event when mbx task is busy
-                        * handling. This shall be cleared when mbx task just
-                        * enters handling state.
-                        */
-                       hdev->mbx_event_pending = true;
-
                        /* we will drop the async msg if we find ARQ as full
                         * and continue with next message
                         */
@@ -298,11 +291,6 @@ void hclgevf_mbx_async_handler(struct hclgevf_dev *hdev)
        u8 flag;
        u8 idx;
 
-       /* we can safely clear it now as we are at start of the async message
-        * processing
-        */
-       hdev->mbx_event_pending = false;
-
        tail = hdev->arq.tail;
 
        /* process all the async queue messages */