From patchwork Tue Jun 26 11:37:20 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yongseok Koh X-Patchwork-Id: 41562 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9D5ED1B435; Tue, 26 Jun 2018 13:37:44 +0200 (CEST) Received: from EUR03-VE1-obe.outbound.protection.outlook.com (mail-eopbgr50056.outbound.protection.outlook.com [40.107.5.56]) by dpdk.org (Postfix) with ESMTP id 08E631B3B6 for ; Tue, 26 Jun 2018 13:37:44 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=beimZS+QHAbws+FEEmEmyJwUpKBHWo2XMnJ9FvvGAiM=; b=hI6/D0PqhPFoPuDvf06wiM7pQd9dG8reakX913Lbtva39birGzlJS9CNg/Yksd5FIUACrNdKvSCxTCnSArnJm4W410MElmZjk8kh6gDpDbttNF7EvrSqVFB1lsflps8qOsxORZ0tuTtMdBs9+gI/4CHvudm8Pooc/OMJEJzxsmI= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=yskoh@mellanox.com; Received: from mellanox.com (209.116.155.178) by DB6PR0501MB2038.eurprd05.prod.outlook.com (2603:10a6:4:6::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.906.20; Tue, 26 Jun 2018 11:37:41 +0000 From: Yongseok Koh To: adrien.mazarguil@6wind.com, nelio.laranjeiro@6wind.com Cc: dev@dpdk.org, shahafs@mellanox.com, Yongseok Koh Date: Tue, 26 Jun 2018 04:37:20 -0700 Message-Id: <20180626113723.33035-1-yskoh@mellanox.com> X-Mailer: git-send-email 2.11.0 MIME-Version: 1.0 X-Originating-IP: [209.116.155.178] X-ClientProxiedBy: BYAPR07CA0040.namprd07.prod.outlook.com (2603:10b6:a03:60::17) To DB6PR0501MB2038.eurprd05.prod.outlook.com (2603:10a6:4:6::20) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 86af0f74-5102-4f64-5832-08d5db593ac2 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(8989117)(4534165)(4627221)(201703031133081)(201702281549075)(8990107)(5600026)(711020)(48565401081)(2017052603328)(7153060)(7193020); SRVR:DB6PR0501MB2038; X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2038; 3:Db2KpJE9dj6jYZIobfu8nyNlP2G6HnEga48sxQQepkIBS6fKX23CIUe8XBRopff+DXpHgYzCR3dpaEqPGpElIwWwUgBKkOsDn+1gMLUahfdsmVUFkzffI/RpBh/6E2zMtZlHfmcm9xEnXy04BsON9HbjkN3QGa2Mu5sBNaCD2BHirEm2nfFeYdHn+2Aq2TJJrA+qLzhayUQbKs794OJKmWa6dtdBRHOT4+t2r9E5mR1Z7VfmZKVhARgaodNvOkYj; 25:pkENNklc10sP3/4OKQgKJ7niZkZWZuA+2u3sbHTSEOceZQyJdovnn0XsPVI1lYeWI3kAcaAcIObzL7dVl3Cop3mE8s23X/dBRMFQc0O1zUzeOhALOIsDiRp92i9iUD+t/dZOJmGlgdEn7rKBFzy0gmiisXf740NqkQA4IZ+kO9eKq9MIRJO3+M9eyM5hDxGvMO6fDTFfS/pvwlLBCig9EDbiW8jdzfhoC2OHn8/xdWgJwEQSD0ealpNSSxFDwRqdO2GaotjFu9H/zUun9C3lSKTQkQ6dfEiGhFfPZg3YlRxFwHqbTzRG58wfHUS1z2eDg+XkdpnFeHPPxtVbroKXCA==; 31:Tm1+Q7qv03CY0DU1z7Wz4xudLYvAmQyqzUEDVb1E0QPmop7Tbp6QV0vcMOuFerLeM4NEk0jRSVu7oPEbaRqeUrSgTAoFhTqYvK+MeT/6hhfI3z4yzlND0P9BHU9WrdjKW5a9Cb1fQMykyGvkEmkYdhhiMZYAiWE9CLlB47aa9mGOlk8A6DkjfOLgewPftvpba69PR9pdZdo1Tp3MzXEBYBjaJLMPzb+HtwoCWgLkLRk= X-MS-TrafficTypeDiagnostic: DB6PR0501MB2038: X-LD-Processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtAddr X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2038; 20:PVi9X7/+IlJ8hHsfN+WrytRagMLxD5cgzNun64Yap+ZxYuZ38setwl0/L+kQqvIUociGKsH1I9fKda1DYPpFjwAH7EbtG7EHRnNCX07o01qhvKyvCF1PqvKNnwjsTyNyzzEmWID/iR7ljG3tHWhdD8lVtF1zDe9eRVruBNrsTOxu/5NNI1iEx46CqHNDBbgj5TVzgahGd7fg/h1Ga1/cVZdINwPyjaR2zKj69n8I1ijtrn+I8h5p2o+wG3dCpPWLo0DYMoLV4dhew/Nzh+geJZrpjtsLz7gYqaoPnFhIQZQrbUlz8HRSJG/PMEpbvzL2Ckkq0wHieL28JDTZMh9zVRzig5uYlhIIsU5dJtLotTDJHXWxRFw83U8lTI9mux5hHPFkE4QiD+npnifQM1k6eFxqUP1J2zLzO1VAndKs38qQ9R9GtCq6qmDZLrmI9YAfgP26+gqb89/tlY57qlUYT65s+Ufmr1l71vTdIay/8uD6n2gmBNJcOAKscRywiu6p; 4:uDfBP2ieN3b6iPaW+fNoMBvVWDaZWQ2PBM4NuzKow3gGK2DFYbV0atJooowsRBkvKf7vwIOYNaUQiAkjLVXq5hXEwCcAEYBm7JmTmMAoTCrXRIMe+qeW3qpnjgJIkLkrRBiMKauH0GFwmx6zPsLcWpJkBA7FlycL6RHO+tEhj26cEeFYU/kTNAez6Gfg/fOX19WNYeOaUNxAa322ofTNrN15LhsQxfTN/mfgjCIsvRrkJOczp1t/tKtvBVqxfBpaNcSbj+XbQL77b/QEJROyOyj4MJmJwv7Uq6QhqAFUiuVKTv9PFA8iHcJa7CPwf72g X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(788757137089); X-MS-Exchange-SenderADCheck: 1 X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(8121501046)(5005006)(3002001)(3231254)(944501410)(52105095)(93006095)(93001095)(10201501046)(6055026)(149027)(150027)(6041310)(20161123558120)(20161123560045)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(6072148)(201708071742011)(7699016); SRVR:DB6PR0501MB2038; BCL:0; PCL:0; RULEID:; SRVR:DB6PR0501MB2038; X-Forefront-PRVS: 071518EF63 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(136003)(39860400002)(366004)(346002)(376002)(396003)(199004)(189003)(16526019)(48376002)(16586007)(26005)(386003)(86362001)(50466002)(97736004)(6666003)(186003)(50226002)(53936002)(486006)(36756003)(21086003)(316002)(105586002)(51416003)(55016002)(7696005)(106356001)(107886003)(52116002)(956004)(476003)(5660300001)(4326008)(2616005)(68736007)(25786009)(8936002)(6116002)(478600001)(3846002)(81156014)(81166006)(1076002)(7736002)(8676002)(305945005)(2906002)(69596002)(47776003)(66066001); DIR:OUT; SFP:1101; SCL:1; SRVR:DB6PR0501MB2038; H:mellanox.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: mellanox.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; DB6PR0501MB2038; 23:26CGkbNXhlzbAU27slxMYecssLjm650y6ZI2fjC?= sOw4MpNeO6XlwJAMIQ7SY9CFDAhpSRhNHFxvTseW9HISiyjLeMWuuwM1Gc9SqzMS+SC0KCoH2fAeJ+t8/x0IMdojxUQSKZjm884qUVBbr0k/YuTBeaIT/5/9shs09dgHLNsD6xYie7QMyQ+yOJbpdlpaFzyj1dUIf59/68abUzFRe5j/rjHMeahSun7Rh+aBhSzWsf7Yz4sItlAYKio6ULvno+m8zwhWt0eP0+iuPNU8Wogxz0jIj0/d6E49rC1OpHIxULS/ZghHevsYUoZpxZ+Mdb+u7zkV5WJbYzx75gbHJ/FQwhs8Raus/SYJW6G6mPuFmrDDQQ/+0VnWdyz2CrvfADEgkzKdm6eKyssaZ1IXTZ+KneE4EaKLZJhfCdukV4cOaAPy4Y3RvN46ryDG9N9Cyp/IyI6OyQ2N87xPwLgERdwwx0fKylrseARD7NhBwh+sQb34V/+0H0HFBpyU+SlFB2udRS5a+IsHc2UDG//8ZsylZq4Pss9EHMxKLNw6T+AssOKvPhQBaoEgA5/dtP/kejFAYKu67P9siJEIpQPnyXxbg4Q5C/P73QFShvpIvSCwx9kfsJU5Eb4PSmuBCUMcIaQe1Z+hqVf/wHLiLToW5YPUSz2KbLc1MXwQNknPTnwfsYcdfluVH8Ni54LKWEq4CMwLexpHgWohV8cobaw4oM3lob5lBKjvilu+917vQEcyjH4ffC0LaF9/rN4qsnZlVfU6c1omgRGeylD0g+jxj6tjT7OZTsW8TA5ImK+65tatdyv5yvDEqmGzy5pbt7ErE9JaqE9v7DQ/+yyy+rmsPQi2zHFHJVSd1mH8YbU1cNdbPnvazSMlvG3/mVOeLCjT+Vxg6smFpB1cIaKi7wuPeCLGzxOqKkmSYVFQ+qsx093GQ0/NS+J6iR17RMMNO6rtP0mJ2wuQAnd8FQ1TAI/EAEybB3uSIgMI6Bz8i3n4toZImbloEqeY/SWdU0JRdiSv1mZZBqOE6QFgoHkX6fFLh+d0rZGdySjbgfaeacaYA63/PH00/nCu1vhpt8vGRR+choJdzbZIk/ZVnETnKHN4CNehVc9l6/OLM0dFb189DZg/aSs2ANbeYLRIcJ2RiEMZgXvAcVQUn3bTWCO7I+0sSkg== X-Microsoft-Antispam-Message-Info: K73rdAL2KaozePgNCvQckJAaQMWCbwZW362gxVk+UtWYk9V1uw3d2ADt86GndlZJ4BwwtRNSXyCi8qFahz4XIh28zh+1JPwZnhgqIWf6hLlPr9vS3MkCqgRrXZz2fzUyDTAOtOm++/zKZXAbbK5E50rIji1mQYXxq3h0gan8kqfTHSU9m9+UuIch/GcNESPtxxvxUzZj/d8faScbZep8GKee0KB13VPr0glY3NnAnLxxl26IKJxbxcGOooannlSURhbJZ4IY4UrAl7POL1CFBQ6WzBsnm5OpNMCIS13ORO/tVzbIwTS2KRriS+OgXtFOl52BkEpQkvu9RGknEfBHZKiY/N3P+58ajyQr1rVZKRs= X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2038; 6:J2aVfA5sB6tIxlvDQLlQN+MhHzEsL/KuW+ZAZyfpywA80zDrZQW8UBCgJfA5fK47D/3WF5eP15w+sB8dy6SUtzsEwchUX+mZAu9vowYKQIoGp0jn1uDduencNhzxkGd58PsVnWwX5Ag4e4AIdswCCgVi2qwMtMXjycFG3oqZABM74Ty8huQGYay5m14g9Jakg0wPPfy0RO7wlr8dP1Jr+EdvX/nJyzUbuPk3XfnVXPNmYbrBSR9/w/DTBPAzLg2rDZxs9ySv0Xd2V9BEdaeY4nSjQqciqyn37p8ei5rb0rXnX1zQ3wqcy3WP1Cgi+rTJx39s2U2RHpgfdsSqoCbpBKRX5VliJHEu7dpXtrtpAUciSlYqvDjFPaIDWPC1s8G0mgbOrUKhUVF+dyDzRv1Pj6teI8pNvOzxCDk7tL9E+h+RCREZtAMZSkdE4O8dyBUvv+HbTrN2/LJbswCElA7jKA==; 5:2z0ZI0y7b5rEMKOEXCQXPv6thJ31zcONVKPq1IkiIYltKW6Q+4gYd7x0taOLxbECNyLopMe1MSIiU5gq8m4CO+X50p3eDdN6kc7laC5ZwbXTKP698Be8nYNSYLiWZAxcfvPNAoziSxHFDjTpD7bqo9WLcl8pVUl+oBEkCBLoedM=; 24:mcl1e7Hn9YQoZNa1NxIB/pQxku0abjmaFjfKgZWuNQtmUhDpXlFCRKWqlWsSdyhMyqq3O++w35mRN8IvrZ3Dkw4nkfSj9LhPkBBBbx2Vi7w= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DB6PR0501MB2038; 7:p1N0LmbJdes50QNr3XzerXNvBdjETiKzeHTw18pxWndzUyu7DjEDRwLU/fBvEMA6kSkfkLcNOD87z/nXJ/gxcA2x1ELyYHU7Qqqn9yOO4HV0KVgb1R43U6NF0ZLv9tcrqd3xWZWpSovqqryOFRbmzA/P3tNzHRY0x2BlA4Zs8NAc+EXZUB+feCJ3BKqH5iEhZchbJJDrUyj6puDlrpEWIBK1xoLWnc1pSOd1sb+PzS2FofLnRvU9WjPHfODijzia X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jun 2018 11:37:41.7181 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 86af0f74-5102-4f64-5832-08d5db593ac2 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0501MB2038 Subject: [dpdk-dev] [PATCH 1/4] net/mlx5: change return value of Rx completion poll X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" mlx5_rx_poll_len() returns Rx hash result extracted from either mini CQE or regular CQE. As mini CQE may not have the hash result if configured otherwise, it shouldn't assume the first DWORD of mini CQE is always hash result. mlx5_rx_poll_len() is changed to return pointer to the mini CQE if compressed. Signed-off-by: Yongseok Koh Acked-by: Shahaf Shuler --- drivers/net/mlx5/mlx5_rxtx.c | 33 +++++++++++++++++++++------------ 1 file changed, 21 insertions(+), 12 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c index 527859461..a4e15c519 100644 --- a/drivers/net/mlx5/mlx5_rxtx.c +++ b/drivers/net/mlx5/mlx5_rxtx.c @@ -38,7 +38,7 @@ rxq_cq_to_pkt_type(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe); static __rte_always_inline int mlx5_rx_poll_len(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe, - uint16_t cqe_cnt, uint32_t *rss_hash); + uint16_t cqe_cnt, volatile struct mlx5_mini_cqe8 **mcqe); static __rte_always_inline uint32_t rxq_cq_to_ol_flags(volatile struct mlx5_cqe *cqe); @@ -1722,8 +1722,9 @@ rxq_cq_to_pkt_type(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe) * Pointer to RX queue. * @param cqe * CQE to process. - * @param[out] rss_hash - * Packet RSS Hash result. + * @param[out] mcqe + * Store pointer to mini-CQE if compressed. Otherwise, the pointer is not + * written. * * @return * Packet size in bytes (0 if there is none), -1 in case of completion @@ -1731,7 +1732,7 @@ rxq_cq_to_pkt_type(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe) */ static inline int mlx5_rx_poll_len(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe, - uint16_t cqe_cnt, uint32_t *rss_hash) + uint16_t cqe_cnt, volatile struct mlx5_mini_cqe8 **mcqe) { struct rxq_zip *zip = &rxq->zip; uint16_t cqe_n = cqe_cnt + 1; @@ -1745,7 +1746,7 @@ mlx5_rx_poll_len(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe, (uintptr_t)(&(*rxq->cqes)[zip->ca & cqe_cnt].pkt_info); len = rte_be_to_cpu_32((*mc)[zip->ai & 7].byte_cnt); - *rss_hash = rte_be_to_cpu_32((*mc)[zip->ai & 7].rx_hash_result); + *mcqe = &(*mc)[zip->ai & 7]; if ((++zip->ai & 7) == 0) { /* Invalidate consumed CQEs */ idx = zip->ca; @@ -1810,7 +1811,7 @@ mlx5_rx_poll_len(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe, zip->cq_ci = rxq->cq_ci + zip->cqe_cnt; /* Get packet size to return. */ len = rte_be_to_cpu_32((*mc)[0].byte_cnt); - *rss_hash = rte_be_to_cpu_32((*mc)[0].rx_hash_result); + *mcqe = &(*mc)[0]; zip->ai = 1; /* Prefetch all the entries to be invalidated */ idx = zip->ca; @@ -1821,7 +1822,6 @@ mlx5_rx_poll_len(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cqe, } } else { len = rte_be_to_cpu_32(cqe->byte_cnt); - *rss_hash = rte_be_to_cpu_32(cqe->rx_hash_res); } /* Error while receiving packet. */ if (unlikely(MLX5_CQE_OPCODE(op_own) == MLX5_CQE_RESP_ERR)) @@ -1934,7 +1934,8 @@ mlx5_rx_burst(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) volatile struct mlx5_wqe_data_seg *wqe = &((volatile struct mlx5_wqe_data_seg *)rxq->wqes)[idx]; struct rte_mbuf *rep = (*rxq->elts)[idx]; - uint32_t rss_hash_res = 0; + volatile struct mlx5_mini_cqe8 *mcqe = NULL; + uint32_t rss_hash_res; if (pkt) NEXT(seg) = rep; @@ -1964,8 +1965,7 @@ mlx5_rx_burst(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) } if (!pkt) { cqe = &(*rxq->cqes)[rxq->cq_ci & cqe_cnt]; - len = mlx5_rx_poll_len(rxq, cqe, cqe_cnt, - &rss_hash_res); + len = mlx5_rx_poll_len(rxq, cqe, cqe_cnt, &mcqe); if (!len) { rte_mbuf_raw_free(rep); break; @@ -1979,6 +1979,10 @@ mlx5_rx_burst(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) pkt = seg; assert(len >= (rxq->crc_present << 2)); pkt->ol_flags = 0; + /* If compressed, take hash result from mini-CQE. */ + rss_hash_res = rte_be_to_cpu_32(mcqe == NULL ? + cqe->rx_hash_res : + mcqe->rx_hash_result); rxq_cq_to_mbuf(rxq, pkt, cqe, rss_hash_res); if (rxq->crc_present) len -= ETHER_CRC_LEN; @@ -2115,7 +2119,8 @@ mlx5_rx_burst_mprq(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) uint16_t consumed_strd; uint32_t offset; uint32_t byte_cnt; - uint32_t rss_hash_res = 0; + volatile struct mlx5_mini_cqe8 *mcqe = NULL; + uint32_t rss_hash_res; if (strd_idx == strd_n) { /* Replace WQE only if the buffer is still in use. */ @@ -2142,7 +2147,7 @@ mlx5_rx_burst_mprq(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) buf = (*rxq->mprq_bufs)[rq_ci & wq_mask]; } cqe = &(*rxq->cqes)[rxq->cq_ci & cq_mask]; - ret = mlx5_rx_poll_len(rxq, cqe, cq_mask, &rss_hash_res); + ret = mlx5_rx_poll_len(rxq, cqe, cq_mask, &mcqe); if (!ret) break; if (unlikely(ret == -1)) { @@ -2237,6 +2242,10 @@ mlx5_rx_burst_mprq(void *dpdk_rxq, struct rte_mbuf **pkts, uint16_t pkts_n) continue; } } + /* If compressed, take hash result from mini-CQE. */ + rss_hash_res = rte_be_to_cpu_32(mcqe == NULL ? + cqe->rx_hash_res : + mcqe->rx_hash_result); rxq_cq_to_mbuf(rxq, pkt, cqe, rss_hash_res); PKT_LEN(pkt) = len; DATA_LEN(pkt) = len;