RE: [PATCH v2] eal: avoid issues in macro expansion of alignment

2023-07-14 Thread Morten Brørup
...@marvell.com Emne: Re: [PATCH v2] eal: avoid issues in macro expansion of alignment On Wed, 5 Jul 2023 00:16:50 +0200Morten Brørup wrote:> > From: Stephen Hemminger [mailto:step...@networkplumber.org]> > Sent: Tuesday, 4 July 2023 18.01> > > > On Tue, 4 Jul 2023 10:43:40 +0200&

Re: [PATCH v2] eal: avoid issues in macro expansion of alignment

2023-07-14 Thread Stephen Hemminger
On Wed, 5 Jul 2023 00:16:50 +0200 Morten Brørup wrote: > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Tuesday, 4 July 2023 18.01 > > > > On Tue, 4 Jul 2023 10:43:40 +0200 > > Morten Brørup wrote: > > > > > > From: Stephen Hemminger [mailto:step...@networkplumber.or

RE: [PATCH v2] eal: avoid issues in macro expansion of alignment

2023-07-04 Thread Morten Brørup
> From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Tuesday, 4 July 2023 18.01 > > On Tue, 4 Jul 2023 10:43:40 +0200 > Morten Brørup wrote: > > > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > > Sent: Tuesday, 4 July 2023 01.24 > > > > > > RTE_ALIGN_MUL_NEAR

Re: [PATCH v2] eal: avoid issues in macro expansion of alignment

2023-07-04 Thread Stephen Hemminger
On Tue, 4 Jul 2023 10:43:40 +0200 Morten Brørup wrote: > > From: Stephen Hemminger [mailto:step...@networkplumber.org] > > Sent: Tuesday, 4 July 2023 01.24 > > > > RTE_ALIGN_MUL_NEAR is a macro so the cycle argument could > > get evaluated twice causing some potential skew. Fix by > > computing

RE: [PATCH v2] eal: avoid issues in macro expansion of alignment

2023-07-04 Thread Morten Brørup
> From: Stephen Hemminger [mailto:step...@networkplumber.org] > Sent: Tuesday, 4 July 2023 01.24 > > RTE_ALIGN_MUL_NEAR is a macro so the cycle argument could > get evaluated twice causing some potential skew. Fix by > computing value once. > > Suggested by patch to fix side effects. > > Fixes:

[PATCH v2] eal: avoid issues in macro expansion of alignment

2023-07-03 Thread Stephen Hemminger
RTE_ALIGN_MUL_NEAR is a macro so the cycle argument could get evaluated twice causing some potential skew. Fix by computing value once. Suggested by patch to fix side effects. Fixes: 5cbd14b3e5f9 ("eal: roundup TSC frequency when estimating") Cc: pbhagavat...@marvell.com Signed-off-by: Stephen H