Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-17 Thread Mike Holmes
This series does not apply cleanly with apply-and-build looks like it needs refactoring so that each patch builds when it is applied. ls ~/incoming/bill/ API-NEXT-PATCHv10-1-4-api-tm-add-tm-API-definitions.patch API-NEXT-PATCHv10-3-4-linux-generic-tm-add-tm-to-build.patch API-NEXT-PATCHv10-2-4-lin

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-17 Thread Maxim Uvarov
Problem with building 32 bit app: /* A hash table entry is LOGICALLY either empty, a pointer to a 64-byte * aligned name_tbl_entry_t OR a pointer to a 64-byte aligned secondary hash * table. Since the bottom 6-bits of this value are not needed to hold the * address, these 6 bits are used

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-13 Thread Maxim Uvarov
g-odp-boun...@lists.linaro.org] On Behalf Of EXT Maxim Uvarov Sent: Friday, November 13, 2015 9:54 AM To: Alexandru Badicioiu Cc: LNG ODP Mailman List Subject: Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager On 11/13/2015 10:29, Alexandru Badicioiu wrote: Compile with -march=armv8-a+crc.

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-13 Thread Savolainen, Petri (Nokia - FI/Espoo)
andru Badicioiu > Cc: LNG ODP Mailman List > Subject: Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager > > On 11/13/2015 10:29, Alexandru Badicioiu wrote: > > Compile with -march=armv8-a+crc. > > > > Alex > > > ok, now arm64 looks like work, but

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-13 Thread Alexandru Badicioiu
The crc32 definition should be between : #elif defined(__arm__) || defined(__aarch64__) #else #error "Need to define PLATFORM_DEPENDENT_HASH32 macro" #endif so it should not affect a x86 build. On 13 November 2015 at 09:54, Maxim Uvarov wrote: > On 11/13/2015 10:29, Alexandru Badicioiu wrote:

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-12 Thread Maxim Uvarov
On 11/13/2015 10:29, Alexandru Badicioiu wrote: Compile with -march=armv8-a+crc. Alex ok, now arm64 looks like work, but other arches broken: ./configure --with-platform=linux-generic --prefix=/builds/check-odp-new.git/new-build --enable-test-perf --with-openssl-path=/builds/check-odp-new.g

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-12 Thread Alexandru Badicioiu
Compile with -march=armv8-a+crc. Alex On 13 November 2015 at 09:23, Maxim Uvarov wrote: > now it compiles, but fails on execution in virtual machine: > > ./configure --with-platform=linux-generic > --prefix=/builds/check-odp-new.git/new-build --host=aarch64-linux-gnu > --enable-test-perf > --wi

Re: [lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-12 Thread Maxim Uvarov
now it compiles, but fails on execution in virtual machine: ./configure --with-platform=linux-generic --prefix=/builds/check-odp-new.git/new-build --host=aarch64-linux-gnu --enable-test-perf --with-openssl-path=/builds/check-odp-new.git/installed/arm64/openssl-OpenSSL_1_0_1h --enable-test-val

[lng-odp] [API-NEXT PATCHv10 0/4] Egress Traffic Manager

2015-11-12 Thread Bill Fischofer
Changes in v10 - Add back ARM definitions for _crc32w() from Alex that got dropped in v9 Note: Checkpatch complains about the _asm() line. Says spaces are needed but if spaces are added then complains that spaces are prohibited. Rules need to be updated to ignore these sort of constructs. Chang