From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hemant Agrawal Subject: Re: [PATCH v2] bus/dpaa: fix dpaa returning IOVA as PA by default Date: Tue, 6 Feb 2018 22:58:29 +0530 Message-ID: <0cc5b397-dbc1-de66-76ad-5ad47707a88a@nxp.com> References: <20180206162156.2104-1-pbhagavatula@caviumnetworks.com> <20180206172248.5067-1-pbhagavatula@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Pavan Nikhilesh , jerin.jacob@caviumnetworks.com, santosh.shukla@caviumnetworks.com, thomas@monjalon.net, ferruh.yigit@intel.com Return-path: Received: from EUR01-HE1-obe.outbound.protection.outlook.com (mail-he1eur01on0077.outbound.protection.outlook.com [104.47.0.77]) by dpdk.org (Postfix) with ESMTP id EC74E1B6D2 for ; Tue, 6 Feb 2018 18:28:42 +0100 (CET) In-Reply-To: <20180206172248.5067-1-pbhagavatula@caviumnetworks.com> Content-Language: en-US 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 2/6/2018 10:52 PM, Pavan Nikhilesh wrote: > Fix dpaa bus returning IOVA as PA even when it is not running on dpaa > platform. > > Fixes: 1ee9569576f6 ("config: enable dpaaX drivers for generic ARMv8") > Fixes: d5a4e3a00c4a ("bus/dpaa: set IOVA mode as physical") > > Signed-off-by: Pavan Nikhilesh > --- > > v2 Changes: > - redo commit log. > > Currently all armv8 platforms are broken due to this. > > drivers/bus/dpaa/dpaa_bus.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/bus/dpaa/dpaa_bus.c b/drivers/bus/dpaa/dpaa_bus.c > index 290b967da..f2bb3b158 100644 > --- a/drivers/bus/dpaa/dpaa_bus.c > +++ b/drivers/bus/dpaa/dpaa_bus.c > @@ -541,6 +541,10 @@ rte_dpaa_find_device(const struct rte_device *start, rte_dev_cmp_t cmp, > static enum rte_iova_mode > rte_dpaa_get_iommu_class(void) > { > + if ((access(DPAA_DEV_PATH1, F_OK) != 0) && > + (access(DPAA_DEV_PATH2, F_OK) != 0)) { > + return RTE_IOVA_DC; > + } > return RTE_IOVA_PA; > } > > -- > 2.14.1 > > Acked-by: Hemant Agrawal