From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 76F66CA9EB6 for ; Wed, 23 Oct 2019 14:47:26 +0000 (UTC) Received: from dpdk.org (dpdk.org [92.243.14.124]) by mail.kernel.org (Postfix) with ESMTP id F0B452086D for ; Wed, 23 Oct 2019 14:47:25 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org F0B452086D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=6wind.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=dev-bounces@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 540461C243; Wed, 23 Oct 2019 16:47:25 +0200 (CEST) Received: from proxy.6wind.com (host.76.145.23.62.rev.coltfrance.com [62.23.145.76]) by dpdk.org (Postfix) with ESMTP id C3DD11C23D for ; Wed, 23 Oct 2019 16:47:24 +0200 (CEST) Received: from glumotte.dev.6wind.com (unknown [10.16.0.195]) by proxy.6wind.com (Postfix) with ESMTP id A26B133588B; Wed, 23 Oct 2019 16:47:24 +0200 (CEST) Date: Wed, 23 Oct 2019 16:47:24 +0200 From: Olivier Matz To: Jerin Jacob Cc: Vamsi Krishna Attunuru , Andrew Rybchenko , Ferruh Yigit , "thomas@monjalon.net" , Jerin Jacob Kollanukkaran , Kiran Kumar Kokkilagadda , "anatoly.burakov@intel.com" , "stephen@networkplumber.org" , "dev@dpdk.org" Message-ID: <20191023144724.GO25286@glumotte.dev.6wind.com> References: <20190816061252.17214-1-vattunuru@marvell.com> <20191021080324.10659-1-vattunuru@marvell.com> <20191021080324.10659-3-vattunuru@marvell.com> <4bd1acf5-2da2-b2da-2b0c-7ee243d5aeb9@intel.com> <77f8eaf0-52ca-1295-973d-c8085f7b7736@intel.com> <08c426d1-6fc9-1c3f-02d4-8632a8e3c337@solarflare.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Subject: Re: [dpdk-dev] [EXT] Re: [PATCH v11 2/4] eal: add legacy kni option 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" Hi, On Wed, Oct 23, 2019 at 03:42:39PM +0530, Jerin Jacob wrote: > On Tue, Oct 22, 2019 at 7:01 PM Vamsi Krishna Attunuru > wrote: > > > > Hi Ferruh, > > > > Can you please explain the problems in using kni dedicated mbuf alloc routines while enabling kni iova=va mode. Please see the below discussion with Andrew. He wanted to know the problems in having newer APIs. > > > While waiting for the Ferruh reply, I would like to summarise the > current status > > # In order to make KNI work with IOVA as VA, We need to make sure > mempool pool _object_ should not span across two huge pages > > # This problem can be fixed by, either of: > > a) Introduce a flag in mempool to define this constraint, so that, > when only needed, this constraint enforced and this is in line > with existing semantics of addressing such problems in mempool > > b) Instead of creating a flag, Make this behavior by default in > mempool for IOVA as VA case > > Upside: > b1) There is no need for specific mempool_create for KNI. > > Downside: > b2) Not align with existing mempool API semantics > b3) There will be a trivial amount of memory waste as we can not > allocate from the edge. Considering the normal huge > page memory size is 1G or 512MB this not a real issue. > > c) Make IOVA as PA when KNI kernel module is loaded > > Upside: > c1) Doing option (a) would call for new KNI specific mempool create > API i.e existing KNI applications need a one-line change in > application to make it work with release 19.11 or later. > > Downslide: > c2) Driver which needs RTE_PCI_DRV_NEED_IOVA_AS_VA can not work with KNI > c3) Need root privilege to run KNI as IOVA as PA need root privilege > > For the next year, we expect applications to work 19.11 without any > code change. My personal opinion to make go with option (a) > and update the release notes to document the change any it simple > one-line change. > > The selection of (a) vs (b) is between KNI and Mempool maintainers. > Could we please reach a consensus? Or can we discuss this TB meeting? > > We are going back and forth on this feature on for the last 3 > releases. Now that, we solved all the technical problems, please help > us > to decide (a) vs (b) to make forward progress. Thank you for the summary. What is not clear to me is if (a) or (b) may break an existing application, and if yes, in which case.