[dpdk-dev] 答复: [PATCH] eal/ppc64: add support for rte pause
Jerin Jacob
jerin.jacob at caviumnetworks.com
Fri Oct 12 06:12:57 CEST 2018
-----Original Message-----
> Date: Fri, 12 Oct 2018 10:24:16 +0800
> From: Chao Zhu <chaozhu at linux.vnet.ibm.com>
> To: 'Jerin Jacob' <jerin.jacob at caviumnetworks.com>
> CC: dev at dpdk.org, thomas at monjalon.net, gowrishankar.m at linux.vnet.ibm.com,
> ola.liljedahl at arm.com
> Subject: 答复: [dpdk-dev] [PATCH] eal/ppc64: add support for rte pause
> X-Mailer: Microsoft Outlook 16.0
>
>
> -----邮件原件-----
> 发件人: Jerin Jacob <jerin.jacob at caviumnetworks.com>
> 发送时间: 2018年10月7日 14:19
> 收件人: Chao Zhu <chaozhu at linux.vnet.ibm.com>
> 抄送: dev at dpdk.org; thomas at monjalon.net; gowrishankar.m at linux.vnet.ibm.com;
> ola.liljedahl at arm.com; Jerin Jacob <jerin.jacob at caviumnetworks.com>
> 主题: [dpdk-dev] [PATCH] eal/ppc64: add support for rte pause
>
> Add support for rte_pause() implementation for ppc64.
>
> Signed-off-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
> ---
>
> The reference implementation for Linux's cpu_relax() for ppc64 is at
> https://elixir.bootlin.com/linux/latest/source/arch/powerpc/include/asm/proc
> essor.h#L440
>
> ---
> lib/librte_eal/common/include/arch/ppc_64/rte_pause.h | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/lib/librte_eal/common/include/arch/ppc_64/rte_pause.h
> b/lib/librte_eal/common/include/arch/ppc_64/rte_pause.h
> index 8bd835764..16e47ce22 100644
> --- a/lib/librte_eal/common/include/arch/ppc_64/rte_pause.h
> +++ b/lib/librte_eal/common/include/arch/ppc_64/rte_pause.h
> @@ -9,10 +9,17 @@
> extern "C" {
> #endif
>
> +#include "rte_atomic.h"
> +
> #include "generic/rte_pause.h"
>
> static inline void rte_pause(void)
> {
> + /* Set hardware multi-threading low priority */
> + asm volatile("or 1,1,1");
> + /* Set hardware multi-threading medium priority */
> + asm volatile("or 2,2,2");
> + rte_compiler_barrier();
> }
>
> #ifdef __cplusplus
> --
> 2.19.0
>
> Acked-by: Chao Zhu <chaozhu at linux.vnet.ibm.com>
Thomas,
Just noticed, due to some reason, The above Acked-by: is not reflected in patchwork.
http://patches.dpdk.org/patch/46195/
Probably because, it was not a "inline" reply which patchwork couldn't parse it.
>
>
More information about the dev
mailing list