Re: [lng-odp] [API-NEXT PATCH] linux-generic: tm: use odp_hash_crc32c() api to avoid arch issues

2015-12-08 Thread Maxim Uvarov
Merged this patch. Now only issue left is fix: hash_main: no such file I think something wrong in Makefile.am, not sure if CI also captures that issues, but we will see. That is different issue. Maxim. On 12/03/2015 00:56, Bill Fischofer wrote: Change the internal hash_name_and_kind()

Re: [lng-odp] [API-NEXT PATCH] linux-generic: tm: use odp_hash_crc32c() api to avoid arch issues

2015-12-08 Thread Bill Fischofer
hash_main is the binary output from compiling the hash CUnit test. That's should have nothing to do with TM. On Tue, Dec 8, 2015 at 9:35 AM, Maxim Uvarov wrote: > Merged this patch. > > Now only issue left is fix: > hash_main: no such file > I think something wrong in

Re: [lng-odp] [API-NEXT PATCH] linux-generic: tm: use odp_hash_crc32c() api to avoid arch issues

2015-12-04 Thread Maxim Uvarov
Bill, here is the same problem which I had with mine updated IPC patches. odp-check says that there is no hash_main binary so test fails. I think something like that is needed. But it does not help: +++ b/test/validation/hash/Makefile.am @@ -3,7 +3,7 @@ include ../Makefile.inc

Re: [lng-odp] [API-NEXT PATCH] linux-generic: tm: use odp_hash_crc32c() api to avoid arch issues

2015-12-04 Thread Bill Fischofer
Per our discussion a few moments ago, When I run the following: GIT_BRANCH=api-next PATCH_DIR=$HOME/linaro/tmcrcfix CLEANUP=0 ./build.sh it executes cleanly for me, so I'm not sure what the problem is. On Fri, Dec 4, 2015 at 5:52 AM, Maxim Uvarov wrote: > Bill, here

Re: [lng-odp] [API-NEXT PATCH] linux-generic: tm: use odp_hash_crc32c() api to avoid arch issues

2015-12-03 Thread Bill Fischofer
Ping. This patch needs a review as it fixes critical bug https://bugs.linaro.org/show_bug.cgi?id=1930 that is breaking CI for api-next. Thanks. On Wed, Dec 2, 2015 at 3:56 PM, Bill Fischofer wrote: > Change the internal hash_name_and_kind() function to eliminate the

[lng-odp] [API-NEXT PATCH] linux-generic: tm: use odp_hash_crc32c() api to avoid arch issues

2015-12-02 Thread Bill Fischofer
Change the internal hash_name_and_kind() function to eliminate the use of architecture-specific hash instructions. This eliminates build issues surrounding ARM variants. Future optimizations will use the arch directory consistent with other ODP modules. Signed-off-by: Bill Fischofer