From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Subject: Re: [RFC PATCH 0/2] Fix examples/distributor build issue
 for non x86
Date: Fri, 29 Jan 2016 22:23:28 +0530
Message-ID: <20160129165327.GA11725@localhost.localdomain>
References: <1449415470-25545-1-git-send-email-jerin.jacob@caviumnetworks.com>
 <20160129032140.GA25541@localhost.localdomain>
 <20160129150337.GA18356@bricha3-MOBL3>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Cc: dev@dpdk.org, viktorin@rehivetech.com
To: Bruce Richardson <bruce.richardson@intel.com>
Return-path: <dev-bounces@dpdk.org>
Received: from na01-bl2-obe.outbound.protection.outlook.com
 (mail-bl2on0096.outbound.protection.outlook.com [65.55.169.96])
 by dpdk.org (Postfix) with ESMTP id CEFFBC70C
 for <dev@dpdk.org>; Fri, 29 Jan 2016 17:53:50 +0100 (CET)
Content-Disposition: inline
In-Reply-To: <20160129150337.GA18356@bricha3-MOBL3>
List-Id: patches and discussions about DPDK <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>

On Fri, Jan 29, 2016 at 08:03:37AM -0700, Bruce Richardson wrote:
> On Fri, Jan 29, 2016 at 08:51:41AM +0530, Jerin Jacob wrote:
> > On Sun, Dec 06, 2015 at 08:54:28PM +0530, Jerin Jacob wrote:
> > > Introduced rte_prefetch_non_temporal() to remove IA specific _mm_prefect(addr, 0)
> > > gcc intrinsic and build examples/distributor for non 86 platform
> > 
> > ping for review.
> > 
> 
> Is there much performance difference between making this a prefetch NT vs making
> it an rte_prefetch0 on the platforms you have tested?

Not much difference. But, I think its worth keep the abstraction as
IA and arm64 architecture supports it.

Jerin

> 
> /Bruce
> 
> > > 
> > > Not sure the rte_prefetch_non_temporal mapping correct for
> > > all the platforms. Architecture maintainers please check the mapping for
> > > rte_prefetch_non_temporal() for specific architecures
> > > 
> > > Jerin Jacob (2):
> > >   eal: introduce rte_prefetch_non_temporal
> > >   examples/distributor: remove IA specific __mm_prefetch
> > > 
> > >  examples/distributor/main.c                              |  9 +++++----
> > >  lib/librte_eal/common/include/arch/arm/rte_prefetch_32.h |  5 +++++
> > >  lib/librte_eal/common/include/arch/arm/rte_prefetch_64.h |  5 +++++
> > >  lib/librte_eal/common/include/arch/ppc_64/rte_prefetch.h |  5 +++++
> > >  lib/librte_eal/common/include/arch/tile/rte_prefetch.h   |  5 +++++
> > >  lib/librte_eal/common/include/arch/x86/rte_prefetch.h    |  5 +++++
> > >  lib/librte_eal/common/include/generic/rte_prefetch.h     | 12 ++++++++++++
> > >  7 files changed, 42 insertions(+), 4 deletions(-)
> > > 
> > > --
> > > 2.1.0
> > >