Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-10 Thread Ananyev, Konstantin
Hi Jia, > -Original Message- > From: Jia He [mailto:hejia...@gmail.com] > Sent: Friday, November 10, 2017 2:06 AM > To: Ananyev, Konstantin ; Jianbo Liu > > Cc: Richardson, Bruce ; > jerin.ja...@caviumnetworks.com; dev@dpdk.org; olivier.m...@6wind.com; > hemant.agra...@nxp.com; jia...@h

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-09 Thread Jianbo Liu
The 11/10/2017 10:06, Jia He wrote: > > > On 11/9/2017 5:38 PM, Ananyev, Konstantin Wrote: > > > >>-Original Message- > >>From: Jianbo Liu [mailto:jianbo@arm.com] > >>Sent: Thursday, November 9, 2017 4:56 AM > >>To: Jia He > >>Cc: Richardson, Bruce ; > >>jerin.ja...@caviumnetworks.com

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-09 Thread Jia He
On 11/9/2017 5:38 PM, Ananyev, Konstantin Wrote: -Original Message- From: Jianbo Liu [mailto:jianbo@arm.com] Sent: Thursday, November 9, 2017 4:56 AM To: Jia He Cc: Richardson, Bruce ; jerin.ja...@caviumnetworks.com; dev@dpdk.org; olivier.m...@6wind.com; Ananyev, Konstantin ; he

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-09 Thread Ananyev, Konstantin
> -Original Message- > From: Jianbo Liu [mailto:jianbo@arm.com] > Sent: Thursday, November 9, 2017 9:58 AM > To: Ananyev, Konstantin > Cc: Jia He ; Richardson, Bruce > ; jerin.ja...@caviumnetworks.com; dev@dpdk.org; > olivier.m...@6wind.com; hemant.agra...@nxp.com; jia...@hxt-semite

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-09 Thread Jianbo Liu
The 11/09/2017 09:38, Ananyev, Konstantin wrote: > > > > -Original Message- > > From: Jianbo Liu [mailto:jianbo@arm.com] > > Sent: Thursday, November 9, 2017 4:56 AM > > To: Jia He > > Cc: Richardson, Bruce ; > > jerin.ja...@caviumnetworks.com; dev@dpdk.org; olivier.m...@6wind.com; >

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-09 Thread Ananyev, Konstantin
> -Original Message- > From: Jianbo Liu [mailto:jianbo@arm.com] > Sent: Thursday, November 9, 2017 4:56 AM > To: Jia He > Cc: Richardson, Bruce ; > jerin.ja...@caviumnetworks.com; dev@dpdk.org; olivier.m...@6wind.com; > Ananyev, Konstantin ; hemant.agra...@nxp.com; > jia...@hxt-sem

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-08 Thread Jianbo Liu
The 11/09/2017 12:43, Jia He wrote: > Hi Jianbo > > > On 11/9/2017 11:21 AM, Jianbo Liu Wrote: > >The 11/09/2017 11:14, Jia He wrote: > >> > >>On 11/9/2017 9:22 AM, Jia He Wrote: > >>>Hi Bruce > >>> > >>> > >>>On 11/8/2017 6:28 PM, Bruce Richardson Wrote: > On Wed, Nov 08, 2017 at 06:17:10AM +0

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-08 Thread Jia He
Hi Jianbo On 11/9/2017 11:21 AM, Jianbo Liu Wrote: The 11/09/2017 11:14, Jia He wrote: On 11/9/2017 9:22 AM, Jia He Wrote: Hi Bruce On 11/8/2017 6:28 PM, Bruce Richardson Wrote: On Wed, Nov 08, 2017 at 06:17:10AM +, Jia He wrote: for the code as follows: if (condition) rte_smp_r

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-08 Thread Jianbo Liu
The 11/09/2017 11:14, Jia He wrote: > > > On 11/9/2017 9:22 AM, Jia He Wrote: > >Hi Bruce > > > > > >On 11/8/2017 6:28 PM, Bruce Richardson Wrote: > >>On Wed, Nov 08, 2017 at 06:17:10AM +, Jia He wrote: > >>>for the code as follows: > >>>if (condition) > >>>rte_smp_rmb(); > >>>else > >>>   

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-08 Thread Jia He
On 11/9/2017 9:22 AM, Jia He Wrote: Hi Bruce On 11/8/2017 6:28 PM, Bruce Richardson Wrote: On Wed, Nov 08, 2017 at 06:17:10AM +, Jia He wrote: for the code as follows: if (condition) rte_smp_rmb(); else rte_smp_wmb(); Without this patch, compiler will report this error: error: '

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-08 Thread Jia He
Hi Bruce On 11/8/2017 6:28 PM, Bruce Richardson Wrote: On Wed, Nov 08, 2017 at 06:17:10AM +, Jia He wrote: for the code as follows: if (condition) rte_smp_rmb(); else rte_smp_wmb(); Without this patch, compiler will report this error: error: 'else' without a previous 'if'

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-08 Thread Bruce Richardson
On Wed, Nov 08, 2017 at 06:17:10AM +, Jia He wrote: > for the code as follows: > if (condition) > rte_smp_rmb(); > else > rte_smp_wmb(); > Without this patch, compiler will report this error: > error: 'else' without a previous 'if' > > Signed-off-by: Jia He > Signed-off-by: jia...

Re: [dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-07 Thread Jerin Jacob
-Original Message- > Date: Wed, 8 Nov 2017 06:17:10 + > From: Jia He > To: jerin.ja...@caviumnetworks.com, dev@dpdk.org, olivier.m...@6wind.com > Cc: konstantin.anan...@intel.com, bruce.richard...@intel.com, > jianbo@arm.com, hemant.agra...@nxp.com, Jia He , > jia...@hxt-semitec

[dpdk-dev] [PATCH 1/3] eal/arm64: remove the braces {} for dmb(), dsb()

2017-11-07 Thread Jia He
for the code as follows: if (condition) rte_smp_rmb(); else rte_smp_wmb(); Without this patch, compiler will report this error: error: 'else' without a previous 'if' Signed-off-by: Jia He Signed-off-by: jia...@hxt-semitech.com --- lib/librte_eal/common/include/arch/arm/rte_atomic