Message: 4 Date: Wed, 24 Feb 2021 11:24:27 +0100 From: Thomas Monjalon thomas@monjalon.net To: oulijun oulijun@huawei.com Cc: ferruh.yigit@intel.com, dev@dpdk.org, linuxarm@openeuler.org, juraj.linkes@pantheon.tech, bruce.richardson@intel.com Subject: Re: [dpdk-dev] [PATCH 1/2] config/arm: fix Hisilicon kunpeng920 SoC build Message-ID: 5309744.5Ik2CEZAj8@thomas Content-Type: text/plain; charset="UTF-8"
24/02/2021 10:17, oulijun:
? 2021/2/24 15:41, Thomas Monjalon ??:
24/02/2021 02:34, oulijun:
? 2021/2/10 17:41, Thomas Monjalon ??:
03/02/2021 13:46, Lijun Ou:
From: Chengchang Tang tangchengchang@huawei.com
Because of the '9ca2f16' have merged, the current hns3 pmd driver can not be directly complied on the kunpeng920 server board. Therefore, we need to fix the meson build. Besides, add kunpeng 920 SoC meson cross compile target.
Fixes: 9ca2f16faa7f ("config/arm: isolate generic build")
Why do you think this patch is fixing the one above? It looks just a new config, not a fix. Am I missing something?
I'm sorry to see you so late. In the meantime, we are celebrating the Spring Festival. This patch fixes the problem. If the patch is not added, the latest version cannot be directly compiled on the Kunpeng 930 server board.In addition, the cross compilation configuration file is added.
Please can you explain what was removed which breaks your compilation?
I understand that implementation_id is identified in 9ca2f16. If the configuration file does not contain implementation_id, it will be displayed as unsupported.
The error as follows: config/arm/meson.build:227:2: ERROR: Problem encountered: Unsupported Arm implementer: 0x48. Please add support for it or use the generic (-Dmachine=generic) build.
Besides, we use the -Dmachine=generic, the bug is not resolved. the cmd as: meson build or meson -Dmachine=generic build
What is the result of a generic build?
Hello,
Following dpdk commit 91c730fd4e093cd, the generic compilation for Arm is selected with '-Dmachine=default'
The comments in config/arm/meson.build should be updated to follow the change in default machine name: https://git.dpdk.org/dpdk/tree/config/arm/meson.build#n241 https://git.dpdk.org/dpdk/tree/config/arm/meson.build#n256
Regards
Thierry