Re: [PATCH] Add nginx

2017-07-18 Thread Justin Cinkelj
The v2 patch: The real sigsuspend patch (4 of them) are not required. But, the stubs for sigsuspend, ioctl(FIOASYNC) and fcntl(F_SETOWN) are now required :) In nginx.conf, I removed keepalive_timeout, as it is not really required. And in makefile $(MAKE) is used instead of 'make -j4', so that par

Re: [PATCH] Add nginx

2017-07-16 Thread Justin Cinkelj
On 07/16/2017 03:14 PM, Nadav Har'El wrote: On Fri, Jul 14, 2017 at 12:02 PM, Justin Cinkelj mailto:justin.cink...@xlab.si>> wrote: Based on @myechuri work, https://github.com/myechuri/osv-apps/commits/nginx-osv Thanks (t

Re: [PATCH] Add nginx

2017-07-16 Thread Nadav Har'El
On Fri, Jul 14, 2017 at 12:02 PM, Justin Cinkelj wrote: > Based on @myechuri work, > https://github.com/myechuri/osv-apps/commits/nginx-osv Thanks (to both of you)! I want to commit this patch (we can always improve it later), but have one question first - does this patch depend on any of your

[PATCH] Add nginx

2017-07-14 Thread Justin Cinkelj
Based on @myechuri work, https://github.com/myechuri/osv-apps/commits/nginx-osv A single nginx worker thread is run, so only one CPU core is utilized. Example configuration includes very long keepalive_timeout/requests setting to avoid possible problems with OSV and TCP client port reuse. With ex