add more details how I tested mips and ppc codes: on my x86 machine, ./configure --host=powerpc, then arch/powerpc/* are compiled and create a pseudo /tmp/cpuinfo following the powerpc format, and let my code to parse this file instead of the /proc/cpuinfo on my x86
for mips, I did the same thing to verify the code (only have to comment out some assembly code in odp_cpu_arch.c to pass compilation) by this way I tested my patches on my x86, it should work I think. On 20 January 2016 at 17:20, <hongbo.zh...@linaro.org> wrote: > From: Hongbo Zhang <hongbo.zh...@linaro.org> > > v6->v7 changes: > - update symlink in patch 7 > > v5->v6 changes: > - eliminate checkpatch.pl warnings > > v4->v5 changes: > - rebase to latest api-next branch > > v3->v4 changes: > - update patch 9 of platform naming > - add new patch 12 to rename cpu_hz to cpu_hz_max > > v2->v3 changes: > use "api: cpu:" tag in patch 8/11 title instead of "linux-generic: sysinfo" > > v1->v2 changes: > - don't create arch/arm/ since there isn't implementation now, use arch/linux > as default choice > - create symlink to arch/linux/odp_cpu_cycles.c for powerpc, if absent this > arch cannot be compiled > - add some clean-ups patches 8~11, these patches are against the previous > ones > so send them together for better review and merge. > > v1 notes: > This patch set separates the CPU info codes into their own platform sepcific > files. > It is common sence that the top general layer call an uniform interface to > initialize some plarform specific data structures, and this uniform interface > is implemented in their own platform specific files. > This patch set makes it. > > Hongbo Zhang (12): > linux-generic: sysinfo: move cpu_arch_str to odp_system_info_t > linux-generic: sysinfo: use uniform call odp_sysinfo_parser > linux-generic: sysinfo: rename odp_cpu_hz_current with odp_ prefix > linux-generic: sysinfo: move x86 system info codes to its plarform > file > linux-generic: sysinfo: move MIPS system info codes to its plarform > file > linux-generic: sysinfo: move ARM system info codes to default arch > file > linux-generic: sysinfo: move PowerPC system info codes to its plarform > file > api: cpu: make frequency API return 0 on failure > linux-generic: sysinfo: set values for cpu_arch_str > linux-generic: sysinfo: apply per-CPU implementation to MIPS > linux-generic: sysinfo: apply per-CPU implementation to PowerPC > linux-generic: sysinfo: rename variable cpu_hz to cpu_hz_max > > configure.ac | 1 + > include/odp/api/cpu.h | 4 + > platform/linux-generic/Makefile.am | 10 +- > .../linux-generic/arch/linux/odp_sysinfo_parse.c | 19 ++ > .../linux-generic/arch/mips64/odp_sysinfo_parse.c | 64 ++++++ > platform/linux-generic/arch/powerpc/odp/cpu_arch.h | 1 + > platform/linux-generic/arch/powerpc/odp_cpu_arch.c | 1 + > .../linux-generic/arch/powerpc/odp_sysinfo_parse.c | 63 ++++++ > .../linux-generic/arch/x86/odp_sysinfo_parse.c | 73 +++++++ > platform/linux-generic/include/odp_internal.h | 7 +- > platform/linux-generic/odp_system_info.c | 220 > +-------------------- > 11 files changed, 246 insertions(+), 217 deletions(-) > create mode 100644 platform/linux-generic/arch/linux/odp_sysinfo_parse.c > create mode 100644 platform/linux-generic/arch/mips64/odp_sysinfo_parse.c > create mode 120000 platform/linux-generic/arch/powerpc/odp/cpu_arch.h > create mode 120000 platform/linux-generic/arch/powerpc/odp_cpu_arch.c > create mode 100644 platform/linux-generic/arch/powerpc/odp_sysinfo_parse.c > create mode 100644 platform/linux-generic/arch/x86/odp_sysinfo_parse.c > > -- > 2.1.4 > _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp