[dpdk-dev] [PATCH] eal:ppc: fix incorrect ifdef for ppc_64

Burakov, Anatoly anatoly.burakov at intel.com
Thu Oct 17 18:18:17 CEST 2019


On 16-Oct-19 9:45 PM, David Christensen wrote:
>>> An ifdef present in eal_memory.c references "RTE_ARCH_PPC64" when
>>> it should actually use "RTE_ARCH_PPC_64".  Simple testing revealed
>>> that both the PPC_64 and non-PPC_64 versions of the code involved
>>> work, but the PPC_64 version of the code is retained to be
>>> consistent with other instances in the same file where mmapped
>>> memory is accessed in reverse order on Power platforms.
>>
>> The change itself is not that scary, but just reading this commitlog I
>> fail to see the impact for an application.
>> Can you share some light?
>>
> 
> As far as I can tell there is no impact on any applications.  The old 
> code, which walked through the list in a forward direction, worked 
> perfectly well with testpmd and DPDK pktgen applications on Power systems.
> 
> With the ifdef fixed, the core walks the list in the reverse direction 
> as intended, the code still worked (i.e. no errors or problems were 
> observed in the same test applications).
> 
> I'm not completely familiar with why memseg lists must be traversed in 
> the reverse direction for Power systems.  It might be something specific 
> to Power 8 systems which I'm not actually supporting on DPDK, only the 
> Power 9 systems that I use for for development and testing.
> 
> Dave
> 

If the code makes no difference anyway, should we just take it out so?

-- 
Thanks,
Anatoly


More information about the dev mailing list