Maxim You missed some dates

+++ b/platform/linux-generic/test/shmem/shmem_odp.c
@@ -0,0 +1,74 @@
+/* Copyright (c) 2014, Linaro Limited
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier:     BSD-3-Clause
+ */
+

+++ b/platform/linux-generic/test/shmem/shmem_odp.h
@@ -0,0 +1,7 @@
+/* Copyright (c) 2014, Linaro Limited
+ * All rights reserved.
+ *
+ * SPDX-License-Identifier:     BSD-3-Clause
+ */
+



On 23 March 2016 at 03:20, Maxim Uvarov <maxim.uva...@linaro.org> wrote:

> On 03/23/16 03:04, Bill Fischofer wrote:
>
>> Maxim, I had added my review in my March 2nd reply.
>>
>>
>   CC       shmem_linux.o
> ../../../../../platform/linux-generic/test/shmem/shmem_linux.c:48:25:
> fatal error: shmem_linux.h: No such file or directory
>  #include "shmem_linux.h"
>
>
> So final fix for latest patch is:
> --- a/platform/linux-generic/test/shmem/Makefile.am
> +++ b/platform/linux-generic/test/shmem/Makefile.am
> @@ -16,3 +16,5 @@ shmem_odp_CFLAGS = $(AM_CFLAGS) \
>                    $(INCODP)
>  shmem_odp_LDFLAGS = $(AM_LDFLAGS)
>  shmem_odp_LDADD = $(LIBCUNIT_COMMON) $(LIBODP)
> +
> +noinst_HEADERS = shmem_common.h shmem_linux.h shmem_odp.h
> diff --git a/platform/linux-generic/test/shmem/shmem_linux.c
> b/platform/linux-generic/test/shmem/shmem_linux.c
> index 516bbe4..12266cc 100644
> --- a/platform/linux-generic/test/shmem/shmem_linux.c
> +++ b/platform/linux-generic/test/shmem/shmem_linux.c
> @@ -1,4 +1,4 @@
> -/* Copyright (c) 2014, Linaro Limited
> +/* Copyright (c) 2016, Linaro Limited
>   * All rights reserved.
>   *
>   * SPDX-License-Identifier:     BSD-3-Clause
>
>
> Merged with corresponding fixes.
>
> Maxim.
>
>
>
> On Mon, Mar 21, 2016 at 7:52 AM, Maxim Uvarov <maxim.uva...@linaro.org
>> <mailto:maxim.uva...@linaro.org>> wrote:
>>
>>     does somebody also want to review that test?
>>
>>     Maxim.
>>
>>
>>     On 03/02/16 17:52, Christophe Milard wrote:
>>
>>         (Note: requires: linux-generic: shmem: odp scope in /dev/shmem)
>>         Adding a platform side test to test sharing memory between ODP
>>         and linux.
>>         This test is more interresting for its side effects:
>>         *first complete test on the platform side building ODP and
>>         platform
>>           programs (other platform tests so far were just wrappers to
>>         generic
>>           platform-agnostic test)
>>         *tests that the shared memory device name (under /dev/shm/) is
>>         scoped
>>           (under linux)
>>
>>         Christophe Milard (2):
>>            linux-generic: test: adding odp includes
>>            linux-generic: test: ODP_SHM_PROC test
>>
>>           platform/linux-generic/m4/configure.m4           |  1 +
>>           platform/linux-generic/test/Makefile.am          |  4 +-
>>           platform/linux-generic/test/Makefile.inc         | 16 +++
>>           platform/linux-generic/test/shmem/.gitignore     |  2 +
>>           platform/linux-generic/test/shmem/Makefile.am    | 18 +++
>>           platform/linux-generic/test/shmem/shmem.h        | 21 +++
>>           platform/linux-generic/test/shmem/shmem_common.h | 23 ++++
>>           platform/linux-generic/test/shmem/shmem_linux.c  | 156
>>         +++++++++++++++++++++++
>>           platform/linux-generic/test/shmem/shmem_linux.h  |  9 ++
>>           platform/linux-generic/test/shmem/shmem_odp.c    | 74
>>         +++++++++++
>>           platform/linux-generic/test/shmem/shmem_odp.h    |  7 +
>>           11 files changed, 330 insertions(+), 1 deletion(-)
>>           create mode 100644 platform/linux-generic/test/Makefile.inc
>>           create mode 100644 platform/linux-generic/test/shmem/.gitignore
>>           create mode 100644 platform/linux-generic/test/shmem/Makefile.am
>>           create mode 100644 platform/linux-generic/test/shmem/shmem.h
>>           create mode 100644
>>         platform/linux-generic/test/shmem/shmem_common.h
>>           create mode 100644
>>         platform/linux-generic/test/shmem/shmem_linux.c
>>           create mode 100644
>>         platform/linux-generic/test/shmem/shmem_linux.h
>>           create mode 100644 platform/linux-generic/test/shmem/shmem_odp.c
>>           create mode 100644 platform/linux-generic/test/shmem/shmem_odp.h
>>
>>
>>
>>
>


-- 
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
"Work should be fun and collaborative, the rest follows"
_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to