From patchwork Sat Jul 27 14:57:17 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Harman Kalra X-Patchwork-Id: 57202 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 51B791C0B0; Sat, 27 Jul 2019 16:57:22 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by dpdk.org (Postfix) with ESMTP id 8033B1C06A for ; Sat, 27 Jul 2019 16:57:21 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id x6REuViT011241 for ; Sat, 27 Jul 2019 07:57:20 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : content-type : content-transfer-encoding : mime-version; s=pfpt0818; bh=vkgdez1xYm4xj679Ucu03iAN8Z6a2zRvtc4Aq66RoHM=; b=YmX7EgM/wx9q+WVJlcP6cX1Xu7Qf4a3qvnff/gyHy/Mj+tUVs+Z51pMi0J/67Bh/I97q KpsAzJcnMSKiuBTo2UCujA+mU5/Ws9UH10DIwQhBkKBw5MagquQ0XadZOxM359zKmX2x /908Rv6FXcZ+K69ElJjJM+sShXfVywZL8BrLu6YgRekSc8J6/5g687s69uTjHwZFl49q LdiL3kamGdtdXpQ//YP8EjTF3nZA3cd9PCnNPtJVxZvCJaM5R8qXge61VZmgjUWil9r6 mg32gMnU9wSYVumkCs+s3zs43dPEIDyJwPxSUBi9ysj2nL9ksFt6XUDRX0S/883vNek2 rA== Received: from sc-exch04.marvell.com ([199.233.58.184]) by mx0a-0016f401.pphosted.com with ESMTP id 2u0kyprwnn-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for ; Sat, 27 Jul 2019 07:57:20 -0700 Received: from SC-EXCH02.marvell.com (10.93.176.82) by SC-EXCH04.marvell.com (10.93.176.84) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Sat, 27 Jul 2019 07:57:19 -0700 Received: from NAM05-DM3-obe.outbound.protection.outlook.com (104.47.49.50) by SC-EXCH02.marvell.com (10.93.176.82) with Microsoft SMTP Server (TLS) id 15.0.1367.3 via Frontend Transport; Sat, 27 Jul 2019 07:57:19 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Vob/shGS2u+ehUSdzc2zBeW3YoQb2G1omcvQGvUZcql1duZ8VMKvSjV1Aq4IWMgnWv63dBJUN5kQBnhI0xPz1CfZKcc8+7cGJsMjJ+T2V0L+tPOx2BNagYdYLS6ChnxIVjAJ/bBRYLXgUQVz9tFGSz/D8OXq0G8oduc7Iy3nWon4tGGuAKNyiFVtHvE7MZuYTWRvfATuYZX05fAwytdq9Lvw4zpNTd84uLBylvHK+qRaN08vOfUyit4Hq0c2zxlE5ptkwcoAbWnJu+mTE0ugjxb43a+up1Y4uvFrMNEJIPzfuS4kUG88DwDGzWFLs7LHYqbByE9KFLwHjVl7S/OBEw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vkgdez1xYm4xj679Ucu03iAN8Z6a2zRvtc4Aq66RoHM=; b=WGIzcjWezZWtKAAsvNUDsRUBhGJs/1kd1CPDLiITUOOqpXbRBdKwrfNMUcZP34Sdv30vAU2eRnkvN9ve/UmN1KbqnyalwIfc5SkRNzKNoessLyNQesl9tcPT7yAb0ebD8sVnn0tiiShOAVX7kE6tMTKFEGA2IrgzakHQPIwqAgqluJOoc2kQIYBKKQ0vL+IbF5ItpHbibKL4WzP54MM2S7/TKsSqlq7kVbKfzDT2zNMc4Ir2qiDPDq1cSXt70I4F4PkIj6qJXERKVSJWWXo0fSniCLNw92goda6+k6cU2PfgDOnFC6pJ5jl9MqaOHXi7QgH+7PfAi/N62snbM0WYVw== ARC-Authentication-Results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=marvell.com;dmarc=pass action=none header.from=marvell.com;dkim=pass header.d=marvell.com;arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.onmicrosoft.com; s=selector2-marvell-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vkgdez1xYm4xj679Ucu03iAN8Z6a2zRvtc4Aq66RoHM=; b=tZc5LTFq8jasYesaPjLpBAmG1I8p8XTSkVrkeWMDd9UOeX2jtIJvwEnvdycNNG1ygxDXOAizCT438PqYMT4tNaL8b/eqzaQlSEMI30uQM75nktduhsmmuiUcur/VEmYJjzJ77hVKP9xHL+ZsQjLK74v/K9OOAbzUugkWwDHcWTI= Received: from MN2PR18MB2848.namprd18.prod.outlook.com (20.179.21.149) by MN2PR18MB2542.namprd18.prod.outlook.com (20.179.82.221) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2115.13; Sat, 27 Jul 2019 14:57:17 +0000 Received: from MN2PR18MB2848.namprd18.prod.outlook.com ([fe80::4447:9459:5386:2e18]) by MN2PR18MB2848.namprd18.prod.outlook.com ([fe80::4447:9459:5386:2e18%7]) with mapi id 15.20.2115.005; Sat, 27 Jul 2019 14:57:17 +0000 From: Harman Kalra To: Pavan Nikhilesh Bhagavatula , "Jerin Jacob Kollanukkaran" , Nithin Kumar Dabilpuram , Kiran Kumar Kokkilagadda CC: "dev@dpdk.org" , Harman Kalra Thread-Topic: [PATCH 1/2] net/octeontx2: fix ptp performance issue Thread-Index: AQHVRIuVmbhtgQ4+m0aaomJ3Zw/pjQ== Date: Sat, 27 Jul 2019 14:57:17 +0000 Message-ID: <1564239400-2919-1-git-send-email-hkalra@marvell.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: PN1PR01CA0118.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00::34) To MN2PR18MB2848.namprd18.prod.outlook.com (2603:10b6:208:3e::21) x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.7.4 x-originating-ip: [115.113.156.2] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: b8422fd9-789f-4977-3464-08d712a2b7b5 x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(7168020)(4627221)(201703031133081)(201702281549075)(8990200)(5600148)(711020)(4605104)(1401327)(2017052603328)(7193020); SRVR:MN2PR18MB2542; x-ms-traffictypediagnostic: MN2PR18MB2542: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:4303; x-forefront-prvs: 01110342A5 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(376002)(396003)(366004)(39850400004)(346002)(136003)(189003)(199004)(7736002)(305945005)(54906003)(68736007)(110136005)(50226002)(81166006)(8676002)(8936002)(81156014)(6636002)(5660300002)(25786009)(53936002)(478600001)(256004)(14444005)(66066001)(6512007)(71190400001)(71200400001)(2906002)(6116002)(3846002)(316002)(26005)(4326008)(186003)(6486002)(64756008)(66446008)(2616005)(66556008)(476003)(486006)(66476007)(6436002)(86362001)(99286004)(66946007)(107886003)(14454004)(36756003)(6506007)(386003)(55236004)(102836004)(52116002); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR18MB2542; H:MN2PR18MB2848.namprd18.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: marvell.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: QjKMriJD/nClZ1xwxZbxFkprI0J8dgSD6ObqicBJYVlIX60KwWjRoj/if19YESv3elG9iwRc4yNnt5kqPrh9dKEESj/FUg9RdW5vSZnzMXMqVCH2E513RJkth00hC308MPymHV6tRrpcSCSk/oGXZy/Zin1PxRlEWKQB7Kd2OFya+fl/q+RCSNl4IYZbXHI9UBLCYNjvRdMsbiTWqh3WpHxlPeE/ITPrwBZs1GArTh+J8KOPXintu82M0BWASRjl4scyza8a0f9kBK3Umsu0GRHc2K3eCVHGdFuFPwXWji7z/1MyKlUuL9Y0lG7Mf9iMoodrw8pz/ihrtgOqhdRj4xu9z/7ccIhbQSBU0ptZ21Q2w/ESWIjay9yzoras4iT7stFLU+0VYTWLmbMMy6VtK2f5GNp/s6lZ+QjxXmtAto4= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: b8422fd9-789f-4977-3464-08d712a2b7b5 X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Jul 2019 14:57:17.4800 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: hkalra@marvell.com X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR18MB2542 X-OriginatorOrg: marvell.com X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:5.22.84,1.0.8 definitions=2019-07-27_12:2019-07-26,2019-07-27 signatures=0 Subject: [dpdk-dev] [PATCH 1/2] net/octeontx2: fix ptp performance issue 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" A huge drop in per core MPPS value was observed when PTP stack is enabled. The reason behind the bottleneck is HW serialises the transfer of all SQEs, which seeks timestamp capture, on the same send DMA path. Hence only those packets which requires timestamp capture should set SETTSTAMP in send mem alg. With this patch timestamping would be done only for those packets with PKT_TX_IEEE1588_TMST set. Fixes: fb3ae0951abd ("net/octeontx2: support Tx") Fixes: 8980a153006b ("event/octeontx2: support PTP for SSO") Signed-off-by: Harman Kalra --- drivers/event/octeontx2/otx2_evdev.h | 7 ++++++- drivers/event/octeontx2/otx2_worker.h | 10 +++++++-- drivers/event/octeontx2/otx2_worker_dual.h | 14 +++++++++++-- drivers/net/octeontx2/otx2_ethdev.c | 2 -- drivers/net/octeontx2/otx2_rx.c | 3 ++- drivers/net/octeontx2/otx2_rx.h | 24 +++++++++++++--------- drivers/net/octeontx2/otx2_tx.h | 19 +++++++++++++---- 7 files changed, 57 insertions(+), 22 deletions(-) diff --git a/drivers/event/octeontx2/otx2_evdev.h b/drivers/event/octeontx2/otx2_evdev.h index 9c9718f6f..5cd80e3b2 100644 --- a/drivers/event/octeontx2/otx2_evdev.h +++ b/drivers/event/octeontx2/otx2_evdev.h @@ -25,6 +25,7 @@ #define OTX2_SSO_SQB_LIMIT (0x180) #define OTX2_SSO_XAQ_SLACK (8) #define OTX2_SSO_XAQ_CACHE_CNT (0x7) +#define OTX2_SSO_WQE_SG_PTR (9) /* SSO LF register offsets (BAR2) */ #define SSO_LF_GGRP_OP_ADD_WORK0 (0x0ull) @@ -222,10 +223,14 @@ otx2_wqe_to_mbuf(uint64_t get_work1, const uint64_t mbuf, uint8_t port_id, const void * const lookup_mem) { struct nix_wqe_hdr_s *wqe = (struct nix_wqe_hdr_s *)get_work1; + uint64_t val = mbuf_init.value | (uint64_t)port_id << 48; + + if (flags & NIX_RX_OFFLOAD_TSTAMP_F) + val |= NIX_TIMESYNC_RX_OFFSET; otx2_nix_cqe_to_mbuf((struct nix_cqe_hdr_s *)wqe, tag, (struct rte_mbuf *)mbuf, lookup_mem, - mbuf_init.value | (uint64_t)port_id << 48, flags); + val, flags); } diff --git a/drivers/event/octeontx2/otx2_worker.h b/drivers/event/octeontx2/otx2_worker.h index 3c847d223..76f91bb59 100644 --- a/drivers/event/octeontx2/otx2_worker.h +++ b/drivers/event/octeontx2/otx2_worker.h @@ -18,6 +18,7 @@ otx2_ssogws_get_work(struct otx2_ssogws *ws, struct rte_event *ev, const uint32_t flags, const void * const lookup_mem) { union otx2_sso_event event; + uint64_t tstamp_ptr; uint64_t get_work1; uint64_t mbuf; @@ -69,8 +70,10 @@ otx2_ssogws_get_work(struct otx2_ssogws *ws, struct rte_event *ev, otx2_wqe_to_mbuf(get_work1, mbuf, event.sub_event_type, (uint32_t) event.get_work0, flags, lookup_mem); /* Extracting tstamp, if PTP enabled*/ + tstamp_ptr = *(uint64_t *)(((struct nix_wqe_hdr_s *)get_work1) + + OTX2_SSO_WQE_SG_PTR); otx2_nix_mbuf_to_tstamp((struct rte_mbuf *)mbuf, ws->tstamp, - flags); + flags, (uint64_t *)tstamp_ptr); get_work1 = mbuf; } @@ -86,6 +89,7 @@ otx2_ssogws_get_work_empty(struct otx2_ssogws *ws, struct rte_event *ev, const uint32_t flags) { union otx2_sso_event event; + uint64_t tstamp_ptr; uint64_t get_work1; uint64_t mbuf; @@ -131,8 +135,10 @@ otx2_ssogws_get_work_empty(struct otx2_ssogws *ws, struct rte_event *ev, otx2_wqe_to_mbuf(get_work1, mbuf, event.sub_event_type, (uint32_t) event.get_work0, flags, NULL); /* Extracting tstamp, if PTP enabled*/ + tstamp_ptr = *(uint64_t *)(((struct nix_wqe_hdr_s *)get_work1) + + OTX2_SSO_WQE_SG_PTR); otx2_nix_mbuf_to_tstamp((struct rte_mbuf *)mbuf, ws->tstamp, - flags); + flags, (uint64_t *)tstamp_ptr); get_work1 = mbuf; } diff --git a/drivers/event/octeontx2/otx2_worker_dual.h b/drivers/event/octeontx2/otx2_worker_dual.h index 4a72f424d..5134e3d52 100644 --- a/drivers/event/octeontx2/otx2_worker_dual.h +++ b/drivers/event/octeontx2/otx2_worker_dual.h @@ -21,6 +21,7 @@ otx2_ssogws_dual_get_work(struct otx2_ssogws_state *ws, { const uint64_t set_gw = BIT_ULL(16) | 1; union otx2_sso_event event; + uint64_t tstamp_ptr; uint64_t get_work1; uint64_t mbuf; @@ -70,8 +71,17 @@ otx2_ssogws_dual_get_work(struct otx2_ssogws_state *ws, event.event_type == RTE_EVENT_TYPE_ETHDEV) { otx2_wqe_to_mbuf(get_work1, mbuf, event.sub_event_type, (uint32_t) event.get_work0, flags, lookup_mem); - /* Extracting tstamp, if PTP enabled*/ - otx2_nix_mbuf_to_tstamp((struct rte_mbuf *)mbuf, tstamp, flags); + /* Extracting tstamp, if PTP enabled. CGX will prepend the + * timestamp at starting of packet data and it can be derieved + * from WQE 9 dword which corresponds to SG iova. + * rte_pktmbuf_mtod_offset can be used for this purpose but it + * brings down the performance as it reads mbuf->buf_addr which + * is not part of cache in general fast path. + */ + tstamp_ptr = *(uint64_t *)(((struct nix_wqe_hdr_s *)get_work1) + + OTX2_SSO_WQE_SG_PTR); + otx2_nix_mbuf_to_tstamp((struct rte_mbuf *)mbuf, tstamp, flags, + (uint64_t *)tstamp_ptr); get_work1 = mbuf; } diff --git a/drivers/net/octeontx2/otx2_ethdev.c b/drivers/net/octeontx2/otx2_ethdev.c index b018b25b7..595c8003a 100644 --- a/drivers/net/octeontx2/otx2_ethdev.c +++ b/drivers/net/octeontx2/otx2_ethdev.c @@ -874,8 +874,6 @@ otx2_nix_form_default_desc(struct otx2_eth_txq *txq) send_mem = (struct nix_send_mem_s *)(txq->cmd + (send_hdr->w0.sizem1 << 1)); send_mem->subdc = NIX_SUBDC_MEM; - send_mem->dsz = 0x0; - send_mem->wmem = 0x1; send_mem->alg = NIX_SENDMEMALG_SETTSTMP; send_mem->addr = txq->dev->tstamp.tx_tstamp_iova; } diff --git a/drivers/net/octeontx2/otx2_rx.c b/drivers/net/octeontx2/otx2_rx.c index deefe9588..701efc858 100644 --- a/drivers/net/octeontx2/otx2_rx.c +++ b/drivers/net/octeontx2/otx2_rx.c @@ -68,7 +68,8 @@ nix_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, otx2_nix_cqe_to_mbuf(cq, cq->tag, mbuf, lookup_mem, mbuf_init, flags); - otx2_nix_mbuf_to_tstamp(mbuf, rxq->tstamp, flags); + otx2_nix_mbuf_to_tstamp(mbuf, rxq->tstamp, flags, + (uint64_t *)((uint8_t *)mbuf + data_off)); rx_pkts[packets++] = mbuf; otx2_prefetch_store_keep(mbuf); head++; diff --git a/drivers/net/octeontx2/otx2_rx.h b/drivers/net/octeontx2/otx2_rx.h index e150f38d7..d12e8b809 100644 --- a/drivers/net/octeontx2/otx2_rx.h +++ b/drivers/net/octeontx2/otx2_rx.h @@ -50,22 +50,26 @@ union mbuf_initializer { static __rte_always_inline void otx2_nix_mbuf_to_tstamp(struct rte_mbuf *mbuf, - struct otx2_timesync_info *tstamp, const uint16_t flag) + struct otx2_timesync_info *tstamp, const uint16_t flag, + uint64_t *tstamp_ptr) { if ((flag & NIX_RX_OFFLOAD_TSTAMP_F) && - mbuf->packet_type == RTE_PTYPE_L2_ETHER_TIMESYNC && (mbuf->data_off == RTE_PKTMBUF_HEADROOM + NIX_TIMESYNC_RX_OFFSET)) { - uint64_t *tstamp_ptr; - /* Deal with rx timestamp */ - tstamp_ptr = rte_pktmbuf_mtod_offset(mbuf, uint64_t *, - -NIX_TIMESYNC_RX_OFFSET); + /* Reading the rx timestamp inserted by CGX, viz at + * starting of the packet data. + */ mbuf->timestamp = rte_be_to_cpu_64(*tstamp_ptr); - tstamp->rx_tstamp = mbuf->timestamp; - tstamp->rx_ready = 1; - mbuf->ol_flags |= PKT_RX_IEEE1588_PTP | PKT_RX_IEEE1588_TMST - | PKT_RX_TIMESTAMP; + /* PKT_RX_IEEE1588_TMST flag needs to be set only in case + * PTP packets are received. + */ + if (mbuf->packet_type == RTE_PTYPE_L2_ETHER_TIMESYNC) { + tstamp->rx_tstamp = mbuf->timestamp; + tstamp->rx_ready = 1; + mbuf->ol_flags |= PKT_RX_IEEE1588_PTP | + PKT_RX_IEEE1588_TMST | PKT_RX_TIMESTAMP; + } } } diff --git a/drivers/net/octeontx2/otx2_tx.h b/drivers/net/octeontx2/otx2_tx.h index b75a220ea..494ba3884 100644 --- a/drivers/net/octeontx2/otx2_tx.h +++ b/drivers/net/octeontx2/otx2_tx.h @@ -43,18 +43,29 @@ otx2_nix_xmit_prepare_tstamp(uint64_t *cmd, const uint64_t *send_mem_desc, if (flags & NIX_TX_OFFLOAD_TSTAMP_F) { struct nix_send_mem_s *send_mem; uint16_t off = (no_segdw - 1) << 1; + const uint8_t is_ol_tstamp = !(ol_flags & PKT_TX_IEEE1588_TMST); send_mem = (struct nix_send_mem_s *)(cmd + off); - if (flags & NIX_TX_MULTI_SEG_F) + if (flags & NIX_TX_MULTI_SEG_F) { /* Retrieving the default desc values */ cmd[off] = send_mem_desc[6]; + /* Using compiler barier to avoid voilation of C + * aliasing rules. + */ + rte_compiler_barrier(); + } + /* Packets for which PKT_TX_IEEE1588_TMST is not set, tx tstamp - * should not be updated at tx tstamp registered address, rather - * a dummy address which is eight bytes ahead would be updated + * should not be recorded, hence changing the alg type to + * NIX_SENDMEMALG_SET and also changing send mem addr field to + * next 8 bytes as it corrpt the actual tx tstamp registered + * address. */ + send_mem->alg = NIX_SENDMEMALG_SETTSTMP - (is_ol_tstamp); + send_mem->addr = (rte_iova_t)((uint64_t *)send_mem_desc[7] + - !(ol_flags & PKT_TX_IEEE1588_TMST)); + (is_ol_tstamp)); } } From patchwork Sat Jul 27 14:57:26 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Harman Kalra X-Patchwork-Id: 57203 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 C65831C2CA; Sat, 27 Jul 2019 16:57:32 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by dpdk.org (Postfix) with ESMTP id DFEB31C2C4 for ; Sat, 27 Jul 2019 16:57:30 +0200 (CEST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id x6REtbj1030305 for ; Sat, 27 Jul 2019 07:57:30 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : content-transfer-encoding : mime-version; s=pfpt0818; bh=u0KhsJGq3SHyRFW0s8lMwyEU6Kb6+b/BpjoIFAAsczc=; b=e/HjsLNRvZ1Tzu5cv4J1ughj+42zRdc7sUSSRHpNiILnB9P55Y6YdsNwYtW64PMUR7nH Y8hHUiO+IYqD4eDqOHLgsX1yUO7n7/9cDawt3qlJNID2O7qhHM7b8JjoKADiRXN4ArFs gnLOWoegPEaLYQxmgv5FUlJs0aXlosOlvNwbsNaQgum4TudAzCodIqdfh7eU0L0AC/Bd xD/N6aQcj6qINGp3IArzgXeHXuRrjmxCc1P57chBJbZpfqOw6VZIFpy6aIUj1HBsXQJA wKeYTZZi0G9x6t3+Gws4o7JrJnjzdocCLHyUQkBLEio9rXB4S3TKSv458GA5qjQHngMv OA== Received: from sc-exch04.marvell.com ([199.233.58.184]) by mx0b-0016f401.pphosted.com with ESMTP id 2u0p4krekw-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for ; Sat, 27 Jul 2019 07:57:30 -0700 Received: from SC-EXCH03.marvell.com (10.93.176.83) by SC-EXCH04.marvell.com (10.93.176.84) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Sat, 27 Jul 2019 07:57:28 -0700 Received: from NAM05-DM3-obe.outbound.protection.outlook.com (104.47.49.58) by SC-EXCH03.marvell.com (10.93.176.83) with Microsoft SMTP Server (TLS) id 15.0.1367.3 via Frontend Transport; Sat, 27 Jul 2019 07:57:28 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XjdYrqUcBBb1Pvq6gZIPCfU6Ouof/zA1MDFMLpRk7u5lb+377kSScwklq4QkI/sahyPkrWhz1IXPJXlfa9vo0kk4Rh1OpGg/t9+G6FnkPXOfuat0cEDZwjiMZwQXnVUbeVhGR2kITfBZu7SUkCqybAvzGMCAOCqcHjS9W641Yu6OWV06TQ0Q1kLJLlaiMzTzZhB/T13vAo1PhbCdensY9xc7vPJJxrSQsn+fjmTekmt8dGHWw3Q0y4tFMS/C15FiIaACcyIhdZ7chRaT0xTHGZptso3kadVJJGiQOKlalTwVxAYuWW17in/qTTuIYGS6wZlBjhDFUrPmrnOFK21luw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=u0KhsJGq3SHyRFW0s8lMwyEU6Kb6+b/BpjoIFAAsczc=; b=mDJfs90xE/ssbDU715dUQHRQJrUCG4qSFTPptiDyz+SyUb1PPpXiircROnPoVUF2GclsFwtw/pm3IsF5SNbCnforW6FoHDIMnpx4gSnfWDEtEPeoCv745BiU+jCNGUjiErAtKdD8lK3oE9ypwPj8zoZVTcKdpZGagQi0FL554O0RS105o2q1EU5sG9NarXUUR6sbCn9lS71Lv7apH9xDXrNztPoLTJaxSzboRfBo87kJZkERlEylbef/9DVc52qakWtH3Q5jOjsym5jeV8nI0e7QFpcZOK5ojK0pDugstE4GeoPZTOvg6D9bfuyJwYT4j3KCTsAPw0TyAPMUPDD2ag== ARC-Authentication-Results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=marvell.com;dmarc=pass action=none header.from=marvell.com;dkim=pass header.d=marvell.com;arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.onmicrosoft.com; s=selector2-marvell-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=u0KhsJGq3SHyRFW0s8lMwyEU6Kb6+b/BpjoIFAAsczc=; b=RAVCM3V9pqG8tUk7pK8KHSlrVLJoeb8PYNjuOydpZ8aphyvwSNqpGcgbFr6Bw6N8+FYldVSfZEWdWiFO+3QqdiPHOXAN07D3R0m5ndwTYNT1smE1eUp0XTBI2rjOp7gc+9o2t+SyXyThbe1xDGra+KXQxHVwzsrnyLIaGLirAo8= Received: from MN2PR18MB2848.namprd18.prod.outlook.com (20.179.21.149) by MN2PR18MB2542.namprd18.prod.outlook.com (20.179.82.221) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2115.13; Sat, 27 Jul 2019 14:57:27 +0000 Received: from MN2PR18MB2848.namprd18.prod.outlook.com ([fe80::4447:9459:5386:2e18]) by MN2PR18MB2848.namprd18.prod.outlook.com ([fe80::4447:9459:5386:2e18%7]) with mapi id 15.20.2115.005; Sat, 27 Jul 2019 14:57:27 +0000 From: Harman Kalra To: Jerin Jacob Kollanukkaran , Nithin Kumar Dabilpuram , Vamsi Krishna Attunuru , Kiran Kumar Kokkilagadda CC: "dev@dpdk.org" , Harman Kalra Thread-Topic: [PATCH 2/2] net/octeontx2: support read clock API Thread-Index: AQHVRIuax/PbF55GWUK0y3SGB4NLVw== Date: Sat, 27 Jul 2019 14:57:26 +0000 Message-ID: <1564239400-2919-2-git-send-email-hkalra@marvell.com> References: <1564239400-2919-1-git-send-email-hkalra@marvell.com> In-Reply-To: <1564239400-2919-1-git-send-email-hkalra@marvell.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: PN1PR01CA0118.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00::34) To MN2PR18MB2848.namprd18.prod.outlook.com (2603:10b6:208:3e::21) x-ms-exchange-messagesentrepresentingtype: 1 x-mailer: git-send-email 2.7.4 x-originating-ip: [115.113.156.2] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 3f2d5db5-3fba-49fc-0fd5-08d712a2bd5b x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(7168020)(4627221)(201703031133081)(201702281549075)(8990200)(5600148)(711020)(4605104)(1401327)(2017052603328)(7193020); SRVR:MN2PR18MB2542; x-ms-traffictypediagnostic: MN2PR18MB2542: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8273; x-forefront-prvs: 01110342A5 x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(376002)(396003)(366004)(39850400004)(346002)(136003)(189003)(199004)(7736002)(305945005)(54906003)(68736007)(110136005)(50226002)(81166006)(8676002)(8936002)(81156014)(6636002)(5660300002)(25786009)(53936002)(478600001)(256004)(14444005)(66066001)(6512007)(71190400001)(71200400001)(2906002)(6116002)(3846002)(316002)(26005)(4326008)(186003)(6486002)(446003)(11346002)(64756008)(66446008)(2616005)(66556008)(476003)(486006)(66476007)(6436002)(86362001)(99286004)(66946007)(107886003)(76176011)(14454004)(36756003)(6506007)(386003)(55236004)(102836004)(52116002); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR18MB2542; H:MN2PR18MB2848.namprd18.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: marvell.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: ZXWuKBC3vChvGocDlLB5slqeMuPwWWuS1Ddf0kWAPUTCnPQ7m6c28a6CCSVFTFp2byacNqz2FHO0sF3th5Xi/b/54O9dp4MRHz9iDrWv8baPTIAukuJnUqCl0mVGz0MaC7EjB7tzbbjcD20HxB9kA/71deYnprw/Gahg6RJr9uKLXB6pOPZqXo+ZMeE/Gk+gqMXqMoEddw1n6rYXKih5F8g93ze01v9KLSx8F6IPzPgPcI453hb6Ixag6OKXSaatLFtbgaabXPKDOjIm0UOZGsxw7bZ0MKFb9LM8tdyFuWSuEBLOMIpo8bjjmT+rZh+IcFBZFeiq4Iafq8gfH5vjBdd0TkoSYj4aRnrIc1Qazfl2hirtBGR4txkFU8gv9fKmGgHmVC+jsdeMv4kcNYh6xDiR8bMN5FctrxJayJMIwko= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: 3f2d5db5-3fba-49fc-0fd5-08d712a2bd5b X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Jul 2019 14:57:26.9396 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: hkalra@marvell.com X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR18MB2542 X-OriginatorOrg: marvell.com X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:5.22.84,1.0.8 definitions=2019-07-27_12:2019-07-26,2019-07-27 signatures=0 Subject: [dpdk-dev] [PATCH 2/2] net/octeontx2: support read clock API 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" This patch implements read clock api whose purpose is to return raw clock ticks. Using this API real time ticks spent in processing a packet can be known: - mbuf->timestamp Signed-off-by: Harman Kalra --- drivers/common/octeontx2/otx2_mbox.h | 2 + drivers/net/octeontx2/otx2_ethdev.c | 86 ++++++++++++++++++++++++++++ drivers/net/octeontx2/otx2_ethdev.h | 4 ++ drivers/net/octeontx2/otx2_ptp.c | 30 ++++++++++ 4 files changed, 122 insertions(+) diff --git a/drivers/common/octeontx2/otx2_mbox.h b/drivers/common/octeontx2/otx2_mbox.h index c0bb676b2..b2c59c86e 100644 --- a/drivers/common/octeontx2/otx2_mbox.h +++ b/drivers/common/octeontx2/otx2_mbox.h @@ -1354,11 +1354,13 @@ struct ptp_req { struct mbox_msghdr hdr; uint8_t __otx2_io op; int64_t __otx2_io scaled_ppm; + uint8_t __otx2_io is_pmu; }; struct ptp_rsp { struct mbox_msghdr hdr; uint64_t __otx2_io clk; + uint64_t __otx2_io tsc; }; struct get_hw_cap_rsp { diff --git a/drivers/net/octeontx2/otx2_ethdev.c b/drivers/net/octeontx2/otx2_ethdev.c index 595c8003a..799e67480 100644 --- a/drivers/net/octeontx2/otx2_ethdev.c +++ b/drivers/net/octeontx2/otx2_ethdev.c @@ -521,6 +521,17 @@ otx2_nix_rx_queue_setup(struct rte_eth_dev *eth_dev, uint16_t rq, eth_dev->data->rx_queues[rq] = rxq; eth_dev->data->rx_queue_state[rq] = RTE_ETH_QUEUE_STATE_STOPPED; + + /* Calculating delta and freq mult between PTP HI clock and rdtsc. + * These are needed for deriving PTP HI clock value from tsc counter. + */ + if ((dev->rx_offloads & DEV_RX_OFFLOAD_TIMESTAMP) || + otx2_ethdev_is_ptp_en(dev)) { + rc = otx2_nix_raw_clock_rdtsc_conv(dev); + if (rc) + otx2_err("Failed to calculate delta and freq mult"); + } + return 0; free_rxq: @@ -1186,6 +1197,79 @@ nix_set_nop_rxtx_function(struct rte_eth_dev *eth_dev) rte_mb(); } +static int +nix_read_raw_clock(struct otx2_eth_dev *dev, uint64_t *clock, uint64_t *tsc, + uint8_t is_pmu) +{ + struct otx2_mbox *mbox = dev->mbox; + struct ptp_req *req; + struct ptp_rsp *rsp; + int rc = 0; + + req = otx2_mbox_alloc_msg_ptp_op(mbox); + req->op = PTP_OP_GET_CLOCK; + req->is_pmu = is_pmu; + rc = otx2_mbox_process_msg(mbox, (void *)&rsp); + if (rc) + goto done; + + *clock = rsp->clk; + *tsc = rsp->tsc; + +done: + return rc; +} + +/* This function calculates two parameters "clk_freq_mult" and + * "clk_delta" which is useful in deriving PTP HI clock from + * rdtsc value. + */ +int +otx2_nix_raw_clock_rdtsc_conv(struct otx2_eth_dev *dev) +{ + uint64_t ticks_base = 0, ticks = 0, t_freq = 0, tsc = 0; + uint8_t retval = 0, val; + + /* Calculating the frequency at which PTP HI clock is running */ + retval = nix_read_raw_clock(dev, &ticks_base, &tsc, false); + if (retval != 0) { + otx2_err("Failed to read the raw clock value: %d", retval); + goto done; + } + + rte_delay_ms(100); + + retval = nix_read_raw_clock(dev, &ticks, &tsc, false); + if (retval != 0) { + otx2_err("Failed to read the raw clock value: %d", retval); + goto done; + } + + t_freq = (ticks - ticks_base) * 10; + + /* Calculating the freq multiplier viz the ratio between the + * frequency at which PTP HI clock works and rdtsc clock runs + */ + dev->clk_freq_mult = + (double)pow(10, floor(log10(t_freq))) / rte_get_timer_hz(); + + val = false; +#ifdef RTE_ARM_EAL_RDTSC_USE_PMU + val = true; +#endif + retval = nix_read_raw_clock(dev, &ticks, &tsc, val); + if (retval != 0) { + otx2_err("Failed to read the raw clock value: %d", retval); + goto done; + } + + /* Calculating delta between PTP HI clock and rdtsc */ + dev->clk_delta = ((uint64_t)(ticks / dev->clk_freq_mult) - tsc); + +done: + return retval; +} + static int otx2_nix_configure(struct rte_eth_dev *eth_dev) { @@ -1363,6 +1447,7 @@ otx2_nix_configure(struct rte_eth_dev *eth_dev) dev->configured = 1; dev->configured_nb_rx_qs = data->nb_rx_queues; dev->configured_nb_tx_qs = data->nb_tx_queues; + return 0; cq_fini: @@ -1649,6 +1734,7 @@ static const struct eth_dev_ops otx2_eth_dev_ops = { .vlan_pvid_set = otx2_nix_vlan_pvid_set, .rx_queue_intr_enable = otx2_nix_rx_queue_intr_enable, .rx_queue_intr_disable = otx2_nix_rx_queue_intr_disable, + .read_clock = otx2_nix_read_clock, }; static inline int diff --git a/drivers/net/octeontx2/otx2_ethdev.h b/drivers/net/octeontx2/otx2_ethdev.h index 863d4877f..a2bd0ffcf 100644 --- a/drivers/net/octeontx2/otx2_ethdev.h +++ b/drivers/net/octeontx2/otx2_ethdev.h @@ -300,6 +300,8 @@ struct otx2_eth_dev { struct rte_timecounter systime_tc; struct rte_timecounter rx_tstamp_tc; struct rte_timecounter tx_tstamp_tc; + double clk_freq_mult; + uint64_t clk_delta; } __rte_cache_aligned; struct otx2_eth_txq { @@ -527,5 +529,7 @@ int otx2_nix_timesync_write_time(struct rte_eth_dev *eth_dev, int otx2_nix_timesync_read_time(struct rte_eth_dev *eth_dev, struct timespec *ts); int otx2_eth_dev_ptp_info_update(struct otx2_dev *dev, bool ptp_en); +int otx2_nix_read_clock(struct rte_eth_dev *eth_dev, uint64_t *time); +int otx2_nix_raw_clock_rdtsc_conv(struct otx2_eth_dev *dev); #endif /* __OTX2_ETHDEV_H__ */ diff --git a/drivers/net/octeontx2/otx2_ptp.c b/drivers/net/octeontx2/otx2_ptp.c index 0186c629a..3f54cfeaf 100644 --- a/drivers/net/octeontx2/otx2_ptp.c +++ b/drivers/net/octeontx2/otx2_ptp.c @@ -224,6 +224,13 @@ otx2_nix_timesync_adjust_time(struct rte_eth_dev *eth_dev, int64_t delta) rc = otx2_mbox_process_msg(mbox, (void *)&rsp); if (rc) return rc; + /* Since the frequency of PTP comp register is tuned, delta and + * freq mult calculation for deriving PTP_HI from rdtsc should + * be done again. + */ + rc = otx2_nix_raw_clock_rdtsc_conv(dev); + if (rc) + otx2_err("Failed to calculate delta and freq mult"); } dev->systime_tc.nsec += delta; dev->rx_tstamp_tc.nsec += delta; @@ -271,3 +278,26 @@ otx2_nix_timesync_read_time(struct rte_eth_dev *eth_dev, struct timespec *ts) return 0; } + + +int +otx2_nix_read_clock(struct rte_eth_dev *eth_dev, uint64_t *clock) +{ + struct otx2_eth_dev *dev = otx2_eth_pmd_priv(eth_dev); + + if (!otx2_ethdev_is_ptp_en(dev)) { + otx2_err("PTP should be enabled."); + return -EINVAL; + } + + /* This API returns the raw PTP HI clock value. Since LFs doesn't + * have direct access to PTP registers and it requires mbox msg + * to AF for this value. In fastpath reading this value for every + * packet (which involes mbox call) becomes very expensive, hence + * we should be able to derive PTP HI clock value from rdtsc by + * using freq_mult and clk_delta calculated during configure stage. + */ + *clock = (rte_rdtsc() + dev->clk_delta) * dev->clk_freq_mult; + + return 0; +}