[Buildroot] [PATCH 3/5] package/nginx: add thread pool support

[Martin Bark]

Thomas,

On 1 May 2016 at 22:42, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Hello,
>
> On Sun,  1 May 2016 20:52:23 +0100, Martin Bark wrote:
>
>>  NGINX_CONF_OPTS += \
>> -     $(if $(BR2_PACKAGE_NGINX_FILE_AIO),--with-file-aio)
>> +     $(if $(BR2_PACKAGE_NGINX_FILE_AIO),--with-file-aio) \
>> +     $(if $(BR2_PACKAGE_NGINX_THREADS),--with-threads)
>
> Just curious, any reason to not enable this as soon as
> BR2_TOOLCHAIN_HAS_THREADS=y ?

Well it does add code so if you just want nginx as a static webserver
on an embedded device you probably don't need this.  Just for interest
there is some info on nginx thread pools here
https://www.nginx.com/blog/thread-pools-boost-performance-9x/.  I
really just added the option for completeness.

What is missing is a dependency on BR2_TOOLCHAIN_HAS_THREADS for
BR2_PACKAGE_NGINX_THREADS in the Config.in so I'll resubmit with that
fixed.

Thanks

Martin

>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com