mwarning / zerotier-openwrt

A OpenWrt package for ZeroTier One - Pull requests are welcome!
697 stars 146 forks source link

SSE2 instruction set not enabled #1

Closed raarts closed 9 years ago

raarts commented 9 years ago

Hi,

This would not compile for me, I get the following:

In file included from node/Salsa20.hpp:20:0,
                 from node/Identity.cpp:36:
/home/raarts/openwrt/staging_dir/toolchain-i386_i486_gcc-4.8-linaro_uClibc-0.9.33.2/lib/gcc/i486-openwrt-linux-uclibc/4.8.3/include/emmintrin.h:31:3: error: #error "SSE2 instruction set not enabled"
 # error "SSE2 instruction set not enabled"
   ^
In file included from node/Identity.cpp:36:0:
node/Salsa20.hpp:82:3: error: '__m128i' does not name a type
   __m128i v[4];
   ^

Now I can disable SSE2 in the zerotier-one Makefile, by just removing it, but I am wondering why it compiled for you.

mwarning commented 9 years ago

That is odd. It compiles for me. I use Debian 8.0 right now and OpenWrt 14.07 (Barrier Breaker). Did you disable it by adding -mno-sse?

raarts commented 9 years ago

Hi,

I re-did everything, but same problem. Note I chose the x86 Target, because I run it on virtualbox.

Ron

On Tue, Feb 3, 2015 at 9:48 AM, Moritz Warning notifications@github.com wrote:

That is odd. I compiles for me. I use Debian 8.0 right now and OpenWrt 14.07 (Barrier Breaker). Did you disable it by adding -mno-sse?

— Reply to this email directly or view it on GitHub https://github.com/mwarning/zerotier-openwrt/issues/1#issuecomment-72662515 .

raarts commented 9 years ago

BTW, I am an openwrt noob, so I wouldn't know where to specify -mno-sse

mwarning commented 9 years ago

I think the x86 target is the "culprit". Zerotier probably enables sse2 for x86. I will do some testing. Thanks for the find!

raarts commented 9 years ago

By the way, I had to install gettext as well, you might want to add that to the apt-get line in README.md

mwarning commented 9 years ago

Both issues should be fixed now. Thanks a lot for reporting!