Message ID | 1493811091-26226-1-git-send-email-hemant.agrawal@nxp.com (mailing list archive) |
---|---|
State | Rejected, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> 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 874D268A9; Wed, 3 May 2017 13:31:50 +0200 (CEST) Received: from NAM01-BN3-obe.outbound.protection.outlook.com (mail-bn3nam01on0060.outbound.protection.outlook.com [104.47.33.60]) by dpdk.org (Postfix) with ESMTP id 8CB1368A1 for <dev@dpdk.org>; Wed, 3 May 2017 13:31:48 +0200 (CEST) Received: from BN6PR03CA0081.namprd03.prod.outlook.com (10.164.122.147) by BN3PR0301MB0900.namprd03.prod.outlook.com (10.160.156.142) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1061.12; Wed, 3 May 2017 11:31:47 +0000 Received: from BY2FFO11FD020.protection.gbl (2a01:111:f400:7c0c::156) by BN6PR03CA0081.outlook.office365.com (2603:10b6:405:6f::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1061.12 via Frontend Transport; Wed, 3 May 2017 11:31:47 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; dpdk.org; dkim=none (message not signed) header.d=none;dpdk.org; dmarc=fail action=none header.from=nxp.com; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BY2FFO11FD020.mail.protection.outlook.com (10.1.14.137) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.1047.9 via Frontend Transport; Wed, 3 May 2017 11:31:46 +0000 Received: from bf-netperf1.ap.freescale.net (bf-netperf1.ap.freescale.net [10.232.134.28]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id v43BVhQ9024594; Wed, 3 May 2017 04:31:44 -0700 From: Hemant Agrawal <hemant.agrawal@nxp.com> To: <ferruh.yigit@intel.com> CC: <dev@dpdk.org> Date: Wed, 3 May 2017 17:01:31 +0530 Message-ID: <1493811091-26226-1-git-send-email-hemant.agrawal@nxp.com> X-Mailer: git-send-email 1.9.1 X-EOPAttributedMessage: 0 X-Matching-Connectors: 131382847066579559; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.168.50; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(979002)(6009001)(336005)(39450400003)(39850400002)(39840400002)(39860400002)(39410400002)(39380400002)(39400400002)(2980300002)(1109001)(1110001)(339900001)(189002)(199003)(9170700003)(104016004)(356003)(53936002)(4326008)(305945005)(5003940100001)(2906002)(106466001)(33646002)(2351001)(105606002)(50986999)(15650500001)(498600001)(5660300001)(8936002)(6916009)(50226002)(81166006)(86362001)(189998001)(8676002)(38730400002)(36756003)(47776003)(110136004)(50466002)(77096006)(85426001)(48376002)(6666003)(969003)(989001)(999001)(1009001)(1019001); DIR:OUT; SFP:1101; SCL:1; SRVR:BN3PR0301MB0900; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; MLV:ovrnspm; MX:1; A:1; PTR:InfoDomainNonexistent; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BY2FFO11FD020; 1:WoybALL7EO3dBxub4lgmlutcmUE06y57/SwgEYtk31DqjXuxNyHPfxKC/ADDrfnK459/yHgl7xMtq4rBqlUsUcjI5I9Djf4BMPL6CoyopvkVEQg57B4bzJglupBnpLD6tXLdQrUALUJakCredpt9Vm2ygm4bW/vDbV7VJsZ5gzjrkNFr4f6xVbHjxrcsyL9Y4l/7yNSWmd5lf3DspQIrKu96hTsxC22t6Y5Vgg6efo3O2k0zM4ERr+2wUBvsgC/C30bnkt7iBSediq9Ooswdgpt5MUunzEQrDKeRVyqI9orIz/0UwmOcP6o1DrqsvEnhMXnpp5lhSKPqK41tUPYqiqlOXT7zyq4CK8JgeO0qKC60BmmOQ0fAmrVzm1LCbi11jiPG2GGF0EglXenJXOW5lx3STlIFruPaYPeYzL/4k7rhvm4iMQHtAGp4mKsawDfuZ3KZ4NwILYYIKbfTcxkQrz0jpNp4vh5+7ICPQuWofabozF9LPsoib4BaUwcSCcLRzXxD/fjRRMZgwUE+p30A739pAn3q6M9LOW6qxHx/RkVWUFXOtIA+fA5WJ5khSoLuwlb/kaGp1TMTt262OGz/YNjxJehUeSTSLp0Z5ApkZlKm8BImQFO6iMxheFzTBoTlWWXIXkNryE41kEQz41xnQe819X+AX/lXhnR3vHXVRwI= MIME-Version: 1.0 Content-Type: text/plain X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 85fd3659-e60f-4186-35fe-08d49217fb5d X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(201703131430075)(201703131517081); SRVR:BN3PR0301MB0900; X-Microsoft-Exchange-Diagnostics: 1; BN3PR0301MB0900; 3:LUQsLg7mFF8t2y+UU6Kthw8CmAEX5849A6CTbQQFwpgX+v5pY6LjdVhQ/KHPVdT+nNbQgp79gB3KrkZAdmAA0chGV3GqJB3RHtIRcRRW8DlxTiWoJZOtjbaOC6eEckoA57hXxLQf2p92XfFJcBg0M6mn86gZbGWo+eBDRukUjvzX0MUBYH/pxCNhMaTNgEb/95s3JnwCXpMhNCP3B8yK246q4U4opxzYD8/ETpbXvplsk06qxhmBPWNaDgvIDdYzixchvpJDsDi7C3Pb+PzjZ21ojsff5B+1KTvVCjKdmMjOyl0NFSIu/jSoPwuVDYxGtzofseeY7D02kG8x6X8eXdrloOBHwy0SE+3vdJtlgjBe57MOye7ihPlaANAOsLPT2Rs5FM8Ph0dIYAV3UvXPnf7eDUI691cRsIWwNvw+WjS/3sXuOoazEtMQp9lNfe7F; 25:T9a6a2TkAGtJsPE+DOr74VE33GhO0pzdm+A2qnJvaT8wEvinSx1HU0SkKc2P6PaNZZK7bnEuOtelUc+k4OWSDA4sGMoA3Pucfco4AzcXQWqc7mxk2GpaPAbsux7yHQhbB64Xw97Fbp55F1y2vhcn8ueaIa4+Lom9pK+3+DebLBNjOXqzhxnHtvaXw7ZZm9WGjMjk+VIQqzSKvQWDJPuS1LwONSUa/F+7fpY8jUOl1tI0u7n64WiehGkXFnez5TcNMXneyHNhIpzO4Sh5jHjHBqvQ8htLmQKg6vFAhHbhFZJrdLZqNcv/YuoNM3FaMFBJ6gqT7JR3LULSdtPLUYZBiSxpS+8/HvSjcg3AZfdJFF4Vx64k3etz/kdYauPFNwuE/LjpV+9vXfqqfxf4faSM/BgwEKX88L5bp3Tn93UQI8xdFEtyOzCwgN8HkstZixK/1kbi2/6feCwFWghE0DXDEA== X-Microsoft-Exchange-Diagnostics: 1; BN3PR0301MB0900; 31:QyIeDN8Se+4m0SYH5NUHWqSTf0Ick+mXAK/3YERe6SkLzE/1oyAXc/Ebmn8In5q1jXULRFjzNeh6Sc/kHRaGBRwqKRfYA6CNcYft7BSIP4UPgU+ELE3pa30iX2O1S0VPMYfc+7mfTEo9IoAuAJfTWRD/L3mVd1+czitOv5UbdNLrBJSJSgfW6mbnXMc8Qda563y13K3SWsXQ7l+SEX+xFmjDgcH9XVDUX4cxtSRWNpU1l9AkNSQMXNihiRg89m4UwWdx8gKUyohJYfWlc/8UYVgrEjQhFAPP/1oOcrTrswQ= X-Microsoft-Antispam-PRVS: <BN3PR0301MB09002E862064A703C257F71489160@BN3PR0301MB0900.namprd03.prod.outlook.com> X-Exchange-Antispam-Report-Test: UriScan:(185117386973197); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6095135)(601004)(2401047)(13015025)(13017025)(8121501046)(5005006)(13024025)(13018025)(13023025)(3002001)(10201501046)(93006095)(93001095)(6055026)(6096035)(20161123563025)(201703131430075)(201703131448075)(201703131433075)(201703161259150)(201703151042153)(20161123565025)(20161123559100)(20161123561025)(20161123556025); SRVR:BN3PR0301MB0900; BCL:0; PCL:0; RULEID:(400006); SRVR:BN3PR0301MB0900; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN3PR0301MB0900; 4:2RaNCQhQSIwH3qC+mCFFX4MJTBx4BpkX6Hi2AShx?= hVsGZoBPhhs3KNLCNRiqwLRXxn3JLAF+SSpS+SOqPGEGrvhdQ2stG2qaFATJPqM5WyH2IRrMpIporAGH98N4tbDKvKO7+qXHkFBWdJci4fSzPQK48PJQ2PKGpkgYVoLRri8WJ0Yjj1agD2ogkfTU2WvSlkZ0LP4aAxs2oLogfl2RpGgpuOMCpYqo7MtfaPiPsbPGQ0LQpbKEsPKrnphdS+LhGtBYQMgUK5pKijXS+L/xM6uNpCbTzNpufU2stpVrKPMupo3vjYbh6VOTaHON8ZUEG7ylTU3mLxLy6VCCgmOtXSXoyg08/BQltt9/OYAgHlRw+uoyygNne8D/ubLIAQ+EeRjZDSJdxFEb3JKLu3uM/r75MTksNvEkq2uXGWNOfLPEwUV9FVRaAxTFYvN1kA758RJHUHWEVPxH3VhLwrtbixpQm9ToX/wffa0UkO5WAYnSHaGGS5fOnNpWgDCa965u1yGeoncNe0HbVP5/LxgfM2GW747dDc9JcZa/JRqW/Hm1gE66T1Czdb3ozIOAVIVgH80m0WDxqHJq5/r5+spsvJypUtjJZqUVmEfmKRuIhJTk66R8hUnWqFe2GX7vFomMTE+UT15KJ0uoIgoxefVhfYkadH4n8co/y7BYgdmQaswjyRsXxXfQw0bJ2ZZK/+TI3wXwOHQnbELMhTqD9ixYIAdKgMJupMQuBiIoeLv9FZMpGa5nTjItBLs7FjuebuVOToCAj6DKHIlLolXsTAbX7/fKa37iHpE59gO5CsAUfjE7vLLpcCrH6mOMjq4XE7XHCWIz1eOv79EdcYYXLvqvg2f/u8PaH6qRGTJUopGFqAaa8hAwHHmmYFt6+uDrLF5j1DhXhBt81kZLAyuFNN2U412rR0uYNhRDWikPXqovqbng3iXppycLtfW08ipYPQ/WzivLYN1ZTWDl3n1KWSCSyw== X-Forefront-PRVS: 029651C7A1 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; BN3PR0301MB0900; 23:ozl10w2ycmcrhjNjLXwrlxwgWqj4eVLwnPQsIgW?= cqHkYNoQ+LtNu/5PDevdch2ALcQKPxp7l1+ZbvfEeMniIOPPq/sv4Asnk9tafSXMdnb9v+SWEzCD8qzDsoK2P8WAOKUON+HYtgzNRa1C4wPzyMZJvB5In1yVyKHVb3HF/7zfdih4oQb/uXhARTrXjmf2mNZSjhujhnnPjeMwtsOus8xcAE7Nw3E4hwzx0dO6zbhjSx2t3wmqNgWOUojeG5fCUsdyPq6jGaJRX0E/way4AwMYN0YWWyZKLmVtKYdoxc+ER5j5JkkHGXqyk1rPfIxcHwQPrpxQfooxv/hFKPQB5nnEw7Cf64HPyLkPIICmlvuuoLxcqPFsYgjcX81sDwxn0en8nrFJdk8iCETYxH9cwALJgjNmOOr9fw4zBWmMKqnaQC1QX13zqXwiPTOqSOxbrMixc9bg4R1V1355eoMpTUhTLFzzc2EElQjivjsrM/tjpRfpepBBpvFcFrxRCk/4PdkmMO3R1cf83S/g+e0jhYze+62fnnXFHaoIita/ZT+vDV18ZiaR1mnVjM8kyLkGPX5MW8XwdkEnnLHZPHEIVVVj60rlcimqEWG6n7KxiT5i9rMWTBXFNTjrtfipGYLetpXDqVjuH0M+YQ1uRCA2aT1nKk5dV7j/cz5pmqPX6O46Rai6qIjYz163woEDUnDpDW1llJwlpVwJrGcp9v/2VvkZT9A601XP7WgFbK5SSzUJ6Mw4mFKxqp+k/3kxDca8SfVMj1pQtl2JphznlZQwtm+5/PtJB5ZBqwAJtv+LKBXydJTDKlfBeTDWWtCyK8itp2tVpRupvbfBV/H9jC7PVUvVHYqKqjx0GiSlbEB8M4QhAk3VxMlfkbBPnUyW4IjAMyZMLaimSa2ELDbfs2OcAtaAo2mDyPk7byJLgO1WCpr4hL+bthhNkqNHkGiAAO4qszUfFhAFO0ClrbBJYpCUzWFPHw8Wzpoqr2Apvb/kF7xJjE0p6C9C6Br8JurBFzoWR7+0GEqwHtyd+JWeLsbEfXV/yugfMah2uQBP2XpfE2dy+ao9GDJ+4QCVH51zcDYrZVleAJMP8bR3BWV0TpfwLvX3nUHlp3zebpWI53sriVVdotk05k3+2xpRAUhuEmuaZebKnFd3PxMEVNPT4AkgV2Wf4KC8vogWblbKG4YqlAyzO7RtR+YYuMatxjNj25JfKE79TxLanq9Tf190d0/kPwWfBmPQhceeayGMHjHHZPn1Kk0KsZ9HDvBzhT2HlYnwq X-Microsoft-Exchange-Diagnostics: 1; BN3PR0301MB0900; 6:ADJ8gVMCjaV0PElMEQx2wkVuACixZZjgQUL/cN/iJwUjKsreD1OieGpxjQ/sQ2/78Tqd0y+m2LJmtJGcwAkRa8C1FEslUe80YUZcU/8bLOnayKShmsRi+jKCgc0sglXjJEaQ8dNNRVGDLhMbOYvneq4ox1PrZq1S5RCwZaquWLb2XZbJkIetKF+7roepAioouhdOzDgCgFgmiRj3tcEEiT+lC0zTdqeUj3hT6dSn4WefdiCV8Z8Tt/ii6KrieQJfUAUmFxF4SOcnoxUQqVcVXX1ZByi5ZuaojkJGekXoHRX0ySd7JzZZB7LgM5pclOSwSZmu3HvDnbXE9S1zHb9Qdbx8dKsh8QzQSPC2Wz3ns8y+HsBO6uOmqs8azweSrz1htOYidEzqGV9ckc0Alu32C9xQ7EZgxP9rwJ2HXV6tNo+bxGLdrfWly6iNO+nqeHM8/xWd2pcz5ndHkxDmqnG+vl2yCNExnTih+d/VJ/PU4Sv+XR2zmP3cXqvBwFmAAXAB/Zn9fz07gq7SNZG4D5M2Mw==; 5:8hAV4uZc7B865uHLOmusGgTv6MvGOpi+Wj2odjbi5gnc1prx1X5zRyu790vYLErwAx/LvWTKuJE3oFX4n6mmLl9FvlDqVji+v6w+CicKZzCvQbgOpfbhM3GKMca1LjfmjK5t/SQSekpNSuEolZKmQb26TvBet4ZkBIGFUwxV+u0tUHqQDv0vWHfdS76zp02M; 24:Uet06ANQUqpyxMX5VyuOAjZpx9KMt1rLeaASxbWou+Ip3xmmq/rA3+GQmGpRTyqZJH6PVi6CEPCr5pFAafr8jfXLyiwyb5Vh95PujRO2tkM= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; BN3PR0301MB0900; 7:p68nlF3kL/Js1rpaYmYVNLE+gpA3CmbymAU/E2//g/peR4Vyk2PL3ivS47cs9rVc4GEKJTCP7d2ughcnlh5Taq3hVnI8+21t0uCpx7BoTw1Kag31+Vnldn+2GAmTUKQ1mXPlyz3MMon5v2htHRwz357aCzq6TBwJc234it2fZ2PXWHOXlaBqYdCXp0x5C6NOLSGwXMXCYi9CvSbsjcteFHdhOhh+PDNI5ltHpAEkJRxoH6ooXxIxv9cSXBanzOijx305TKtZfUqOziga1OgzaM/xfPYjWK1rZCzULYLun8mlD7vzeCbUvR2qV+5i0OgE9SzdImu4v0/s1+wKzWFKNA== X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 May 2017 11:31:46.3459 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0301MB0900 Subject: [dpdk-dev] [PATCH] doc: notice for changes in kni structures X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
ci/Intel-compilation | success | Compilation OK |
Commit Message
Hemant Agrawal
May 3, 2017, 11:31 a.m. UTC
Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
doc/guides/rel_notes/deprecation.rst | 7 +++++++
1 file changed, 7 insertions(+)
Comments
On Wed, 3 May 2017 17:01:31 +0530 Hemant Agrawal <hemant.agrawal@nxp.com> wrote: > Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com> > --- > doc/guides/rel_notes/deprecation.rst | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst > index a3e7c72..0c1ef2c 100644 > --- a/doc/guides/rel_notes/deprecation.rst > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -81,3 +81,10 @@ Deprecation Notices > > - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` > - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` > + > +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, > + gso_size, promiscusity configuration. > + some of the kni structure will be changed to support additional functionality > + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, > + ``rte_kni_mbu`` to support the configured gso_size, > + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. Why are these exposed as something applications should care about? The data structures in rte_kni_common.h are not an API
On 5/3/2017 4:50 PM, Stephen Hemminger wrote: > On Wed, 3 May 2017 17:01:31 +0530 > Hemant Agrawal <hemant.agrawal@nxp.com> wrote: > >> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com> >> --- >> doc/guides/rel_notes/deprecation.rst | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst >> index a3e7c72..0c1ef2c 100644 >> --- a/doc/guides/rel_notes/deprecation.rst >> +++ b/doc/guides/rel_notes/deprecation.rst >> @@ -81,3 +81,10 @@ Deprecation Notices >> >> - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` >> - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` >> + >> +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, >> + gso_size, promiscusity configuration. >> + some of the kni structure will be changed to support additional functionality >> + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, >> + ``rte_kni_mbu`` to support the configured gso_size, >> + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. > > Why are these exposed as something applications should care about? > The data structures in rte_kni_common.h are not an API +1, rte_kni_common.h is a contract between user - kernel space, not part of API.
On 5/3/2017 12:31 PM, Hemant Agrawal wrote: > Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com> > --- > doc/guides/rel_notes/deprecation.rst | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst > index a3e7c72..0c1ef2c 100644 > --- a/doc/guides/rel_notes/deprecation.rst > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -81,3 +81,10 @@ Deprecation Notices > > - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` > - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` > + > +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, > + gso_size, promiscusity configuration. > + some of the kni structure will be changed to support additional functionality > + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, rte_kni_request is between KNI library and KNI kernel module, shouldn't be part of API. > + ``rte_kni_mbu`` to support the configured gso_size, Again, rte_kni_mbuf should be only concern of KNI kernel module. > + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. rte_kni_device_info also between KNI library and KNI kernel module. I think deprecation notice not required for above ones. But you KNI patchset updates rte_kni_conf and rte_kni_ops. These are part of KNI API and changing them cause ABI breakage, but if new fields appended in these structs, this will not cause an ABI breakage, and I think that is better to do instead of deprecation notice, what do you think? And apart from above ABI issues, adding new fields to "rte_kni_ops" means DPDK application that use KNI should implement them, right? So this suggest everyone require to set promiscuity of KNI device should implement this. Can't we find another way that all can benefit from a common implementation? Thanks, ferruh
On 5/4/2017 10:20 PM, Ferruh Yigit wrote: > On 5/3/2017 12:31 PM, Hemant Agrawal wrote: >> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com> >> --- >> doc/guides/rel_notes/deprecation.rst | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst >> index a3e7c72..0c1ef2c 100644 >> --- a/doc/guides/rel_notes/deprecation.rst >> +++ b/doc/guides/rel_notes/deprecation.rst >> @@ -81,3 +81,10 @@ Deprecation Notices >> >> - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` >> - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` >> + >> +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, >> + gso_size, promiscusity configuration. >> + some of the kni structure will be changed to support additional functionality >> + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, > > rte_kni_request is between KNI library and KNI kernel module, shouldn't > be part of API. > >> + ``rte_kni_mbu`` to support the configured gso_size, > > Again, rte_kni_mbuf should be only concern of KNI kernel module. > >> + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. > > rte_kni_device_info also between KNI library and KNI kernel module. > > I think deprecation notice not required for above ones. > > But you KNI patchset updates rte_kni_conf and rte_kni_ops. > These are part of KNI API and changing them cause ABI breakage, > but if new fields appended in these structs, this will not cause an ABI > breakage, and I think that is better to do instead of deprecation > notice, what do you think? I agree. > > > And apart from above ABI issues, > adding new fields to "rte_kni_ops" means DPDK application that use KNI > should implement them, right? Well, it depend, if the application is interested in this information or not? > So this suggest everyone require to set promiscuity of KNI device should > implement this. yes! > Can't we find another way that all can benefit from a common implementation? how you want it differently? Any ideas? > > Thanks, > ferruh >
On 5/3/2017 9:20 PM, Stephen Hemminger wrote: > On Wed, 3 May 2017 17:01:31 +0530 > Hemant Agrawal <hemant.agrawal@nxp.com> wrote: > >> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com> >> --- >> doc/guides/rel_notes/deprecation.rst | 7 +++++++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst >> index a3e7c72..0c1ef2c 100644 >> --- a/doc/guides/rel_notes/deprecation.rst >> +++ b/doc/guides/rel_notes/deprecation.rst >> @@ -81,3 +81,10 @@ Deprecation Notices >> >> - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` >> - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` >> + >> +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, >> + gso_size, promiscusity configuration. >> + some of the kni structure will be changed to support additional functionality >> + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, >> + ``rte_kni_mbu`` to support the configured gso_size, >> + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. > > Why are these exposed as something applications should care about? > The data structures in rte_kni_common.h are not an API > Agreed. This is not needed.
On 5/8/2017 10:46 AM, Hemant Agrawal wrote: > On 5/4/2017 10:20 PM, Ferruh Yigit wrote: >> On 5/3/2017 12:31 PM, Hemant Agrawal wrote: >>> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com> >>> --- >>> doc/guides/rel_notes/deprecation.rst | 7 +++++++ >>> 1 file changed, 7 insertions(+) >>> >>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst >>> index a3e7c72..0c1ef2c 100644 >>> --- a/doc/guides/rel_notes/deprecation.rst >>> +++ b/doc/guides/rel_notes/deprecation.rst >>> @@ -81,3 +81,10 @@ Deprecation Notices >>> >>> - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` >>> - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` >>> + >>> +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, >>> + gso_size, promiscusity configuration. >>> + some of the kni structure will be changed to support additional functionality >>> + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, >> >> rte_kni_request is between KNI library and KNI kernel module, shouldn't >> be part of API. >> >>> + ``rte_kni_mbu`` to support the configured gso_size, >> >> Again, rte_kni_mbuf should be only concern of KNI kernel module. >> >>> + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr. >> >> rte_kni_device_info also between KNI library and KNI kernel module. >> >> I think deprecation notice not required for above ones. >> >> But you KNI patchset updates rte_kni_conf and rte_kni_ops. >> These are part of KNI API and changing them cause ABI breakage, >> but if new fields appended in these structs, this will not cause an ABI >> breakage, and I think that is better to do instead of deprecation >> notice, what do you think? > > I agree. >> >> >> And apart from above ABI issues, >> adding new fields to "rte_kni_ops" means DPDK application that use KNI >> should implement them, right? > > Well, it depend, if the application is interested in this information or > not? > >> So this suggest everyone require to set promiscuity of KNI device should >> implement this. > > yes! > >> Can't we find another way that all can benefit from a common implementation? > > how you want it differently? Any ideas? Can having default implementations in librte_kni work? Would applications be doing something different, lets say to set MTU? > > >> >> Thanks, >> ferruh >> > >
diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst index a3e7c72..0c1ef2c 100644 --- a/doc/guides/rel_notes/deprecation.rst +++ b/doc/guides/rel_notes/deprecation.rst @@ -81,3 +81,10 @@ Deprecation Notices - ``rte_crpytodev_scheduler_mode_get``, replaced by ``rte_cryptodev_scheduler_mode_get`` - ``rte_crpytodev_scheduler_mode_set``, replaced by ``rte_cryptodev_scheduler_mode_set`` + +* kni: additional functionality is planned to be added in kni to support mtu, macaddr, + gso_size, promiscusity configuration. + some of the kni structure will be changed to support additional functionality + e.g ``rte_kni_request`` to support promiscusity`` and mac_addr, + ``rte_kni_mbu`` to support the configured gso_size, + ``rte_kni_device_info`` and ``rte_kni_conf`` to also support mtu and macaddr.