net: hns3: add support for forwarding packet to queues of specified TC when flow...
authorJian Shen <shenjian15@huawei.com>
Thu, 10 Dec 2020 03:42:08 +0000 (11:42 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Dec 2020 04:33:19 +0000 (20:33 -0800)
For some new device, it supports forwarding packet to queues
of specified TC when flow director rule hit. So extend the
command handle to support it.

Signed-off-by: Jian Shen <shenjian15@huawei.com>
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.c
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.h

index 85986c7..b728be4 100644 (file)
@@ -359,6 +359,8 @@ static void hclge_parse_capability(struct hclge_dev *hdev,
                set_bit(HNAE3_DEV_SUPPORT_HW_TX_CSUM_B, ae_dev->caps);
        if (hnae3_get_bit(caps, HCLGE_CAP_UDP_TUNNEL_CSUM_B))
                set_bit(HNAE3_DEV_SUPPORT_UDP_TUNNEL_CSUM_B, ae_dev->caps);
+       if (hnae3_get_bit(caps, HCLGE_CAP_FD_FORWARD_TC_B))
+               set_bit(HNAE3_DEV_SUPPORT_FD_FORWARD_TC_B, ae_dev->caps);
 }
 
 static enum hclge_cmd_status
index 52a6f9b..df5417d 100644 (file)
@@ -1051,6 +1051,9 @@ struct hclge_fd_tcam_config_3_cmd {
 #define HCLGE_FD_AD_WR_RULE_ID_B       0
 #define HCLGE_FD_AD_RULE_ID_S          1
 #define HCLGE_FD_AD_RULE_ID_M          GENMASK(13, 1)
+#define HCLGE_FD_AD_TC_OVRD_B          16
+#define HCLGE_FD_AD_TC_SIZE_S          17
+#define HCLGE_FD_AD_TC_SIZE_M          GENMASK(20, 17)
 
 struct hclge_fd_ad_config_cmd {
        u8 stage;
index 366920b..0b6102e 100644 (file)
@@ -5099,6 +5099,7 @@ static int hclge_fd_tcam_config(struct hclge_dev *hdev, u8 stage, bool sel_x,
 static int hclge_fd_ad_config(struct hclge_dev *hdev, u8 stage, int loc,
                              struct hclge_fd_ad_data *action)
 {
+       struct hnae3_ae_dev *ae_dev = pci_get_drvdata(hdev->pdev);
        struct hclge_fd_ad_config_cmd *req;
        struct hclge_desc desc;
        u64 ad_data = 0;
@@ -5114,6 +5115,12 @@ static int hclge_fd_ad_config(struct hclge_dev *hdev, u8 stage, int loc,
                      action->write_rule_id_to_bd);
        hnae3_set_field(ad_data, HCLGE_FD_AD_RULE_ID_M, HCLGE_FD_AD_RULE_ID_S,
                        action->rule_id);
+       if (test_bit(HNAE3_DEV_SUPPORT_FD_FORWARD_TC_B, ae_dev->caps)) {
+               hnae3_set_bit(ad_data, HCLGE_FD_AD_TC_OVRD_B,
+                             action->override_tc);
+               hnae3_set_field(ad_data, HCLGE_FD_AD_TC_SIZE_M,
+                               HCLGE_FD_AD_TC_SIZE_S, (u32)action->tc_size);
+       }
        ad_data <<= 32;
        hnae3_set_bit(ad_data, HCLGE_FD_AD_DROP_B, action->drop_packet);
        hnae3_set_bit(ad_data, HCLGE_FD_AD_DIRECT_QID_B,
@@ -5357,16 +5364,22 @@ static int hclge_config_key(struct hclge_dev *hdev, u8 stage,
 static int hclge_config_action(struct hclge_dev *hdev, u8 stage,
                               struct hclge_fd_rule *rule)
 {
+       struct hclge_vport *vport = hdev->vport;
+       struct hnae3_knic_private_info *kinfo = &vport->nic.kinfo;
        struct hclge_fd_ad_data ad_data;
 
+       memset(&ad_data, 0, sizeof(struct hclge_fd_ad_data));
        ad_data.ad_id = rule->location;
 
        if (rule->action == HCLGE_FD_ACTION_DROP_PACKET) {
                ad_data.drop_packet = true;
-               ad_data.forward_to_direct_queue = false;
-               ad_data.queue_id = 0;
+       } else if (rule->action == HCLGE_FD_ACTION_SELECT_TC) {
+               ad_data.override_tc = true;
+               ad_data.queue_id =
+                       kinfo->tc_info.tqp_offset[rule->tc];
+               ad_data.tc_size =
+                       ilog2(kinfo->tc_info.tqp_count[rule->tc]);
        } else {
-               ad_data.drop_packet = false;
                ad_data.forward_to_direct_queue = true;
                ad_data.queue_id = rule->queue_id;
        }
@@ -5937,7 +5950,7 @@ static int hclge_add_fd_entry(struct hnae3_handle *handle,
                        return -EINVAL;
                }
 
-               action = HCLGE_FD_ACTION_ACCEPT_PACKET;
+               action = HCLGE_FD_ACTION_SELECT_QUEUE;
                q_index = ring;
        }
 
index b3c1301..a481064 100644 (file)
@@ -572,8 +572,9 @@ enum HCLGE_FD_PACKET_TYPE {
 };
 
 enum HCLGE_FD_ACTION {
-       HCLGE_FD_ACTION_ACCEPT_PACKET,
+       HCLGE_FD_ACTION_SELECT_QUEUE,
        HCLGE_FD_ACTION_DROP_PACKET,
+       HCLGE_FD_ACTION_SELECT_TC,
 };
 
 struct hclge_fd_key_cfg {
@@ -619,6 +620,7 @@ struct hclge_fd_rule {
        u32 unused_tuple;
        u32 flow_type;
        u8 action;
+       u8 tc;
        u16 vf_id;
        u16 queue_id;
        u16 location;
@@ -637,6 +639,8 @@ struct hclge_fd_ad_data {
        u8 write_rule_id_to_bd;
        u8 next_input_key;
        u16 rule_id;
+       u16 tc_size;
+       u8 override_tc;
 };
 
 enum HCLGE_MAC_NODE_STATE {