diff mbox

Pull request: validation last modules from platform side

Message ID 55BA938C.5050107@linaro.org
State New
Headers show

Commit Message

Ivan Khoronzhuk July 30, 2015, 9:13 p.m. UTC 
Hi Christophe,

Could you please clarify how to build validation tests from platform
side. As I see all tests stop to build only because i do:


The test/Makefile.am is called because subdir "test" is in root
Makefile.am

You've added TESTS list in
platform/linux-generic/test/Makefile.am
but for now they are built only because test/Makefile.am contains subdir
"validation". Is it supposed that it's built now from platform side
already or it's still under development?


On 08.07.15 18:07, Christophe Milard wrote:
> Hi Maxim,
> Hope this is what you need for a proper pull request.
> Let me know if anything is wrong/could be improved in this pull-req.
>
> All these patches have been reviewed by Stuart and do -hopefully-
> not contain any big surprises. just the continuation of the
> "run from platform side" for the last test/validation modules.
> Thanks,
> Christophe.
>
> The following changes since commit 493b618e8ad3184a97910485ad782732ec382979:
>
>    linux-generic: schedule: fix double free (2015-07-03 15:09:24 +0300)
>
> are available in the git repository at:
>
>    https://git.linaro.org/people/christophe.milard/odp.git test_module_timer
>
> for you to fetch changes up to 46fc2ab3225159d2bababe0d7f020009ebbb69f0:
>
>    validation: call to timer_main from platform (2015-07-08 16:18:41 +0200)
>
> ----------------------------------------------------------------
> Christophe Milard (20):
>        validation: removed errno from odp_ver_abt_log_dbg
>        validation: cosmetic change in odp_errno.c
>        validation: new module errno
>        validation: renaming in odp_errno.c
>        validation: renaming system to time
>        validation: single src for odp_ver_abt_log_dbg
>        validation: mv odp_ver_abt_log_dbg to system
>        validation: own main in odp_system
>        validation: renaming in system.c
>        validation: creating own dir and lib for system
>        validation: call to system_main from platform side
>        validation: own main in odp_shared_memory.c
>        validation: renaming in odp_shared_memory.c
>        validation: creating own dir and lib for shmem
>        validation: call to shmem_main from platform side.
>        validation: own main in odp_timer.c
>        validation: renaming in odp_timer.c
>        validation: cosmetic changes in odp_timer.c
>        validation: creating own dir and lib for timer
>        validation: call to timer_main from platform
>
>   configure.ac                                       |   6 +-
>   platform/linux-generic/test/Makefile.am            |   8 +-
>   test/validation/.gitignore                         |   3 -
>   test/validation/Makefile.am                        |  27 ++---
>   test/validation/errno/.gitignore                   |   2 +
>   test/validation/errno/Makefile.am                  |   8 ++
>   .../{ver_abt_log_dbg/odp_errno.c => errno/errno.c} |  17 +++-
>   test/validation/errno/errno.h                      |   7 ++
>   test/validation/errno/errno_main.c                 |  12 +++
>   test/validation/shmem/.gitignore                   |   2 +
>   test/validation/shmem/Makefile.am                  |   8 ++
>   .../{odp_shared_memory.c => shmem/shmem.c}         |  15 ++-
>   test/validation/shmem/shmem.h                      |   7 ++
>   test/validation/shmem/shmem_main.c                 |  12 +++
>   test/validation/system/.gitignore                  |   2 +-
>   test/validation/system/system.c                    | 112 +++++++++++++--------
>   test/validation/system/system.h                    |   2 +-
>   test/validation/system/system_main.c               |   2 +-
>   test/validation/time/.gitignore                    |   2 +
>   test/validation/time/Makefile.am                   |   8 ++
>   test/validation/time/time.c                        |  79 +++++++++++++++
>   test/validation/time/time.h                        |   7 ++
>   test/validation/time/time_main.c                   |  12 +++
>   test/validation/timer/.gitignore                   |   2 +
>   test/validation/timer/Makefile.am                  |   8 ++
>   test/validation/{odp_timer.c => timer/timer.c}     |  63 ++++++------
>   test/validation/timer/timer.h                      |   7 ++
>   test/validation/timer/timer_main.c                 |  12 +++
>   test/validation/ver_abt_log_dbg/odp_errno.h        |   9 --
>   test/validation/ver_abt_log_dbg/odp_system.c       |  94 -----------------
>   test/validation/ver_abt_log_dbg/odp_system.h       |   9 --
>   .../ver_abt_log_dbg/odp_ver_abt_log_dbg.c          |  16 ---
>   32 files changed, 343 insertions(+), 237 deletions(-)
>   create mode 100644 test/validation/errno/.gitignore
>   create mode 100644 test/validation/errno/Makefile.am
>   rename test/validation/{ver_abt_log_dbg/odp_errno.c => errno/errno.c} (55%)
>   create mode 100644 test/validation/errno/errno.h
>   create mode 100644 test/validation/errno/errno_main.c
>   create mode 100644 test/validation/shmem/.gitignore
>   create mode 100644 test/validation/shmem/Makefile.am
>   rename test/validation/{odp_shared_memory.c => shmem/shmem.c} (86%)
>   create mode 100644 test/validation/shmem/shmem.h
>   create mode 100644 test/validation/shmem/shmem_main.c
>   create mode 100644 test/validation/time/.gitignore
>   create mode 100644 test/validation/time/Makefile.am
>   create mode 100644 test/validation/time/time.c
>   create mode 100644 test/validation/time/time.h
>   create mode 100644 test/validation/time/time_main.c
>   create mode 100644 test/validation/timer/.gitignore
>   create mode 100644 test/validation/timer/Makefile.am
>   rename test/validation/{odp_timer.c => timer/timer.c} (90%)
>   create mode 100644 test/validation/timer/timer.h
>   create mode 100644 test/validation/timer/timer_main.c
>   delete mode 100644 test/validation/ver_abt_log_dbg/odp_errno.h
>   delete mode 100644 test/validation/ver_abt_log_dbg/odp_system.c
>   delete mode 100644 test/validation/ver_abt_log_dbg/odp_system.h
>   delete mode 100644 test/validation/ver_abt_log_dbg/odp_ver_abt_log_dbg.c
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
diff mbox

Patch

--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -1,5 +1,5 @@ 
  SUBDIRS = api_test performance miscellaneous

  if cunit_support
-    SUBDIRS += validation
+#    SUBDIRS += validation
  endif