erasure_code/gf_vect_mad_test fails with avx2

[daxtens]

If I change gf_vect_mad_test to test the avx2 implementation, like so

#ifndef FUNCTION_UNDER_TEST                                                                                                                                                                                                                
# define FUNCTION_UNDER_TEST gf_vect_mad_avx2                                                                                                                                                                                              
# define REF_FUNCTION gf_vect_dot_prod                                                                                                                                                                                                     
# define VECT 1                                                                                                                                                                                                                            
#endif

I get the following error:

Fail rand gf_vect_mad_avx2 test0 size=16
 40 a2 e2 15 38 32 96 af 18 f7 12 de 5c 8c 39  1

dprod_base: 1d 57 49 fd 6e 87 24 9f 67 79 5c 73 e7 a3 cc 98 be
dprod_dut:  0  0  0  0  0  0  0  0  0  0  0  0  0  0  0  0 be

The tests pass if I use avx or sse.

This is broken on all the variants of gf_Nvect_mad_avx2 that I have tested.

[gbtucker]

This is an arch-specific test and these specific low-level functions have different alignment restrictions. To use with the test you need to also define align_size=32 like so.

#ifndef ALIGN_SIZE
-# define ALIGN_SIZE 16
+# define ALIGN_SIZE 32
#endif

[daxtens]

OK, good to know - I fix up the alignment!

…

On Tue, Mar 21, 2017 at 11:41 AM, Greg Tucker ***@***.***> wrote: This is an arch-specific test and these specific low-level functions have different alignment restrictions. To use with the test you need to also define align_size=32 like so. #ifndef ALIGN_SIZE -# define ALIGN_SIZE 16 +# define ALIGN_SIZE 32 #endif — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#19 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADJ5BewVOg4C0-20EzsSvyhYtZ7EzNm1ks5rnxzRgaJpZM4MjLHA> .