net/mana: support building the driver on arm64

Message ID 1724711938-3108-1-git-send-email-longli@linuxonhyperv.com (mailing list archive)
State Accepted
Delegated to: Ferruh Yigit
Headers
Series net/mana: support building the driver on arm64 |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing fail Testing issues
ci/intel-Functional success Functional PASS
ci/iol-marvell-Functional success Functional Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS

Commit Message

Long Li Aug. 26, 2024, 10:38 p.m. UTC
From: Long Li <longli@microsoft.com>

The driver has been verified on Linux arm64. Enable this build option and
add a missing header file for arm64.

Signed-off-by: Long Li <longli@microsoft.com>
---
 drivers/net/mana/meson.build | 4 ++--
 drivers/net/mana/mp.c        | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)
  

Comments

David Marchand Aug. 27, 2024, 12:54 p.m. UTC | #1
Hello,

On Tue, Aug 27, 2024 at 12:01 PM Ferruh Yigit <ferruh.yigit@amd.com> wrote:
>
> On 8/26/2024 11:38 PM, longli@linuxonhyperv.com wrote:
> > From: Long Li <longli@microsoft.com>
> >
> > The driver has been verified on Linux arm64. Enable this build option and
> > add a missing header file for arm64.
> >
> > Signed-off-by: Long Li <longli@microsoft.com>
> >
>
> Hi Long,
>
> I don't see this patch in the mail list and patchwork, it can be because
> of the email address it has been sent, fyi.

The mail was waiting in the moderation queue.
Long, please fix your send-email setup, or send with a mail address
registered to the dev@ ml.


Thanks.
  
Ferruh Yigit Aug. 27, 2024, 2:35 p.m. UTC | #2
On 8/26/2024 11:38 PM, longli@linuxonhyperv.com wrote:
> From: Long Li <longli@microsoft.com>
> 
> The driver has been verified on Linux arm64. Enable this build option and
> add a missing header file for arm64.
> 
> Signed-off-by: Long Li <longli@microsoft.com>
>

Applied to dpdk-next-net/main, thanks.
  
Long Li Aug. 27, 2024, 11:59 p.m. UTC | #3
> > Hi Long,
> >
> > I don't see this patch in the mail list and patchwork, it can be
> > because of the email address it has been sent, fyi.
> 
> The mail was waiting in the moderation queue.
> Long, please fix your send-email setup, or send with a mail address registered
> to the dev@ ml.

I think the patch has made it to patchwork:
https://patchwork.dpdk.org/project/dpdk/patch/1724711938-3108-1-git-send-email-longli@linuxonhyperv.com/

The sender is longli@linuxonhyperv.com, the author is longli@microsoft.com. I hope that is okay.

Long
  
David Marchand Aug. 28, 2024, 6:02 a.m. UTC | #4
On Wed, Aug 28, 2024 at 1:59 AM Long Li <longli@microsoft.com> wrote:
> > The mail was waiting in the moderation queue.
> > Long, please fix your send-email setup, or send with a mail address registered
> > to the dev@ ml.
>
> I think the patch has made it to patchwork:
> https://patchwork.dpdk.org/project/dpdk/patch/1724711938-3108-1-git-send-email-longli@linuxonhyperv.com/
>

I released this mail from the moderation queue, so the patch could
make it to patchwork.

> The sender is longli@linuxonhyperv.com, the author is longli@microsoft.com. I hope that is okay.

The @linuxonhyperv.com address one is not registered to the dev ml,
which is the reason why the mail got moderated.

Hope it is clear now..
  
Long Li Aug. 28, 2024, 8:18 a.m. UTC | #5
> I released this mail from the moderation queue, so the patch could make it to
> patchwork.
> 
> > The sender is longli@linuxonhyperv.com, the author is
> longli@microsoft.com. I hope that is okay.
> 
> The @linuxonhyperv.com address one is not registered to the dev ml, which is
> the reason why the mail got moderated.
> 
> Hope it is clear now..

Thank you, I have subscribed as longli@linuxonhyperv.com. I don't know the reason why it got bounced and membership disabled in the past.

Long
  

Patch

diff --git a/drivers/net/mana/meson.build b/drivers/net/mana/meson.build
index 2d72eca5a8..330d30b2ff 100644
--- a/drivers/net/mana/meson.build
+++ b/drivers/net/mana/meson.build
@@ -1,9 +1,9 @@ 
 # SPDX-License-Identifier: BSD-3-Clause
 # Copyright(c) 2022 Microsoft Corporation
 
-if not is_linux or not dpdk_conf.has('RTE_ARCH_X86')
+if not is_linux or not (dpdk_conf.has('RTE_ARCH_X86') or dpdk_conf.has('RTE_ARCH_ARM64'))
     build = false
-    reason = 'only supported on x86 Linux'
+    reason = 'only supported on x86 or arm64 Linux'
     subdir_done()
 endif
 
diff --git a/drivers/net/mana/mp.c b/drivers/net/mana/mp.c
index 738487f65a..34b45ed832 100644
--- a/drivers/net/mana/mp.c
+++ b/drivers/net/mana/mp.c
@@ -5,6 +5,7 @@ 
 #include <rte_malloc.h>
 #include <ethdev_driver.h>
 #include <rte_log.h>
+#include <stdlib.h>
 
 #include <infiniband/verbs.h>