From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 5B8FCA054A; Wed, 7 Sep 2022 23:33:23 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3AA5A40143; Wed, 7 Sep 2022 23:33:23 +0200 (CEST) Received: from smartserver.smartsharesystems.com (smartserver.smartsharesystems.com [77.243.40.215]) by mails.dpdk.org (Postfix) with ESMTP id CD448400D6 for ; Wed, 7 Sep 2022 23:33:21 +0200 (CEST) Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Subject: RE: [PATCH v2 1/5] build: add meson option to configure IOVA mode as VA Content-Transfer-Encoding: quoted-printable X-MimeOLE: Produced By Microsoft Exchange V6.5 Date: Wed, 7 Sep 2022 23:33:20 +0200 Message-ID: <98CBD80474FA8B44BF855DF32C47DC35D87301@smartserver.smartshare.dk> In-Reply-To: X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH v2 1/5] build: add meson option to configure IOVA mode as VA Thread-Index: AdjCz+b/5gU3WYq2QheJtiSHQBMt4QAMVHrw References: <20220829151626.2101336-1-sthotton@marvell.com> <20220907134340.3629224-2-sthotton@marvell.com> <20220907083109.1cfe432c@hermes.local> From: =?iso-8859-1?Q?Morten_Br=F8rup?= To: "Bruce Richardson" , "Stephen Hemminger" , "Shijith Thotton" Cc: , , , , , , "Nicolas Chautru" , "Ciara Power" , "Konstantin Ananyev" , "Chengwen Feng" , "Kevin Laatz" , "Reshma Pattan" , "Maxime Coquelin" , "Chenbo Xia" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org > From: Bruce Richardson [mailto:bruce.richardson@intel.com] > Sent: Wednesday, 7 September 2022 17.38 >=20 > On Wed, Sep 07, 2022 at 08:31:09AM -0700, Stephen Hemminger wrote: > > On Wed, 7 Sep 2022 19:13:36 +0530 Shijith Thotton > > > wrote: > > > > > IOVA mode in DPDK is either PA or VA. The new build option > iova_as_va > > > configures the mode to VA at compile time and prevents setting it > to PA > > > at runtime. For now, all drivers which are not always enabled are > > > disabled with this option. Supported driver can set the flag > > > pmd_iova_as_va in its build file to enable build. > > > > > > mbuf structure holds the physical (PA) and virtual address (VA) of > a > > > buffer. if IOVA mode is set to VA, PA is redundant as it is the > same as > > > VA. So PA field need not be updated and marked invalid if the = build > is > > > configured to use only VA. > > > > > > Signed-off-by: Shijith Thotton > > > > NAK This works ok for use cases where NFV is building local version > of > > DPDK, but meson options dont work for distro vendors. > > > I think for something of this magnitude a build-time option is > definitely > the way to go. Sure, it won't be available in distro-packages, but for > something like this it's just too invasive - and rather niche IMHO - = to > make a runtime option. +1