From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 05C27A0471 for ; Thu, 18 Jul 2019 08:45:44 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C634F1D9E; Thu, 18 Jul 2019 08:45:43 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by dpdk.org (Postfix) with ESMTP id B90551B53 for ; Thu, 18 Jul 2019 08:45:42 +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 x6I6jfHC028982; Wed, 17 Jul 2019 23:45:41 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=pfpt0818; bh=HoRxHQTsI81emdA6lrI0dDtL9hcxZhqKDuKcXAnUI/c=; b=kSSUwVCq7Waoy71aHIKTFXOlyO9BTi+/Q26Hm64eepo+wmSOP8o+mxdRSkrUYBEwZkMN 862ErX82/sBt8sDuPJGPMJCOJSR+2yIhUQeoX85gfifjo0u4goV8L5eAvmDI7sKIUmAU 7kEEqagoUNeP9J+6VmyUWQa8DyDePUhQNHcYxeBmBPoVXFkSUblfrIsJ7AW3jgrPVKp6 um5ShsAVklhhsM8tCkt5tCCdSaXYvw894bsL98m38qhjZyedRuukY+3EEtcPzYPu/561 jPyhdv6bYI2kDgjokwXhsj3epAgtBsQLnniRzllNquBNQsc6o7U4BhdHTgqww9eTcpiB 4Q== Received: from sc-exch04.marvell.com ([199.233.58.184]) by mx0b-0016f401.pphosted.com with ESMTP id 2ts0a2aw9j-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Wed, 17 Jul 2019 23:45:41 -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; Wed, 17 Jul 2019 23:45:39 -0700 Received: from maili.marvell.com (10.93.176.43) by SC-EXCH03.marvell.com (10.93.176.83) with Microsoft SMTP Server id 15.0.1367.3 via Frontend Transport; Wed, 17 Jul 2019 23:45:39 -0700 Received: from jerin-lab.marvell.com (jerin-lab.marvell.com [10.28.34.14]) by maili.marvell.com (Postfix) with ESMTP id 2DBD33F703F; Wed, 17 Jul 2019 23:45:37 -0700 (PDT) From: To: CC: , , , Jerin Jacob Date: Thu, 18 Jul 2019 12:15:38 +0530 Message-ID: <20190718064543.33383-1-jerinj@marvell.com> X-Mailer: git-send-email 2.22.0 In-Reply-To: <20190716134609.40930-1-jerinj@marvell.com> References: <20190716134609.40930-1-jerinj@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:5.22.84,1.0.8 definitions=2019-07-18_02:2019-07-18,2019-07-18 signatures=0 Subject: [dpdk-dev] [PATCH v3 0/4] Fixes on IOVA mode selection 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" From: Jerin Jacob Orginal V1 cover letter from David Marchand: Following the issues reported by Jerin and the discussion that emerged from it, here are fixes to restore and document the behavior of the EAL and the pci bus driver. I pondered all the arguments and tried to have the less changes possible. I can't find a need for a flag to just announce support of physical addresses from the pmd point of view. So it ended up with something really close to what Jerin had suggested. But the problem is that this is still unfinished wrt the documentation. I will be offline for 10 days and we need this to move forward, so sending anyway. v3: - Patch 2/4 - Remove personal appeals in log messages(Anatoly) - Patch 4/4 - Added following documentation (Anatoly) a) #. It is easy to enable large amount of IOVA-contiguous memory use-cases with IOVA in VA mode. in the reasons for VA as default b) As a note, If the device driver needs IOVA as VA and it cannot work with IOVA as PA then the driver must request the PCI bus layer using ``RTE_PCI_DRV_NEED_IOVA_AS_VA`` requirement flag. Absence of this flag, dictates, the driver must support both IOVA as PA and VA modes. v2: - Changed RTE_PCI_DRV_IOVA_AS_VA flag name as RTE_PCI_DRV_NEED_IOVA_AS_VA(patch 3/4) - Changed IOVA mode as VA for default case(patch 4/4) with documentation - Tested the patch series on octeontx2 platform David Marchand (2): Revert "bus/pci: add Mellanox kernel driver type" eal: fix IOVA mode selection as VA for pci drivers Jerin Jacob (2): eal: change RTE_PCI_DRV_IOVA_AS_VA flag name eal: select IOVA mode as VA for default case .../prog_guide/env_abstraction_layer.rst | 45 +++++++++++++++++++ drivers/bus/pci/linux/pci.c | 24 +++------- drivers/bus/pci/pci_common.c | 30 ++++++++++--- drivers/bus/pci/rte_bus_pci.h | 4 +- drivers/event/octeontx/timvf_probe.c | 2 +- drivers/event/octeontx2/otx2_evdev.c | 2 +- drivers/mempool/octeontx/octeontx_fpavf.c | 2 +- drivers/mempool/octeontx2/otx2_mempool.c | 2 +- drivers/net/atlantic/atl_ethdev.c | 3 +- drivers/net/bnxt/bnxt_ethdev.c | 3 +- drivers/net/e1000/em_ethdev.c | 3 +- drivers/net/e1000/igb_ethdev.c | 5 +-- drivers/net/enic/enic_ethdev.c | 3 +- drivers/net/fm10k/fm10k_ethdev.c | 3 +- drivers/net/i40e/i40e_ethdev.c | 3 +- drivers/net/i40e/i40e_ethdev_vf.c | 2 +- drivers/net/iavf/iavf_ethdev.c | 3 +- drivers/net/ice/ice_ethdev.c | 3 +- drivers/net/ixgbe/ixgbe_ethdev.c | 5 +-- drivers/net/mlx4/mlx4.c | 3 +- drivers/net/mlx5/mlx5.c | 2 +- drivers/net/nfp/nfp_net.c | 6 +-- drivers/net/octeontx2/otx2_ethdev.c | 7 +-- drivers/net/qede/qede_ethdev.c | 6 +-- drivers/raw/ioat/ioat_rawdev.c | 3 +- drivers/raw/octeontx2_dma/otx2_dpi_rawdev.c | 2 +- lib/librte_eal/common/eal_common_bus.c | 30 +++++++++++-- lib/librte_eal/common/include/rte_dev.h | 1 - lib/librte_eal/linux/eal/eal.c | 6 +-- 29 files changed, 132 insertions(+), 81 deletions(-) -- 2.22.0