* Re: [dpdk-dev] [PATCH] config/arm: fix typo in meson config
2021-02-01 22:41 ` [dpdk-dev] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
@ 2021-02-01 23:22 ` Honnappa Nagarahalli
2021-02-02 2:19 ` Ruifeng Wang
2021-02-02 7:12 ` Juraj Linkeš
2 siblings, 0 replies; 8+ messages in thread
From: Honnappa Nagarahalli @ 2021-02-01 23:22 UTC (permalink / raw)
To: Ferruh Yigit, Jan Viktorin, Ruifeng Wang, jerinj,
Bruce Richardson, Juraj Linkeš,
Vimal Chungath
Cc: dev, thomas, stable, Thierry Herbelot, nd, Honnappa Nagarahalli, nd
<snip>
> Subject: [PATCH] config/arm: fix typo in meson config
>
> s/implementor/implementer
>
> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
> Fixes: c51401832147 ("config/arm: add Graviton2")
> Cc: stable@dpdk.org
>
> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
Thanks Ferruh. I remember discussing this on the list. I am fine with 'implementEr' as that is what is used in Arm public specs.
> ---
> Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: Ruifeng Wang <ruifeng.wang@arm.com>
> Cc: vcchunga@amazon.com
> ---
> config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
> config/arm/arm64_graviton2_linux_gcc | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> index aa5ee0132065..ae3fb30c5da8 100644
> --- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> +++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> @@ -14,7 +14,7 @@ cpu = 'armv8-a'
> endian = 'little'
>
> [properties]
> -implementor_id = 'generic'
> -implementor_pn = 'default'
> +implementer_id = 'generic'
> +implementer_pn = 'default'
> c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
> c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-
> toolchain=/usr'] diff --git a/config/arm/arm64_graviton2_linux_gcc
> b/config/arm/arm64_graviton2_linux_gcc
> index cfe2397975fc..123f14eed29e 100644
> --- a/config/arm/arm64_graviton2_linux_gcc
> +++ b/config/arm/arm64_graviton2_linux_gcc
> @@ -13,8 +13,8 @@ cpu = 'armv8-a'
> endian = 'little'
>
> [properties]
> -implementor_id = '0x41'
> -implementor_pn = '0xd0c'
> +implementer_id = '0x41'
> +implementer_pn = '0xd0c'
> max_lcores = 64
> max_numa_nodes = 1
> numa = false
> --
> 2.29.2
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-dev] [PATCH] config/arm: fix typo in meson config
2021-02-01 22:41 ` [dpdk-dev] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
2021-02-01 23:22 ` Honnappa Nagarahalli
@ 2021-02-02 2:19 ` Ruifeng Wang
2021-02-02 8:08 ` Ferruh Yigit
2021-02-02 7:12 ` Juraj Linkeš
2 siblings, 1 reply; 8+ messages in thread
From: Ruifeng Wang @ 2021-02-02 2:19 UTC (permalink / raw)
To: Ferruh Yigit, Jan Viktorin, jerinj, Bruce Richardson,
Juraj Linkeš,
Honnappa Nagarahalli, Vimal Chungath
Cc: dev, thomas, stable, Thierry Herbelot, nd
> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> Sent: Tuesday, February 2, 2021 6:42 AM
> To: Jan Viktorin <viktorin@rehivetech.com>; Ruifeng Wang
> <Ruifeng.Wang@arm.com>; jerinj@marvell.com; Bruce Richardson
> <bruce.richardson@intel.com>; Juraj Linkeš <juraj.linkes@pantheon.tech>;
> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; Vimal Chungath
> <vcchunga@amazon.com>
> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org;
> thomas@monjalon.net; stable@dpdk.org; Thierry Herbelot
> <thierry.herbelot@6wind.com>
> Subject: [PATCH] config/arm: fix typo in meson config
>
> s/implementor/implementer
>
> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
> Fixes: c51401832147 ("config/arm: add Graviton2")
'implementor' -> 'implementer' change was made in 7870ae8994c6.
It seems 7870ae8994c6 missed Graviton2 config file which was added earlier.
With the fix tag update,
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
> Cc: stable@dpdk.org
>
> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
> Cc: Ruifeng Wang <ruifeng.wang@arm.com>
> Cc: vcchunga@amazon.com
> ---
> config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
> config/arm/arm64_graviton2_linux_gcc | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> index aa5ee0132065..ae3fb30c5da8 100644
> --- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
> +++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
> @@ -14,7 +14,7 @@ cpu = 'armv8-a'
> endian = 'little'
>
> [properties]
> -implementor_id = 'generic'
> -implementor_pn = 'default'
> +implementer_id = 'generic'
> +implementer_pn = 'default'
> c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
> c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-
> toolchain=/usr'] diff --git a/config/arm/arm64_graviton2_linux_gcc
> b/config/arm/arm64_graviton2_linux_gcc
> index cfe2397975fc..123f14eed29e 100644
> --- a/config/arm/arm64_graviton2_linux_gcc
> +++ b/config/arm/arm64_graviton2_linux_gcc
> @@ -13,8 +13,8 @@ cpu = 'armv8-a'
> endian = 'little'
>
> [properties]
> -implementor_id = '0x41'
> -implementor_pn = '0xd0c'
> +implementer_id = '0x41'
> +implementer_pn = '0xd0c'
> max_lcores = 64
> max_numa_nodes = 1
> numa = false
> --
> 2.29.2
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-dev] [PATCH] config/arm: fix typo in meson config
2021-02-02 2:19 ` Ruifeng Wang
@ 2021-02-02 8:08 ` Ferruh Yigit
0 siblings, 0 replies; 8+ messages in thread
From: Ferruh Yigit @ 2021-02-02 8:08 UTC (permalink / raw)
To: Ruifeng Wang, Jan Viktorin, jerinj, Bruce Richardson,
Juraj Linkeš,
Honnappa Nagarahalli, Vimal Chungath
Cc: dev, thomas, stable, Thierry Herbelot, nd
On 2/2/2021 2:19 AM, Ruifeng Wang wrote:
>> -----Original Message-----
>> From: Ferruh Yigit <ferruh.yigit@intel.com>
>> Sent: Tuesday, February 2, 2021 6:42 AM
>> To: Jan Viktorin <viktorin@rehivetech.com>; Ruifeng Wang
>> <Ruifeng.Wang@arm.com>; jerinj@marvell.com; Bruce Richardson
>> <bruce.richardson@intel.com>; Juraj Linkeš <juraj.linkes@pantheon.tech>;
>> Honnappa Nagarahalli <Honnappa.Nagarahalli@arm.com>; Vimal Chungath
>> <vcchunga@amazon.com>
>> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org;
>> thomas@monjalon.net; stable@dpdk.org; Thierry Herbelot
>> <thierry.herbelot@6wind.com>
>> Subject: [PATCH] config/arm: fix typo in meson config
>>
>> s/implementor/implementer
>>
>> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
>> Fixes: c51401832147 ("config/arm: add Graviton2")
>
> 'implementor' -> 'implementer' change was made in 7870ae8994c6.
> It seems 7870ae8994c6 missed Graviton2 config file which was added earlier.
>
Right, I missed the rename patch, so it should be:
Fixes: 7870ae8994c6 ("config/arm: rename variables")
And that rename patch does 'implementor_pn' -> 'part_number', so I will update
this patch to do the same.
> With the fix tag update,
> Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
>
>> Cc: stable@dpdk.org
>>
>> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>> Cc: Juraj Linkeš <juraj.linkes@pantheon.tech>
>> Cc: Ruifeng Wang <ruifeng.wang@arm.com>
>> Cc: vcchunga@amazon.com
>> ---
>> config/arm/arm64_armv8_linux_clang_ubuntu1804 | 4 ++--
>> config/arm/arm64_graviton2_linux_gcc | 4 ++--
>> 2 files changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> b/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> index aa5ee0132065..ae3fb30c5da8 100644
>> --- a/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> +++ b/config/arm/arm64_armv8_linux_clang_ubuntu1804
>> @@ -14,7 +14,7 @@ cpu = 'armv8-a'
>> endian = 'little'
>>
>> [properties]
>> -implementor_id = 'generic'
>> -implementor_pn = 'default'
>> +implementer_id = 'generic'
>> +implementer_pn = 'default'
>> c_args = ['-target', 'aarch64-linux-gnu', '--sysroot', '/usr/aarch64-linux-gnu']
>> c_link_args = ['-target', 'aarch64-linux-gnu', '-fuse-ld=lld', '--gcc-
>> toolchain=/usr'] diff --git a/config/arm/arm64_graviton2_linux_gcc
>> b/config/arm/arm64_graviton2_linux_gcc
>> index cfe2397975fc..123f14eed29e 100644
>> --- a/config/arm/arm64_graviton2_linux_gcc
>> +++ b/config/arm/arm64_graviton2_linux_gcc
>> @@ -13,8 +13,8 @@ cpu = 'armv8-a'
>> endian = 'little'
>>
>> [properties]
>> -implementor_id = '0x41'
>> -implementor_pn = '0xd0c'
>> +implementer_id = '0x41'
>> +implementer_pn = '0xd0c'
>> max_lcores = 64
>> max_numa_nodes = 1
>> numa = false
>> --
>> 2.29.2
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [dpdk-dev] [PATCH] config/arm: fix typo in meson config
2021-02-01 22:41 ` [dpdk-dev] [PATCH] config/arm: fix typo in meson config Ferruh Yigit
2021-02-01 23:22 ` Honnappa Nagarahalli
2021-02-02 2:19 ` Ruifeng Wang
@ 2021-02-02 7:12 ` Juraj Linkeš
2 siblings, 0 replies; 8+ messages in thread
From: Juraj Linkeš @ 2021-02-02 7:12 UTC (permalink / raw)
To: Ferruh Yigit, Jan Viktorin, Ruifeng Wang, Jerin Jacob,
Bruce Richardson, Honnappa Nagarahalli, Vimal Chungath
Cc: dev, Thomas Monjalon, stable, Thierry Herbelot
> -----Original Message-----
> From: Ferruh Yigit <ferruh.yigit@intel.com>
> Sent: Monday, February 1, 2021 11:42 PM
> To: Jan Viktorin <viktorin@rehivetech.com>; Ruifeng Wang
> <ruifeng.wang@arm.com>; Jerin Jacob <jerinj@marvell.com>; Bruce Richardson
> <bruce.richardson@intel.com>; Juraj Linkeš <juraj.linkes@pantheon.tech>;
> Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>; Vimal Chungath
> <vcchunga@amazon.com>
> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org; Thomas Monjalon
> <thomas@monjalon.net>; stable@dpdk.org; Thierry Herbelot
> <thierry.herbelot@6wind.com>
> Subject: [PATCH] config/arm: fix typo in meson config
>
> s/implementor/implementer
>
> Fixes: 3d01d65ba0d3 ("config: add aarch64 clang for Ubuntu 18.04")
> Fixes: c51401832147 ("config/arm: add Graviton2")
> Cc: stable@dpdk.org
>
> Reported-by: Thierry Herbelot <thierry.herbelot@6wind.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
Thanks for submitting this,
Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
^ permalink raw reply [flat|nested] 8+ messages in thread