From: "Boyer, Andrew" <Andrew.Boyer@amd.com>
To: Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>
Cc: "Boyer, Andrew" <Andrew.Boyer@amd.com>,
"dev@dpdk.org" <dev@dpdk.org>, "Patel, Neel" <Neel.Patel@amd.com>,
"R, Mohamed Shah" <mohamedshah.r@amd.com>,
Alfredo Cardigliano <cardigliano@ntop.org>, nd <nd@arm.com>
Subject: Re: [PATCH 3/3] net/ionic: add vdev support for embedded applications
Date: Wed, 21 Feb 2024 01:16:53 +0000 [thread overview]
Message-ID: <8AAD45FE-EB20-4357-9C85-806B8A44F86E@amd.com> (raw)
In-Reply-To: <DD807C47-218E-44DE-A012-6F5539BA7E01@arm.com>
[-- Attachment #1: Type: text/plain, Size: 4543 bytes --]
On Feb 19, 2024, at 9:02 PM, Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com> wrote:
Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
On Feb 16, 2024, at 11:07 AM, Andrew Boyer <andrew.boyer@amd.com> wrote:
Add support for running DPDK applications directly on AMD Pensando
embedded HW. The platform exposes the device BARs through UIO. The
UIO code in the common/ionic library walks the sysfs filesystem
to identify the relevant BARs and map them into process memory.
The SoCs are named 'Capri' and 'Elba'.
The vdev device interface code is located in ionic_dev_vdev.c.
Some datapath operations are #ifdef-ed out to save on resources when
running in embedded mode.
Some controlpath operations are skipped by the ionic_is_embedded()
helper function.
Before ringing the doorbell, use an ARM 'dsb st' barrier. The normal
barrier inside rte_write64() is insufficient on these devices due to
a chip errata.
Signed-off-by: Andrew Boyer <andrew.boyer@amd.com>
Signed-off-by: Neel Patel <neel.patel@amd.com>
Signed-off-by: R Mohamed Shah <mohamedshah.r@amd.com>
Signed-off-by: Alfredo Cardigliano <cardigliano@ntop.org>
---
config/arm/arm64_capri_linux_gcc | 16 +++
config/arm/arm64_elba_linux_gcc | 16 +++
config/arm/meson.build | 42 ++++++++
drivers/net/ionic/ionic.h | 2 +-
drivers/net/ionic/ionic_dev.h | 17 ++++
drivers/net/ionic/ionic_dev_vdev.c | 156 +++++++++++++++++++++++++++++
drivers/net/ionic/ionic_ethdev.c | 7 ++
drivers/net/ionic/ionic_lif.c | 19 ++++
drivers/net/ionic/ionic_rxtx.h | 4 +
drivers/net/ionic/meson.build | 1 +
10 files changed, 279 insertions(+), 1 deletion(-)
create mode 100644 config/arm/arm64_capri_linux_gcc
create mode 100644 config/arm/arm64_elba_linux_gcc
create mode 100644 drivers/net/ionic/ionic_dev_vdev.c
diff --git a/config/arm/arm64_capri_linux_gcc b/config/arm/arm64_capri_linux_gcc
new file mode 100644
index 0000000000..1a6313e684
--- /dev/null
+++ b/config/arm/arm64_capri_linux_gcc
@@ -0,0 +1,16 @@
+[binaries]
+c = ['ccache', 'aarch64-linux-gnu-gcc']
+cpp = ['ccache', 'aarch64-linux-gnu-g++']
+ar = 'aarch64-linux-gnu-gcc-ar'
+strip = 'aarch64-linux-gnu-strip'
+pkgconfig = 'aarch64-linux-gnu-pkg-config'
+pcap-config = ''
+
+[host_machine]
+system = 'linux'
+cpu_family = 'aarch64'
+cpu = 'armv8-a'
+endian = 'little'
+
+[properties]
+platform = 'capri'
diff --git a/config/arm/arm64_elba_linux_gcc b/config/arm/arm64_elba_linux_gcc
new file mode 100644
index 0000000000..4d891bd5a7
--- /dev/null
+++ b/config/arm/arm64_elba_linux_gcc
@@ -0,0 +1,16 @@
+[binaries]
+c = ['ccache', 'aarch64-linux-gnu-gcc']
+cpp = ['ccache', 'aarch64-linux-gnu-g++']
+ar = 'aarch64-linux-gnu-gcc-ar'
+strip = 'aarch64-linux-gnu-strip'
+pkgconfig = 'aarch64-linux-gnu-pkg-config'
+pcap-config = ''
+
+[host_machine]
+system = 'linux'
+cpu_family = 'aarch64'
+cpu = 'armv8-a'
+endian = 'little'
+
+[properties]
+platform = 'elba'
diff --git a/config/arm/meson.build b/config/arm/meson.build
index 36f21d2259..2326021fed 100644
--- a/config/arm/meson.build
+++ b/config/arm/meson.build
@@ -165,6 +165,33 @@ implementer_cavium = {
}
}
+implementer_ionic = {
+ 'description': 'AMD Pensando',
+ 'flags': [
+ ['RTE_MAX_NUMA_NODES', 1],
+ ['RTE_CACHE_LINE_SIZE', 64],
+ ['RTE_LIBRTE_VHOST_NUMA', false],
+ ['RTE_EAL_NUMA_AWARE_HUGEPAGES', false],
+ ['RTE_LIBRTE_IONIC_PMD_EMBEDDED', true],
+ ],
+ 'part_number_config': {
+ '0xc1': {
+ 'compiler_options': ['-mcpu=cortex-a72'],
+ 'flags': [
+ ['RTE_MAX_LCORE', 4],
+ ['RTE_LIBRTE_IONIC_PMD_BARRIER_ERRATA', true],
+ ]
+ },
+ '0xc2': {
+ 'compiler_options': ['-mcpu=cortex-a72'],
+ 'flags': [
+ ['RTE_MAX_LCORE', 16],
+ ['RTE_LIBRTE_IONIC_PMD_BARRIER_ERRATA', true],
+ ]
+ }
+ }
+}
Can you place it such that it is ordered alphabetically? (I understand that currently things are not ordered alphabetically, I have plans to fix that)
I switched from 'soc_ionic_capri' and 'soc_ionic_elba' to the shorter 'soc_capri' and 'soc_elba', to better match other vendors, and sorted my entries. Please see V2 and let me know if anything else needs to change.
Thanks!
Andrew
[-- Attachment #2: Type: text/html, Size: 9534 bytes --]
next prev parent reply other threads:[~2024-02-21 1:17 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-16 17:07 [PATCH 0/3] net/ionic, common/ionic: add vdev support Andrew Boyer
2024-02-16 17:07 ` [PATCH 1/3] common/ionic: create common code library for ionic Andrew Boyer
2024-02-16 17:07 ` [PATCH 2/3] net/ionic: remove duplicate barriers Andrew Boyer
2024-02-19 22:16 ` Wathsala Wathawana Vithanage
2024-02-20 2:02 ` Wathsala Wathawana Vithanage
2024-02-20 18:31 ` Boyer, Andrew
2024-02-16 17:07 ` [PATCH 3/3] net/ionic: add vdev support for embedded applications Andrew Boyer
2024-02-19 15:24 ` Ferruh Yigit
2024-02-19 22:02 ` Boyer, Andrew
2024-02-20 2:02 ` Honnappa Nagarahalli
2024-02-21 1:16 ` Boyer, Andrew [this message]
2024-02-20 8:28 ` Ferruh Yigit
2024-02-20 14:39 ` Honnappa Nagarahalli
2024-02-16 19:39 ` [PATCH 0/3] net/ionic, common/ionic: add vdev support Ferruh Yigit
2024-02-20 20:42 ` [PATCH v2 " Andrew Boyer
2024-02-20 20:42 ` [PATCH v2 1/3] common/ionic: create common code library for ionic Andrew Boyer
2024-02-20 20:42 ` [PATCH v2 2/3] net/ionic: remove duplicate barriers Andrew Boyer
2024-02-20 20:42 ` [PATCH v2 3/3] net/ionic: add vdev support for embedded applications Andrew Boyer
2024-02-23 1:52 ` Honnappa Nagarahalli
2024-02-21 16:36 ` [PATCH v2 0/3] net/ionic, common/ionic: add vdev support Ferruh Yigit
2024-02-23 13:48 ` Ferruh Yigit
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=8AAD45FE-EB20-4357-9C85-806B8A44F86E@amd.com \
--to=andrew.boyer@amd.com \
--cc=Honnappa.Nagarahalli@arm.com \
--cc=Neel.Patel@amd.com \
--cc=cardigliano@ntop.org \
--cc=dev@dpdk.org \
--cc=mohamedshah.r@amd.com \
--cc=nd@arm.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 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).