For the series:

Reviewed-by: Christophe Milard <christophe.mil...@linaro.org>


On 18 January 2017 at 23:01, Mike Holmes <mike.hol...@linaro.org> wrote:
> Starting with some clean up, rename the Linux specific files and then move 
> them
> to a platform specific directory. A new configure option is introduced
> "with-helper-platform", this defaults to match the existing odp selector
> "with-platform" so that there is no operational change.
>
> With the move made the default is to build only the portable helper API, if 
> the
> legacy and previously unused Linux thread APIs are needed they can be built in
> with --enable-helper-extn
>
> Then remove the dependence on the test directory.
>
> v2:
> sort some lists into alphabetical order (Maxim)
> addtional code simplification removing a .h helper(Christophe)
>
> v3:
> rebase
> whitespace error is there due to matching existing formatting in file
>
> Mike Holmes (4):
>   configure: use helper configure.m4
>   helper: cleanup Linux rename to thread
>   helper: move thread implementation under platform
>   helper: remove dependence on test dir
>
>  configure.ac                                       |  19 +-
>  example/Makefile.inc                               |   2 +-
>  example/classifier/odp_classifier.c                |   2 +-
>  example/generator/odp_generator.c                  |   2 +-
>  example/ipsec/odp_ipsec.c                          |   2 +-
>  example/l2fwd_simple/odp_l2fwd_simple.c            |   2 +-
>  example/l3fwd/odp_l3fwd.c                          |   2 +-
>  example/packet/odp_pktio.c                         |   2 +-
>  example/switch/odp_switch.c                        |   2 +-
>  example/time/time_global_test.c                    |   5 +-
>  example/timer/odp_timer_test.c                     |   2 +-
>  helper/Makefile.am                                 |  22 +-
>  .../helper/platform/linux-generic/threads_extn.h   | 112 ++++++++
>  helper/include/odp/helper/{linux.h => threads.h}   |  78 -----
>  helper/m4/configure.m4                             |  14 +
>  helper/platform/linux-generic/thread.c             | 313 
> +++++++++++++++++++++
>  helper/test/Makefile.am                            |  43 ++-
>  helper/test/chksum.c                               |  18 +-
>  helper/test/cuckootable.c                          |   4 +-
>  helper/test/iplookuptable.c                        |   4 +-
>  helper/test/linux-generic/Makefile.am              |   5 +
>  helper/test/{ => linux-generic}/process.c          |  16 +-
>  helper/test/{ => linux-generic}/thread.c           |  16 +-
>  helper/test/odpthreads.c                           |  20 +-
>  helper/test/parse.c                                |  86 +++---
>  helper/test/table.c                                |  12 +-
>  helper/{linux.c => threads.c}                      | 240 +---------------
>  ...inux.pc.in => libodphelper-linux-generic.pc.in} |   4 +-
>  test/Makefile.inc                                  |   2 +-
>  test/common_plat/common/odp_cunit_common.c         |   2 +-
>  .../common_plat/miscellaneous/odp_api_from_cpp.cpp |   2 +-
>  test/common_plat/performance/odp_crypto.c          |   2 +-
>  test/common_plat/performance/odp_l2fwd.c           |   2 +-
>  test/common_plat/performance/odp_pktio_perf.c      |   2 +-
>  test/common_plat/performance/odp_sched_latency.c   |   2 +-
>  test/common_plat/performance/odp_scheduling.c      |   2 +-
>  test/common_plat/validation/api/Makefile.inc       |   2 +-
>  test/common_plat/validation/api/timer/timer.c      |   2 +-
>  test/linux-generic/Makefile.inc                    |   2 +-
>  test/linux-generic/mmap_vlan_ins/mmap_vlan_ins.c   |   2 +-
>  test/linux-generic/pktio_ipc/ipc_common.h          |   2 +-
>  test/linux-generic/ring/ring_stress.c              |   2 +-
>  42 files changed, 633 insertions(+), 444 deletions(-)
>  create mode 100644 
> helper/include/odp/helper/platform/linux-generic/threads_extn.h
>  rename helper/include/odp/helper/{linux.h => threads.h} (74%)
>  create mode 100644 helper/platform/linux-generic/thread.c
>  create mode 100644 helper/test/linux-generic/Makefile.am
>  rename helper/test/{ => linux-generic}/process.c (84%)
>  rename helper/test/{ => linux-generic}/thread.c (84%)
>  rename helper/{linux.c => threads.c} (67%)
>  rename pkgconfig/{libodphelper-linux.pc.in => 
> libodphelper-linux-generic.pc.in} (72%)
>
> --
> 2.9.3

Reply via email to