Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NGINX building from openresty requires newer SSE4.2 compatible CPU #649

Closed
tobybellwood opened this issue Sep 28, 2018 · 4 comments
Closed

Comments

@tobybellwood
Copy link
Member

Just something I encountered - now that the NGINX images have switched from alpine to openresty, it has introduced a requirement for SSE4.2 compatible CPUs on the host machine (pretty much anything 2011-onwards - but you'll need to check) - some reference at https://github.com/openresty/docker-openresty#tips--pitfalls

It will manifest itself as the NGINX container building, but not deploying (immediate fail with a exit code 132.

Mounting the failed container and running nginx directly will result in an illegal instruction error.

Luckily Ebay is a good hunting ground for old brand-new CPUs...

@Schnitzel
Copy link
Contributor

@tobybellwood
wow nice found! Do you have cases where this affects local development?

I went down the rabbithole and researched that, looks like there is a performance loss if we run an OpenResty without SSE4.2 flags on a SSE4.2 compatible machine (but actually only for the LUA stuff, so not the nginx part of openresty).

There is though an Issue: openresty/docker-openresty#39 and PR openresty/luajit2#20 that will teach LUA how to decide to use SEE4.2 in runtime instead of buildtime.

Until then I think the easiest would be to run our own openresty version without SSE4.2 flags.

@tobybellwood
Copy link
Member Author

I honestly think it's not worth the effort to re-engineer around a problem that no-one else has reported - if more people do, then we should pursue - I more left this here in case anyone else comes across it!

For the record, my brand-new 2013-era FX-8320 CPU does the job just fine now!

@Schnitzel
Copy link
Contributor

sounds good! thanks for the report, closing this then :)

@rzr
Copy link

rzr commented Oct 11, 2019

openresty/luajit2#75 may help to support non SSE-4.2 CPU

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants