DPDK patches and discussions
 help / color / mirror / Atom feed
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
To: "Hunt, David" <david.hunt@intel.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH 00/12] DPDK armv8-a support
Date: Tue, 3 Nov 2015 22:08:41 +0530	[thread overview]
Message-ID: <20151103163838.GA18587@localhost.localdomain> (raw)
In-Reply-To: <5638C202.1050205@intel.com>

On Tue, Nov 03, 2015 at 02:17:38PM +0000, Hunt, David wrote:
> On 03/11/2015 13:09, Jerin Jacob wrote:
> >This is the v1 patchset for ARMv8 that now sits on top of the v6 patch
> >of the ARMv7 code by RehiveTech. It adds code into the same arm include
> >directory, reducing code duplication.
> >
> >Tested on an ThunderX arm 64-bit arm server board, with PCI slots. Passes traffic
> >between two physical ports on an Intel 82599 dual-port 10Gig NIC. Should
> >work with many other NICS as long as long as there is no unaligned access to
> >device memory but not yet untested.
> 
> I have your patchset building and running on an X-Gene based 8-core MP30AR0
> system, passing traffic between two ports on and 82599 also.
> 

Thanks.

> >Notes on arm64 kernel configuration:
> >
> >   Tested on using Ubuntu 14.04 LTS with a 3.18 kernel and igb_uio.
> >   ARM64 kernels does not have functional resource mapping of PCI memory
> >   (PCI_MMAP), so the pci driver needs to be patched to enable this. The
> >   symptom of this is when /sys/bus/pci/devices/0000:0X:00.Y directory is
> >   missing the resource0...N files for mmapping the device memory.
> >
> >   Following patch fixes the PCI resource mapping issue om armv8.
> >   Its not yet up streamed.We are in the process of up streaming it.
> >
> >   http://lists.infradead.org/pipermail/linux-arm-kernel/2015-July/358906.html
> 
> Good to see that there's a patch on the way for this. That fix looks almost
> exactly the same as the hack I did to my kernel :)
> 
> I had a couple of small issues when patching/building:
> 
> 1. Three of the files had an extra blank line at the end. Maybe worth
> running checkpatch on the patches. 'git am' was complaining.

I will fix it in next version.

> 
> 2. I had problems compiling two drivers because they were attempting to
> include tmmintrin.h:
> 
> ...dpdk/drivers/net/fm10k/fm10k_rxtx_vec.c:41:23: fatal error: tmmintrin.h:
> No such file or directory
> 
> ...dpdk/drivers/net/i40e/i40e_rxtx_vec.c:43:23: fatal error: tmmintrin.h: No
> such file or directory
>

 
> To avoid this, I added the following two lines into
> defconfig_arm64-armv8a-linuxapp-gcc
> 
> CONFIG_RTE_LIBRTE_FM10K_PMD=n
> CONFIG_RTE_LIBRTE_I40E_PMD=n

the patch was based on 82fb702077f67585d64a07de0080e5cb6a924a72 which
don't have these changes. I will add these in next version.
 
> and then it built fine, and I can run testpmd with my 82599's and run
> autotests.

I ran autotest, "Mbuf autotest" stress failure is due strong vs weak ordering
issue. I will send the next version based on new patch being discussed
on ml.


> 
> Thanks for that.
> Dave.

  reply	other threads:[~2015-11-03 16:39 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-03 13:09 Jerin Jacob
2015-11-03 13:09 ` [dpdk-dev] [PATCH 01/12] eal: arm64: add armv8-a version of rte_atomic_64.h Jerin Jacob
2015-11-03 13:09   ` [dpdk-dev] [PATCH 02/12] eal: arm64: add armv8-a version of rte_cpuflags_64.h Jerin Jacob
2015-11-03 13:09     ` [dpdk-dev] [PATCH 03/12] eal: arm64: add armv8-a version of rte_prefetch_64.h Jerin Jacob
2015-11-03 13:09       ` [dpdk-dev] [PATCH 04/12] eal: arm64: add armv8-a version of rte_cycles_64.h Jerin Jacob
2015-11-03 13:09         ` [dpdk-dev] [PATCH 05/12] eal: arm64: rte_memcpy_64.h version based libc memcpy Jerin Jacob
2015-11-03 13:09           ` [dpdk-dev] [PATCH 06/12] eal: arm: ret_vector.h improvements Jerin Jacob
2015-11-03 13:09             ` [dpdk-dev] [PATCH 07/12] app: test: added the new cpu flags of arm64 in test_cpuflags test case Jerin Jacob
2015-11-03 13:09               ` [dpdk-dev] [PATCH 08/12] acl: arm64: acl implementation using NEON gcc intrinsic Jerin Jacob
2015-11-03 13:09                 ` [dpdk-dev] [PATCH 09/12] mk: add support for armv8 on top of armv7 Jerin Jacob
2015-11-03 13:09                   ` [dpdk-dev] [PATCH 10/12] mk: add support for thunderx machine target based armv8-a Jerin Jacob
2015-11-03 13:09                     ` [dpdk-dev] [PATCH 11/12] updated release note for armv8 support for DPDK 2.2 Jerin Jacob
2015-11-03 13:09                       ` [dpdk-dev] [PATCH 12/12] maintainers: claim responsibility for ARMv8 Jerin Jacob
2015-11-03 14:17 ` [dpdk-dev] [PATCH 00/12] DPDK armv8-a support Hunt, David
2015-11-03 16:38   ` Jerin Jacob [this message]
2015-11-05 14:39     ` Hunt, David

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20151103163838.GA18587@localhost.localdomain \
    --to=jerin.jacob@caviumnetworks.com \
    --cc=david.hunt@intel.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).