All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Hunt, David" <david.hunt@intel.com>
To: Jerin Jacob <jerin.jacob@caviumnetworks.com>, dev@dpdk.org
Subject: Re: [PATCH 00/12] DPDK armv8-a support
Date: Tue, 3 Nov 2015 14:17:38 +0000	[thread overview]
Message-ID: <5638C202.1050205@intel.com> (raw)
In-Reply-To: <1446556153-18845-1-git-send-email-jerin.jacob@caviumnetworks.com>

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.

> 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.

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

and then it built fine, and I can run testpmd with my 82599's and run 
autotests.

Thanks for that.
Dave.

  parent reply	other threads:[~2015-11-03 14:19 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-03 13:09 [PATCH 00/12] DPDK armv8-a support Jerin Jacob
2015-11-03 13:09 ` [PATCH 01/12] eal: arm64: add armv8-a version of rte_atomic_64.h Jerin Jacob
2015-11-03 13:09   ` [PATCH 02/12] eal: arm64: add armv8-a version of rte_cpuflags_64.h Jerin Jacob
2015-11-03 13:09     ` [PATCH 03/12] eal: arm64: add armv8-a version of rte_prefetch_64.h Jerin Jacob
2015-11-03 13:09       ` [PATCH 04/12] eal: arm64: add armv8-a version of rte_cycles_64.h Jerin Jacob
2015-11-03 13:09         ` [PATCH 05/12] eal: arm64: rte_memcpy_64.h version based libc memcpy Jerin Jacob
2015-11-03 13:09           ` [PATCH 06/12] eal: arm: ret_vector.h improvements Jerin Jacob
2015-11-03 13:09             ` [PATCH 07/12] app: test: added the new cpu flags of arm64 in test_cpuflags test case Jerin Jacob
2015-11-03 13:09               ` [PATCH 08/12] acl: arm64: acl implementation using NEON gcc intrinsic Jerin Jacob
2015-11-03 13:09                 ` [PATCH 09/12] mk: add support for armv8 on top of armv7 Jerin Jacob
2015-11-03 13:09                   ` [PATCH 10/12] mk: add support for thunderx machine target based armv8-a Jerin Jacob
2015-11-03 13:09                     ` [PATCH 11/12] updated release note for armv8 support for DPDK 2.2 Jerin Jacob
2015-11-03 13:09                       ` [PATCH 12/12] maintainers: claim responsibility for ARMv8 Jerin Jacob
2015-11-03 14:17 ` Hunt, David [this message]
2015-11-03 16:38   ` [PATCH 00/12] DPDK armv8-a support Jerin Jacob
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=5638C202.1050205@intel.com \
    --to=david.hunt@intel.com \
    --cc=dev@dpdk.org \
    --cc=jerin.jacob@caviumnetworks.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.