-----Original Message----- > Date: Fri, 12 Oct 2018 10:24:16 +0800 > From: Chao Zhu <[email protected]> > To: 'Jerin Jacob' <[email protected]> > CC: [email protected], [email protected], [email protected], > [email protected] > Subject: 答复: [dpdk-dev] [PATCH] eal/ppc64: add support for rte pause > X-Mailer: Microsoft Outlook 16.0 > > > -----邮件原件----- > 发件人: Jerin Jacob <[email protected]> > 发送时间: 2018年10月7日 14:19 > 收件人: Chao Zhu <[email protected]> > 抄送: [email protected]; [email protected]; [email protected]; > [email protected]; Jerin Jacob <[email protected]> > 主题: [dpdk-dev] [PATCH] eal/ppc64: add support for rte pause > > Add support for rte_pause() implementation for ppc64. > > Signed-off-by: Jerin Jacob <[email protected]> > --- > > 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 <[email protected]>
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. > >
