From patchwork Mon Feb 6 05:29:37 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jerin Jacob X-Patchwork-Id: 20182 X-Patchwork-Delegate: bruce.richardson@intel.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id E3BDF14EC; Mon, 6 Feb 2017 06:34:03 +0100 (CET) Received: from NAM01-BY2-obe.outbound.protection.outlook.com (mail-by2nam01on0046.outbound.protection.outlook.com [104.47.34.46]) by dpdk.org (Postfix) with ESMTP id 531BC201 for ; Mon, 6 Feb 2017 06:34:01 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=of+icyi6HoissZmiIxVrtkZXv96p57vRjYCWJq+4GAw=; b=Z3Fr8RWVOHcubHh4Mwudj2JOWmR6elKATYWx0FYKxzS7VqP8YRYm6i39fcgw+hYZ2hZHH+IFc9wJRpcq4ANu5hBANNP1nbHlzWqi/Xu4PSQWd2ZZg4h/CUzUczHbJF4+l/BzDZhD40VAPA70ZfY14Te4Cd9LRKt1ZhIF1CygEYo= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jerin.Jacob@cavium.com; Received: from localhost.localdomain.localdomain (122.171.53.66) by BLUPR0701MB1714.namprd07.prod.outlook.com (10.163.85.140) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.888.16; Mon, 6 Feb 2017 05:33:56 +0000 From: Jerin Jacob To: CC: , , , , , Jerin Jacob Date: Mon, 6 Feb 2017 10:59:37 +0530 Message-ID: <1486358977-16542-1-git-send-email-jerin.jacob@caviumnetworks.com> X-Mailer: git-send-email 2.5.5 MIME-Version: 1.0 X-Originating-IP: [122.171.53.66] X-ClientProxiedBy: MA1PR01CA0006.INDPRD01.PROD.OUTLOOK.COM (10.164.117.13) To BLUPR0701MB1714.namprd07.prod.outlook.com (10.163.85.140) X-MS-Office365-Filtering-Correlation-Id: d8b7b42f-56a2-4bbc-6a1c-08d44e51c03e X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001); SRVR:BLUPR0701MB1714; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1714; 3:tB+8FKLrZzE4Agu02e5G5ERhbAU/iXRMHN58SKPuh78YdwRw2Bc4vWhJCMCSzpx3eDnOVS6FXvJrLFYAmBjLjAeKwnAUFPbVD3kaKSebygExulX0jdMVU5OhqfbOlrRUTm74ofJ+Lka1FXXKylIeysqHrtzW48whjKxtWtq6mLUmeHfeeuNYvERmMgZK33rZgzmwwR9hwBwWJURC2JEF+DWEYxns0MtgZYfpQ794E+mzAuzM1jeNJvXtngO7K82mp//crP2lnxf7YPspg/LnpA==; 25:xWUjpcK3mLmOueNOuag0bEbV+E7iAZP9W2PtNCGAzHHuIiU6Tp1M+bP4K+WRySpR9TE8XyOdmX18x8MKIhFfx9PuUR5RtKZ9xYdmMEzVJz+TNpzwRVmuQj+C0T60A/tXxz+uGfyxoQSB/4P5itsQmNa9Pwvzx+t7no9LMvoEyVN5+a6kxREPDEczfWRtNiJaSo//XTVj1IyGldkCA0i/zGrhYqbyM1esJXRkaBtZU/KPsodxXWL4ZfFbE+QfzapwOfuDZnjUphFnuDsF7Z59rtU2Ld7KWzDy2jKCFxodrWiMuI5eaYcA8Zv39X5t9X2EQ70Xgk2pzgNR17Oq5RFX9ygervsoIPhYm1zjNvC4+RipFkAql68FSn4M/GRmyIBchl8aCEUXKMRRv0fjn6KdY9MEII7ll8A3yt6pScAleXcdjrBc+0PbfruPQaeoYS6aV1wLZxxbX0+PAXR1cS5jNQ== X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1714; 31:Z/biM/DgSCNAJ+wQfKHgi8PUAwm3GLxqEAT1c9wORnn/8PXIrh/ifmS+GzIeqZrbM6urEl9vGymtrf3yElFE+ameeGkR/QW2S8Vk3g9Tky5Np4q20WkGWHGjKr5RooQt4dF9zKPw670IpucNgx1GuI2hb1uTOBeI/pRMAtsqwlCmenabRLIpA14vBmHSPDdyHiB2FxmcAJ4tJR8a4SPvPi3H4vFuKf5DyM9oFtOa77pC+35QLeci2gBMOflptHOY; 20:71HD2DK4sxl/n7eSpA38diz7PXmlkB8vZoJQNlA9fTpFU0VGHUvVMzkrsQaSWj7wZvsSXrxtNx4KMl0eDJ1PssoyNwWqKivU/nyZn0GiwleQuDS8Md+dI+hh/Sbe9pKLwIrTYxW8BlunNyjDWS6ZmQeTdMlL9kNwLjd+rJ/8JrQ/XP0m1M1j0hEtUr0gV7Pc208/a22k/PP1SOoLa9NelB35nKJrVf2xMf/Z5P6saVOOvyPrO9PC+NkM/phn+NdhTUkx1Apu21C4GAgyhKecLFrxMKsRLteR5BBKsUK2d1zmtRZVWiDB/NWe6F/LWdkOsjExjcjVUISGmjweB16PTBUxjuTfql2JTqxnTKyzzVPf60Y8q/vxcuA8d3BUqntsmNCStv2HBAdk9UwwDrdCnNEHsUW/VnEcI1eCvAxHSon0LeVXZwpMwuyBJBVT4//cqcu6mGeczBb9ZJvr6CqxG++dI39s6Zk3lLtTk9R/6kgzrrfUV0ndOjRMEeXKY49SP1jAwD7ALE0m0aKip2SAWEEgDz3vNDNiOp1rfTQXVRpzzzDgrGg4teyqxSg/IqPA5/dVY3o4V8QT+6Ml5m8XA3UWMrhzyn77eg99HFqLEeg= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(185117386973197); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(20170203043)(10201501046)(3002001)(6041248)(20161123564025)(20161123560025)(20161123562025)(20161123558025)(20161123555025)(6072148); SRVR:BLUPR0701MB1714; BCL:0; PCL:0; RULEID:; SRVR:BLUPR0701MB1714; X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1714; 4:H5YH6xQgs/NJf3kT+aNPg+Di/exyu0xzKW6Z6p/TycukRqQwzMO1XKuw7ixRITqUla7S14KkXTdAHyQ9apsGcctRDdwypoTWPJvA8ufHbnY8UU761hLkqM/TP/cBEAuMR8CaPSAITgaDaE3Vhge7+LljRGF1UlGoeNwmhb66Fxc8ITUGGIfa0no/FFJvT2gpkog/Y1IbrvXYAez492LnixjW5Fx6RCClWa/hHOT+24WSQEWPps5je0lW5ONVwhcnipoT+3qeDUn2p9yDzKFNZZT6UjSovquY9QCPUuWMvXSn22D2ymP8VUZp8N5XTXMFaoIwZ2Xvzm0ql4Tyc50dI5ilapMkzsHIWadEJKx2FtVQbkPExoUzPEkJhjCUR+Wweo2ktehqpEyZ0IuZkZzTwdZ2OUPfvJP5/YTtELuylnyMkJIBAQL4YL3t/oRnOL93e8ml9f6qoDqfC4W1gHox6trapmorFA6xY6224TiWMpU9ltB7/65FHbsMiszECFafudpzicy6MYFgoSpxaP+T8dNQL16zo8WJeX5UO70QxHIAX96u5jqr3k6eQU94vprPzInk8xLj7JfQTL3Uh2mJbBUKZsrmLxZzHhM0msJ9mYsctv6/b5qfhKnYh+TuorxMQ0VW9ok9SCEtAy8aRehdnv3gyoymdi+MubkZnGmiWqHRlcw172X6O3so3f+mPDvO X-Forefront-PRVS: 0210479ED8 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(4630300001)(6069001)(6009001)(7916002)(39450400003)(189002)(199003)(42186005)(97736004)(50986999)(5003940100001)(6666003)(53936002)(47776003)(38730400001)(107886002)(6916009)(42882006)(2351001)(4326007)(36756003)(6486002)(66066001)(110136003)(101416001)(6506006)(3846002)(6116002)(81156014)(2906002)(305945005)(6512007)(50226002)(189998001)(25786008)(54906002)(81166006)(7736002)(8676002)(4001430100002)(92566002)(48376002)(106356001)(105586002)(68736007)(50466002)(33646002)(8656002)(5660300001); DIR:OUT; SFP:1101; SCL:1; SRVR:BLUPR0701MB1714; H:localhost.localdomain.localdomain; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BLUPR0701MB1714; 23:nL5z/1p0rhuIBFl7S9hm9/xtxdexiTdevDrrM5z?= ORrs2KIMPbnDUkp0rpb8g7gufIJtCyG028x3Nbt8dZBRMQ+nWNoAzChry8tDI02CDH1ombRBV8JtpIscw8azTg2UIrH9P5uVOatjXEOP/kC/LvE2BVen9PrjbhWTNOobHMghWB6+1SuKWSE6VXYvdBADdd6vTzGk9luMF+lUZrn3QTpcBfNOmO77d5gVuh6jiqF1g5ZmJzMT9lEXn/Ot9oXaJY4q61/4RBOjsGhQv86DVzyOZN91fq9hb3i0k1EntqnxxWLV+UflaYG7aJhArP4KzBQijZW4JGE5LWvON3UUAclyXXCGA8UcuNsHULjooqbL65M+Z8lsHNOObMSz5xIneoViHFwbpbOwuITgYsykZ/22tSGg3CWLY0dh/vQrBEigimv3ih/u2ZFUdyhrrecTY0RT59Lnclzu7Xbus9mbF/Wt4oeWTVYEtBpVV5M9JXqDuGvtMMKMsWTWzWTHKl15ezDrHWNSjdBpihoqr4wi/eMrkRC/4YooUDOeKGkGfiGk4IOzkht906WbznyPnjeq64/O+0Vdr6UXGw4rALVRyNVahLM6HyIwZoElQnUplk2xaiNBmBTWtjh7Odes3YJaTf0ENprwLJqEZgYy0kVL0Ylem7JIn6NBAaLZ1ig95XT2EpOvuu8sZvDW80hJTpxMY3spRCQ8nsf/CwNpLbcQCb9h8ng6yCfWVqH9ZSiwBt3uNOBLc2PvKAR88Wh0C4HRpSieOSzOLA9EaN3P4YRJsqc8ybmx3jcur3QpsEAVhC1rCOe/3poGkQO6qQgDE+9xW6Qjs0nrc3Ryzrc7etyzjrjoMrL45b6vIHrCtWaX2rmt1tPC5PEBemyCeqmNC9AX1tKSNRCCaZlUgUEk+ung8GyOn+KRC9ULbv/H6CC41k/c46pWMaM89IUdh5xVGhYmCtpasoKo77H5CzYrDNlVxQ8FFeEZLTUi1QP3zoW0CLyJsXB5KABwFhZGTjxja/r6xXjiSubFkHA82aSncs3A4dKn9Z4jpgWKMsz3+kmAJQazLW/FtsaB/UFSf4xNnNu3LFAQGYMfCfuCo0kuoGmibU/Ltn5FkFqSvwdv+MWk3zPjmfgmO2CICvMiaztCZqYc/kjdOZuBw2iXnIxE6/SqLqhRQNUdDfXu6n5W8UXKRHFI= X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1714; 6:wn1+3TwUaoL5Iz6VW5r96ukNew2QZ8K8cqQTfUI/1TJjh1nhFf5RtVynR036asdne9QSGntpPTeV90jsdckyralAoxJh/7eUdq1LIqcD2eMTymqKT7DfqVzNJUy4ARpX8o2QWPFVsJ8qp2O9cCdDXUAIn3awcItyzmRUKkuHiAShQgEdrmQKahi6eZNZxt1mZDuR69aQoJzVaJ2b2zTwxIsY9PrPCmdr33DZmOHbPHiNTOsXw6qHcjF1sjSjAl9lk+EhYkg38RQyc3r5x04hYv7FpXRmqanUoPfPUGZzIqBlzGsk9RxzVDNVRmej8lMjQ9lda33sBtP6sG0r9kjeE47iPZOeSPyKNC0Vy5c6n56u8HYyxuwUbcaD3n7/PLJViMaCDBaiTzPT2KNkvvpsOw==; 5:d9UJ2tpCkN12UzPB2sWiAja7ACbRXq41LJEoHp18TVH5kJPglRohIWmMDhZU135iXiviQoZApwpmryRpzeQsay5TpZBB6MHk2eM3N38gdRcq4UCEubVTO91Tf9vB5SUQ4J1ThmvlQo9vYLPywOs7Zg==; 24:3Xzf+zxvHG6yu+esDkxyF0xOneKUVXlc66Scr1QxNfVHkZrz3JMUnvZ3qQuL7I+B8FNr9W1vx5FXHcLgsenPT+75Vw/gVQ6CDAANXaZ4JiA= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BLUPR0701MB1714; 7:p33FyrzL5EbnpVBUIPDI+GO3fS0WzEThIBTxghecLisOYyq88vZw/EJgkTZXFC5UjWPFb3qQC3NDoRiHOol/dxNavroAcjSlbJVm79i8n9jL2XfIlvpyz7+9L6LtfnzK1fYYNjOvq0yFGf3EyfwDfyQ1aepE251FBo4vNdrzd2ys7gCZymwnzN0k3gg2pJAYLT9Ltel69hg4Qn197vhqrZ1bCsDVQW6Y/fPKzFhSVpHWSh96gU7IBPSY8m6H7XSiDAbZVJymhQHLGwJ1f7214Nl408URW1nCu85h8NMNvhXm528jQqNE6/mO5VpAPoFiE/4nIak1KNsFpTYOESsKWZpF0o2U8wEZSr9v7pFOIfJLDVuTZI+YQJBy0TNMk8LAJWObbUd+9Efzc9Ql6Fj+qZRbFAjVnzrDltALaUadHvaZSTj+yLG2zJjP7xoEMEDocpkoTn7oiPnoWo6iZQfN3+LrkhMqyt/uAA4vRlo+ccocynGwkDLk1UQddy78iCzSbjFaZSt0Sc5eUmyxlfw7UQ== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Feb 2017 05:33:56.1079 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR0701MB1714 Subject: [dpdk-dev] [PATCH] eventdev: limit port link operation to configured queues 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" On port_setup, the link_map is updated only for configured number of event queues. Limit the port_links_get scan only to configured number of event queues. Also, Limit the port link and unlink queue validation to configured number of event queues. Fixes: 8ca610e0487c ("eventdev: implement the northbound APIs") Reported-by: Nipun Gupta Signed-off-by: Jerin Jacob Acked-by: Nipun Gupta --- lib/librte_eventdev/rte_eventdev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/librte_eventdev/rte_eventdev.c b/lib/librte_eventdev/rte_eventdev.c index 2c5277a..09fc274 100644 --- a/lib/librte_eventdev/rte_eventdev.c +++ b/lib/librte_eventdev/rte_eventdev.c @@ -798,7 +798,7 @@ rte_event_port_link(uint8_t dev_id, uint8_t port_id, } for (i = 0; i < nb_links; i++) - if (queues[i] >= RTE_EVENT_MAX_QUEUES_PER_DEV) + if (queues[i] >= dev->data->nb_queues) return -EINVAL; diag = (*dev->dev_ops->port_link)(dev->data->ports[port_id], queues, @@ -843,7 +843,7 @@ rte_event_port_unlink(uint8_t dev_id, uint8_t port_id, } for (i = 0; i < nb_unlinks; i++) - if (queues[i] >= RTE_EVENT_MAX_QUEUES_PER_DEV) + if (queues[i] >= dev->data->nb_queues) return -EINVAL; diag = (*dev->dev_ops->port_unlink)(dev->data->ports[port_id], queues, @@ -879,7 +879,7 @@ rte_event_port_links_get(uint8_t dev_id, uint8_t port_id, links_map = dev->data->links_map; /* Point links_map to this port specific area */ links_map += (port_id * RTE_EVENT_MAX_QUEUES_PER_DEV); - for (i = 0; i < RTE_EVENT_MAX_QUEUES_PER_DEV; i++) { + for (i = 0; i < dev->data->nb_queues; i++) { if (links_map[i] != EVENT_QUEUE_SERVICE_PRIORITY_INVALID) { queues[count] = i; priorities[count] = (uint8_t)links_map[i];